Codezeilen "auslagern"

[dennisT]

Hallo, ich habe ein Programm geschrieben was momentan ca 3000Zeilen hat. Ich habe mir überlegt, um es übersichtlicher zu gestalten, bestimmte Zeilen "auszulagern" sprich diese in eine neue .java Datei zu speichern.

Hier mal ein Beispielcode:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.table.*;
import java.io.*;
import java.text.*;
import java.util.*;
import java.sql.*;
import javax.swing.border.*;


public class Compafree extends JFrame  {

  Container cp;
  private static Compafree fenster = new Compafree("");
  String serverdaten[] = new String[10];
  JTextField jtf_loginname;
  JPasswordField jpf_loginpasswort;
  String loginNameAusDB="";
  int fullwidth = 0;
  JButton jb_login = new JButton("Login");

  JPanel jp_center = new JPanel();
  JPanel jp_loginbuttons = new JPanel();
  JPanel jp_logo = new JPanel();

  public Compafree (String title) {
    super(title);
    cp=getContentPane();
    Lauscher lausch = new Lauscher();

    jtf_loginname = new JTextField(10);
    jpf_loginpasswort = new JPasswordField(10);

    JLabel jl_loginname = new JLabel("Name:",JLabel.CENTER);
    JLabel jl_loginpasswort = new JLabel("Passwort:",JLabel.CENTER);

    jb_login.addActionListener(lausch);

    JPanel jp_login = new JPanel();
           jp_login.setLayout(new GridLayout(4,1));
           jp_login.add(jl_loginname);
           jp_login.add(jtf_loginname);
           jp_login.add(jl_loginpasswort);
           jp_login.add(jpf_loginpasswort);

    jp_center.add(jp_login);
    cp.add(jp_center,"Center");
    cp.add(jb_login,"South");
    
    this.setDefaultCloseOperation(EXIT_ON_CLOSE);
  }

  class Lauscher implements ActionListener {
    public void actionPerformed(ActionEvent e) {

/////////////////////////////////////////LOGIN////////////////////////////////////////////////////////
      if( e.getSource() == jb_login){

        String loginPWAusDB="";
        String loginname = jtf_loginname.getText();
        String loginpassname = new String(jpf_loginpasswort.getPassword());
        
          try{
            Connection con = DriverManager.getConnection("jdbc:mysql://"+serverdaten[0]+"/"+serverdaten[1],serverdaten[2],serverdaten[3]);
            Statement stmt = con.createStatement();
            ResultSet rslt = stmt.executeQuery("select * from ttm_user where Name = '"+loginname+"'");
            while (rslt.next()) {
              loginNameAusDB=(rslt.getString("Name"));
              loginPWAusDB=(rslt.getString("Passwort"));
            }
            stmt.close();
            con.close();
          }
          catch (Exception f){}

          if(loginNameAusDB.equals(loginname)&&loginPWAusDB.equals(loginpassname)){
          
            jp_center.removeAll();
          }
      }
//////////////////////////////////////////////////////////////////////////////////////////////////////
    }
  }
/////////////////////////////////////////////MAIN/////////////////////////////////////////////////////////
    public static void main(String[] args) throws Exception {
      fenster.setSize(200,200);
      fenster.setLocationRelativeTo(null);
      fenster.setVisible(true);
    }
  }

Wäre es nun möglich diesen Abschnitt:

/////////////////////////////////////////LOGIN////////////////////////////////////////////////////////
      if( e.getSource() == jb_login){

        String loginPWAusDB="";
        String loginname = jtf_loginname.getText();
        String loginpassname = new String(jpf_loginpasswort.getPassword());
        
          try{
            Connection con = DriverManager.getConnection("jdbc:mysql://"+serverdaten[0]+"/"+serverdaten[1],serverdaten[2],serverdaten[3]);
            Statement stmt = con.createStatement();
            ResultSet rslt = stmt.executeQuery("select * from ttm_user where Name = '"+loginname+"'");
            while (rslt.next()) {
              loginNameAusDB=(rslt.getString("Name"));
              loginPWAusDB=(rslt.getString("Passwort"));
            }
            stmt.close();
            con.close();
          }
          catch (Exception f){}

          if(loginNameAusDB.equals(loginname)&&loginPWAusDB.equals(loginpassname)){
          
            jp_center.removeAll();
          }
      }
//////////////////////////////////////////////////////////////////////////////////////////////////////

In eine neue java Datei zu speichern?
Ich habs schon selber versucht aber dann konnte er nie die JTextFields erkennen. ? Wäre für hilfe dankbar.

 

[Landei]

So nicht. Du könntest eine Klasse namens Lauscher.java machen

public class Lauscher implements ActionListener {
  private JTextField jtf_loginname;
  //und was du sonst noch aus der Hauptklasse verwendest

  public Lauscher(JTextField jtf_loginname, /*und was du sonst noch aus der Hauptklasse verwendest*/) {
      this.jtf_loginname = jtf_loginname;
      //der Rest auch so
  }
  ...
  //das bleibt alles
}

Im Hauptprogramm musst du dann dem Lauscher-Konstruktor alle Felder "mitgeben", die er so braucht.

 

[Guardi]

Ähem, das ist aber quick and dirty gelöst.
Code für DB hat nichts in Klassen verloren in denen Swing-Komponenten sind.
Also mach jeweils eine Klasse aus DB, Swing und ActionListener.

Wenn du es allerdings richtig machen willst.
MVC-Pattern und Facade-Pattern anschauen und versuchen umzusetzen.

 

[Empire Phoenix]

Ein erster schrit währe es, das der actionlistener nur noch eine normal methode aufrüft wo er alles was benötigt wird weitergibt (aka textfelder und so weiter), damit ist dann das ganze zumindest schonmal etwas übersichtlicher.

Idealerweise sollte gui aber eigentlich keine logic enthalten. (Zumindest bei anwendung von MVC)

 

[Landei]

Ähem, das ist aber quick and dirty gelöst.

Na ja, wichtig ist, dass man mit dem Refactoring erst einmal irgendwo anfängt, und das kann bei einem 3000-Zeilen-Monster recht schwierig sein. Wenn man schon "fertige" innere Klassen hat, nimmt man eben die, und macht zu äußeren. Natürlich sollte man dabei nicht stehenbleiben, sondern zu versuchen, die neuen Strukturen weiter zu verbessern. Für weitergehende Tipps stehen hier aber nicht genügend Informationen zur Verfügung.