Discussion :

[lostsoul]

Bonjour,

J'ai un prog qui recupere les mails sur un serveur.
D'abord une premiere fois et il stock les uid dans un buffer.

Ensuite, toutes les n minutes, et stock les uid dans un autre buffer.

Puis, comparaison des buffers pour savoir s'il y a de nouveau messages.

Tout marche bien a la premiere verification, puis, si j'envoie un premier mail, il le detecte bien en nouveau.
Mais ensuite, il ne detecte plus les nouveaux mails, et je pense que ça vient des buffers.

Voila le bout de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
            store.connect(pop, user, pass);
            folder = store.getFolder("INBOX");
            folder.open(Folder.READ_WRITE);
            Message messages[] = folder.getMessages(); 
            for (i = 0, n = messages.length; i < n; i++) { 
                jTextArea1.append(i + ": " + messages[i].getFrom()[0] + "\t" + messages[i].getSubject() + "\n");
                statusMessageLabel.setText(i + 1 + " messages pour : " + user);
 
            fp.add(UIDFolder.FetchProfileItem.UID);
            folder.fetch(folder.getMessages(), fp);
            String cle2 = fp.getItems().toString();
            cle2 = cle2.substring(cle2.lastIndexOf("@")+1);
            buf2.append(cle2);       
            if(buf2.equals(buf1)  == false) {
                JOptionPane.showMessageDialog(null, " Vous avez un nouveau message ", "Information", JOptionPane.INFORMATION_MESSAGE, new ImageIcon("mail.png"));
 
            else 
                statusMessageLabel.setText("Pas de nouveaux messages");      
            buf1 = null;
            buf2 = buf1;
            buf2 = null;
            folder.close(false);
            store.close();

Je vide le premier buffer, puis met le buffer 2 dans le buffer 1 et vide le buffer 2.

Si quelqu'un voit mon erreur ?
Merci d'avance,
Tres cordialement,

ANDRE Ani

[JoeChip]

if(buf2.equals(buf1) == false)

peut s'écrire

if(!buf2.equals(buf1))

par ailleurs

for (i = 0, n = messages.length; i < n; i++) {
jTextArea1.append(i + ": " + messages[i].getFrom()[0] + "\t" + messages[i].getSubject() + "\n");
statusMessageLabel.setText(i + 1 + " messages pour : " + user);

il manque une accolade non ? et, amha, ce serait plus clair écrit ainsi :

i=0;
for (Message message : messages) {
jTextArea1.append(++i + ": " + message.getFrom()[0] + "\t" + message.getSubject() + "\n");
statusMessageLabel.setText(i + 1 + " messages pour : " + user);

Par ailleurs, "jTextArea1" n'est pas un nom très causant... et

buf1 = null;
buf2 = buf1;
buf2 = null;

est redondant, il me semble : quand tu fais "buff2 = null", buff2 est déjà à null, non ?

Mais tout cela est difficile à analyser, car il n'y a pas les décalarations des variables, donc j'ai ptet tout faux

[lostsoul]

Merci d'avoir regarde ;-)
J'avoue que c'est un peu le bazard...

buf1 et buf2 sont les stringbuffers dans lesquels je stock les uid des mails (en string).

jtextarea1 permet juste d'afficher les mails (from et subject).

Le soucis vient des stringbuffer.

Je veux, apres la premiere comparaison, vider le buf1, mettre la valeur de buf2 dans buf1 (pour la prochaine comparaison), puis vider buf2.

Je ne sais pas si je suis tres clair encore ?

Voila le code pour essayer d'etre plus clair :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
            buf2 = null;
            FetchProfile fp = new FetchProfile();
            jTextArea1.setText("");
            user = "user";
            pass = "pass";
            pop = "pop";           
            props = new Properties();
            session = Session.getDefaultInstance(props, null);
            store = session.getStore("pop3");
            store.connect(pop, user, pass);
            folder = store.getFolder("INBOX");
            folder.open(Folder.READ_WRITE);
            Message messages[] = folder.getMessages(); 
            for (int i = 0, n = messages.length; i < n; i++) { 
                jTextArea1.append(i + ": " + messages[i].getFrom()[0] + "\t" + messages[i].getSubject() + "\n");
                statusMessageLabel.setText(i + 1 + " messages pour : " + user);
            }
            fp.add(UIDFolder.FetchProfileItem.UID);
            folder.fetch(folder.getMessages(), fp);
            String cle2 = fp.getItems().toString();
            cle2 = cle2.substring(cle2.lastIndexOf("@")+1);
            buf2.append(cle2);       
            if(!buf2.equals(buf1)) {
                JOptionPane.showMessageDialog(null, " Vous avez un nouveau message ;-) ", "Information", JOptionPane.INFORMATION_MESSAGE, new ImageIcon("mail.png"));
            }
            else 
                statusMessageLabel.setText("Pas de nouveaux messages");             
            buf2 = buf1;
            folder.close(false);
            store.close();

La partie qui affiche les messages marche bien elle, les nouveaux messages sont bien affiches a chaque verif. C'est la detection des nouveaux qui se fait mal...

[therwald]

Alors, en mettant les buffers à nul, tu jette le buffer, mais il a une méthode pour le vider ce qui évite de refaire la réservation de place...
Par ailleurs, quand tu fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
buf1=null
buf2=buf1
buf2=null

tu:
jettes le buffer 1 (et son contenu)
met la référence sur le buffer 2 à null en la rendant égale à la référence sur le buffer 1, que tu viens de jeter (donc jette le buffer2)
puis remet à nul la référence sur le buffer 2 que tu viens d'effacer...
ça ne transfère pas buf1 vers buf2 avant de vider buf2...

[therwald]

Pour ce que tu veux faire (contenu buf1 -> buf2, buf 2 vidé si j'ai bien compris) je te conseille plutôt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
buf2.setLength(0); //vide le buffer2
buf2.append(buf1); //transfère le contenu de 1 vers 2
buf1.setLength(0); //vider le buffer1

[lostsoul]

Merci bien, mais finalement, j'ai reussi ;-)
Et je ne recupere plus les UID car il change a chaque fois, mais les sujets des mails.
Mais je vais voir avec buf.setLenght(), je n'y avais pas pense mais ça me parait plus propre, merci encore.