Notendurchschnitt wird falsch berechnet

            int sum = 0 ;
            int quantity=0;
            double average = 0;
                
                
            for (int idx = 0; idx < marks.GetLength(0); idx++) //Schleife für die  Berechnung der Durchschnitte von jedem Schüler
            {
                for (int i = 0; i < marks[idx].Length; i++) 
                {
                    sum +=  marks[idx][i];
                    quantity++;
                }
                average = (double)sum / quantity;
                Console.WriteLine("Der Durchschnitt des  "+ (idx+1) +" Schülers beträgt:{0}",average);
                average = 0;
                
            }

LG, DerAskTyp

Hallo,

weil quantity und sum nicht zurück auf 0 gesetzt werden?

Gruß, MarsStein

Dein Code, den Teil den wir hier sehen können zumindest, sieht noch nicht so gut aus.
marks scheint mir ein Konstrukt eines Arrays von Arrays zu sein.
Wäre es nicht sinnvoller anstelle dieser Kombination, die zwar funktioniert es schwerer zu lesen ist, nicht marks als eine Liste mit einem entsprechenden Container Objekt umzusetzen?
Dann ist der Code auch verständlicher.

Hilft dir später auch, wenn du deinen alten Code mal nach X Monaten wieder anfassen musst.
In einem kleinen Projekt vielleicht kein Problem aber in großen sollte man da schon sauber arbeiten.
Würde es dir ans Herz legen dir das anzugewöhnen, macht das lesen des eigenen Codes um längen leichter 😃

T-Virus

[Artikel] Debugger: Wie verwende ich den von Visual Studio?

und für logische Probleme hilft oft: back to roots -> Blatt und Bleistift -> Prozess gedanklich abbilden

*seufz* Genau für so etwas wurde mal LinQ eingeführt.

Einzeiler.

var marksPerStudent = marks.Select((p,index) => new { Index = index+1, AverageMark = p.Average() }).ToList();

marksPerStudent.ForEach(p => Console.WriteLine("Schüler: {0}, Durchschnitt: {1:N}", p.Index, p.AverageMark));

LaTino

*seufz*
Naja.... mal bei den Tatsachen bleiben.

Bei nem Anfänger jetzt mit Linq zu kommen, der nicht mal das Zurücksetzen von Variablenwerten erkannt hat: definitiv falsche Zielgruppe und stiftet mehr Verwirrung als Lösung.

Hm. Mag sein, dass das an meinem speziellen Arbeitsumfeld liegt.
(Resharper läuft hier überall, und der schlägt bei den üblichen, ich nenn' sie mal "Anfängerschleifen", automatisch Linq-Vereinfachungen vor. Unsere Anfänger lernen das also von Anfang an einfach nur durch häufiges Sehen UND sie lernen gleich noch, dass es sich um eine kürzere und sicherere Schreibweise handelt. Solche Fehler wie oben kriege ich deshalb so gut wie nie zu Gesicht.)

Also, @DerAskTyp: wenn du deine foreach-Anweisungen im Schlaf beherrschst, DANN erinner' dich dran, das mal jemand Linq erwähnt hat, und schau's dir an 😉.

LaTino