heu gros probleme logique et algorithme

Question

"hello world !"
voila jai un "petit" probleme,  j'ai cr&#233;&#233; un code qui me permet de faire un traceroute a partir dun ping, avec un Ttl de 1 et en l'augmentant de 1 a chaque routeur travers&#233;.
bref, jai dans mon code une erreur, lorsque quand je fait un tracert ca indique "*",ici ca bloque lors de l'affectation de adr_passerelle=reply.Adress.ToString(); , car cette valeur ne peut pas etre nulle, alors, j'ai mis une condition "if" mais le probleme cest :
>> lorsque jexc&#233;cute le programme, ca me dit erreur a la ligne adr_passerelle et ca ne m&#233;xc&#233;cute pas le "if" cat il est plac&#233; apr&#233;s, et si je le place avant, adr_passerelle ne sera pas connue.
voila, ca fait 3jours que je suis dessus, j'en peux plus la..
SVP aidez moi...



string adr_passerelle = reply.Address.ToString();



long round_trip_time = reply.RoundtripTime;


if (adr_passerelle "") adr_passerelle 
"*";



MessageBox.Show(
"reply.Adress test = " + adr_passerelle, 
"reply.adress", 
MessageBoxButtons.OK, 
MessageBoxIcon.Error);

cs_coq · Answer

Salut, 

Ton probl&#232;me c'est que reply.Address vaut null, donc l'appel de ToString() &#233;choue.

if ( reply.Address != null )
{
   string adr_passerelle = reply.Address.ToString();
   long round_trip_time = reply.RoundtripTime;   if (adr_passerelle "") adr_passerelle "*";
      MessageBox.Show("reply.Adress test = " + adr_passerelle, "reply.adress", MessageBoxButtons.OK, MessageBoxIcon.Error);
}
else
{
   // ...
}

/*
coq
MVP Visual C#
*/

Nikoui · Answer

Je ne suis pas sur que ton if puisse r&#233;soudre le probl&#232;me : il y a une diff&#233;rence entre 'null' et ""... Le premier concerne un objet, le second une chaine vide.

Je pense plutot que ton erreur viens :
- soit de l'objet 'reply', qui est null lorsque tu arrive &#224; la ligne qui plante
- soit de l'objet 'reply.Address' qui est null lui aussi a ce moment la...

Tu devrais trouver cette erreur tr&#232;s rapidement en d&#233;buguant, sinon essaye deja de mettre cela :

adr_passerelle = "*";
if(reply != null)
{
    if(reply.Address != null)
    {
        adr_passerelle = reply.Address.ToString();
    }
}

long round_trip_time [...]

Nikoui · Answer

(arf pas assez rapide ^^)

cs_coq · Answer

4s de diff&#233;rence...je crois qu'on a pas fait pire encore ;-)

Sinon effectivement je n'ai pas penser &#224; reply, &#231;a me parressait &#233;vident qu'il ne soit pas null mais &#231;a peut &#234;tre le cas.
Cependant ton test peut &#234;tre raccourci : 
if(reply != null)
{
    if(reply.Address != null)
    {
       adr_passerelle = reply.Address.ToString();
    }
}

devient

if(reply != null && reply.Address != null)
{
   adr_passerelle = reply.Address.ToString();
}

:-)

/*
coq
MVP Visual C#
*/

billut · Answer

Voici mon code en un peu plus complet :
args cest l'argument que je met, l'adresse Ip de destination..
Le probleme cest dans le Else, il faudrait que j'arrive a mettre que le reply.Adress soit &#233;gal a "*" et non a null

if (reply.Address != 
null)


{



string adr_passerelle = reply.Address.ToString();    
//adresse de passerelle



long round_trip_time = reply.RoundtripTime;          
//dur&#233;e en ms de la requete ping


if (adr_passerelle "") adr_passerelle 
"*";



MessageBox.Show(
"reply.Adresse = "+ adr_passerelle, 
"Fin", 
MessageBoxButtons.OK, 
MessageBoxIcon.Exclamation);



if (args == reply.Address.ToString()) 
break;



if (args == adr_passerelle) 
MessageBox.Show(
"args=adresse_passerelle", 
"Fin", 
MessageBoxButtons.OK, 
MessageBoxIcon.Information);


}



 


else    
// reply.adress == null


{



string adr_passerelle = reply.Address.ToString();



long round_trip_time = reply.RoundtripTime;


 if (adr_passerelle "") adr_passerelle "*";



MessageBox.Show(
"reply.Adress test = " + adr_passerelle, 
"reply.adress", 
MessageBoxButtons.OK, 
MessageBoxIcon.Error);


}



Ttl++;


options.Ttl += 1;


}

voici lerreur g&#233;n&#233;r&#233;e :

Merci

Nikoui · Answer

Dans ce cas, si reply.Address est null et que tu ne veux pas qu'il le soit... il te faut le r&#233;cr&#233;er :

Si c'est reply qui est null :
reply = new JeSaisPasQuoi();

Si c'est reply address :
reply.Address = new JeSaisPasQuoi();

Tout d&#233;pend du type de replyAddress...

cs_coq · Answer

On ne peut pas voir ton image, elle est locale &#224; ta machine.

/*
coq
MVP Visual C#
*/

cs_coq · Answer

* est cens&#233; repr&#233;senter quoi ?
"N'importe quelle adresse" ?

/*
coq
MVP Visual C#
*/

billut · Answer

non, * cest pareil que lorsqu'on fait un traceroute sous le shell, lorsque l'on ne peut pas joindre l'hote, ca met *, ici cest pareil, lorsque Reply.Adress == null alors Reply.Adress. devrait prendre cette valeur "*".

Nikoui · Answer

De quel type est ton objet 'reply' ?

billut · Answer

voici :

Ping pingSender = 
new 
Ping();

PingOptions options = 
new 
PingOptions();
options.DontFragment = 
true;
options.Ttl = Ttl;

string data = 
"bonjour ceci est le string data ";
b
yte[] buffer = 
Encoding.ASCII.GetBytes(data);

int timeout = 120;



PingReply reply = pingSender.Send(args, timeout, buffer, options);

billut · Answer

voici  je met tout comme ca..:

Ping pingSender = 
new 
Ping();

PingOptions options = 
new 
PingOptions();
options.DontFragment = 
true;
options.Ttl = Ttl;

string data = 
"bonjour ceci est le string data ";
b
yte[] buffer = 
Encoding.ASCII.GetBytes(data);

int timeout = 120;



PingReply reply = pingSender.Send(args, timeout, buffer, options);

cs_coq · Answer

Dans le cas de la commande, * est juste une repr&#233;sentation graphique.
Une instance d'IPAddress ne peut pas prendre cette valeur, par contre tu peux probablement te servir du champs statique None pour repr&#233;senter *.

/*
coq
MVP Visual C#
*/

Nikoui · Answer

reply.Address est de type IpAddress, donc tu peux essayer cela :

if(reply.Address == null)
{
  reply.Address = new IPAddress(IPAddress.Broadcast);
}

Ce qui te metra une adresse de broadcast...

Mais je ne sais pas si c'est que tu cherches en fait.... Tu ne peux pas mettre "*" dans une adresse IP. Si le "*" te sert a d&#233;tecter comme tu le dis que l'hote n'est pas joignable... et bien dans ce cas laisse ton code comme il est :

si reply.Address == null >> alors l'hote n'est pas joignable et tu fait le traitement que tu dois faire dans ce cas.

billut · Answer

est-ce que tu peux m'expliquer un peu plus ? dsl de t'emb&#233;ter avec tout ca, mais moi et la programmation ca fait plus de 2...merci

Heu gros probleme logique et algorithme

15 réponses

Votre réponse

Discussions similaires