JList Problem : Synchronisation mit Vector klappt nicht :(

[187nitro]

Hallo liebe Java-Gemeinde,

in der maske.java :

public class maske extends javax.swing.JFrame {

    
    public void addtoList(Vector email) {
            
            email.add("lol");
            jList1.setListData(email);
      
    }

.... blabla....

private void cmd_searchActionPerformed(java.awt.event.ActionEvent evt) {
addtoList();
}

-----------------------------------------------------------------------------

Wenn ich den Button klicke funzt auch alles ...
So nun zu meinem Problem.

Wenn ich addtoList z.B. von xyz.java aufrufe dann ist im Vector email alle
übergebenen Daten vorhanden ( system.out ) sehe ich alles.

Wenn ich jetzt aber

jList1.setListData(email);

aufrufe. Passiert nichts d.h. die jList nimmt die Daten nicht auf.


Meine Frage : Wieso funktioniert es nicht wenn ich aus xyz.java die Methode aufrufe ?

Vielen Dank für eure Antworten im vorraus!

 

[SlaterB]

vielleicht passiert schon etwas nur was anderes als du erwartest,

z.B. könntest du zwei verschiedene maske-Objekte haben, eines wird angezeigt,
ein anderes ist nicht sichtbar und Änderungen dort bringen wenig für den Bildschirm

oder eine von tausenden anderen Ursachen
-> vollständiges Programm und Ablaufbeschreibung posten, dann ist alles sonnenklar

 

[187nitro]

/*
 * maske.java
 *
 * Created on 19. September 2008, 17:01
 */

package deratzowjava;
import java.util.*;
import javax.swing.DefaultListModel;

/**
 *
 * @author  Mir xD
 */
public class maske extends javax.swing.JFrame {

    /** Creates new form maske
     * @param email 
     */
    
    public void addtoList() {
        DefaultListModel liste = new DefaultListModel();
        jList1.setModel(liste);
        liste.addElement("idjfiosjfois");
        liste.addElement("idjdsadfiosjfois");
        liste.addElement("idjfiasdasdosjfois");
        liste.addElement("idjfioasdassjfois");
        
        jList1.validate();
      
    }
    
    
    public maske() {
        initComponents();
       
    }

    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    @SuppressWarnings("unchecked")
    // <editor-fold defaultstate="collapsed" desc="Generated Code">                          
    private void initComponents() {

        jPanel1 = new javax.swing.JPanel();
        cmd_search = new javax.swing.JButton();
        jLabel1 = new javax.swing.JLabel();
        text_search = new javax.swing.JTextField();
        cmd_end = new javax.swing.JButton();
        jScrollPane1 = new javax.swing.JScrollPane();
        jList1 = new javax.swing.JList();

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);

        jPanel1.setBorder(javax.swing.BorderFactory.createTitledBorder("Main"));

        cmd_search.setText("Webseite durchsuchen");
        cmd_search.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                cmd_searchActionPerformed(evt);
            }
        });

        jLabel1.setFont(new java.awt.Font("Tahoma", 1, 11));
        jLabel1.setText("Webseite druchsuchen:");

        cmd_end.setText("Beenden");
        cmd_end.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                cmd_endActionPerformed(evt);
            }
        });

        javax.swing.GroupLayout jPanel1Layout = new javax.swing.GroupLayout(jPanel1);
        jPanel1.setLayout(jPanel1Layout);
        jPanel1Layout.setHorizontalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addContainerGap()
                .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addComponent(jLabel1, javax.swing.GroupLayout.DEFAULT_SIZE, 171, Short.MAX_VALUE)
                    .addGroup(jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.TRAILING, false)
                        .addComponent(cmd_end, javax.swing.GroupLayout.Alignment.LEADING, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                        .addComponent(cmd_search, javax.swing.GroupLayout.Alignment.LEADING, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                        .addComponent(text_search, javax.swing.GroupLayout.Alignment.LEADING, javax.swing.GroupLayout.DEFAULT_SIZE, 155, Short.MAX_VALUE)))
                .addContainerGap())
        );
        jPanel1Layout.setVerticalGroup(
            jPanel1Layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(jPanel1Layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jLabel1)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(text_search, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(cmd_search)
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addComponent(cmd_end)
                .addContainerGap(javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
        );

        jList1.setSelectionMode(javax.swing.ListSelectionModel.SINGLE_SELECTION);
        jScrollPane1.setViewportView(jList1);

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(getContentPane());
        getContentPane().setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addContainerGap()
                .addComponent(jPanel1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                .addGap(18, 18, 18)
                .addComponent(jScrollPane1, javax.swing.GroupLayout.DEFAULT_SIZE, 563, Short.MAX_VALUE)
                .addContainerGap())
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup()
                .addContainerGap(21, Short.MAX_VALUE)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(layout.createSequentialGroup()
                        .addComponent(jScrollPane1, javax.swing.GroupLayout.PREFERRED_SIZE, 423, javax.swing.GroupLayout.PREFERRED_SIZE)
                        .addContainerGap())
                    .addGroup(javax.swing.GroupLayout.Alignment.TRAILING, layout.createSequentialGroup()
                        .addComponent(jPanel1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                        .addGap(284, 284, 284))))
        );

        pack();
    }// </editor-fold>                        

private void cmd_endActionPerformed(java.awt.event.ActionEvent evt) {                                        
// TODO add your handling code here:
    System.exit(0);
}                                       

private void cmd_searchActionPerformed(java.awt.event.ActionEvent evt) {                                           
// TODO add your handling code here:
    getEmails mail = new getEmails();
    [b]addtoList();   //Damit geht es! aber ich will die Methode gerne auslagern![/b]
        try {

           [b] //mail.getQuelltext(text_search.getText());  So sollte es eigentlich gehen... tut es aber nicht[/b]
        } catch (Exception ex) {
            System.out.println("Fehler -> "+ex);
        }
  
}                                          

    /**
    * @param args the command line arguments
    */
    public static void main(String args[]) {
        java.awt.EventQueue.invokeLater(new Runnable() {
            public void run() {
                new maske().setVisible(true);
            }
        });
    }

    // Variables declaration - do not modify                     
    private javax.swing.JButton cmd_end;
    private javax.swing.JButton cmd_search;
    private javax.swing.JLabel jLabel1;
    private javax.swing.JList jList1;
    private javax.swing.JPanel jPanel1;
    private javax.swing.JScrollPane jScrollPane1;
    private javax.swing.JTextField text_search;
    // End of variables declaration                   

}

SO nun die andere Datei

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */

package deratzowjava;
import java.net.*;
import java.io.*;
import java.util.*;

/**
 *
 * @author nitro
 */
public class getEmails {

    
    public void getQuelltext(String strUrl) throws Exception {
        
        URL url                 = new URL(strUrl);
        URLConnection  urlConn  = url.openConnection();
        StringBuffer content    = new StringBuffer();
        String inputLine;
        
        BufferedReader in       = new BufferedReader (new InputStreamReader(urlConn.getInputStream()));

        while ((inputLine = in.readLine()) != null)
        {
            content.append(inputLine);
            content.append("\n");
        }
        in.close();
        parseCode(content.toString());
 
    }
    
    
public void parseCode(String code) {
    Vector vec = new Vector();
    int pos =1;
    int ende=1;
            
    while (code.indexOf("mailto:",pos+1) != -1)
    {
        pos     = code.indexOf("mailto:",pos+1);
        ende    = code.indexOf("\"",pos+2);
       vec.add(code.substring(pos+7, ende));
      
    }
    
//    for (int i = 0; i < vec.size(); i++) {
//       System.out.println(vec.get(i).toString()); }
        
        maske maske = new maske();
        [b]maske.addtoList(); //Eigentlich wird noch was übergeben, hab ich zu Testzwecken rausgenommen also nicht wundern ^^[/b]
        
   
    
   
}
    
    
    
    
    
}

Hoffe ihr könnt mir da helfen ! Wäre super !
Danke shconmal im vorraus!

 

[SlaterB]

ich habe die Lösung in der ersten Antwort exakt erraten

 

[Gast]

hm ok jetzt verstehe ich was du meinst. wie rufe ich addtolist denn auf, damit ich zugriff auf die "schon vorhandene maske" habe?
vielen Dank schonmal

 

[SlaterB]

der Aufruf ist ganz normal wie bekannt:

richtigeMaske.addtoList();

interessant ist nur, die Referenz auf das richtige maske-Objekt zu erhalten,
das ist je nach Ausgangslage eine Kunst,

hier erzeugst du ja das getEmails-Objekt (schrecklicher Klassenname..)
in Zeile 142 von maske:
> getEmails mail = new getEmails();

da ist es ein leichtes, maske (also this, das eigene Objekt) als Parameter zu übergeben und im Konstruktor von getEmails abzuspeichern

 

[187nitro]

Ah es klappt
Vielen Dank für deine Hilfe