Position d'un caractère

Bjour tous,

j'ai un souci avec une fonction type strpos de php que j'ai faite en C. Elle doit retourner la position d'un caractère dans une chaine. La voici :

Code:

---

1 2 3 4

int substrlen (char *chaine, char *car) { return (strstr (chaine, car) - chaine); }

---

Je dois en fait retrouver la position d'un retour chariot dans une chaine pour modifier le caractère (en fait, un header http que je dois couper en lignes). Jusque là, pas de problème, ma fonction me retrouve bien la position de mon caractère. Sauf que, quand j'essaye de modifier mon caractère, je me prends une erreur de segmentation en pleine figure (aïeuhhh, ça picote). Ca ressemble à ça :

Code:

---

1 2 3 4

char *data = mon_header_http ; int i = substrlen (data, "\n"); strncpy (data+i, "\0", 1) ;

---

Quand je change la valeur de i pour mettre un bête chiffre, ça fonctionne.

J'arrive pas du tout où ça bloque. Quelqu'un de plus expérimenté voir mon erreur ? J'espère que je suis clair

Merci d'avance

Enjoy

The lsd

Est-ce que tu as alloué de la mémoire pour 'data' ?
Si tu as affecté une chaîne telle quelle à ton pointeur 'data' il est normal que tu aies une erreur de segmentation parce que la chaîne pointée n'est pas modifiable.

Pourquoi tu utilises strstr et pas strchr ? strchr recherche 1 caractère dans ta chaîne, strstr recherche une chaîne dans une autre.

Plutôt que d'utiliser strncpy tu pourrais tout simplement affecter ta valeur : data[i] = '\0';

Citation:

---

Envoyé par Pouet_forever

Est-ce que tu as alloué de la mémoire pour 'data' ?
Si tu as affecté une chaîne telle quelle à ton pointeur 'data' il est normal que tu aies une erreur de segmentation parce que la chaîne pointée n'est pas modifiable.

---

ben je l'ai affecté, euhhh oui normalement... 'fin je pense quoi ! Vu que quand je modifie la chaine en mettant

Code:

---

1 2	int i = 20; strncpy (data+i, "\0", 1) ;

---

la chaîne data est bien modifiée. Donc ça devrait être bon. Et je vais poser la question alakon, comment je l'alloue, avec malloc ? Sauf que je connais pas la taille de ma chaine, donc c'pas pratique.

Citation:

---

Envoyé par Pouet_forever

Pourquoi tu utilises strstr et pas strchr ? strchr recherche 1 caractère dans ta chaîne, strstr recherche une chaîne dans une autre.

---

Parce que ! C'est le première que j'ai trouvé, donc j'ai pris celle ci :p Mais bon, je vais surement avoir besoin de trouver \r, \n, ou \r\n, donc ça dérange pas.

Citation:

---

Envoyé par Pouet_forever

Plutôt que d'utiliser strncpy tu pourrais tout simplement affecter ta valeur : data[i] = '\0';

---

C'est pas un peu crade comme méthode, ça fait pas propre je trouve, mais je me trompe peut être ?

Enjoy

The lsd

Citation:

---

Envoyé par the lsd

ben je l'ai affecté, euhhh oui normalement... 'fin je pense quoi ! Vu que quand je modifie la chaine en mettant

Code:

---

1 2	int i = 20; strncpy (data+i, "\0", 1) ;

---

la chaîne data est bien modifiée. Donc ça devrait être bon.

---

Ouch cte lacune. En partant comme ça tu vas droit au mur.
Pour gagner en rapidité, le C ne vérifie jamais rien, partant du principe que "le programmeur sait ce qu'il fait". Ce qui t'autorise donc, en terme de code, à écrire 15 int dans un tableau de 10. Les 5 autres iront écraser une autre variable.
Donc ça compilera, ça pourra même marcher un temps. Mais un jour ou l'autre... :aie::aie::aie:

C'est à toi de toujours t'assurer que ce que tu stockes a bien la place d'être stocké. Et que si tu tapes dans tab[n] alors n est bien inclus dans les limites du tableau.

Citation:

---

Envoyé par the lsd

Et je vais poser la question alakon, comment je l'alloue, avec malloc ? Sauf que je connais pas la taille de ma chaine, donc c'pas pratique.

---

2 solutions
1) tu connais pas la taille mais tu peux la connaitre (par le biais d'une fonction d'info quelconque) donc tu fais char *chaine=malloc(n * sizeof(char))
2) tu connais pas la taille et tu ne peux pas la connaitre. Alors il te faut programmer un algo de ce genre

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

déclarer un char *chaine=NULL, un size_t taille_allouée=0 et un size_t indice=0 tant que lecture 1 caractère ok faire si indice=taille_allouée alors augmenter taille_allouée de B caractères chaine=realloc(chaine, taille_allouée * sizeof(char)) fin si   chaine[indice]=caractère_lu indice++ fin faire

---

De cette façon, t'es toujours sûr d'avoir assez de mémoire. Et en utilisant des tampons de taille "B" ça évite les realloc (lourds) à chaque fois que tu dois insérer un élément (surtout que tu peux, si tu le désires quand-même, positionner B à 1). Et en initialisant ton tableau à NULL et tes variables à 0, t'es sûr que t'auras un premier malloc avant toute insertion (le premier if est forcément vrai et quand le tableau initial est à NULL, realloc se comporte comme malloc).

Citation:

---

Envoyé par Sve@r

Ouch cte lacune. En partant comme ça tu vas droit au mur.
Pour gagner en rapidité, le C ne vérifie jamais rien, partant du principe que "le programmeur sait ce qu'il fait". Ce qui t'autorise donc, en terme de code, à écrire 15 int dans un tableau de 10. Les 5 autres iront écraser une autre variable.
Donc ça compilera, ça pourra même marcher un temps. Mais un jour ou l'autre... :aie::aie::aie:

C'est à toi de toujours t'assurer que ce que tu stockes a bien la place d'être stocké. Et que si tu tapes dans tab[n] alors n est bien inclus dans les limites du tableau.

---

Ouais, je sais ça, mais je suis toujours dans les bornes de ma variables, puisque i est forcément inférieur au nombre de caractère de data. Donc ça devrais forcément être bon.

Citation:

---

Envoyé par Sve@r

2 solutions
1) tu connais pas la taille mais tu peux la connaitre (par le biais d'une fonction d'info quelconque) donc tu fais char *chaine=malloc(n * sizeof(char))
2) tu connais pas la taille et tu ne peux pas la connaitre. Alors il te faut programmer un algo de ce genre

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

déclarer un char *chaine=NULL, un size_t taille_allouée=0 et un size_t indice=0 tant que lecture 1 caractère ok faire si indice=taille_allouée alors augmenter taille_allouée de B caractères chaine=realloc(chaine, taille_allouée * sizeof(char)) fin si   chaine[indice]=caractère_lu indice++ fin faire

---

De cette façon, t'es toujours sûr d'avoir assez de mémoire. Et en utilisant des tampons de taille "B" ça évite les realloc (lourds) à chaque fois que tu dois insérer un élément (surtout que tu peux, si tu le désires quand-même, positionner B à 1). Et en initialisant ton tableau à NULL et tes variables à 0, t'es sûr que t'auras un premier malloc avant toute insertion (le premier if est forcément vrai et quand le tableau initial est à NULL, realloc se comporte comme malloc).

---

Bon, ça ok, je sais faire. Mais est ce qu'il faut que je le fasse quand même pour mon char *data ?

Enjoy

The lsd

Bon, alors, après plusieurs tests, je suis heureux de vous apprendre que... rien n'a changé, ça ne fonctionne toujours pas. malloc ou pas, strncpy ou directement avec data[i].
Ca ne peut venir que de ma fonction substrlen. Quand je la change, pour mettre return 20 par exemple, tout va bien. Mais je vois pas d'erreur dans cette fonction. Quelqu'un en voit une ?

Enjoy

The lsd

Citation:

---

Quelqu'un en voit une ?

---

Une énorme, oui ! Tu ne vérifies pas si strstr retourne NULL ou pas dans ta fonction. Il ne faut jamais partir du principe que le caractère ou la sous-chaîne à rechercher sera forcément trouvé(e). Car sinon on court à la catastrophe... :lol:

~~~~~~~~~~

Code:

---

1 2 3

char *data = mon_header_http ; int i = substrlen (data, "\n"); strncpy (data+i, "\0", 1) ;

---

Pourquoi t'embêter à compliquer les choses alors qu'il y a plus simple ? Ca alourdit inutilement le code et les ressources. Et côté optimisation c'est pas terrible.

Tu peux faire quelque chose comme ça :

Code:

---

1 2 3 4

char *cherche_caractere ; cherche_caractere = strchr(mon_header_http , '\n'); if ( cherche_caractere != NULL ) *cherche_caractere = '\0' ;

---

Ca économise déjà un appel de fonction, le code est plus simple et plus lisible.
A partir de là, aucun bug possible, sauf si ton '\0' de fin de chaîne dépasse les limites de ta chaîne ou alors si mon_header_http pointe n'importe où (et là, on ne peut pas deviner, vu qu'on n'a pas le code). Donc, si strchr crashe ou si le programme a un comportement bizarre (style, si des variables changent de valeur sans raison apparente), tu sauras pourquoi... ;)

Bon, ok effectivement, vu sous cet angle, mon code est un peu compliqué. J'ai essayé ta solution, tout fonctionne correctement.

En fait, c'est moi qui ai été un peu bête ! strstr me renvoyait forcément un NULL, mais avec tout le flot de données, il passait inaperçu...:roll:

Je mets donc en résolu.

Merci à vous 3 !

Enjoy

The lsd