Select nimmt nicht alle Where /AND befehlen an

[Emrullah]

Hallo alle zusammen hab es schon öfter in meinem Code mit der Where und AND Funktion aber leider in diesem fall nimmt es mir entweder das Jahr oder die Ausgabe nicht an (welcher als 2 steht wird ignoriert) ich weiß nicht woran es liegen kann.

hab es schon öfter gelöscht und neu geschrieben bzw. von einer anderen stelle kopiert wo es funktioniert.

die Ausgabe -1 muss sein da in der Datenbank fängt die zahl von 0 an aber im Panel fängt die zahl von 1 an.

 public void aboziele(int Ausgabe,int Jahr,int DatenbankID,int Heftname){
          Ausgabe -=1;
            Connection con;
                       con= mySQL.Aboziel_Erstelle_verbindung();
        
        
            System.out.println("Panels.DatenTabelle.aboziele(Ausgabe)      "+Ausgabe+"  Jahr  "+Jahr+"   DatenbankID  "+DatenbankID+"  HeftID  "+Heftname);
                      
                        try {
                            
                    
            Statement stmt =con.createStatement();
            ResultSet rs =stmt.executeQuery
                    ("SELECT ABOZiel FROM abos WHERE " +
                                                                            "Ausgabe ='" +Ausgabe+
                                                                            "AND Jahr=" +Jahr+
                                                                            "AND FirmenID="+DatenbankID+                                    
                                                                            "AND HeftID=" +Heftname+"'");
                                
            String[] temp = new String[1];
                                 //Aboziel
                        if(rs.next()) {
                temp[0] = ""+ rs.getInt(1);
                              
                              
                              System.out.println("Panels.DatenTabelle.aboziele(0)"+rs.getInt(1));
                             if(temp[0]==null){
                                 abozielzahl=0;
                             }else{
                                
                                    abozielzahl=rs.getInt(1);
                             }
                            

                                   System.out.println("Panels.DatenTabelle.aboziele(Aboziele)         "+abozielzahl);
                                    
                        }
                
                            
            mySQL.schliesse_verbindung();
            
            
        }catch(Exception e) {
            System.out.println("firmen : Konnte keine Verbindung hergestellt werden");
                e.printStackTrace();
                
        }
               }

im output gibt er mir das aus

Panels.DatenTabelle.aboziele(Ausgabe)      0  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)1500
Panels.DatenTabelle.aboziele(Aboziele)         1500

    
Panels.DatenTabelle.aboziele(Ausgabe)      1  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)2000
Panels.DatenTabelle.aboziele(Aboziele)         2000

    
Panels.DatenTabelle.aboziele(Ausgabe)      2  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)1000
Panels.DatenTabelle.aboziele(Aboziele)         1000

    
Panels.DatenTabelle.aboziele(Ausgabe)      3  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

    
Panels.DatenTabelle.aboziele(Ausgabe)      4  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

    
Panels.DatenTabelle.aboziele(Ausgabe)      5  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

    
Panels.DatenTabelle.aboziele(Ausgabe)      6  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

    
Panels.DatenTabelle.aboziele(Ausgabe)      7  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

    
Panels.DatenTabelle.aboziele(Ausgabe)      8  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

    
Panels.DatenTabelle.aboziele(Ausgabe)      9  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0


Panels.DatenTabelle.aboziele(Ausgabe)      10  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)2000
Panels.DatenTabelle.aboziele(Aboziele)         2000

    
Panels.DatenTabelle.aboziele(Ausgabe)      11  Jahr  1990   DatenbankID  1  HeftID  4
Panels.DatenTabelle.aboziele(0)0
Panels.DatenTabelle.aboziele(Aboziele)         0

wenn ich Ausgabe an erster stelle habe nimmt er das Jahr 2020
wenn ich das Jahr an erster stelle habe nimmt er nur die erste Ausgabe(0)

 

[Mart]

du hast sehr wahrscheinlich einen Hochkommata Fehler irgendwo aber wie im anderen post beschrieben müsstest du es grundsätzlich mit prepared statements machne dann gehts auch wesentlich einfahcer und du kannst sogar gewährleisten dass der befehl richtig übersetzt wird

 

[kneitzel]

du hast sehr wahrscheinlich einen Hochkommata Fehler irgendwo aber wie im anderen post beschrieben müsstest du es grundsätzlich mit prepared statements machne dann gehts auch wesentlich einfahcer und du kannst sogar gewährleisten dass der befehl richtig übersetzt wird

Der Hochkomma-Fehler ist auch sofort offensichtlich:

                    ("SELECT ABOZiel FROM abos WHERE " +
                                                                            "Ausgabe ='" +Ausgabe+
                                                                            "AND Jahr=" +Jahr+
                                                                            "AND FirmenID="+DatenbankID+                                   
                                                                            "AND HeftID=" +Heftname+"'");

Er startet bei Ausgabe und endet erst beim Heftnamen.

Daher ist eher die Frage, wieso er da überhaupt einen Treffer bekommt


Aber bei dem Code stellen sich mir eh die Nackenhaare auf... z.B. (etwas gekürzt):

               temp[0] = ""+ rs.getInt(1);
                             if(temp[0]==null){

Wie soll temp[0] denn null sein, wenn es mit dem leeren String + einem Int Wert initialisiert wird?

Und ich hoffe, dass die Einrückung nur bei der Übertragung ins Forum so zerhauen wurde.

Aber @Mart hat die Lösung zum ersten Problem sehr gut und präzise genannt.

 

[Mart]

wenn du Eclipse benutzt kannst du mit => strg + umschalt + f <= deinen code automatisch formatieren lassen
du kannst auch mit rechtsklick aufs projekt => source => code formatieren dann formatierts das ganze projekt


jede andere Java IDE wird es dir auch IRGENDWIE so machen lassen.. musst halt gucken was die shortcuts sind
( kann man sogar dann einstellen dass es dir unnötige imports weg macht, @ovverride annotationen hinzufügt, überall super.xyz wo es geht usw )
machts halt einfacher

 

[Neumi5694]

Etwas einfacher machst du dir solche längeren Abfragen, wenn du mit einem Formatter arbeitest. Da sieht man solche Fehler deutlich schneller.
Sicherer werden die Abfragen auf jeden Fall, wenn du ein PreparedStatement verwendest. Dann musst du dich auch nicht um Hochkommas scheren.