using goto in nested loops

[cody]

for (int serie=1; serie < TurnierGruppe.AktuelleSerie.Serie; serie++)
{
//outer: here is works
foreach (Tischbesetzung tb in
TurnierGruppe.Serien[serie].Tischebesetzungen)
{
for (int i=0; i<tb.SpielerAnzahl; i++)
{
if (tb.Spieler[i].Spieler==this)
{
gespielteSerien.Add(tb.Spieler[i]);
goto outer;
}
}
}
outer:
}

Iam getting an error that a semicolon is missing after the label. If I move
the label to the top of the loop it works.
I guess C# doesn't allow labels with no code following directly, if so I'd
like to know the reason.

The problem for me is now if I'd move the label to the top of the loop the
enumerator will not be moved to the next element which would cause serious
trouble for me.

--
cody

Freeware Tools, Games and Humour
http://www.deutronium.de.vu || http://www.deutronium.tk

[Jeti]

i know, this is not an answer, but why dont you do this (use flag & break):
bool flag = true;

for (int serie=1; serie < TurnierGruppe.AktuelleSerie.Serie && flag; serie++) {
//outer: here is works
foreach (Tischbesetzung tb in
TurnierGruppe.Serien[serie].Tischebesetzungen)
{
for (int i=0; i<tb.SpielerAnzahl; i++)
{
if (tb.Spieler[i].Spieler==this)
{
gespielteSerien.Add(tb.Spieler[i]);
flag = false; break; }
}
}
outer:
}

[cody]

> i know, this is not an answer, but why dont you do this (use flag &
break):

bool flag = true;

for (int serie=1; serie < TurnierGruppe.AktuelleSerie.Serie && flag;

serie++)

{
//outer: here is works
foreach (Tischbesetzung tb in
TurnierGruppe.Serien[serie].Tischebesetzungen)
{
for (int i=0; i<tb.SpielerAnzahl; i++)
{
if (tb.Spieler[i].Spieler==this)
{
gespielteSerien.Add(tb.Spieler[i]);
flag = false;

break;

}
}
}
outer:
}

Using flags degrades performance, is error prone and less readable than even
goto.
Now the question arises why C# doesn't allow labeled breaks like Java.

--
cody

Freeware Tools, Games and Humour
http://www.deutronium.de.vu || http://www.deutronium.tk

[Morten Wennevik]

Hi Cody,

A label needs to have some code it refers to.

Try putting an empty code block {} after outer:

outer: {}

Happy coding!
Morten Wennevik [C# MVP]

[C# Learner]

cody wrote:

[...]
Iam getting an error that a semicolon is missing after the label. If I move
the label to the top of the loop it works.
I guess C# doesn't allow labels with no code following directly, if so I'd
like to know the reason.

The problem for me is now if I'd move the label to the top of the loop the
enumerator will not be moved to the next element which would cause serious
trouble for me.

Just add a semi-colon after the colon:

outer:;

C# labels require a statement, so give it a blank statement.

[cody]

> Just add a semi-colon after the colon:

outer:;

C# labels require a statement, so give it a blank statement.

How funny :)

--
cody

Freeware Tools, Games and Humour
http://www.deutronium.de.vu || http://www.deutronium.tk

[Morten Wennevik]

Even simpler than an empty code block :)

Happy coding!
Morten Wennevik [C# MVP]

[Daniel Jin]

then add a semicolon. "outer: ;" if you don't like the look of it, do "outer: continue;" I don't like goto, never had to use it. but I guess using it to get yourself out of very deeply nested loops is not THAT bad

----- cody wrote: ----

for (int serie=1; serie < TurnierGruppe.AktuelleSerie.Serie; serie++

//outer: here is work
foreach (Tischbesetzung tb i
TurnierGruppe.Serien[serie].Tischebesetzungen

for (int i=0; i<tb.SpielerAnzahl; i++

if (tb.Spieler[i].Spieler==this

gespielteSerien.Add(tb.Spieler[i])
goto outer

outer
Iam getting an error that a semicolon is missing after the label. If I mov
the label to the top of the loop it works
I guess C# doesn't allow labels with no code following directly, if so I'
like to know the reason

The problem for me is now if I'd move the label to the top of the loop th
enumerator will not be moved to the next element which would cause seriou
trouble for me

-
cod

Freeware Tools, Games and Humou
http://www.deutronium.de.vu || http://www.deutronium.t

[Olex Malko]

Try to change "outer:" to "outer:;".

"cody" <no****************@gmx.net> wrote in message
news:O8**************@TK2MSFTNGP12.phx.gbl...

for (int serie=1; serie < TurnierGruppe.AktuelleSerie.Serie; serie++)
{
//outer: here is works
foreach (Tischbesetzung tb in
TurnierGruppe.Serien[serie].Tischebesetzungen)
{
for (int i=0; i<tb.SpielerAnzahl; i++)
{
if (tb.Spieler[i].Spieler==this)
{
gespielteSerien.Add(tb.Spieler[i]);
goto outer;
}
}
}
outer:
}

Iam getting an error that a semicolon is missing after the label. If I move the label to the top of the loop it works.
I guess C# doesn't allow labels with no code following directly, if so I'd
like to know the reason.

The problem for me is now if I'd move the label to the top of the loop the
enumerator will not be moved to the next element which would cause serious
trouble for me.

--
cody

Freeware Tools, Games and Humour
http://www.deutronium.de.vu || http://www.deutronium.tk

[cody]

> I don't like goto, never had to use it. but I guess using it to get
yourself out of very deeply nested loops is not THAT bad.
Yes, in this case a goto is the best solution. Iam still wondering why .NET
doesn't support labeled breaks like Java.

--
cody

Freeware Tools, Games and Humour
http://www.deutronium.de.vu || http://www.deutronium.tk