zugriffsmethoden

[juergenerwin]

Hallo Leute,

bräuchte mal euren Rat. Habe eine KLasse, in der ich Parameter aus einer Datenbank auslese und in eine ArrayList schreibe.

  public void initUnternehmen() {
    try {
      MainProgram.DBANSCon.rs = MainProgram.DBANSCon.stmt.executeQuery(
        "select unternehmen_id, unternehmensbezeichnung from jf_unternehmen");
      while (MainProgram.DBANSCon.rs.next()) {
        unternehmenID.add(MainProgram.DBANSCon.rs.getString("unternehmen_id").trim());
        unternehmensBezeichnung.add(MainProgram.DBANSCon.rs.getString("unternehmensbezeichnung").trim());
      }
    }
    catch (SQLException ex) {
      ex.printStackTrace();
    }
  }

  public ArrayList getUnternehmenID () {
    return unternehmenID;
  }

  public ArrayList getUnternehmensBezeichnung() {
    return unternehmensBezeichnung;
  }

Mein Problem: Solange ich die Methode initUnternehmen nicht aufrufe kann natürlich auch in den Zugriffsmethoden nichts stehen. Darf ich dann eigentlich "public" Zugriffsmethoden machen wenn ich diese unter eine Bedingung stelle (in diesem Fall aufrufen der Methode initUnternehmen) ?

Danke

 

[KSG9|sebastian]

woah..

du darfst schon public-methoden machen...aber sowas hier:
MainProgram.DBANSCon.rs.getString("unternehmen_id")
solltest du vermeiden.

1. solltest du die ergebnisse aus nem resultset in ne andere datenstruktur übermitteln
2. warum machst du alles statisch (ich nehm mal an dass DBANSCon.rs statisch ist, wenn nicht: warum machst du das rs public? )

überführ die daten aus der datenbank in deine eigene klassen, schmeiss die in ne collection und holl dir von ner anderen klasse dann die collection!

 

[puddah]

Ruf initUnternehmen() einfach im Konstruktor auf... des weiteren Empfehle ich dir ebenfalls die Ratschläge von KSG9|plak einzuhalten.