initialisieren eines arrays richtiger Größe und mit geeignetem Datentyp

[Cookiieee]

Hallo, ist die Zeile 6, so richtig?
Ich habe das Programm auch schon laufen lassen, aber da steht dass da "public static void main(String [] args)" fehlt, kommt, dass dann nach "public class Framed" oder wo?

Aufgabenstellung:
Unten ist ein fast vollständiges Programm gegeben. Das Programm soll zwei Vierecke bestehend aus den Buchstaben 'A' (Rand) und 'B' (Füllung) ausgeben.

In der ersten "Doppel-Schleife" wird bereits ein Viereck ausgegeben und dabei in ein 2D-Array gespeichert. In der zweiten "Doppel-Schleife" wird dann der Inhalt dieses Arrays ausgegeben.

Die Aufgabe besteht nun lediglich darin in Zeile 6 ein Array namens "array" richtiger Größe und von geeignetem Datentyp zu initialisieren, damit das Programm fehlerfrei und wie beschrieben abläuft.

public class Framed {
    public static int breite = 6;
    public static int hoehe = 3;
   
    public static void frame() {
        //Hier soll das Array deklariert und initialisiert werden!
        for(int i = 0; i <= (hoehe+1); i++) {
            for(int j = 0; j <= (breite+1); j++) {
                if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {
                    System.out.print("A");
                    array[j][i] = 'A';
                }
                else {
                    System.out.print("B");
                    array[j][i] = 'B';
                }
            }
            System.out.print("\n");
        }

        System.out.print("\n---Und hier jetzt nochmal das Viereck aus dem Array---\n\n");

        for(int i = 0; i <= (hoehe+1); i++){
            for(int j = 0; j <= (breite+1); j++){
                System.out.print(array[j][i]);
            }
            System.out.print("\n");
        }
    }
}

Bearbeitetes Programm:

public class Viereck {
            public static int breite = 6;
            public static int hoehe = 3;
           
            public static void frame() {
                char[][] array = new char[breite][hoehe];
                for(int i = 0; i <= (hoehe+1); i++) {
                    for(int j = 0; j <= (breite+1); j++) {
                        if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {
                            System.out.print("A");
                            array[j][i] = 'A';
                        }
                        else {
                            System.out.print("B");
                            array[j][i] = 'B';
                        }
                    }
                    System.out.print("\n");
                }

                System.out.print("\n---Und hier jetzt nochmal das Viereck aus dem Array---\n\n");

                for(int i = 0; i <= (hoehe+1); i++){
                    for(int j = 0; j <= (breite+1); j++){
                        System.out.print(array[j][i]);
                    }
                    System.out.print("\n");
                }
            }
        }

 

[mihe7]

aber da steht dass da "public static void main(String [] args)" fehlt, kommt, dass dann nach "public class Framed" oder wo?

Ja, zwischen die geschweiften Klammern. Die main-Methode ist die Methode, die beim Start des Programms ausgeführt wird.

 

[kneitzel]

Und du solltest einmal anschauen, auf welche Elemente in den Schleifen zugegriffen wird. Wie groß muss das Array dann sein?

 

[Lion.King]

Es kommen folgende Fehlermeldungen, wenn man das Programm mit der main-Methode laufen lässt:
/Viereck.java:3: error: illegal start of expression
public static int breite = 6;
^
/Viereck.java:33: error: class, interface, or enum expected
}
^
2 errors

Wie kann man sie beheben?

 

[kneitzel]

Zeig uns die ganze Datei - vermutlich ist der Code in keiner Klasse oder so ...

Und zum Posten von Code bitte den Knopf </> oberhalb des Eingabebereichs nutzen.

 

[Cookiieee]

Und du solltest einmal anschauen, auf welche Elemente in den Schleifen zugegriffen wird. Wie groß muss das Array dann sein?

Also bei mir kommt für diesen Code:

public class Viereck {
    
    public static void main(String[] args) {
            public static int breite = 6;
            public static int hoehe = 3;
            
            public static void frame() {
                char[][] array = new char[breite][hoehe];
                for(int i = 0; i <= (hoehe+1); i++) {
                    for(int j = 0; j <= (breite+1); j++) {
                        if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {
                            System.out.print("A");
                            array[j][i] = 'A';
                        }
                        else {
                            System.out.print("B");
                            array[j][i] = 'B';
                        }
                    }
                    System.out.print("\n");
                }

                System.out.print("\n---Und hier jetzt nochmal das Viereck aus dem Array---\n\n");

                for(int i = 0; i <= (hoehe+1); i++){
                    for(int j = 0; j <= (breite+1); j++){
                        System.out.print(array[j][i]);
                    }
                    System.out.print("\n");
                }
            }
        }

Diese Fehlermeldung:
javac /tmp/ZdYi4mK5Co/Viereck.java
/tmp/ZdYi4mK5Co/Viereck.java:4: error: illegal start of expression
public static int breite = 6;
^
1 error

 

[thecain]

Zähl doch mal die öffnenden und schliessenden Klammern...

Und methoden kann man nicht in der main anlegen...

 

[Cookiieee]

Vielleicht eine etwas blöde Frage, aber muss ich jetzt das " public static void main(String[] args) {" wegmachen oder wie ist das gemeint...?
Also da du ja meintest dass man Methoden nicht in der main anlegen kann...

Zähl doch mal die öffnenden und schliessenden Klammern...

Und methoden kann man nicht in der main anlegen...

 

[thecain]

Nein, du sollst deine andere Methode nicht innerhalb der main Methode schreiben.

 

[mihe7]

@Cookiieee die Sache ist ganz einfach:

public class Viereck {
    // Klassenattribute
    public static int breite = 6;
    public static int hoehe = 3;
            
    // main-Methode (Programmeinstiegspunkt)
    // wird beim Programmstart automatisch aufgerufen
    public static void main(String[] args) {
        frame(); // rufe frame auf
    }

    public static void frame() {
        // Dein Code.
    }
}

 

[Cookiieee]

Ahh okay, jetzt verstehe ich Danke dir!! Habe das gerade auch so ergänzt und laufen lassen und da ist immer noch eine Fehlermeldung
und zwar das, weiß ehrlich gesagt nicht, was das aussagt:🙃

AAAAAAAException in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 6 out of bounds for length 6
at Viereck.frame(Viereck.java:16)
at Viereck.main(Viereck.java:7)

Das ist der Code:

public class Viereck {
    
            public static int breite = 6;
            public static int hoehe = 3;
            
            public static void main(String[] args) {
                frame();
            }
            
            public static void frame() {
                char[][] array = new char[breite][hoehe];
                for(int i = 0; i <= (hoehe+1); i++) {
                    for(int j = 0; j <= (breite+1); j++) {
                        if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {
                            System.out.print("A");
                            array[j][i] = 'A';
                        }
                        else {
                            System.out.print("B");
                            array[j][i] = 'B';
                        }
                    }
                    System.out.print("\n");
                }

                System.out.print("\n---Und hier jetzt nochmal das Viereck aus dem Array---\n\n");

                for(int i = 0; i <= (hoehe+1); i++){
                    for(int j = 0; j <= (breite+1); j++){
                        System.out.print(array[j][i]);
                    }
                    System.out.print("\n");
                }
            }
        }

 

[mihe7]

und zwar das, weiß ehrlich gesagt nicht, was das aussagt:🙃

Die Fehlermeldung sagt, dass Du auf Index 6 eines Arrays zugegriffen hast, das nur 6 Elemente enthält. Hintergrund ist, dass der Index (nicht nur) in Java ab 0 gezählt wird. Ein Array der Größe 6 hat also Indizes von 0 bis 5. Versuchst Du auf -1 oder auf 6, 10 oder 100 zuzugreifen, bekommst Du eine ArrayIndexOutOfBoundsException (übersetzt etwa ArrayIndexAußerhalbDerGrenzenAusnahme).

 

[Cookiieee]

Sorry für die vielen Fragen..
Die Fehlermeldung ist ja bei 7 und 16, was muss ich denn hinzufügen oder wegmachen um diesen Error wegzubekommen?

 

[mihe7]

Die Fehlermeldung ist ja bei 7 und 16,

Ne, das ist ein Stack-Trace, der den Aufrufstapel anzeigt:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 6 out of bounds for length 6
at Viereck.frame(Viereck.java:16)
at Viereck.main(Viereck.java:7)

D. h. in Zeile 16 trat die Ausnahme in Methode frame() auf und diese Methode wurde von main aus in Zeile 7 aufgerufen.

Das Problem liegt also nur in Zeile 16:

array[j][i] = 'A';

Hier ist j oder i eben ein ungültiger Index. Schau Dir mal die Größe Deines Arrays und Deine Bedingungen in den for-Schleifen und if-Statements an.

 

[kneitzel]

Und du solltest einmal anschauen, auf welche Elemente in den Schleifen zugegriffen wird. Wie groß muss das Array dann sein?

Option 1:
Es wird in den Schleifen auf die Array Elemente i / j zugegriffen.

Daher schau einmal, bis wohin diese Schleifen zählen. Dann hast du den höchsten Index und kannst die Größe bestimmen.

Option 2:
Was genau macht der Code? Hast Du Dir da einmal überlegt? Wenn man sich die Problemdomain überlegt, dann kommt man auch auf so Dinge wie die richtige Größe.

Option 3:
Wenn man absolut keine Ahnung hat, dann könnte man einfach das Array vergrößern.
Die 6 kommt von der Breite, also erzeugt man erst einmal ein Array mit Breite 7 ...
Das ist aber natürlich die schlechteste aller Optionen. Null Verständnis, was der Code machen soll ...: null Verständnis, was der Code macht ... Korrektheit ist nur dadurch definiert, dass es keine Exception gibt.
Aber bei der Aufgabe ist auch der Ansatz möglich.

 

[Barista]

char[][] array = new char[breite][hoehe];

Vielleicht muss das Array ein bisschen größer sein, weil mit breite und hoehe ohne Rand gemeint ist, mal so als Hinweis.

 

[Cookiieee]

Vielleicht muss das Array ein bisschen größer sein, weil mit breite und hoehe ohne Rand gemeint ist, mal so als Hinweis.

Was meinst du mit größer?😅Also an sich musste ich ja in der Zeile 6 nur etwas schreiben, dann wäre das Programm fertig, verstehe nur nicht was der Fehler ist...

 

[mihe7]

verstehe nur nicht was der Fehler ist...

Nehmen wir mal Dein i:

for(int i = 0; i <= (hoehe+1); i++) {

Das läuft bis inkl. hoehe+1, Dein Array hat aber eine Größe hoehe. Wie soll das funktionieren?

 

[Cookiieee]

Nehmen wir mal Dein i:

Das läuft bis inkl. hoehe+1, Dein Array hat aber eine Größe hoehe. Wie soll das funktionieren?

Ahhhh okay jetzt verstehe ich, Dankeschööön!!!

jetzt kommt das raus...

AAAAAAA
ABBBBBB
ABBBBBB
ABBBBBB

---Und hier jetzt nochmal das Viereck aus dem Array---

AAAAAAAException in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 7 out of bounds for length 7
at Viereck.frame(Viereck.java:27)
at Viereck.main(Viereck.java:6)

Habe es auch schon korrigiert, müsste das dann nicht schon so gehen oder was mache ich wieder falsch hahah?🙃

public class Viereck {
   
            public static int breite = 6;
            public static int hoehe = 3;
            public static void main(String[] args) {
                frame();}
            public static void frame() {
                char[][] array = new char[breite+1][hoehe+1];
                for(int i = 0; i < (hoehe+1); i++) {
                    for(int j = 0; j < (breite+1); j++) {
                        if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {
                            System.out.print("A");
                            array[j][i] = 'A';
                        }
                        else {
                            System.out.print("B");
                            array[j][i] = 'B';
                        }
                    }
                    System.out.print("\n");
                }

                System.out.print("\n---Und hier jetzt nochmal das Viereck aus dem Array---\n\n");

                for(int i = 0; i <= (hoehe+1); i++){
                    for(int j = 0; j <= (breite+1); j++){
                        System.out.print(array[j][i]);
                    }
                    System.out.print("\n");
                }
            }
        }

 

[mihe7]

In 25 und 26 hast Du noch ein <= statt eines <.

 

[kneitzel]

Also ich weiss nicht, wieso ich überhaupt etwas schreibe ... 3 Optionen habe ich genannt und die interessieren Dich überhaupt nicht ...

a) Vielleicht einfach noch eins größer machen?

b) Wenn du ein Fehld mit Breite x Höhe hast und da links/rechts und oben/unten jeweils ein Feld mehr brauchst: Dann ist es jeweils +2!

c) Wenn die schleife bis einschließlich breite+1 / hoehe+1 läuft und ein Array Indizes von 0 ... size-1 hat, dann brauchst Du für ein index breite+1 ein Array der größe breite+2. Ebenso bei der hoehe.

 

[kneitzel]

In 25 und 26 hast Du noch ein <= statt eines <.

Das soll ja auch so bleiben. Es war ein Fehler von ihm, an dem Code an der Stelle etwas zu ändern. Erkennt man ja auch am Inhalt in der ersten verschachtelten Schleife:
if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {

 

[mihe7]

Das soll ja auch so bleiben.

Ah, ich hätte mir vielleicht mal die Aufgabe durchlesen sollen 🙂

 

[kneitzel]

Ah, ich hätte mir vielleicht mal die Aufgabe durchlesen sollen 🙂

Wieso? Er liest ja auch nicht, was ich schreibe 😛 Das ist dann also mehr als gerechtfertigt 🙂

 

[Cookiieee]

Das soll ja auch so bleiben. Es war ein Fehler von ihm, an dem Code an der Stelle etwas zu ändern. Erkennt man ja auch am Inhalt in der ersten verschachtelten Schleife:
if((i == 0 || j == 0) || ( i == (hoehe+1) || j == (breite+1))) {

Wieso mein Fehler...? 😅
Der Code wurde mir so zugeteilt und ich musste, wie oben schon beschrieben nur die 6.Zeile erstellen und somit wäre das Programm fertig..

 

[kneitzel]

in #1 war es noch:

    public static void frame() {
        //Hier soll das Array deklariert und initialisiert werden!
        for(int i = 0; i <= (hoehe+1); i++) {
            for(int j = 0; j <= (breite+1); j++) {

in #19 war es dann:

            public static void frame() {
                char[][] array = new char[breite+1][hoehe+1];
                for(int i = 0; i < (hoehe+1); i++) {
                    for(int j = 0; j < (breite+1); j++) {

Du hast also nicht nur den Kommentar ersetzt sondern Du hast auch aus den beiden <= ein < gemacht. Und genau das sollst Du eben nicht machen, sondern lediglich (wie Du richtig geschrieben hast):

wie oben schon beschrieben nur die 6.Zeile erstellen

 

[Cookiieee]

Ahsoo okay, Dankeschöön habe es selber nicht bemerkt 😄