[reseaux]Net::Ping et $ret

**Warren@TI** · 20/12/2004, 11h56

Bonjour,

J'exécutte ce code sans probleme:

Code :

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

1
2
3
 
($ret, $duration, $ip) = $p->ping($host,2);
printf("$host [ip: $ip] is alive (packet return time: %.2f ms)\n", 1000 * $duration) if ($ret);

Je ne comprend pas vraiment ce qu'il y a derière la variable $ret, si je teste:

Code :

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

1
2
3
4
 
($ret, $duration, $ip) = $p->ping($host,2);
if $ret
{printf("$host [ip: $ip] is alive (packet return time: %.2f ms)\n", 1000 * $duration);}

Cela ne fonctionne plus. Quelqu'un sait de quoi il retourne? et éventuellement comment ajouter un else dans le code qui marche?

Merci

**Fanch.g** · 20/12/2004, 17h30

D'apres http://search.cpan.org/~bbb/Net-Ping...ib/Net/Ping.pm

Tu devrais plutot faire :

Code :

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

1
2
3
4
  ($ret, $duration, $ip) = $p->ping($host, 5.5);
    printf("$host [ip: $ip] is alive (packet return time: %.2f ms)\n", 1000 * $duration)
      if $ret;
    $p->close();

Mais je comprends pas bien pourquoi ton code marchais pas

surtout en lisant ca

Otherwise, the success flag will be 1 if the host is reachable and 0 if it is not

**Warren@TI** · 20/12/2004, 18h06

Le code que tu me suggères est celui que je cite en 1er (exepté mon timeout plus bas) et qui marche aussi chez moi mais je souhaites faire un else dans le cas ou le ping ne marche pas.

Comme toi, je ne comprend pas ou ce situe le soucis :-/

**Jedai** · 21/12/2004, 13h18

Simple, il faut lire la doc et tu t'aperçevra qu'il faut des parenthèses autour du test du if() { } (elle ne sont d'ailleurs pas indispensable si tu mets le if après, pourtant tu les as mises...), donc pour un else :

Code :

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

1
2
3
4
5
6
7
 
($ret, $duration, $ip) = $p->ping($host,2);
if ( $ret ) {
    printf("$host [ip: $ip] is alive (packet return time: %.2f ms)\n", 1000 * $duration);
} else {
 
}

Tout simplement...

--
Jedaï

**Warren@TI** · 21/12/2004, 19h03

Merci, je testerai cela demain.

**Warren@TI** · 22/12/2004, 09h36

Code :

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

1
2
3
4
        if ( &ret )
        {
        printf("$host [ip: $ip] is alive (packet return time: %.2f ms)\n", 1000 * $duration);
        } else {print ("testetetetest");}