Discussion :

[Olivier Regnier]

Bonjour

Je développe un petit script qui permet de compter le nombre des messages non lus dans mon dossier principal INBOX ainsi que ses sous-dossiers sur mon serveur imap. Voici le script:

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
 
#!/usr/local/bin/env ruby
 
require 'net/imap'
require 'base64'
 
m0 = "INBOX"
m1 = "INBOX.forums.developpez"
m2 = "INBOX.forums.fug"
m3 = "INBOX.lists.freebsd-announce"
m4 = "INBOX.lists.freebsd-ports"
m5 = "INBOX.lists.freebsd-questions"
m6 = "INBOX.lists.openbox-questions"
m7 = "INBOX.lists.openbsd-announce"
m8 = "INBOX.services.ovh"
m9 = "INBOX.services.free"
m10 = "INBOX.Spam"
 
imap = Net::IMAP.new('imp.nameserver.org')
imap.login('nom@server.org',Base64.decode64('password'))
 
for i in m0;m1;m2;m3;m4;m5;m6;m7;m8;m9;m10
     imap.select(i)
     value = imap.search(['NOT','SEEN'])
     if value.empty? == false
         printf "#{value.nitems}"
     else
         printf "No"
     end
     imap.disconnect
end

Le code compte uniquement le contenu de la variable m0 mais pas les autres

Qu'en pensez-vous ?

Merki

[titoumimi]

essaie peut-être ça :

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
#!/usr/local/bin/env ruby
 
require 'net/imap'
require 'base64'
 
m = ["INBOX",
      "INBOX.forums.developpez",
      "INBOX.forums.fug",
      "INBOX.lists.freebsd-announce",
      "INBOX.lists.freebsd-ports",
      "INBOX.lists.freebsd-questions",
      "INBOX.lists.openbox-questions",
      "INBOX.lists.openbsd-announce",
      "INBOX.services.ovh",
      "INBOX.services.free",
      "INBOX.Spam"]
 
imap = Net::IMAP.new('imp.nameserver.org')
imap.login('nom@server.org',Base64.decode64('password'))
 
m.each { |i|
     imap.select(i)
     value = imap.search(['NOT','SEEN'])
     if value.empty? == false
         printf "#{value.nitems}"
     else
         printf "No"
     end
     imap.disconnect
}

Je suis pas sûr de ton for in (même quasi sûr qu'il marche pas...)

PS : est-ce que ton disconnect dans ta boucle est une bonne idée ?

Edit : j'ai corrigé ma déclaration de tableau, php a fait des dégats sinon, tu peux le déclarer comme suit pour pas toucher de teop à ta syntaxe :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
m = Array.new()
 
m[0] = "INBOX"
m[1] = "INBOX.forums.developpez"
m[2] = "INBOX.forums.fug"
m[3] = "INBOX.lists.freebsd-announce"
m[4] = "INBOX.lists.freebsd-ports"
m[5] = "INBOX.lists.freebsd-questions"
m[6] = "INBOX.lists.openbox-questions"
m[7] = "INBOX.lists.openbsd-announce"
m[8] = "INBOX.services.ovh"
m[9] = "INBOX.services.free"
m[10] = "INBOX.Spam"

[Olivier Regnier]

Je viens de tester le code ci-dessus et j'obtiens le message suivant:

checkimap.rb:6: undefined local variable or method `m' for main:Object (NameError)

Le imap.disconnect est utile car la vérification se produit toutes les 10 minutes. Il faut mieux dans ce cas la fermer directement.

Pour le for in, tu as raison, je viens de regarder sur le wiki et est indiqué qu'il vaut mieux itérer en utilisant les méthodes adaptées.

Merki de ton aide

[titoumimi]

j'ai corrigé mon code sans doute pendant que tu répondait

Mais mettre le disconnect en sortie de boucle me parait plus adapté (sans connaitre ce module), je voit mal comment tu va checker le 2° en te déconnectant en fin de première itération...

[Olivier Regnier]

Tu as entièrement raison pour le disconnect, j'ai corrigé. Le code fonctionne mais il n'additionne pas le nombre de messages.

Lorsque j'exécute le script, j'obtiens l'affichage suivant:

Code :

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

30 0 0 0 0 2 0 0 0 0 0

Je devrais obtenir logiquement 32

[titoumimi]

ça me seble parfaitement normmal, puisque tu réinitialise value à chaque affectation... pour avoir et le total et le détail, fait comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
total = 0
m.each { |i|
     imap.select(i)
     value = imap.search(['NOT','SEEN'])
     if value.empty? == false
         printf "#{value.nitems}"
         total = total + value.nitems
     else
         printf "No"
     end
}
imap.disconnect
puts total

[Olivier Regnier]

Je suis vraiment un newbi de première ceci dit, il calcule bien le nombre total de messages mais l'affichage n'est pas bon, le voici:

Code :

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

33 0 0 0 0 2 0 0 0 0 0 35

Je veux juste le total

[titoumimi]

il suffit que tu suppries ton printf

(et du coup, soit total soit value deviennent inutile..)

[Olivier Regnier]

Je suis d'accord avec toi mais l'affichage n'est toujours pas bon car en sortie, j'obtiens ceci:

Code :

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

00000000038

Tu vas pas me dire que je dois faire de totaux un pour les nouveaux messages et un pour les mailboxes vides ?

Je ne vais pas supprimer non plus le printf "0" car s'il n'y a aucun messages, il faut qu'il affiche 0 new messages!

Merki

[titoumimi]

vire tes deux printf, il affichera 0 puisque total est initialisé à 0 au pire

[Olivier Regnier]

Tout fonctionne, c'est parfait. Merci pour avoir réalisé ce code