warum wird es doppelt ausgegeben

Nico1999

Aktives Mitglied
Java:
public class Figur{
    private int x;
    private int y;
    private int graphik;
    private int movezähler;

public boolean legalmove(int a[][], int jetztx, int jetzty, int dannx, int danny) {//5 =Turmweiß   Schwarz = -Zahlen
        int k=a[jetztx][jetzty];
        if(k==5&&this.x==jetztx&&this.y==jetzty&&a[dannx][danny]<=0&&(jetztx-dannx==0^jetzty-danny==0)&&a[jetztx][jetzty]==5) {//prüfe ob Turmmoven kann
            if(jetztx-dannx<0) {//nach vorne
                for(int i =1; i<dannx-jetztx+1;i++) {//dannx =5 jetztx=3
                    if(a[jetztx+i][jetzty]==0) {//prüfe ob feld frei ist
                        
                    }else {
                        if(a[jetztx+i][jetzty]<0&&dannx-jetztx-i==0){//wenn Feld nicht frei ist prüfe ob es vom Gegner ist und der Zielmove war
                            this.x=dannx;
                            System.out.println("this x="+this.x);
                            return true;
                        }else {
                            System.out.println("Figur kann andere Figuren nicht überspringen 1");
                            return false;
                        }
                    }
                }return true;
            }
        }
    }
}
Java:
public class Spiel{
    public static void main(String[] args) {
        for(Figur a :figurliste) {
                 if(a.legalmove(schachbrett1.getSchachbrett(), figurx, figury, eingabex, eingabey)) {
                     schachbrett1.figurbewegen(brett, figurx, figury, eingabex, eingabey);
                 }else{
                    
                 }
        }
    }
}
wie kann es sein das wenn legalmove = false weil wir in die letzte else schleife mit der Fehlermeldung kommen, dass diese zweimal ausgeben wird. Sollte nach return false; nicht sofort schluss sein (jede Figur sollte eigentlich individuell sein) . wie kann diese Meldung dann zweimal ausgeben werden ?
Danke schonma für eure antworten :)
 

Staarfightaar

Bekanntes Mitglied
Java:
Figur a :figurliste)
ich nehm mal an dass da 2 elemente drin sind und du einfach 2 mal a.legalmove aufrufst

ansonsten zum Code KONVENTIONEN
1. camelCase => legalMove... bei getSchachbrett hast du ja camelcase auch angewandt
2. überdenken von code
Java:
k==5&&this.x==jetztx&&this.y==jetzty&&a[dannx][danny]<=0&&(jetztx-dannx==0^jetzty-danny==0)&&a[jetztx][jetzty]==5
das kann vllt auf eine blöde struktur des programmes hindeuten
3. KEINE magic numbers... magic numbers sind hardgecodete zahlen bzw strings ... hierbei zb a[jetztx][jetzty]==5 <= der 5er ist ne magic number
.. mach ne
private final int FIVE = 5;
oder sonst was draus... auf jedenfall ein feld draus machen
4. Methoden aufteilen ... dein schinken an if else ist riesig mit noch ner for schleife drin...schau mal ne woche drauf dann weist du sicher nicht mehr was das ding macht
5. klassen raus ziehen für moves zb oder Figuren oder feld verhalten
 

Nico1999

Aktives Mitglied
Java:
Figur a :figurliste)
ich nehm mal an dass da 2 elemente drin sind und du einfach 2 mal a.legalmove aufrufst
da sind sogar 32 Elemente in den Figuren array allerdings hat jede Figur einen x und y wert der bei jeder Figur verschieden ist.
1. camelCase => legalMove... bei getSchachbrett hast du ja camelcase auch angewandt
Meinst du damit das man da eine Methode aufruft ?
2. überdenken von code
Java:
k==5&&this.x==jetztx&&this.y==jetzty&&a[dannx][danny]<=0&&(jetztx-dannx==0^jetzty-danny==0)&&a[jetztx][jetzty]==5
das kann vllt auf eine blöde struktur des programmes hindeuten
ich überprüfe halt zuerst ob die Klassevariablen gleich ist der eingabe sind, dann überprüfe ich ob auf dem zielfeld nicht eine eigene Figur steht, dann überprüfe ich ob es theoretisch ein legitimer Move für den Turm (entweder die x oder die y koordinate bewegen sich nicht beide zusammen)
und das letze ist vllt unnötig da eigentlich die Klassenvariablen reichen sollten, da überprüfe ich nochma sicherheitshalber ob das feld auch der Turm also 5 ist.
wenn das eine blöde struktur ist wie soll ich es anders aufbauen ? mit extra methoden die das dann überprüfen?
3. KEINE magic numbers... magic numbers sind hardgecodete zahlen bzw strings ... hierbei zb a[jetztx][jetzty]==5 <= der 5er ist ne magic number
.. mach ne
oder sonst was draus... auf jedenfall ein feld draus machen
könnte auch schreiben a[jetztx][jetzty]==this.graphik
was ist an den magic numbers schlimm ?
würdest du jetzt anstatt dessen das feld a [][] mit FIVE usw ausstatten?
5. klassen raus ziehen für moves zb oder Figuren oder feld verhalten
ich könnte die methode aufspalten in noch eine die überprüft ob frei ist würde allerdings immer noch ne lange methode werden
 

fhoffmann

Top Contributor
Und wenn du in drei Jahren den Code nochmal liest, weißt du immer noch, dass der Turm die Nummer 5 hat?
Da ist doch eine Konstante wie private final int TURM = 5; besser lesbar.
 

Jw456

Top Contributor
Java:
        for(Figur a :figurliste) {
                 if(a.legalmove(schachbrett1.getSchachbrett(), figurx, figury, eingabex, eingabey)) {
                     schachbrett1.figurbewegen(brett, figurx, figury, eingabex, eingabey);
                 }else{

du geht jede Figur in der for Schleife durch und prüft jede Figur mit „legalmove“ ist der Boolean true führst du „schachbrett1.figurbewegen“ aus.
Es könnte auch sein das du beim bewegen auch die „legalmove“ ausführst. Können wir leider nicht sehen.
Oder du führst es irgendwo anders in deinem Code noch mal aus.

Um zuprüfen ob wirklich von der selben Figur kommt lasse dir die Figur und x y Werte mit ausgeben.
 

Nico1999

Aktives Mitglied
Java:
        for(Figur a :figurliste) {
                 if(a.legalmove(schachbrett1.getSchachbrett(), figurx, figury, eingabex, eingabey)) {
                     schachbrett1.figurbewegen(brett, figurx, figury, eingabex, eingabey);
                 }else{

du geht jede Figur in der for Schleife durch und prüft jede Figur mit „legalmove“ ist der Boolean true führst du „schachbrett1.figurbewegen“ aus.
Es könnte auch sein das du beim bewegen auch die „legalmove“ ausführst. Können wir leider nicht sehen.
Oder du führst es irgendwo anders in deinem Code noch mal aus.

Um zuprüfen ob wirklich von der selben Figur kommt lasse dir die Figur und x y Werte mit ausgeben.
okay habs gefunden muss noch this.graphik==TURMW reinschreiben dann klappts danke für eure Hilfe
 
Zuletzt bearbeitet:
Ähnliche Java Themen
  Titel Forum Antworten Datum
S If-Menü wird doppelt ausgegben Allgemeine Java-Themen 4
Sin137 Interface Eingabe wird doppelt angezeigt Allgemeine Java-Themen 2
S Thread wird nach erneutem Instanzieren doppelt gestartet!? Allgemeine Java-Themen 4
N Lwjgl 3d Objekt wird schmaler, wenn es sich dreht Allgemeine Java-Themen 0
D Linux, Java-Version wird nicht erkannt bzw. welche Einstellung fehlt noch? Allgemeine Java-Themen 19
berserkerdq2 Wenn ich einfach eine GIF in den Scenebuilder als Bild reinpacke, wird das dann asl Gif angezeigt Allgemeine Java-Themen 1
G Popup wird nicht sichtbar Allgemeine Java-Themen 9
8u3631984 Funktions Parameter mit Lombok "NonNull" annotieren wird in Jacococ Testcoverage nicht herausgefiltert Allgemeine Java-Themen 3
M Java Überprüfen ob .exe-Datei bereits ausgeführt wird Allgemeine Java-Themen 2
Calli11 Was muss ich hier in die Main schreiben, damit das Programm ausgeführt wird? Allgemeine Java-Themen 4
Sachinbhatt Wie wird die Typumwandlung bei Mehrfachvererbung in Java implementiert? Allgemeine Java-Themen 3
Y MVVM wie wird Eventhandler tausch geregelt Allgemeine Java-Themen 4
berserkerdq2 run-methode eines Threads so programmieren, dass 30x die Sekunde etwas ausgeführt wird. Allgemeine Java-Themen 44
T ImageIcon wird nicht angezeigt Allgemeine Java-Themen 6
N Warum wird die For Schleife nicht betreten Allgemeine Java-Themen 4
A code wird nicht ausgeführt Allgemeine Java-Themen 3
Y Warum wird das JLabel falsch verschoben? Allgemeine Java-Themen 1
T Schaltfläche wird nicht gefunden Allgemeine Java-Themen 4
N Eingabe wird immer als "false" ausgegeben Allgemeine Java-Themen 6
D Firebase retrieve data Problem, Child Element wird nicht angesprochen Allgemeine Java-Themen 0
M Frage-Antwortspiel wie Wer wird Millionär Allgemeine Java-Themen 1
Zrebna Gibt es eine Möglichkeit eine NPE zu vermeiden, wenn null returned wird? Allgemeine Java-Themen 3
R Warum wird mir in der Konsole das "Standard Array" ausgegeben? Allgemeine Java-Themen 2
C FileLock - Exception wird immer geworfen Allgemeine Java-Themen 4
R Was muss ich ändern, damit der Kreis links unten gezeichnet wird? Allgemeine Java-Themen 17
hello_autumn Java_Home geändert auf Java 13, trotzdem wird Java Version 8 angezeigt. Allgemeine Java-Themen 2
Thallius Char +1 wird int? Allgemeine Java-Themen 7
S Wenn eine Klasse zwei Interfaces mit derselben Methodensignatur implementiert: welche wird aufgerufen? Allgemeine Java-Themen 15
F Scrollbar wird nicht angezeigt Allgemeine Java-Themen 0
B Discord Bot - Funktion wird nicht aufgerufen Allgemeine Java-Themen 1
I Temp-Datei wird nicht gelöscht Allgemeine Java-Themen 12
P Variable wird in for - loop nicht richtig hochgezählt Allgemeine Java-Themen 11
S Wird bei den JREs 9, 10, 11+ ueberhaupt noch Bytecode ausgefuehrt..? Allgemeine Java-Themen 8
FRI3ND JFrame wird nicht angezeigt Allgemeine Java-Themen 3
T Plötzlich wird package nicht mehr gefunden Allgemeine Java-Themen 3
X Klassen File-Klasse wird als Directory markiert Allgemeine Java-Themen 8
S Datei wird nicht gefunden Thread.currentThread().getContextClassLoader().getResourceAsStream() Allgemeine Java-Themen 1
mrbig2017 Sleep wird ignoriert und der Thread wartet nicht Allgemeine Java-Themen 1
mrbig2017 Threads wait wird nicht durch notify beendet! Allgemeine Java-Themen 3
kodela Klassen Klassenvariable wird nicht gesetzt Allgemeine Java-Themen 23
M Was geschieht mit Java-Klasse, die aus ArrayList entfernt wird? Allgemeine Java-Themen 10
kodela Drücken der ALT-Taste wird nur bei jedem zweiten Mal erkannt Allgemeine Java-Themen 5
Arif Input/Output Serialisierung - Datei wird nicht erzeugt Allgemeine Java-Themen 3
D Cursor unsichtbar, wenn Kontextmenü aufgerufen wird Allgemeine Java-Themen 5
K Arbeitsspeicher wird langsam voll Allgemeine Java-Themen 6
The Pi Wie oft wird ein Buchstabe in einem Wort wiederholt? Allgemeine Java-Themen 16
N Datentypen If-Statement wird "übersprungen" Allgemeine Java-Themen 2
F Java Mail Problem: Authentifizierung wird nicht immer mitgeschickt Allgemeine Java-Themen 1
T Textarea text wird immer überschrieben Allgemeine Java-Themen 4
C Durch klicken von Button in GUI wird leeres Fenster geöffnet und nicht mein Spiel "Memory" Allgemeine Java-Themen 13
T Java wird nicht gefunden Allgemeine Java-Themen 32
Tacofan GIF-Bild wird "zerstört" Allgemeine Java-Themen 3
Z Exception wird nicht ausgelöst Allgemeine Java-Themen 2
B Threads Timer wird immer schneller Allgemeine Java-Themen 6
N Maven ObjectMapper Error wenn das File gespeichert wird Allgemeine Java-Themen 0
G Methoden Aus einem Event, wo ich weiß, dass es ausgeführt werden wird, eine Get-Methode basteln Allgemeine Java-Themen 8
F Wert in Variable schreiben, wenn diese gerade genutzt wird Allgemeine Java-Themen 10
E Java wird beendet nach paar Sekunden Allgemeine Java-Themen 14
U Erkennen, ob PC benutzt wird Allgemeine Java-Themen 4
O Programm wird einfach "gekillt" Allgemeine Java-Themen 3
P Prüfen ob es Variable mit Namen gibt der als String übergeben wird Allgemeine Java-Themen 7
P Zwei ArrayLists: Ohne die eine überhaupt anzurühren, wird sie verändert Allgemeine Java-Themen 2
B Abfragenteil wird nicht aufgerufen Allgemeine Java-Themen 2
Todesbote JFileChooser im Vordergrund (*.jar wird mittels shell_exec in PHP aufgerufen) Allgemeine Java-Themen 1
T Sicherheitshinweis - Zertifikat wird angefordert Allgemeine Java-Themen 5
L SetBounds wird vor dem Konstruktorcode aufgerufen - Wenn Startleiste links Allgemeine Java-Themen 1
P "Overriden statische Methode" Statische Methode die vererbt wird Allgemeine Java-Themen 5
I CountDown wird durch JOptionPane unterbrochen Allgemeine Java-Themen 11
P Wird double und float auf jedem Computer gleich berechnet? Allgemeine Java-Themen 10
G Tabelle wird nicht richtig dargestellt Allgemeine Java-Themen 9
HarleyDavidson Threads ProgressMonitor wird nicht angezeigt! Allgemeine Java-Themen 6
J OOP Überwachen, ob ein Objekt erzeugt wird Allgemeine Java-Themen 9
S Byte Array welches in Laufzeit aufgelöst wird // Objekt Array Allgemeine Java-Themen 3
A Long variable wird negativ??? Allgemeine Java-Themen 1
J CSV Datei wird ungewollt umformatiert Allgemeine Java-Themen 5
T MEthodenauruf testen, wenn instanz erst erzeugt wird Allgemeine Java-Themen 0
gamebreiti Swing JLabel wird nicht angezeigt Allgemeine Java-Themen 4
I Java Applet wird immer blockiert Allgemeine Java-Themen 3
E Java Editor Klasse wird nicht weiter ausgeführt Allgemeine Java-Themen 4
S Java Applet wird dauerhaft blockiert Allgemeine Java-Themen 1
I Datei wird nicht richtig gelöscht Allgemeine Java-Themen 7
K Methoden Objekt wird nicht erkannt Allgemeine Java-Themen 11
D Input/Output Datei wird lokal aber nicht vom Webserver ausgelesen... Allgemeine Java-Themen 2
F Nach Export wird PDF Datei nicht mehr gefunden Allgemeine Java-Themen 0
L Prüfen, ob Programm über 32bit oder 64bit Java ausgeführt wird Allgemeine Java-Themen 4
D ActionListener wird mit jedem Mal öfter aufgerufen Allgemeine Java-Themen 2
C System.in erhält Input von Tastatur. wo wird das festgelegt? Allgemeine Java-Themen 4
I Projekt wird nicht erstellt Allgemeine Java-Themen 2
T WeakReference/PhantomReference: Mitbekommen WELCHES Objekt nun GC'ed wird Allgemeine Java-Themen 2
A Threads Log4J Logger wird "überschrieben" Allgemeine Java-Themen 3
P Check, welche Java Version installiert bevor ein Programm ausgeführt wird. Allgemeine Java-Themen 12
J batch datei wird nicht aufgerufen Allgemeine Java-Themen 11
M Datei wird nicht vollständig geschrieben (FileOutputStream, OutputStreamWriter, Encoding) Allgemeine Java-Themen 6
P OSGi - Bundle-Methode wird nicht aufgerufen Allgemeine Java-Themen 3
S Antlr Grammatik übersetzt ohne Fehler, dennoch wird Zahl nicht als Eingabe erkannt Allgemeine Java-Themen 4
T Variablen Variable wird als nicht genutzt angezeigt Allgemeine Java-Themen 8
J Slideshow wird im IE überblendet Allgemeine Java-Themen 4
O Socket Object wird scheinbar falsch empfangen Allgemeine Java-Themen 6
J Textdatei auslesen, die gerade benutzt wird Allgemeine Java-Themen 8
M CapsLock abfragen wenn Fenster wieder aktiviert wird Allgemeine Java-Themen 5

Ähnliche Java Themen

Neue Themen


Oben