Best Practice View probleme

[Heiner51]

Hey lieber Postleser,
koenntest du -ja genau du- meinen code mal anschauen und mit mitteilen ob man, dass so programmieren kann!?

package MVC.View;

import java.awt.EventQueue;

import javax.swing.JFrame;

import javax.swing.JList;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;

import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;

import java.awt.BorderLayout;
import javax.swing.JTextField;
import java.awt.FlowLayout;

import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.GridLayout;

import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.awt.event.FocusEvent;
import java.awt.event.FocusListener;

import javax.swing.JPasswordField;


public class Hello extends JFrame implements ActionListener,FocusListener,TableData{


    /**
     *
     */
    private static final long serialVersionUID = 1L;
    private JButton loginButton, rechtsButton, linksButton;
    private JTextField textField;
    private JPasswordField passwordField;
    private JMenu benutzerMenu, auftragMenu;
    private JMenuItem neuItem, loeschenItem, zuweisenItem,erstellenItem, exitItem;
    private JMenuBar menuBar;
    private JList<String> rechtsList,linksList;
    conttroler cont =new conttroler();


    public Hello() {
        super("Management");
        beforLogin();
    }

    private void beforLogin() {
      
      
        setSize(400, 400);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      
      
        setResizable(false);
        loginButton=new JButton("Login");
        textField=new JTextField("Benutzername");
        passwordField=new JPasswordField("Passwort");
        passwordField.setEchoChar((char)0);

        loginButton.addActionListener(this);
        textField.addFocusListener(this);
        passwordField.addFocusListener(this);
        GridBagLayout gbLayout=new GridBagLayout();
        GridBagConstraints gbConstraints=new GridBagConstraints();
        gbConstraints.gridy=0;
        gbConstraints.gridx=0;
        gbConstraints.anchor=GridBagConstraints.WEST;
        gbConstraints.fill=GridBagConstraints.BOTH;

        setLayout(gbLayout);

        add(textField,gbConstraints);
        gbConstraints.gridy++;
        add(passwordField, gbConstraints);
        gbConstraints.gridy++;
        add(loginButton,gbConstraints);
        setVisible(true);
        requestFocus();
    }

    public void login() {

        getContentPane().removeAll();
        setLayout(new BorderLayout());
        setResizable(true);
        setJMenuBar(addMenu());

        JTable table=new JTable(DATA, COLHEADS);      
        //table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
        add(new JScrollPane(table),BorderLayout.CENTER);
        pack();
      
    }
    public void zuweisen() {
        JPanel rechtsPanel=new JPanel(new FlowLayout());
        JPanel linksPanel=new JPanel(new FlowLayout());
        JPanel buttonPanel= new JPanel(new GridLayout(2, 1));
        getContentPane().removeAll();
        linksButton=new JButton("<-");
        rechtsButton=new JButton("->");
        linksButton.addActionListener(this);
        rechtsButton.addActionListener(this);
      
        rechtsPanel.add(rechtsButton);
        buttonPanel.add(rechtsPanel);
        linksPanel.add(linksButton);
        buttonPanel.add(linksPanel);
      
        setLayout(new GridLayout(1, 3));
        add(new JScrollPane(rechtsList));
        add(buttonPanel);
        add(new JScrollPane(linksList));
        pack();
      
      
      

    }
    public JMenuBar addMenu() {
        neuItem=new JMenuItem("Neu");
        loeschenItem=new JMenuItem("Loeschen");
        zuweisenItem=new JMenuItem("Zuweisen");
        erstellenItem=new JMenuItem("Erstellen");
        exitItem=new JMenuItem("EXIT");

        benutzerMenu=new JMenu("Benutzer");
        auftragMenu=new JMenu("Auftrag");

        menuBar=new JMenuBar();

        benutzerMenu.add(neuItem);
        benutzerMenu.add(loeschenItem);
        benutzerMenu.addSeparator();
        benutzerMenu.add(exitItem);
        auftragMenu.add(zuweisenItem);
        auftragMenu.add(erstellenItem);

        neuItem.addActionListener(this);
        loeschenItem.addActionListener(this);
        zuweisenItem.addActionListener(this);
        erstellenItem.addActionListener(this);
        exitItem.addActionListener(this);

        menuBar.add(benutzerMenu);
        menuBar.add(auftragMenu);

        return menuBar;
    }

    public void actionPerformed(ActionEvent e) {
        cont.actionPerformed(e);

    }

    public void focusGained(FocusEvent e) {
        cont.focusGained(e);
    }

    public void focusLost(FocusEvent e) {
        cont.focusLost(e);
    }

}

package MVC.View;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.FocusEvent;
import java.awt.event.FocusListener;

public class conttroler implements ActionListener,FocusListener{

    Hello view;
    public static void main(String[] args) {
        new Hello();
    }
    @Override
    public void actionPerformed(ActionEvent e) {
        // TODO Auto-generated method stub
        System.out.println(e.getActionCommand());
        if (e.getActionCommand()=="Login") {
            view.login();
        }
      
    }

    @Override
    public void focusGained(FocusEvent e) {
        // TODO Auto-generated method stub
      
      
    }
@Override
    public void focusLost(FocusEvent e) {
        // TODO Auto-generated method stub
      
    }
  

}

 

[thecain]

View ist null, weil du es nie initialisierst.

 

[Heiner51]

Jau stimmt soltte wohl mal in die Heia gehen