Modification d'un caractère dans un tableau

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Modification d'un caractère dans un tableau

smurf4xp

Bonjour,

je suis en train de faire un programme pour un de mes cours et ca fait quelques heures maintenant que j'ai la même erreur sans pouvoir la résoudre...

j'obtiens une erreur de segmentation lorsque j'essaie d'assigner une valeur dans le tableau. (ligne 54)

Code :

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <string.h>
/*
*
* Programme visant à générer toutes les possibilités
* de mot de 8 caractères contenant des lettres majuscules
* et/ou minuscules et/ou des chiffres.
*
*/
char *NextOne(char word[]);
char AddOneChar(char tobemod);
int main()
{
char * temp = "00000008";
int i = 0;
for(i=0;i<100;i++)
{
printf("temp = %s",temp);
temp = (char *)NextOne(temp);
}
return 0;
}
/*
*
* Cette méthode renvoie la combinaison de mot de passe suivante.
*
* @PRE : word est un mot de 8 caractères
* @POST : word est la combinaison suivante du mot de 8 caractères.
* l'ordre est le suivant :
* 00000000->00000001->...->0000000z->00000010->...
*
*/
char *NextOne(char word[])
{
char c;
int i = 7;
while(i!=-1)
{
if (word[i]==122)
{
word[i] = '0';
i--;
}
else
{
word[i] = AddOneChar(word[i]); <---- l'erreur est ici
i=7;
break;
}
}
printf("%s\n",word);
return word;
}
/*
*
* Cette méthode renvoie le caractère suivant dans cette
* ordre : d'abord les chiffres de 0 à 9 puis les lettres
* majuscules de A à Z puis enfin les lettres minuscules
* de a à z.
*
* @PRE : tobemod est un caractère de la série [a-zA-Z0-0./]
* @POST : tobemod est le caractère suivant
*
*/
char AddOneChar(char tobemod)
{
switch (tobemod)
{
case '9' : return 'A';
case 'Z' : return 'a';
default : return tobemod+1;
}
}

Je rame à mort, j'espere que quelqu'un d'entre vous pourra m'aider.

Merci d'avance pour toute aide!

Message cité 1 fois

Publicité

tpierron

Mouais, classique. Tu modifies une chaine statique. Les compilos marque la zone occupée par ces chaines en read only. En général, ils en profitent pour optimiser en réutilisant les chaines similaires. Du coup si tu modifiais une chaine, ça pourrait avoir des répercutions dans d'autre partie du code : l'horreur totale.

Il suffit de changer ta ligne 20 en :

Code :

char temp[] = "00000008";

Edit: rhôôooo, le serveur avait mis plein de fotes de grammaires, orthos, typos, ... dans mon post !!

Message édité par tpierron le 16-08-2007 à 20:38:11

smurf4xp

Merci de m'avoir répondu aussi vite,

c'est clair que c'est le genre d'erreur classique en c, c'est pas encore trop la joie avec les pointeurs...

mais bon, maintenant il ne compile plus...

il me dit : c:25: error: incompatible types in assignment

j'ai enlever le (char *) mais rien n'y fait...je suis perdu! :cry:

smurf4xp

c'est bon, ca marche plus ou moins

j'ai changé la ligne 25 par
*temp = NextOne(temp);

maintenant ça compile et ca marche,

mais le problème c'est que j'ai besoin d'appeller la methode NextOne() avec un char * comme arguments et pas un char [] et il me fait à nouveau l'erreur de segmentation... :pfff:

SquiZZ

je ne comprends pas pourquoi ton NextOne retourne quelque chose.
Surtout que ce que tu retournes c'est l'adresse du tableau que tu lui passes en paramètre.
Si tu le déclares en

Code :

void NextOne(char word[])

et que tu retournes rien ça doit bien suffire

smurf4xp

segmentation fault à nouveau...

J'aime beaucoup le c, mais la ya vraiment des jours... incapable de changer un caractère dans un tableau ,c'est quand même la gène...

en tout cas merci pour ta réponse SquiZZ.

J'ai modifié ma methode en void, mais ca ne change rien,

char * temp = ... pose tjrs des problèmes.

Quelqu'n aurait une solution?

SquiZZ

uh, t'as fait ce qu'a dit tpierron ?

smurf4xp

Oui j'ai fait ce qu'il a dit(et ca a marché),
mais le problème c'est que le bout de code que j'ai montré fait parti d'un programme 10fois plus gros,
ca marchait bien au début mais maintenant j'ai cette erreur de segmentation, j'ai pas fait de sauvegarde assez souvent (shame on me)
et je me retrouve bloqué, si je commence à changer la variable à passer en argument ,je devrais changer tout le programme...

donc je me retrouve toujours avec le même problème de base:

- Comment modifier un élement d'un string définit par : char * temp = "TotoLeHeros"?

j'en deviens dingue :pt1cable:

Message cité 1 fois

Deadog

Dain Bramaged

smurf4xp a écrit :

- Comment modifier un élement d'un string définit par : char * temp = "TotoLeHeros"?

on ne peut pas
"TotoLeHeros" est une chaine constante

en faisant ça, tu ne fait que dire : "le pointeur temp pointe sur la chaine constante (donc immuable) "TotoLeHeros"", ce qui ne te permet pas de modifier la chaine puisqu'elle est constante

par contre tu peux faire :

Code :

char *temp = strdup ("TotoLeHeros" );

mais une fois que tu en auras plus besoin il ne faudrat pas que tu oublie de faire

Code :

free(temp);

sinon tu fais simplement

Code :

char temp[] = "TotoLeHeros";

la tu peux la modifier, mais ça t'oblige à changer tout ton programme [:spamafote]

Message édité par Deadog le 17-08-2007 à 02:04:03

smurf4xp

merci pour ta réponse deadog!

tu as raison en declarant comme ceci

Code :

char *temp = "TotoLeHeros" ;

je ne peux plus changer le string... (est cela qu'on appelle une lvalue?)

J'ai finalement pu résoudre mon problème hier (ou ce matin :sweat: ?)

pour cela j'ai fait comme suit:

Code :

char * temp = malloc(sizeof(char)*9)
strncpy(temp,"totoleheros",8);

ce qui resolvait mon problème, mais je vais retenir ta solution (je ne connaissais pas strdup).

Je remercie tout ceux qui m'ont aidé, c'est franchement sympa ce que vous faites.

Continuez comme ca ! :hello: :jap:

Publicité

Deadog

Dain Bramaged

hum, strdup te fait le malloc + le strcpy

ce qui n'est pas plus mal parce que dans ce que tu as fais tu prend un gros risque : tu paris sur le faite que le bout de mémoire alloué par malloc contient initiallement que des 0, ce qui n'est pas toujours le cas.

malloc tu donne un bout de mémoire sans la remettre à zero, donc tu peux très bien obtenir ce qu'il y avant, c'est à dire tout et n'importe quoi, et en particulier pas forcément des 0
ton strncpy s'arrête à 8 et ne met donc pas le 0 final

tiré de la page man de strncpy :

Citation :

La fonction strncpy() est identique, sauf que seuls les n premiers octets de src sont copiés. Ainsi, s’il n’y a pas de caractère nul dans les n premiers octets de src, la chaîne résultante ne disposera pas de caractère nul final.

Chez toi ça a marché uniquement parce que tu as eu la chance que malloc te retourne un bout de mémoire avec que des 0.
Pour éviter ça tu peux utiliser strdup dans ce cas particulier, et plus générallement tu peux utiliser calloc :

Code :

char * temp = calloc(9, sizeof(char));
strncpy(temp,"totoleheros",8);

calloc t'alloue une zone mémoire et te la met à zéro

une autre méthode et de faire ça à chaque fois que tu utilise strncpy :

Code :

char * temp = malloc(sizeof(char)*9);
strncpy(temp,"totoleheros",8);
temp[8] = 0;

Message édité par Deadog le 17-08-2007 à 13:35:29

Emmanuel Delahaye

C is a sharp tool

smurf4xp a écrit :

je suis en train de faire un programme pour un de mes cours et ca fait quelques heures maintenant que j'ai la même erreur sans pouvoir la résoudre...

j'obtiens une erreur de segmentation lorsque j'essaie d'assigner une valeur dans le tableau. (ligne 54)

Il suffit d'utiliser une chaine modifiable :

Code :

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <string.h>
/*
*
* Programme visant à générer toutes les possibilités
* de mot de 8 caractères contenant des lettres majuscules
* et/ou minuscules et/ou des chiffres.
*
*/
/*
*
* Cette méthode renvoie le caractère suivant dans cette
* ordre : d'abord les chiffres de 0 à 9 puis les lettres
* majuscules de A à Z puis enfin les lettres minuscules
* de a à z.
*
* @PRE : tobemod est un caractère de la série [a-zA-Z0-0./]
* @POST : tobemod est le caractère suivant
*
*/
char AddOneChar (char tobemod)
{
switch (tobemod)
{
case '9':
return 'A';
case 'Z':
return 'a';
default:
return tobemod + 1;
}
}
/*
*
* Cette méthode renvoie la combinaison de mot de passe suivante.
*
* @PRE : word est un mot de 8 caractères
* @POST : word est la combinaison suivante du mot de 8 caractères.
* l'ordre est le suivant :
* 00000000->00000001->...->0000000z->00000010->...
*
*/
char *NextOne (char word[])
{
int i = 7;
while (i != -1)
{
if (word[i] == 122)
{
word[i] = '0';
i--;
}
else
{
word[i] = AddOneChar (word[i]);
i = 7;
break;
}
}
printf ("%s\n", word);
return word;
}
int main (void)
{
char temp[] = "00000008";
int i = 0;
for (i = 0; i < 100; i++)
{
printf ("temp = %s", temp);
NextOne (temp);
}
return 0;
}

Message édité par Emmanuel Delahaye le 17-08-2007 à 14:00:13

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

FORUM HardWare.fr

Programmation

Modification d'un caractère dans un tableau

Sujets relatifs
[Resolu] Interdire saisie de caractère	Supprimer une ligne contenant une chaine de caractère
utiliser le tableau super-global $_SESSION	Macro pour supprimer des lignes d'un tableau à partir de valeur
[batch] modifier une ligne d'un texte (doubler un caractere)	Smarty et tableau Mysql
Tableau dynamiquement redimensionné	Macro Excel date et tableau croisé dynamique
Tri de réponses chiffre/caractère [resolu]	Recherche script de modification et sauvegarde d'image
Plus de sujets relatifs à : Modification d'un caractère dans un tableau

Page générée en 0.097 secondes