Probleme mit speichern (GUI)

[tomiu]

Hallo,

warum wenn ich auf speichern1a(Button) klicke, ein Eintrag wird mehrmals gespeichert und nicht nur einmal????

danke im voraus

lg
tomiu

	 ActionListener sp1A=new ActionListener(){
	        public void actionPerformed(ActionEvent e){

	            Lieferant e1=new Lieferant();

	  try{

	    if ((na_me2.getText().trim().length()>0)&&(vorna_me2.getText().trim().length()>0))
	    {
	    	
	    	
	        e1.setName(na_me2.getText().trim());
	         
	        e1.setVorname(vorna_me2.getText().trim());
	         
	        e1.setAdresse(adr_esen2.getText().trim());
	       
	        e1.setTelNr(tel_nr2.getText().trim());

	        LiefManager.sp1(e1);
	    }
	    else
	        JOptionPane.showMessageDialog(null,"Name und Vorname darf nicht leer sein");


	               
	            
	   }catch(Exception r){

	                JOptionPane.showMessageDialog(null,"Falsche Eingabe");
	            }
	            


	          }
	      };

	      sp_eichern1a.addActionListener(sp1A);//speichern1a ist Button

public static void sp1(Lieferant e1){

    Connection con=DB.DB2();

    

    //PreparedStatement pstmt = null;


    try{
  
    	String s="INSERT INTO Lieferant(Name,Vorname,Adresse,Telnr)" +
    			"VALUES(?,?,?,?)";
    	 PreparedStatement pstmt = con.prepareStatement(s);
     
        

         pstmt.setString(1,e1.getName());

         pstmt.setString(2,e1.getVorname());

         pstmt.setString(3,e1.getAdresse());

         pstmt.setString(4,e1.getTelNr());

         pstmt.executeUpdate();

               
         JOptionPane.showMessageDialog(null,"Erfolgreich gespeichert");
    	
        }
        catch(SQLException sqle){
            System.err.println(sqle);

        }
        catch(NullPointerException npe){
            System.err.println(npe);
        }
      }

 

[Runtime]

Mehrmals.

 

[tomiu]

Mehrmals.

Ja..3 mal..manschmal 5, manschmal 9...ich klicke aber nur einmal

 

[gman]

Hi,

kannst dir ja mal ausgeben lassen wie viele Einträge gespeichert wurden:

         int inserts = pstmt.executeUpdate();

               
         JOptionPane.showMessageDialog(null, inserts + " Einträge erfolgreich gespeichert");

 

[Runtime]

Rufst du sp1 nur im ActionListener auf? Warum hast du eingentlich so komische Namen? Code conventions

 

[tomiu]

Hi,

kannst dir ja mal ausgeben lassen wie viele Einträge gespeichert wurden:

         int inserts = pstmt.executeUpdate();

               
         JOptionPane.showMessageDialog(null, inserts + " Einträge erfolgreich gespeichert");

selbes Fenster taucht 3 mal auf..

Rufst du sp1 nur im ActionListener auf? Warum hast du eingentlich so komische Namen? Code conventions

Ja, nur im ActionListener...das mit Code conventions, hast recht...werde später korrigieren

 

[Runtime]

Mir kommen nur folgende Möglichkeiten in den Sinn:
1. Du registriest den ActionListener 3mal
2. Die Methode sp1 wird ausserhalb des ActionListeners aufgerufen.

 

[SlaterB]

wahrscheinlich wurden mehrere gleiche/selber ActionListener registriert,
übernimm mal den Anfang meines ActionListeners hier bzw. durch die drei OptionPanes ist ja klar dass 3x der Listener drankommt

fraglich ist dann der weitere Programmaufbau, aus dem ersten Post ist nicht zu erkennen, ob sp1A mehrmals geadded wird,
vielleicht die gesamte Methode mit diesem Code mehrfach aufgerufen..

public class TestGUI   extends JFrame {
    public TestGUI() {
        JButton b = new JButton("Start");
        ActionListener l = new ActionListener()  {
                public void actionPerformed(ActionEvent e)   {
                    JButton source = (JButton)e.getSource();
                    System.out.println("actionPerformed-Aufruf, source hat Listener: " 
                        + source.getActionListeners().length);
                }
            };
        b.addActionListener(l);
        b.addActionListener(l);
        add(b);

        setSize(200, 200);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setVisible(true);
    }

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

 

[tomiu]

wahrscheinlich wurden mehrere gleiche/selber ActionListener registriert,
übernimm mal den Anfang meines ActionListeners hier bzw. durch die drei OptionPanes ist ja klar dass 3x der Listener drankommt

Stimmt...source hat 3 Listener... habe bemerkt dass der Listener 3mal drankommt nur wenn ich auf zurück klicke (Lief zurück)und dann wieder speichern will

actionPerformed-Aufruf, source hat Listener: 3

hier die codes(ohne Code conventions..sorry )

package gui;
import java.awt.*;
import java.awt.event.*;
import java.util.Vector;

import javax.swing.*;
import javax.swing.border.Border;
import javax.swing.table.DefaultTableModel;

import lieferant.LiefManager;
import lieferant.Lieferant;

public class Gui extends JFrame{

	public Gui(){
		
		Gui1i();
	}
	
	JLabel l_Id1 =new JLabel("LId:");

	   JTextField l_Id2=new JTextField(2);

	   JLabel na_me1 =new JLabel("Name:");

	    JTextField na_me2=new JTextField(15);

	    JLabel vorna_me1 =new JLabel("Vorname:");

	    JTextField vorna_me2=new JTextField(15);

	    JLabel adr_esen1 =new JLabel("Adresse:");

	    JTextField adr_esen2=new JTextField(20);

	    JLabel tel_nr1 =new JLabel("Telnr:");

	    JTextField tel_nr2=new JTextField(20);

	    JButton zu_rueck1=new JButton("Lief zurueck");
	    JButton sp_eichern1=new JButton("Speichern");
	    JButton sp_eichern1a=new JButton("Speichern");

	   public Object colnames[] ={"Name" ,"Vorname", "Adresse","Telnr" };

	    public Object[][] 	data = {
										{"", "", "", ""}

									};

	    private JTable table=new JTable(data,colnames);

	    private JScrollPane s_table=new JScrollPane(table);


	    private String aus1[]={"Lieferant","Schmuck"};

	    JComboBox a_us1=new JComboBox(aus1);
	    
	    public void Gui1i(){

	        Border leer= BorderFactory.createEmptyBorder();


	        setTitle("Lieferant");

	        setLocation(300,300);

	        setLayout(new GridBagLayout());

	        GridBagConstraints c=new GridBagConstraints();

	 
	        c.gridx = 3;

	        c.gridy = 1;

	        c.anchor = GridBagConstraints.CENTER;

	         
	        getContentPane().add(sp_eichern1,c);




	        c.gridx = 1;

	        c.gridy = 6;

	        c.gridwidth = 5 ;

	        getContentPane().add(s_table,c);

	        
	      ActionListener speichern_lieferant=new ActionListener(){
	        public void actionPerformed(ActionEvent e){

	            getContentPane().removeAll();

	            speich1();

	          }
	      };

	      sp_eichern1.addActionListener(speichern_lieferant);

	     

            pack();

	            setVisible(true);

	            setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

	}
	    
	    
	    



	    public void speich1(){

	        setTitle("Lieferanten speichern");

	        
	        setLocation(300,300);

	        setLayout(new GridBagLayout());

	        GridBagConstraints c=new GridBagConstraints();

	       
	      
	        c.gridx = 1;

	        c.gridy = 2;

	        getContentPane().add(na_me1,c);

	        c.gridx = 2;

	        c.gridy = 2;

	        getContentPane().add(na_me2,c);

	        c.gridx = 1;

	        c.gridy = 3;

	        getContentPane().add(vorna_me1,c);

	        c.gridx = 2;

	        c.gridy = 3;

	        getContentPane().add(vorna_me2,c);

	        c.gridx = 1;

	        c.gridy = 4;

	        getContentPane().add(adr_esen1,c);

	        c.gridx = 2;

	        c.gridy = 4;

	        getContentPane().add(adr_esen2,c);

	        c.gridx = 1;

	        c.gridy = 5;

	        getContentPane().add(tel_nr1,c);

	        c.gridx = 2;

	        c.gridy = 5;

	        getContentPane().add(tel_nr2,c);

	        c.gridx = 3;

	        c.gridy = 3;

	        c.anchor = GridBagConstraints.CENTER;

	        getContentPane().add(sp_eichern1a,c);

	        c.gridx = 7;

	        c.gridy = 3;

	        c.anchor = GridBagConstraints.CENTER;

	        getContentPane().add(zu_rueck1,c);


	        c.gridx = 1;

	        c.gridy = 6;

	        c.gridwidth = 5 ;

	        getContentPane().add(s_table,c);

	       
	 ActionListener sp1Action=new ActionListener(){
	        public void actionPerformed(ActionEvent e){

	            Lieferant e1=new Lieferant();
	            JButton source = (JButton)e.getSource();
                System.out.println("actionPerformed-Aufruf, source hat Listener: " 
                    + source.getActionListeners().length);

	  try{

	    if ((na_me2.getText().trim().length()>0)&&(vorna_me2.getText().trim().length()>0))
	    {
	    	
	    	
	        e1.setName(na_me2.getText().trim());
	         
	        e1.setVorname(vorna_me2.getText().trim());
	         
	        e1.setAdresse(adr_esen2.getText().trim());
	       
	        e1.setTelNr(tel_nr2.getText().trim());

	        LiefManager.sp1(e1);
	    }
	    else
	        JOptionPane.showMessageDialog(null,"Name und Vorname darf nicht leer sein");


	               
	            
	   }catch(Exception r){

	                JOptionPane.showMessageDialog(null,"Falsche Eingabe");
	            }
	            


	          }
	      };

	      sp_eichern1a.addActionListener(sp1Action);

	       
          ActionListener zu1=new ActionListener(){
 	          public void actionPerformed(ActionEvent e){

 	              getContentPane().removeAll();
 	              Gui1i();
 	          }
 	        };

 	        zu_rueck1.addActionListener(zu1);

	       pack();

	       setVisible(true);

	       setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

	  }


}

hier ist sie sp1 methode:

package lieferant;

import java.sql.*;

import db.DB;
import java.util.*;

import javax.swing.*;



public class LiefManager {

	     //Speichern
    public static void sp1(Lieferant el1){

    Connection con=DB.DB2();

    

    //PreparedStatement pstmt = null;


    try{
  
    	String s="INSERT INTO Lieferant(Name,Vorname,Adresse,Telnr)" +
    			"VALUES(?,?,?,?)";
    	 PreparedStatement pstmt = con.prepareStatement(s);
     
        

         pstmt.setString(1,el1.getName());

         pstmt.setString(2,el1.getVorname());

         pstmt.setString(3,el1.getAdresse());

         pstmt.setString(4,el1.getTelNr());

         int inserts = pstmt.executeUpdate();

         JOptionPane.showMessageDialog(null, inserts + " Einträge erfolgreich gespeichert");
        
         
        // JOptionPane.showMessageDialog(null,"Erfolgreich gespeichert");
    	
        }
        catch(SQLException sqle){
            System.err.println(sqle);

        }
        catch(NullPointerException npe){
            System.err.println(npe);
        }
      }

}

 

[SlaterB]

besteht eine Frage?
das Problem dürfte offensichtlich (edit: unter anderem) sein, dass bei Zurück
> Gui1i();
aufgerufen wird, Einfügen in die GUI mit Layout usw. ist korrekt nach dem removeAll(), aber es wird auch der Listener neu geadded
(sehe an deinem edit dass du das nun auch erkannt hast, deshalb nochmal: besteht eine Frage oder ist alles geklärt?)

-----

zu deiner Code-Qualität:
Code-Conventions ist eine Sache, aber fällt dir nicht selber auf wie unendlich unleserlich das ganze ist mit Leerzeilen nach JEDER Zeile?
selbst wenn du das nicht aktiv so hast sondern das irgendein Kopier-Programm einfügt, kannst du den Text doch immer noch kurz in einen Editor setzen und (gewiss 5 Min. lang) die Leerzeilen entfernen

und [ java ] statt [ code ] hilft auch ungemein

 

[Michael...]

Wo lernt man so zu programmieren... und blickst Du in Deinem Code überhaupt noch selbst durch?

Zu Deinem Problem:
Jedesmal wenn Du auf den Button sp_eichern1 drückst erzeugst Du einen neuen ActionListener (der letztendlich die Methode zum Speichern in der DB aufruft) und registrierst an sp_eichern1a.

 

[tomiu]

besteht eine Frage?
das Problem dürfte offensichtlich (edit: unter anderem) sein, dass bei Zurück
> Gui1i();
aufgerufen wird, Einfügen in die GUI mit Layout usw. ist korrekt nach dem removeAll(), aber es wird auch der Listener neu geadded
(sehe an deinem edit dass du das nun auch erkannt hast, deshalb nochmal: besteht eine Frage oder ist alles geklärt?)

-----

zu deiner Code-Qualität:
Code-Conventions ist eine Sache, aber fällt dir nicht selber auf wie unendlich unleserlich das ganze ist mit Leerzeilen nach JEDER Zeile?
selbst wenn du das nicht aktiv so hast sondern das irgendein Kopier-Programm einfügt, kannst du den Text doch immer noch kurz in einen Editor setzen und (gewiss 5 Min. lang) die Leerzeilen entfernen

und [ java ] statt [ code ] hilft auch ungemein

ich bin körperlich behindert un deswegen kann nicht immer die Code-Conventions verwenden..es kostet mir sehr viel Zeit(ich schreibe mit Fuß nicht mit Hände)..arbeite mit spezielle Software und mache nur copy/paste wenn es um namen von Objekten oder Variablen geht...

meine Frage: wie kann ich machen dass der Listener nicht geadded wird??

danke im voraus

 

[SlaterB]

da du alle Objekte schon ganz zu Anfang erstellst kannst du auch die Listener zu Beginn, im Konstruktor zuweisen,
verschiebe den Code für die drei Listener und die addActionListener()-Aufrufe alle in den Konstruktor,

vor oder hinter dem
Gui1i();
Aufruf dort

 

[tomiu]

da du alle Objekte schon ganz zu Anfang erstellst kannst du auch die Listener zu Beginn, im Konstruktor zuweisen,
verschiebe den Code für die drei Listener und die addActionListener()-Aufrufe alle in den Konstruktor,

vor oder hinter dem
Gui1i();
Aufruf dort

Danke