Normal
1) Frame.setVisible() gehört NIE in den eigenen Konstruktor sondern IMMER in den Caller !2) Von Frame (bzw Window oder den ganzen Swing-Klassen) zu erben macht keinen Sinn. Denn du willst ja keinen neuen unter-Typ von Frame erstellen sondern lediglich ein normales Frame. Ergo : das extends weg und dafür eine Instanz-Variable vom Typ Frame.3) Nutze lieber Swing. Ist deutlich einfacher zu handhaben als AWT.4) if(x=true) ist weder ein Vergleich (sondern eine Zuweisung) noch sinnvoll. Erstmal müsste es wenn überhaupt if(x==true) heißen, was aber wenig Sinn hat wenn du prüfst ob es wahr ist das x wahr ist. Also prüfst du gleich ob x wahr ist : if(x).Alles in allem ziemlich viele Anfänger-Fehler. Ich würde dir mal einen Blick hirein empfehlen : Galileo Computing :: Java ist auch eine Insel - index
1) Frame.setVisible() gehört NIE in den eigenen Konstruktor sondern IMMER in den Caller !
2) Von Frame (bzw Window oder den ganzen Swing-Klassen) zu erben macht keinen Sinn. Denn du willst ja keinen neuen unter-Typ von Frame erstellen sondern lediglich ein normales Frame. Ergo : das extends weg und dafür eine Instanz-Variable vom Typ Frame.
3) Nutze lieber Swing. Ist deutlich einfacher zu handhaben als AWT.
4) if(x=true) ist weder ein Vergleich (sondern eine Zuweisung) noch sinnvoll. Erstmal müsste es wenn überhaupt if(x==true) heißen, was aber wenig Sinn hat wenn du prüfst ob es wahr ist das x wahr ist. Also prüfst du gleich ob x wahr ist : if(x).
Alles in allem ziemlich viele Anfänger-Fehler. Ich würde dir mal einen Blick hirein empfehlen : Galileo Computing :: Java ist auch eine Insel - index
