Discussion :

[GomuSan]

Bonsoir,
Un autre probleme
Commencons par un exemple :
J'ai une liste d'adresse ip : (en fait j'ai 7447 lignes comme sa)

229.216.99-84.rev.gaoland.net
20151251065.user.veloxzone.com.br
20151251065.user.veloxzone.com.br
20151251065.user.veloxzone.com.br
20151251065.user.veloxzone.com.br
r200-125-62-131-dialup.adsl.anteldata.net.uy
r200-125-62-131-dialup.adsl.anteldata.net.uy

Et moi ce que je veux faire c'est prendre juste la fin, c'est a dire le net, br, uy, etc.... Et compter combien de fois apparaissent chaque lettre

Pour cela, j'ai realisé ce code : Mais quand je compile il m'affiche une erreur a l'execution et aucune a la compilation

[Pacorabanix]

Sur ce sujet il y a des astuces pour le débogage :

http://www.developpez.net/forums/showthread.php?t=443

ça te permettra de nous en dire plus sur ton erreur. (d'ailleurs, c'est quoi l'erreur?) , à moins que des motivés, dans leur grande bonté et infinie sagesse, se penchent sur ton code en détail.

[GomuSan]

Desolé mais tout ce qui est dit pour debogué je l'ai fait, sauf les commentaires.

[Pierre Maurette]

Sans chercher plus loin, vous avez au moins un pétage de pile. Ajoutez

Code :

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

1
2
    printf("%d\n", max * sizeof(struct adresseip));
    return 0;

avant la déclaration de tablo et ce sera plus clair pour vous.
tablo doit être déclaré dynamiquement ou mis en global (hezbollah, c'est très mal, blah, toussa...).

Se méfier des tableaux et des multiplications de tailles, à partir de quelques dizaines d'octets, se poser la question de la mémoire prise dans la pile. Comme d'hab...

[GomuSan]

Sans chercher plus loin, vous avez au moins un pétage de pile. Ajoutez

Code :

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

1
2
    printf("%d\n", max * sizeof(struct adresseip));
    return 0;

avant la déclaration de tablo et ce sera plus clair pour vous.
tablo doit être déclaré dynamiquement ou mis en global (hezbollah, c'est très mal, blah, toussa...).

Se méfier des tableaux et des multiplications de tailles, à partir de quelques dizaines d'octets, se poser la question de la mémoire prise dans la pile. Comme d'hab...

Salut,
Tout d'abord, merci de l'interet que tu portes a mon sujet.
Sinon, un peu plus d'explications seraient les bienvenus. J'ai ajouté le printf juste avant la declaration et toujours erreur a l'execution.
Et le probleme, c'est que la declaration globale, je la proscrit. Quand a l'allocation dynamique, je ne vois pas comment faire, puisque je ne connais pas forcement la taille de mon fichier.

Merci encore
Gomu.

[Emmanuel Delahaye]

Je ne vois pas comment un printf corrigerait mon code.

Le but n'est pas de corriger, mais de te faire prendre conscience de la taille de l'objet demandé, qui est évidemment beaucoup trop gros pour la mémoire automatique.

Et le probleme, c'est que la declaration globale, je la proscrit. Quand a l'allocation dynamique, je ne vois pas comment faire, puisque je ne connais pas forcement la taille de mon fichier.

Tu peux définir une variable statique de portée locale dans main() avec 'static'.
Sinon, il n'y a pas de problème avec la mémoire dynamique. Il suffit de mesurer la taille requise avant l'allocation.

Enfin, on peut allouer dynamiquement une taille fixe.

[GomuSan]

Oui desolé, j'avais pas bien saisi. Donc Comme c'est un gros projet. J'avais deja fais une fonction qui me retourne le nombre de connexions.

Donc j'ai fais appel a cette fonction pour allouer dynamiquement.
Mais toujours le meme probleme :S

[GomuSan]

En fait, j'ai decomposé mon code, pour voir a partir de quel moment sa planté. J'ai remarqué que c'etait le code ou je veux prendre uniquement la fin de la chaine de caracteres :
Je vous explique ce que fais ce code.
J'ai un fichier, avec a chaque ligne une adresse ip :

229.216.99-84.rev.gaoland.net
20151251065.user.veloxzone.com.br
20151251065.user.veloxzone.com.br
20151251065.user.veloxzone.com.br

Moi ce que je veux faire, c'est prendre toutes les lettres qui sont après le point. c'est a dire net, br, fr...
Le probleme, c'est que ce code marche avec une seule ip, et quand je fais une boucle pour le faire sur plusieurs ip, sa plante.


Merci

[Emmanuel Delahaye]

En fait, j'ai decomposé mon code, pour voir a partir de quel moment sa planté. J'ai remarqué que c'etait le code ou je veux prendre uniquement la fin de la chaine de caracteres :

strrchr() ?

[GomuSan]

BIG EDIT :
Merci beaucoup, j'ai reussi a utiliser strrchr
Mais le probleme c'est que sa me prend le point :S

Re EDIT :
Sa marche, j'ai resolu mon probleme.

Je vous tiens au courant pour la suite du code.

Merci beaucoup

[Emmanuel Delahaye]

Ok il renvoie un pointeur. Mais qu'est-ce que j'en fais ?

C'est, à une vache près, l'adresse de la chaine que tu cherches. Ca ne te convient pas ?

Qu'est-ce qui t'empêche de faire un petit essai comme ça pour voir ce qui se passe :

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
 
#include <stdio.h>
#include <string.h>
 
int main (void)
{
   char const *s = "aa.bb.cc";
 
   char *p = strrchr (s, '.');
 
   if (p != NULL)
   {
      printf ("'%s'\n", p);
   }
 
   return 0;
}

Ca donne

Code :

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

1
2
3
4
 
'.cc'
 
Press ENTER to continue.

Si le point ne ne convient, pas, tu affiches à partir de p+1 :

Code :

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

1
2
3
4
5
 
   if (p != NULL)
   {
      printf ("'%s'\n", p + 1);
   }

ce qui donne

Code :

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

1
2
3
4
 
'cc'
 
Press ENTER to continue.

faut un peu apprendre à se prendre en main, on ne sera pas toujours derrière...

[GomuSan]

Erf, Merci mais tu n'avais pas lu mon edit

[GomuSan]

Promis, la derniere fois que je vous embete
Nouveau code. J'ai testé sur papier, j'y ai reflechi bien comme il faut, j'ai essayé de le faire d'une maniere on ne peut plus logique. j'ai testé "a la main"

En fait le probleme vient de la taille je pense.
Lorsqu'au lieu de max, je met par exemple 10, sa marche correctement. Mais avec max, rien du tout, il me dit


Merci

[GomuSan]

Salut,

Meme si personne m'a aidé sur la fin
J'ai reussi a regler le probleme.

Maintenant, mon probleme c'est que sa met 20 secondes a s'afficher...

Comment y remedier?