Semikolon zu viel - ich raffs nicht..

[herrSchlaumann]

Moin zusammen,

ich sitze gerade an der folgenden Aufgabe:

"Ein magisches Quadrat ist eine quadratische Matrix, für die die Summe jeder Zeile, jeder Spalte und der beiden Diagonalen denselben Wert ergibt. Lesen sie eine Zahl n sowie eine n*n Matrix ein und stellen sie fest, ob es ein magisches Quadrat ist".

Hier mein selbst programmierter Code (ich bin Einsteiger, sicherlich kann man den Code an vielen Stellen optimieren):

static int[][] magic(int[][] a) {
        int[][] b = new int[a.length][a[0].length];
      
        int i, j, sumZeile1, sumZeile2, sumSpalte1, sumSpalte2, sumDiag1, sumDiag2;
      
            for (i = 0, j = 0; j < a[0].length; j++){
                sumZeile1 = sumZeile1 + a[i][j]; }
            do {
            for (i++, j++; j < a[0].length; j++){
                sumZeile2 = sumZeile2 + a[i][j]; }
            }while (sumZeile1 == sumZeile2);  
                if (sumZeile1 != sumZeile2)
                    {Out.print("Es handelt sich nicht um ein Magisches Quadrat"); break;}
              
        for (i = 0, j = 0; i < a[0].length; i++){
            sumSpalte1 = sumSpalte1 + a[i][j]; }
            do {
            for (i++, j++; i < a[0].length; i++){
            sumSpalte2 = sumSpalte2 + a[i][j]; }
            }while (sumSpalte1 == sumSpalte2);  
                if (sumSpalte1 != sumSpalte2)
                {Out.print("Es handelt sich nicht um ein Magisches Quadrat"); break;}

        for (i = 0, j = 0; i < a[0].length; i++, j++){
            sumDiag1 = sumDiag1 + a[i][j]; }
        for (i = a.length - 1, j = a.length - 1; i == -1; i--, j--){
            sumDiag2 = sumDiag2 + a[i][j]; }
        if (sumDiag1 != sumDiag2)
        {Out.print("Es handelt sich nicht um ein Magisches Quadrat"); break;}
      
        if (sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1){
            {Out.print("Es handelt sich hier um ein Magisches Quadrat"); break;}
        }
        return b;
    }
}

    public static void main(String[] args) {
        int i, m, n;
        Out.print("Geben Sie eine Zahl für die Länge der Matrix ein. Diese ist gleichzeitig die Breite.");
        int[][] a = new int[i = In.readInt()][i];
        Out.print("Geben Sie nacheinander die einzelnen Werte der Matrix ein.");
        for (m = 0; m < i; m++)
            for (n = 0; n < i; n++)
                a[m][n] = In.readInt();
    }
}

Das zweite Semikolon nach return zeigt er mir als zu viel an, lösche ich das färbt er mir fast meine ganze Methode rot. Ich suche schon lange rum und finde das Problem nicht.
Ob der Code jetzt im Einzelnen richtig ist weiß ich nicht, das konnte ich noch nicht testen.

Ich bedanke mich schon mal für eure Hilfe!

PS: Nicht wundern, ich muss mit den Klassen von Mössenböck arbeiten, Autor von "Sprechen Sie Java?".

 

[Xyz1]

Du hättest einen Block in Block nach if (sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1){, das ist definitiv nicht gewollt. Entferne, in der nachfolgenden Zeile, {}-Klammern.

 

[mrBrown]

nach dem return ist eine geschlossene } zu viel.

Benutzt du einen JavaEditor? Der sollte da noch einige weitere Fehler markieren...

Du hättest einen Block in Block nach if (sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1){, das ist definitiv nicht gewollt. Entferne, in der nachfolgenden Zeile, {}-Klammern.

Ist aber irrelevant^^

 

[herrSchlaumann]

nach dem return ist eine geschlossene } zu viel.

Benutzt du einen JavaEditor? Der sollte da noch einige weitere Fehler markieren...

Ja, ich arbeite mit Eclipse. Dann muss ich doch die Methode überarbeiten.
Mein Problem da ist, dass ich die break-Statements nicht richtig unterbekomme.
Eclipse meckert da und sagt, dass break im Loop sein muss. Was ist da bei mir falsch?

 

[mrBrown]

Ja, ich arbeite mit Eclipse. Dann muss ich doch die Methode überarbeiten.
Mein Problem da ist, dass ich die break-Statements nicht richtig unterbekomme.
Eclipse meckert da und sagt, dass break im Loop sein muss. Was ist da bei mir falsch?

Sie sind nicht in einem Loop.

An den Stellen, an denen das angemeckert wird, sind deine Schleifen schon zu Ende, du bist nur noch in einem if, daraus kann man nicht breaken. break springt nur ans Schleifenende.

 

[herrSchlaumann]

Sie sind nicht in einem Loop.

An den Stellen, an denen das angemeckert wird, sind deine Schleifen schon zu Ende, du bist nur noch in einem if, daraus kann man nicht breaken. break springt nur ans Schleifenende.

Ja natürlich. Wie dumm
Gibt es denn eine andere Möglichkeit (außer den Code umzuschreiben) bei der if-Bedingung abzubrechen, also eine Art von "else break"?

 

[mrBrown]

Was heißt für dich abbrechen? Die Methode beenden wäre return, eine Fehler werfen eine Exception

 

[Xyz1]

Ich habe das mal gemockt,

    static class OUT {
        void print(String s) {
            System.out.println(s);
        }
    }
    static OUT Out = new OUT();

    static class IN {
        int readInt() {
            return 0;
        }
    }
    static IN In = new IN();

    static int[][] magic(int[][] a) {
        int[][] b = new int[a.length][a[0].length];

        int i, j, sumZeile1 = 0, sumZeile2 = 0, sumSpalte1 = 0, sumSpalte2 = 0, sumDiag1 = 0, sumDiag2 = 0;

        for (i = 0, j = 0; j < a[0].length; j++) {
            sumZeile1 = sumZeile1 + a[i][j];
        }
        do {
            for (i++, j++; j < a[0].length; j++) {
                sumZeile2 = sumZeile2 + a[i][j];
            }
        } while (sumZeile1 == sumZeile2);
        if (sumZeile1 != sumZeile2) {
            Out.print("Es handelt sich nicht um ein Magisches Quadrat");
            // break;
        }

        for (i = 0, j = 0; i < a[0].length; i++) {
            sumSpalte1 = sumSpalte1 + a[i][j];
        }
        do {
            for (i++, j++; i < a[0].length; i++) {
                sumSpalte2 = sumSpalte2 + a[i][j];
            }
        } while (sumSpalte1 == sumSpalte2);
        if (sumSpalte1 != sumSpalte2) {
            Out.print("Es handelt sich nicht um ein Magisches Quadrat");
            // break;
        }

        for (i = 0, j = 0; i < a[0].length; i++, j++) {
            sumDiag1 = sumDiag1 + a[i][j];
        }
        for (i = a.length - 1, j = a.length - 1; i == -1; i--, j--) {
            sumDiag2 = sumDiag2 + a[i][j];
        }
        if (sumDiag1 != sumDiag2) {
            Out.print("Es handelt sich nicht um ein Magisches Quadrat");
            // break;
        }

        if (sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1) {
            {
                Out.print("Es handelt sich hier um ein Magisches Quadrat");
                // break;
            }
        }
        return b;
    }

    public static void main(String[] args) {
        int i, m, n;
        Out.print("Geben Sie eine Zahl für die Länge der Matrix ein. Diese ist gleichzeitig die Breite.");
        int[][] a = new int[i = In.readInt()][i];
        Out.print("Geben Sie nacheinander die einzelnen Werte der Matrix ein.");
        for (m = 0; m < i; m++) {
            for (n = 0; n < i; n++) {
                a[m][n] = In.readInt();
            }
        }
    }

DEINE aufgabe ist es jetzt, die Programmablauflogik zu verstehen. Das Buch kenn ich nicht.

 

[Cromewell]

also eine Art von "else break"?

Soweit ich weiß, geht es auch mit

return;

EDIT: Oder eher nicht x)

 

[mrBrown]

Soweit ich weiß, geht es auch mit

return;

EDIT: Oder eher nicht x)

Ginge bei void als Rückgabetyp

 

[Cromewell]

Ist mir dann auch aufgefallen

 

[Xyz1]

Also so schwer ist das nicht, wenn ich länger draufschaue komme ich in Versuchung das zu verstehen.
Zuerst wird gesummt usw.

Was ich aber NICHT verstehe:
1. magic wird nirgendwo aufgerufen,
2. Rückgabe von b ist sinnlos,
3. DU hast das wahrscheinlich KOMPLETT falsch abgeschrieben.

Punkt 3 soll kein vorwurf sein. Ich selber tippe auch alles falsch ab. So viel dazu. ALSO:
Verklag' mich nicht!

(4. Eclipse würd ich dir in deinem "Stadion" nicht empfehlen.)

 

[Cromewell]

(4. Eclipse würd ich dir in deinem "Stadion" nicht empfehlen.)

Stadion ?

 

[Xyz1]

Ja, ich wollte nicht so gemein sein. Viel mir aber auch nix anderes sein.

 

[Cromewell]

Nicht eher Stadium ?

 

[herrSchlaumann]

Vielen Dank für die ganzen Antworten.
Ich habe hier nichts abgeschrieben, sondern selbst programmiert - wenn auch ziemlich verkehrt, wie ich feststellen musste.
Ich habe mich jetzt nochmal hingesetzt und das Programm umgeschrieben - ich hänge euch mal den funktionierenden Code an:

static void magic(int[][] a) {
       
        int i = 0; int j = 0;
        int sumZeile1 = 0; int sumZeile2= 0;
        int sumSpalte1= 0; int sumSpalte2= 0;
        int sumDiag1= 0; int sumDiag2 = 0;
       
            for (i = 0, j = 0; j < a.length; j++){
                sumZeile1 = sumZeile1 + a[i][j]; }
            i++; j= 0;
            while (j < a.length){
            sumZeile2 = sumZeile2 + a[i][j];
            j++;
            }
            if (sumZeile1 != sumZeile2){
                Out.print("Es handelt sich nicht um ein Magisches Quadrat");   
                return;}
           
            for (i = 0, j = 0; i < a.length; i++){
                sumSpalte1 = sumSpalte1 + a[i][j]; }
                i = 0; j++;
            while (i < a.length){
                sumSpalte2 = sumSpalte2 + a[i][j];
                i++;
            }
            if (sumSpalte1 != sumSpalte2) {
                Out.print("Es handelt sich nicht um ein Magisches Quadrat");   
                return;}
                   
        for (i = 0, j = 0; i < a[0].length; i++, j++){
            sumDiag1 = sumDiag1 + a[i][j]; }
        i = 0; j = 0;
        for (i = a.length - 1, j = a.length - 1; i > -1; i--, j--){
            sumDiag2 = sumDiag2 + a[i][j]; }
        if (sumDiag1 != sumDiag2){
        Out.print("Es handelt sich nicht um ein Magisches Quadrat");
        return;}
       
        if (sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1){
            {Out.print("Es handelt sich hier um ein Magisches Quadrat");
            return;
            }
        }
    }
    public static void main(String[] args) {
        int i, m, n;
        Out.print("Geben Sie eine Zahl für die Länge der Matrix ein. Diese ist gleichzeitig die Breite.");
        int[][] a = new int[i = In.readInt()][i];
        Out.print("Geben Sie nacheinander die einzelnen Werte der Matrix ein.");
        for (m = 0; m < i; m++)
            for (n = 0; n < i; n++)
                a[m][n] = In.readInt();
        magic(a);
    }
}

 

[Xyz1]

Ich habe hier nichts abgeschrieben, sondern selbst programmiert [...] Ich habe mich jetzt nochmal hingesetzt und das Programm umgeschrieben[...]

Hoffentlich nicht in die Nesseln!!

[...] ich hänge euch mal den funktionierenden Code an:

Und was sollen wir jetzt machen?? Fast jede zweite Zeile sollte in eine eigene Methode...

 

[InfectedBytes]

und du würdest dir selbst einen groß Gefallen tun, wenn du deinen Code einmal ordentlich einrücken würdest. Dadurch sieht man gleich viel schneller wo denn Probleme liegen könnten.
Schau dir mal zum Vergleich diesen kleinen Ausschnitt an:

if(sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1){
           {Out.print("Es handelt sich hier um ein Magisches Quadrat");
           return;
           }
       }

und einmal in "schöner"

if(sumZeile1 == sumSpalte1 && sumZeile1 == sumDiag1){
  {
    Out.print("Es handelt sich hier um ein Magisches Quadrat");
    return;
  }
}

Hier sieht man jetzt auch, das du einen unnötiges paar geschweifte Klammern benutzt, das schadet zwar nicht, macht es aber wieder unübersichtlicher

Ansonsten, wie @DerWissende schon geschrieben hat, solltest du uns schon sagen was wir jetzt mit deinem Code anfangen sollen. Kommt eine Fehlermeldung? Funktioniert es nicht? Wenn ja, was denn genau? etc.

 

[herrSchlaumann]

Danke für euer Feedback. Der Code funktioniert und ich wollte einfach den überarbeiteten und funktionierenden Code nochmal hochladen für diejenigen, die mir schon geholfen haben und ggf. sehen wollten wie ich das jetzt gelöst habe.

Ich stehe wie schon gesagt noch relativ am Anfang, weshalb es mir z.T. auch schwer fällt, mehrere Methoden zu implementieren. Aber ich nehme eure Punkte gern auf und versuche, da in Zukunft mehr drauf zu achten.