C# Verschachtelte Funktionen

[marivuko]

hallo habe mal ne Frage,
mein Programm funktioniert leider nicht, habe folgende Funktion erstellt, mir wird kein Fehler ausgegeben.

kann man das in c# so schreiben?
oder stimmt da was net von den if und else if her ?

   while( j<50)
            {

                if (x[j] == y[i] && a[i] ==b[j]) 
                {
                    punkte[j] = 3;
                }

                else if ((x[j] > y[j]) && (a[i] > b[i]))
                {
                    punkte[j] = 1;
                }

                else if ((a[j]-b[j]) || a[i] < b[i])
                {
                    punkte[j] = 2;
                }

                else
                {
                    punkte[j] = 0;
                }

            j++;
            i++;
            if ( i == 47)
            {
                i = 0;
            }
            }

 

[SoDaTierchen]

Naja, interessant wäre noch zu wissen, was du machen willst. Da dein Code ja nicht tut was er soll, können wir deinen Code nicht lesen um zu erkennen, was er tun soll.

 

[antred]

hallo habe mal ne Frage,
mein Programm funktioniert leider nicht, habe folgende Funktion erstellt, mir wird kein Fehler ausgegeben.

Wo siehst du da eine Funktion?

 

[InkognitoGER]

Die Syntax scheint korrekt zu sein, da ja auch keine Errors geworfen werden.
Allerdings lässt sich aus diesem unvollständigen Code-Fragment nicht erkennen was da überhaupt ablaufen soll.

Noch ein Tipp nebenbei. Die Variablendeklaration mit a,b,c, etc... ist schlechter Stil. Bezeichne die Variablen nach dem für was sie stehen.

 

[marivuko]

stimmt, ich will werte vergleichen, die in einem Array geschrieben sind,
wenn der Vergleich dann stimmt, sollen die genannten Punkte angezeigt werden,
aber eigentlich geht es mir eher darum ob man das so in c# schreiben kann mit den ganzen verschachtelten ifs

 

[Deamp]

Schön wäre es gewesen wenn die Variablen beschreibene Namen hätten.

Hier mal ne gute Buchempfehlung wo viel über steht über wie Code aussehen sollte.
Clean Code - Robert C. Martin
Eng - http://www.amazon.de/Clean-Code-Handbook-Software-Craftsmanship/dp/0132350882/
De - http://www.amazon.de/Clean-Code-Refactoring-Patterns-Techniken/dp/3826655486/

 

[marivuko]

komisch dachte es könnte an der Syntax liegen, denn er gibt mir in jedem Feld die 3 aus

 

[InkognitoGER]

Vermutlich weil die erste Bedingung immer wahr ist.
Aber um das prüfen zu können, benötigen wir auch den weiteren Code, wo du die Arrays setzt, etc..

 

[Deamp]

2. else if nochmal überprüfen

else if ((a[j]-b[j]) || a < b)

 

[SoDaTierchen]

Die ifs kann man so schachteln, ja. Aber der Code tut vermutlich nicht, was er soll. Du zählst i und j gleichmäßig hoch und setzt i dann willkürlich zurück. Zudem ist bei deinem dritten if ein Minus-Operator, der dort wenig Sinn ergibt.

Das Schachteln ist so möglich, du solltest aber den Code nochmal überdenken