Comparaison de buffers

Version imprimable

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:

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

25/09/2011, 12h33
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 :)

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:

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...

28/09/2011, 15h11
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:

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...
28/09/2011, 15h14
therwald
Pour ce que tu veux faire (contenu buf1 -> buf2, buf 2 vidé si j'ai bien compris) je te conseille plutôt:
Code:

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
28/09/2011, 15h46
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.