ActionListener bei Button?

[Sören]

Hallo, programmiere heute zum ersten mal so 'ne GUI in Java.
Wie mache ich das, dass beim Klick auf den Button "jbutton1" etwas passiert?

Mein Code, aber der geht nicht:

import Network.Network;
import Network.NetworkListener;
import Password.Password;
import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.net.*;
import java.util.StringTokenizer;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class FirstGUI extends JFrame

{
   private JButton button;
   private JPanel panel;
    Network network;
    private String nickname;
    private String passwort;
    private String channel;

   private FirstGUI()
   {
       super("StayOnline by MacPri");
       setSize(400,150);
       setLocation(300,230);
       setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
       setLayout(new BorderLayout(5,5));
string nickname;
string passwort;
string channel;
        nickname = "";
        passwort = "";
        channel = "";

       panel = new JPanel(new GridLayout(5,2));
        final JTextField nickField = new JTextField();
        final JPasswordField passField = new JPasswordField();
        final JTextField channelField = new JTextField();
        JButton jbutton1 = new JButton("StayOnline!");
    

       panel.add(new JLabel(" Nickname:"));
        panel.add(nickField);

        panel.add(new JLabel(" Passwort:"));
        panel.add(passField);

        panel.add(new JLabel(" Channel:"));
        panel.add(channelField);

panel.add(new JLabel(" "));
panel.add(jbutton1);

       getContentPane().add(panel);


       setVisible(true);
   }


jbutton1.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e) {
nickname = nickField.getText();
                    passwort = new String(passField.getPassword());
                    channel = channelField.getText();
       network = new Network(this);
}
});




   public static void main(String[] args)
   {
        FirstGUI g = new FirstGUI();
   }
}

 

[foobar]

Du hast falsch geklammert:

import Network.Network;
import Network.NetworkListener;
import Password.Password;
import java.awt.*;
import java.awt.event.*;

import javax.swing.*;

public class FirstGUI extends JFrame

{
   private JButton button;
   private JPanel panel;
    // Network network;
    private String nickname;
    private String passwort;
    private String channel;

   private FirstGUI()
   {
       super("StayOnline by MacPri");
       setSize(400,150);
       setLocation(300,230);
       setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
       setLayout(new BorderLayout(5,5));
        nickname = "";
        passwort = "";
        channel = "";

        panel = new JPanel(new GridLayout(5,2));
        final JTextField nickField = new JTextField();
        final JPasswordField passField = new JPasswordField();
        final JTextField channelField = new JTextField();
        JButton jbutton1 = new JButton("StayOnline!");
        jbutton1.addActionListener(new ActionListener() 
        {
            public void actionPerformed(ActionEvent e) 
            {
                nickname = nickField.getText();
                            passwort = new String(passField.getPassword());
                            channel = channelField.getText();
               network = new Network(this);
            }
        });


        panel.add(new JLabel(" Nickname:"));
        panel.add(nickField);

        panel.add(new JLabel(" Passwort:"));
        panel.add(passField);

        panel.add(new JLabel(" Channel:"));
        panel.add(channelField);

        panel.add(new JLabel(" "));
        panel.add(jbutton1);

       getContentPane().add(panel);
       setVisible(true);
   }


   public static void main(String[] args)
   {
        new FirstGUI();
   }
}

Ausserdem hast du die Member durch lokale Variablen überschrieben.

 

[Sören]

Danke, klappt soweit ganz gut.
Aber das Aufrufen der Network.class mit network = new Network(this) macht noch Probleme.

Der Code der Network.class:

package Network;

import java.io.*;
import java.net.Socket;

// Referenced classes of package Network:
//            NetworkListener, J, F

public class Network
    implements Runnable
{

    BufferedReader breader;
    OutputStream outStream;
    InputStream is;
    PrintWriter pwriter;
    J code;
    NetworkListener nl;
    public boolean connected;
    boolean verbose;
    F f;

    public void disconnect()
    {
        try
        {
            connected = false;
            nl.connectionLost(this);
            outStream.close();
            is.close();
        }
        catch(Exception exception) { }
    }

    public void setVerbose(boolean b)
    {
        verbose = b;
    }

    public Network(NetworkListener nl)
    {
        code = new J();
        connected = false;
        verbose = true;
        f = new F();
        this.nl = nl;
        try
        {
            Socket socket = new Socket("h-213.61.5.150.host.de.colt.net", 2710);
            is = socket.getInputStream();
            outStream = socket.getOutputStream();
            (new Thread(this)).start();
            sendString("t\000V8.6\000 \0000", 1);
            connected = true;
        }
        catch(Exception e)
        {
            e.printStackTrace();
            connected = false;
            nl.connectionLost(this);
        }
    }

    public void run()
    {
        while(connected) 
        {
            String s1 = f.I(destroy(is));
            System.out.println(s1);
            nl.stringRecieved(this, s1);
        }
    }

    private byte[] destroy(InputStream inputstream)
    {
        try
        {
            int i;
            while(i > 0x19000) 
            {
                i = 0;
                byte abyte0[] = new byte[5];
                abyte0[0] = (byte)inputstream.read();
                if(abyte0[0] >= 0)
                {
                    i = abyte0[0] + 1;
                } else
                {
                    i = (abyte0[0] & 0x1f) + 1;
                    int j = ((abyte0[0] & 0x60) >>> 5) + 1;
                    for(int k = 1; k < j; k++)
                    {
                        i += inputstream.read() << (k - 1) * 8 + 5;
                    }

                }
            }
            byte abyte1[] = new byte[i];
            for(; i > 0; i--)
            {
                abyte1[abyte1.length - i] = (byte)inputstream.read();
            }

            return abyte1;
        }
        catch(Throwable t)
        {
            connected = false;
        }
        nl.connectionLost(this);
        return new byte[2];
    }

    public void sendString(String string, int i)
    {
        if(verbose)
        {
            nl.stringSend(this, string);
        }
        try
        {
            byte abyte0[] = code.I(string);
            byte abyte1[] = currentTimeMillis(abyte0.length);
            byte abyte2[] = new byte[abyte1.length + abyte0.length + i];
            System.arraycopy(abyte1, 0, abyte2, i, abyte1.length);
            System.arraycopy(abyte0, 0, abyte2, abyte1.length + i, abyte0.length);
            outStream.write(abyte2);
        }
        catch(Exception e)
        {
            e.printStackTrace();
        }
    }

    private byte[] currentTimeMillis(int i)
        throws IOException
    {
        if(i > 0x19000)
        {
            throw new IOException("Too long Msg: " + i);
        }
        byte abyte0[] = (byte[])null;
        if(--i >= 0)
        {
            if(i < 128)
            {
                abyte0 = new byte[1];
                abyte0[0] = (byte)i;
            } else
            {
                byte byte0;
                for(byte0 = 0; 32 << (byte0 + 1) * 8 <= i; byte0++) { }
                abyte0 = new byte[byte0 + 2];
                abyte0[0] = (byte)(byte0++ << 5 | 0x80 | i & 0x1f);
                for(int j = 1; j < abyte0.length; j++)
                {
                    abyte0[j] = (byte)(i >>> 8 * (j - 1) + 5);
                }

            }
        }
        return abyte0;
    }
}

P.S: Die FirstGUI.java liegt im Verzeichnis /, die Network.class im Verzeichnis Network/[/code]

 

[André Uhres]

//                network = new Network(this); //this ist kein NetworkListener
                network = new Network(new NetworkListener());  
                new Thread(network).start();

 

[foobar]

Es würde dann aber mehr Sinn machen wenn die GUI-Klasse den Networklistener implementieren würde, damit die GUI Klasse auch auf die Callbacks reagieren kann. Oder du speicherst den Networklistener in einem Member und reagierst dann auf die Events.

 

[Sören]

So, jetzt meldet mir der Compiler aber noch: "Network.NetworkListener is abstract, cannot be instantiated".

FirstGUI.java:

import Network.Network;
import Network.NetworkListener;
import Password.Password;
import java.awt.*;
import java.awt.event.*;

import javax.swing.*;

public class FirstGUI extends JFrame
{
   private JButton button;
   private JPanel panel;
    Network network;
    private String nickname;
    private String passwort;
    private String channel;

   private FirstGUI()
   {
       super("StayOnline by MacPri");
       setSize(400,150);
       setLocation(300,230);
       setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
       setLayout(new BorderLayout(5,5));
        nickname = "";
        passwort = "";
        channel = "";

        panel = new JPanel(new GridLayout(5,2));
        final JTextField nickField = new JTextField();
        final JPasswordField passField = new JPasswordField();
        final JTextField channelField = new JTextField();
        JButton jbutton1 = new JButton("StayOnline!");
        jbutton1.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {

                nickname = nickField.getText();
                            passwort = new String(passField.getPassword());
                            channel = channelField.getText();
                network = new Network(new NetworkListener()); 
                new Thread(network).start();
            }
        });


        panel.add(new JLabel(" Nickname:"));
        panel.add(nickField);

        panel.add(new JLabel(" Passwort:"));
        panel.add(passField);

        panel.add(new JLabel(" Channel:"));
        panel.add(channelField);

        panel.add(new JLabel(" "));
        panel.add(jbutton1);

       getContentPane().add(panel);
       setVisible(true);
   }


   public static void main(String[] args)
   {
        new FirstGUI();
   }
}

 

[André Uhres]

Vielleicht zeigst du mal den Code vom NetworkListener, damit wir wissen was das überhaupt ist, man muss ja sonst alles erraten...

 

[Sören]

NetworkListener.java:

package Network;


// Referenced classes of package Network:
//            Network

public interface NetworkListener
{

    public abstract void stringRecieved(Network network, String s);

    public abstract void connectionLost(Network network);

    public abstract void stringSend(Network network, String s);
}

 

[foobar]

So, jetzt meldet mir der Compiler aber noch: "Network.NetworkListener is abstract, cannot be instantiated".

Deshalb habe ich dir ja geraten den NetworkListener von deiner GUI implementieren zu lassen. Ansonsten funktionieren die Callbacks nicht bzw. machen keinen Sinn.

 

[Sören]

foobar: Ja, bloß ich kenn mich noch nicht soo gut mit Java aus und weiß nicht wie das geht
'ne kleine Erklärung für den Aha-Effekt wäre auch nicht schlecht

 

[foobar]

import Network.Network;
import Network.NetworkListener;
import Password.Password;
import java.awt.*;
import java.awt.event.*;

import javax.swing.*;

public class FirstGUI extends JFrame implements NetworkListener
{
   private JButton button;
   private JPanel panel;
    Network network;
    private String nickname;
    private String passwort;
    private String channel;

   private FirstGUI()
   {
       super("StayOnline by MacPri");
       setSize(400,150);
       setLocation(300,230);
       setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
       setLayout(new BorderLayout(5,5));
        nickname = "";
        passwort = "";
        channel = "";

        panel = new JPanel(new GridLayout(5,2));
        final JTextField nickField = new JTextField();
        final JPasswordField passField = new JPasswordField();
        final JTextField channelField = new JTextField();
        JButton jbutton1 = new JButton("StayOnline!");
        jbutton1.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {

                nickname = nickField.getText();
                            passwort = new String(passField.getPassword());
                            channel = channelField.getText();
                network = new Network(new NetworkListener());
                new Thread(network).start();
            }
        });


        panel.add(new JLabel(" Nickname:"));
        panel.add(nickField);

        panel.add(new JLabel(" Passwort:"));
        panel.add(passField);

        panel.add(new JLabel(" Channel:"));
        panel.add(channelField);

        panel.add(new JLabel(" "));
        panel.add(jbutton1);

       getContentPane().add(panel);
       setVisible(true);
   }

  public void stringRecieved(Network network, String s)
  {
  }

    public void connectionLost(Network network)
   {
   }

    public void stringSend(Network network, String s)
   {
   } 

   public static void main(String[] args)
   {
        new FirstGUI();
   }
}

Jetzt mußt du nur noch die leeren Methoden mit Logik füllen.