Konstruktoraufruf dynamisch auswerten und SQL bauen

[MaikHSW]

Hey zusammen.
Ich habe folgendes Problem bzw. suche eine Lösung:

Ich habe ein Programm mit AWT-GUI und habe ein Maske in der man per Dropdown diverse Dinge wie Vorname, Nachname, Straße und so weiter auswählen kann.
Dies ist natürlich in einer Klasse realisiert.
Wenn ich dann aus dieser Klasse auf den "Suchen" Button klicke wird anhand der gegebenen Parameter ein Konstruktor von der Klasse "Suchen" aufgerufen.
Problem ist, dass einige vielleicht nur den Vornamen suchen, andere den Nachnamen und so weiter.
Somit werte ich erstmal nur ein Kriterium auf, denn so viele mit Namen "Michael" erwarte ich nicht.
Die Klasse suchen muss also folgendes tun:

Sie bekommt die Parameter im Konstruktor und muss auswerten, ob es sich um einen leeren Parameter handelt (User hat ihn nicht gesetzt) oder nicht. Wenn nicht muss aufgrund dieses Parameters ein SQL-Befehl abgesetzt werden der die Datenbank abfragt.
Und genau da komme ich ins Stocken.
Meine erste Idee war eine sehr lange und eigentlich kranke if-Konstruktion zu wählen und die wahrscheinlichsten Paramter nach oben zu setzen. Das würde in den meisten Fällen nur einen geringen Durchlauf erfordern und daher schnell zum Ziel führen.
Nun habe ich einfach als If-Bedingung folgende genommen:

if (Name.length()>0)

Dies sorgt aber bei einem leeren String ganz elegant für einen Nullpointer (was ich nicht direkt verstehen kann).
Heißt also zusammegefasst dass es so nicht läuft.
Aber wie mache ich das am besten?

Ich hoffe, es hat irgendjemand verstanden was ich vor habe und kann mir, eine vermutlich sehr triviale, Lösung an die Hand geben.



Gru0

 

[XHelp]

Was genau willst du denn wissen?
Wie du den SQL Befehl baust? Wie du den Nullpointer umgehst? Oder etwas anderes?
Wenn du die ganzen Parameter mit SQL abdeckst, dann brauchst du doch gar keine if-Abfragen, denn die DBMS hat für dich schon alles abgefragt (zumindestmal sollte es so sein). Oder machst du eine "WHERE 1=1" Abfrage und wertest alles manuell aus? oO
Spielen denn die Verknüpfungen eine Rolle, oder ist es alles "AND"? Du bekommst eine Liste an Parametern, dann machst du sowas wie:

public String getWhereClause(List<ParameterObjekt> params) {
  String s = "";
  for (ParameterObjekt currentParameter : params) {
    s+="`"+currentParameter.getName()+"`='"+currentParameter.getValue()+"' AND ";
  }
  if (s.length()>0) {
    s = s.substring(0, s.length() -5);
  }
  return s;
}

 

[MaikHSW]

Hey XHelp.
Ich brauche eine Lösung für dieses Problem.

Ich baue die SQL-Befehle dann anschließend händisch, bzw. gieße meine Werte nur noch in vorgefertige Befehle ein.
Wenn ich aber ein Gesamt-SQL bauen würde wäre dies tödlich.
Denn ein select * from Tabelle where Name="" and vorname="Hans" wäre unproduktiv wenn ich Hans Meier suche.
Verstehst du?
Ich möchte, wenn der Name gefüllt ist folgendes SQL abschicken:
select * from Tabelle where Name="Meier"

Die Frage ist doch, wie ich am elegantesten auswerte welcher Parameter in der Maske gefüllt wurde und daraus das SQL machen kann.
Das soll dann wieder ein Resultset bauen, welches ich in eine Tabelle parse.

So erfahren bin ich da aber auch nicht, ich übe ja noch aus Spaß.
Wie kann ich alles geschickter machen?


Gruß

 

[XHelp]

Also ich habe meinem Post noch etwas hinzugefügt.

Wenn ich aber ein Gesamt-SQL bauen würde wäre dies tödlich.
Denn ein select * from Tabelle where Name="" and vorname="Hans" wäre unproduktiv wenn ich Hans Meier suche.

Wenn du einen Hans Meier suchst, dann ist es der falsche Befehl, denn so wirst du nur namenlose Hansen rausfiltern.

Ich möchte, wenn der Name gefüllt ist folgendes SQL abschicken:
select * from Tabelle where Name="Meier"

Was genau ist jetzt dein Auswahlkriterium? Immer den 1. Parameter raussuchen oder wie? Bzw. warum filterst du nur nach "Meier", wenn der Benutzer auch den Vornamen spezifiziert hat?

Die Frage ist doch, wie ich am elegantesten auswerte welcher Parameter in der Maske gefüllt wurde und daraus das SQL machen kann.

Was genau stellt deine Maske dar, bzw wie gibst du die Daten weiter?

Das soll dann wieder ein Resultset bauen, welches ich in eine Tabelle parse.

hm?

Im grunde brauchst du ja nur alle eingegebenen Parameter mit einem "und" zu verknüpfen und schon hast du deine Klausel.

 

[MaikHSW]

Hey XHelp.
Danke für deine Antworten.
Ich glaube, ich drücke mich noch schwierig aus.
Ich mache dir heute Abend mal einen Screenshot der Maske und stell noch was an Erläuterungen ein.
Vielleicht ist auch meine Herangehensweise einfach etwas quer....



Gruß

 

[MaikHSW]

Hey.
So, nun habe ich eine mögliche Lösung und zugleich ein neues Problem
Also, die Lösung des Übels war eine Prüfung auf null.
Ich habe bisher nur geprüft ob die Länge größer null ist, dies ist aber tödlich wenn man Nullpointer nicht vorbeugt.
Nun frage ich ab auf null (Parameter == null) und somit ist das Problem gelöst.

Folgende Aufgabenstellung ergibt sich aber daraus:
Ich frage aktuell mit einer if- Bedingung ab.
Sprich ich frage ab, ob der Name gefüllt ist.
Sollte dies so sein nutze ich ein SQL und frage entsprechend ab.
Sollte dies nicht so sein prüft er mit einer nächsten If Bedingung ab ob die Stadt gefüllt ist und so weiter.
Heißt aber dass ich eine ganz verschachtelte If-Bedingung habe welche sehr schwer zu durchdringen und erst recht zu warten ist.
Gibt es da elegantere Lösungen?
Ich will möglichst nach Wahrscheinlichkeit gehen, das heißt Name und Stadt z.B. zuerst.
Denn die Hausnummer oder PLZ sucht man vermutlich eher selten.
Und da ich ja möglichst wenig Schleifendurchläufe oder Tiefe suche....
Gibt es da elegante Lösungen?

Danke.



Gruß

 

[XHelp]

Es existiert keine if-Schleife.
Mir ist immer noch ein Rätsel, was du da überhaupt machst, denn wenn der Benutzer nach Name="Maier" und PLZ="12345", dann musst du ja nur die entsprechende where Klausel bauen "WHERE `Name`='Maier' AND `PLZ`='12345'", und nicht es irgendwie einzeln abfragen.
Deswegen kann ich dir nur das wiederholen, was ich bereits gesagt habe: ja, es gibt eine elegantere Lösung, nämlich where-Klausel bauen.

 

[MaikHSW]

Hey XHelp.
Das mit der Schleife habe ich bereits korrigiert, Gedankenfehler meinerseits.
Natürlich sind where Klauseln der Weg.
Aber ich muss ja erstmal heraus finden, welches Feld der User wirklich gefüllt hat.
Nehmen wir mal beispielhaft folgendes an.

In der Maske Quicksearch gibt es 3 Felder: Name, Vorname, Wohnort.

Nun nutzt Mitarbeiter A die Maske da er einen Anruf bekommt, Kunde ist Herr Meier.
Also will er alle Meiers haben.
Vielleicht will er auch bewusst nach Ralf Meier suchen.

Je nachdem was er will fügt er dies aber auch in die Maske ein.
Die denkbaren Konstellationen habe ich nun schon angeführt.
Wenn ich also nun einfach den Inhalt der Felder übergebe und ein SQL mit where Klauseln habe könnte das zu folgendem Resultat führen: select * from Kunde where Name=Ralf and nachname=meier and wohnort=null
Das würde natürlich nichts bringen. Also muss ich qualifizieren, ob ein Wert gefüllt ist und damit ausgewertet werden kann oder nicht. Für dieses Beispiel wäre der richtige Weg ja folgender: select * from Kunde where Name=ralf and nachname=meier;

Verstehst du was ich meine?
Je nach Konstellation kann das aber auch anders ausehen.



Gruß

 

[XHelp]

Naja, dann gehst du mit einer Schleife alles durch und prüfst es auf "null" (wobei es mich wundert, warum es null ist und nicht "leerer String", aber egal). Wenn es !=null ist, dann verknüpfst du es mit AND, ansonsten lässt du dieses Parameter weg.

 

[MaikHSW]

Hey XHelp.
Eine Schleife?
Ich meine, wir reden hier von 3 Parametern welche entweder null oder gefüllt sein können.
Wie soll man da eine Schleife realisieren? Stehe ich auf dem Schlauch?
Ich würde es fast schon mit if Bedingungen realisieren, das geht bei 3 auch noch.
Aber die erweiterte Suche hat deutlich mehr Bedingungen, das wäre doof....

Danke für deine Hilfe.


Gruß

 

[XHelp]

Also wenn es IMMER nur 3 Parameter sind, dann kannst du es so lassen, ansonsten würde deine Taktik überdenken. Du könntest eine Klasse ParameterContainer machen, die name und key hat (so gesehen Map.Entry) und dem Konstruktor einfach eine Liste von ParameterContainer übergeben. Dann kannst du auch elegant mit ner for Schleife alle durchgehen.