Überladen von Funktionen vermeiden

[sammy2ooo]

Hallo zusammen,

ich habe ein ValueObject mit verschiedenen Werten: vorname, nachname, strasse, plz, etc... und dazu eine Klasse welche mir die Objekte aus einer DB holt. Wie kann ich es vermeiden für jedes Attribut meines ValueObjects eine Methode zu schreiben?

    public void listAddresses(String nachname) {
        Session mySession = HibernateUtil.getSessionFactory().openSession();
        Transaction myTransaction = mySession.beginTransaction();
        
        List result = mySession.createQuery("from Address where nachname = \'" + nachname + "\'").list();
        
        for (Iterator iter = result.iterator(); iter.hasNext(); ) {
        	Address loadedAdr = (Address) iter.next();
        	        	
    		System.out.println("Vorname: " + loadedAdr.getVorname() +
 				   " Nachname: " + loadedAdr.getNachname() +
 				   " Strasse: " + loadedAdr.getStrasse() +
 				   " Hausnr: " + loadedAdr.getHausnr() +
 				   " PLZ: " + loadedAdr.getPlz() +
 				   " Land: " + loadedAdr.getLand());
        }
        
        myTransaction.commit();
        mySession.close(); 
    }

    public void listAddresses(String vorname) {
        Session mySession = HibernateUtil.getSessionFactory().openSession();
        Transaction myTransaction = mySession.beginTransaction();
        
        List result = mySession.createQuery("from Address where nachname = \'" + vorname + "\'").list();
        
        for (Iterator iter = result.iterator(); iter.hasNext(); ) {
        	Address loadedAdr = (Address) iter.next();
        	        	
    		System.out.println("Vorname: " + loadedAdr.getVorname() +
 				   " Nachname: " + loadedAdr.getNachname() +
 				   " Strasse: " + loadedAdr.getStrasse() +
 				   " Hausnr: " + loadedAdr.getHausnr() +
 				   " PLZ: " + loadedAdr.getPlz() +
 				   " Land: " + loadedAdr.getLand());
        }
        
        myTransaction.commit();
        mySession.close(); 
    }

 

[SlaterB]

also auf jeden Fall könntest du jede Operation auf eine Zeile verkürzen:

listAdressesSQL("from Address where nachname = \'" + nachname + "\'");

der ganzen Rest steht in der EINEN Operation listAdressesSQL

--------

aber selbst bei nur einer Zeile pro Operation sind es immer noch n Operationen,

für so einen kleinen Bereich wird es sich kaum lohnen,
aber eine Möglichkeit:

Enums, int-Konstanten und/ oder String-Arrays:

public void listAddresses(Typ typ, String value) { 
listAdressesSQL("from Address where "+field[typ]+" = \'" + value + "\'");
}

wobei der Code von listAdressesSQL dann auch wieder direkt in dieser Operation stehen könnte,
sofern nirgendwo anders gebraucht,

problematisch wirds, wenn du auch int-Felder hast, bei denen die einfachen Anführungszeichen stören würden,
entweder eine zweite Operation speziell für diese oder ein Array mit dieser Information pro Feld
oder ähnliches

 

[bygones]

mhm dein code stueck geht ja schonmal nicht

wenn ueber jedes attribut (vorname, nachname, plz usw) eine Abfrage moeglich sein soll dann brauchst du auch fuer jedes eine Methode.

Ich wuerde das ueber eine Enum machen:

enum AdressFields {
 vorname, nachname, plz, strasse; // usw
}

public void listAddress(AdressFields field, String value) {
        Session mySession = HibernateUtil.getSessionFactory().openSession();
        Transaction myTransaction = mySession.beginTransaction(); 

        List result = mySession.createQuery("from Address where "+ field +" = \'" + value + "\'").list(); 

     for (Iterator iter = result.iterator(); iter.hasNext(); ) {
           Address loadedAdr = (Address) iter.next();
          
          // wie die ausgabe von adress sein soll -> in die toString methode der Klasse            
          System.out.println(loadedAdr);
        }
       
        myTransaction.commit();
        mySession.close(); 
}

oder eben modularer und fuer jedes Feld eine eigene Methode

privatevoid listAddress(String field, String value) {
        Session mySession = HibernateUtil.getSessionFactory().openSession();
        Transaction myTransaction = mySession.beginTransaction(); 

        List result = mySession.createQuery("from Address where "+ field +" = \'" + value + "\'").list(); 

     for (Iterator iter = result.iterator(); iter.hasNext(); ) {
           Address loadedAdr = (Address) iter.next();
          
          // wie die ausgabe von adress sein soll -> in die toString methode der Klasse            
          System.out.println(loadedAdr);
        }
       
        myTransaction.commit();
        mySession.close(); 
}

public void listAddressByLastname(String value) {
   listAddress("nachname", value);
}
// ... und die anderen methoden

 

[sammy2ooo]

danke für die Antworten.

wie würde ich dann einen enum wert als parameter übergeben? So funktioniert das leider nicht.

import pim.data.Address;
import pim.data.AddressManager;
...
AddressManager adrMgr = new AddressManager();
adrMgr.listAddresses(AddressFields.nachname, "Bommelmaier");

 

[SlaterB]

was ist AddressFields (Klasse posten) und was ist die Fehlermeldung?
was erwartet die Operation an Parametern?

poste doch mal eine vollständige Testklasse unabhängig von deinem Code
mit main und allem drumherum keite 30 Zeilen..

 

[bygones]

wenn du es so wie mein vorschlag gemacht hast...

hast du die enums importiert ? bzw wie slaterB schrieb... was genau sit der fehler ?