erreur strcpy à l'execution

bonjour.

Voici un bout de ma fonction, je ne comprends pas pourquoi ensuite ca plante a l'execution.
la compilation s'effectue correctement
le programme plante à la ligne 15 : "Exception non gérée à 0x6da513af (msvcr100d.dll) dans mon_programme.exe*: 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0x40518000."

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

void(char number_played[]) { char lottery[4]; char dest[10]; int good;   printf("\n\n ENTER 3 LAST NUMBERS OF LOTTERY\n\n"); scanf("%s%*c",&lottery); strcpy(dest,lottery+1); // j'essaie ici de copier dans dest les 2 derniers chiffres de lottery en donnant dans la source l'addresse printf("\n%s",dest); // a l'execution, il affiche bien les 2 derniers chiffres   if (strlen(number_played)==2) { good=strcmp(number_played,dest); if good=0 { ... instructions; } } }

---

merci d'avance

Bonjour,

Code:

---

1 2	scanf("%s%*c",lottery);

---

lottery est un tableau, il est donc implicitement convertit en un pointeur sur le premier élément, il n'y a donc pas besoin d'utiliser l'opérateur '&'.

Je te conseille aussi de lire le man de scanf et de fgets.

En effet, tu ne vérifies pas le code de retour de scanf et tu ne gère pas le cas où la chaine de caractère entrée est trop grande pour être stocké dans lottery.

De plus dans ce cas là, on utiliserait de préférence fgets ( /!\ gets est à bannir, toujours préférer fgets à la place )

bonsoir, dans un premier temps je pars du principe que l'utilisateur n'entre pas de chaîne trop longue donc pas de soucis à ce niveau, j'ai enlevé le symbole & mais j'ai toujours mon erreur ! un petit coup de pouce supp required :)

Pourrais-tu nous donner un code minimal reproduisant ton erreur pour voir ce qu'il se passe?

Sinon, pourquoi n'utiliserais-tu pas un débogueur pour comprendre ce que fait réellement ta fonction?

Voici ton code très légèrement corrigé, d'une part selon la correction proposée par Neckara, d'autre part une correction complémentaire sur le test de la valeur de good, qui n'est pas bon, et quelques lignes pour illustrer le fonctionnement : :D

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33

#include <stdio.h> #include <stdlib.h> #include <string.h>   void test(char number_played[]) { char lottery[4]; char dest[10]; int good;   printf("\n\n ENTER 3 LAST NUMBERS OF LOTTERY\n\n"); scanf("%s%*c",lottery); strcpy(dest,lottery+1); // j'essaie ici de copier dans dest les 2 derniers chiffres de lottery en donnant dans la source l'addresse printf("\n%s",dest); // a l'execution, il affiche bien les 2 derniers chiffres   if(strlen(number_played)==2) { good=strcmp(number_played,dest); if (good==0) { printf(" -> Bravo !\n"); } } }   int main() { test("69");   getchar();   return 0; }

---

Je n'ai rien modifié d'autre et ce code fonctionne (je l'ai testé à l'instant) mais, comme Neckara, je te suggère de revoir les fonctions d'entrée-sortie et de contrôler et encadrer les valeurs de retour comme d'entrée ... ;)

merci pour l'aide, après avoir passer au peigne fin tout le code, l'erreur n'était en fait pas du tout ou je la cherchais...

elle s'était glisser dans les instructions

j'avais quelque chose du type

Code:

---

printf("%s%s%s",lottery,lottery); // oubli d'une variable en argument

---

merci quand même ;) bonne journée

Citation:

---

Envoyé par wakka123

merci pour l'aide, après avoir passer au peigne fin tout le code, l'erreur n'était en fait pas du tout ou je la cherchais...

elle s'était glisser dans les instructions

j'avais quelque chose du type

Code:

---

printf("%s%s%s",lottery,lottery); // oubli d'une variable en argument

---

---

Comportement indéterminé. Ca peut faire n'importe quoi y compris très bien fonctionner durant des années tout comme faire planter 600 lignes plus bas sur une instruction tout à fait correcte comme strcpy()...