class, interface or enum expected

[Klassensprecher]

Hallo zusammen...

Ich bin auf eine Fehlermeldung gestoßen mit der ich überhaupt garnichts anfangen kann.
"class, interface or enum expected" kommt bei mir unglaubliche 41 mal ... -.-

Ich poste der übersichtlichkeit halber nicht den ganzen code sondern nur den Ausschnitt.

Es soll eine 9x9 Array erzeugt werden welches mit zufälligen Zahlen befüllt werden soll um dann zu prüfen ob es eine gültige Lösung im Sudoku wäre.

Wenn ich aber meine Methode zum Zeilen und Spalten prüfen aufrufe, kommt besagter Fehler.

static boolean pruefeZeilen(int[][] sudoku) {

int x = 0;
boolean [] zeile = new int [9];

for (int i=0 ; i<9 ; i++) {
    for (int j=0 ; j<9 ; j++) {
        x = x + sudoku[j][i] ;
    }
    if (x==45) zeile[i] = true;
    else zeile[i] = false;
}

if ((zeile[0] && zeile[1] && zeile[2] && zeile[3] && zeile[4] && zeile[5]
     && zeile[6] && zeile[7] && zeile[8]) == true) return true;
else return false; 

}

Ich weiß leider auch garnicht was der Fehler bedeutet


Vielen Dank für Hilfe!!

Grüße

 

[HimBromBeere]

Das liegt an deiner Bedingung in Zeile 14f.
[JAVA=14]
if ((zeile[0] && zeile[1] && zeile[2] && zeile[3] && zeile[4] && zeile[5]
&& zeile[6] && zeile[7] && zeile[8]) == true) return true;
[/code]
Du kannst mittels

&&

nur

boolean

verknüpfen, keine

int

. Dein Zeilen-Aray ist aber nicht vom Typ

boolean

, sondern vom Typ

int

, weswegen deine Zuweisungen in Zeile 10 und 11 auch nicht hinhauen.

 

[nillehammer]

Zwei Tipps zur Codeverschlankung:
- Ausdrücke, die direkt einen boolean-Wert ergeben braucht man nicht mit ==true zu prüfen, es reicht, sie direkt hinzuschreiben.
- Wenn von einem solchen Ausdruck der return-Wert einer Methode abhängt, die selbst wieder boolean zurück gibt, braucht man überhaupt kein if

Und der Fehler kommt durch Schreibfehler in dem Teil des Codes, den man hier nicht sieht. Mögliche Ursachen:
- Bei Klassendefinition das Schlüsselwort class vergessen oder falsch geschrieben
- Durch Fehlerhafte {}-Klammersetzung ist Code aus dem Körper der Klasse rausgerutscht
Also bitte mal komplette Klasse posten.

[EDIT]
Oder ist das am Ende Dein ganzer Code!?!. Dat geht natürlich nicht. Code gehört in Klassen rein.
[/EDIT]

 

[julian.veh]

"boolean [] zeile = new int [9];"

 

[nillehammer]

@Himbrombeere, julian.veh: Den Fehler hatte ich übersehen, aber aufgrund der späteren Benutzung von zeile (zeile = true; ), glaube ich, dass der TO tatsächlich ein boolean-Array initialisieren wollte.

 

[nillehammer]

Und auch hier hätte Google schneller weitergeholfen, als das Forum: Java Error Messages and Solutions: class, interface, or enum expected

 

[HimBromBeere]

[JAVA=4]boolean [] zeile = new int [9];[/code]
Das geht schonmal gleich gar nicht, entweder es ist ein boolean-Array oder ein int...

EDIT: Ooh, Veh, hab´s jetzt erst kapiert. Hättest aber auch ruhig noch zwei einhalb Worte dazu schreiben können

 

[Klassensprecher]

/*
 Ein Programm welches eine 9x9 Matrix erzeugt, diese mit beliebigen Zahlen befüllt und 
 anschließend auf Sudoku korrektheit prüft.
 @Author Aris Konstantinidis Matrikel 3028103
 @Tutor Christian Winger
*/

public class Sudoku {
  public static void main(String[] args) {
  
  int [][] sudoku = initMatrix() ;
  boolean zeile, spalte ;
  
  for (int i=0 ; i<9 ; i++) {
      for (int j=0 ; j<9 ; j++) {
      System.out.printf( "%3d" , sudoku[i][j]); }
      System.out.println(); }
      
   if ((zeile = pruefeZeilen(int[][] sudoku) &&  spalte = pruefeSpalten(int[][] sudoku)) == true) {
       System.out.println("Die Matrix ist eine korrekte Sudoku Lösung !"); }
   else {
       System.out.println("Die Matrix ist KEINE korrekte Sudoku Lösung !"); }
  
  }//end Main




static int [][] initMatrix() {

      int [][] feld = new int [9][9] ;
      
      for (int i=0 ; i < 9 ; i++) {
          for (int j=0 ; j < 9 ; j++) {
              feld [i][j] = (int) (10*Math.random());
          }
      }
      return feld ;
}

static boolean pruefeZeilen(int[][] sudoku) {

int x = 0;
boolean [] zeile = new boolean [9];

for (int i=0 ; i<9 ; i++) {
    for (int j=0 ; j<9 ; j++) {
        x = x + sudoku[j][i] ;
    }
    if (x==45) zeile[i] = true;
    else zeile[i] = false;
}

if ((zeile[0] && zeile[1] && zeile[2] && zeile[3] && zeile[4] && zeile[5]
     && zeile[6] && zeile[7] && zeile[8]) == true) return true;
else return false; 

}


static boolean pruefeSpalten(int[][] sudoku) {

int x = 0;
boolean [] spalte = new boolean [9];

for (int i=0 ; i<9 ; i++) {
    for (int j=0 ; j<9 ; j++) {
        x = x + sudoku[j][i] ;
    }
    if (x==45) spalte[i] = true;
    else spalte [i] = false;
}

if ((spalte[0] && spalte[1] && spalte[2] && spalte[3] && spalte[4] && spalte[5] 
     && spalte[6] && spalte[7] && spalte[8]) == true) return true;
else return false;

}
/*
static boolean pruefeBloecke(int[][] sudoku) {

    for(int z=0 ; z < 3 ; z++) {
        for (int s=0 ; s < 3 ; s++) {
             for (int i=0 ; i < 3 ; i++) {
                  for (int j=0 ; j < 3 ; j++) {
                  
                    
                  
          }//end 4. for
        }//end 3. for
      }// end 2. for 
    }// end 1. for

}

*/      


}//end class

Hier der ganze Code.

Das Ganze ist noch in Arbeit und die 2 Methoden werden in der main noch nicht aufgerufen.
Den Fehler beim Array habe ich korrigiert, und ich wollte natürlich ein boolean Array erzeugen.

Die Fehlermeldung kommt aber trotzdem ...

 

[HimBromBeere]

[JAVA=19]if ((zeile = pruefeZeilen(int[][] sudoku) && spalte = pruefeSpalten(int[][] sudoku)) == true) [/code]Was bitte ist das?!

Also erstmal: Einen Vergleich machst du entweder mit

==

oder mit

equals

(in deinem Fall ist

==

richtig).
Zweitens: Die eckigen Klammern hinter

int

müssen weg (EDIT: Das

int

natürlich auch, in der Klammer darf nur noch

sudoku

stehen).
Drittens: Das

 == true

am Schluss kann auch weg

 

[Klassensprecher]

Hi nochmal!

Danke für deine Hilfe!

if ((zeile = pruefeZeilen(int[][] sudoku) &&  spalte = pruefeSpalten(int[][] sudoku)) == true)

Damit möchte ich ausdrücken das nur wenn das Ergebnis beider Methoden (später dann noch der 3.) true ist, ausgeprintet werden soll das die Sudoku Eigenschaften gegeben sind...

 

[ARadauer]

Programmieren ist aber kein Deutsch Aufsatz ;-)


if ((pruefeZeilen(sudoku) && pruefeSpalten(sudoku))

 

[nillehammer]

Setz Himbrombeeres Tipps mal um, dann gehen die Fehler vielleicht weg. Bei der Deklaration der Methode schreibt man den Datentyp (

int[][]

natürlich hin). Beim Aufruf ist das ein Fehler.

 

[Klassensprecher]

Ich hab die Tipps umgesetzt und die Fehler sind alle weg!

Jetzt setzte ich mich an die 3. Methode und melde mich gegebenenfalls nochmal!


Viiieeelen Dank!

 

[Crian]

Als Generellen Tipp würde ich dir noch geben, das Programm vernünftig einzurücken. Mit einer IDE wie etwa Eclipse kann man genau festlegen, welchen Stil man haben will und Quellcode auf Knopfdruck in diese Form bringen. Aber auch ohne IDE sollte man sich um konsistente Einrückung bemühen.

 

[Kaa27]

Hallo, ich habe eine Anmerkung, bin mir aber nicht sicher,


du schreibst ja, das in einer Zeile die Summe x = 45 sein soll (weil 1+2+3...+9 = 45)
Aber bei random Zahlen kannst du ja auch folgendes bekommen (2+2+2 ... + 9 = 45)

Somit wäre dein Sudoku falsch.


EDIT: okay, wenn spalten,zeilen und blöcke kontrolliert werden, müsste es ja wieder richtig sein.
Sorry.