Discussion :

[wakka123]

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 :

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
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

[Neckara]

Bonjour,

Code :

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

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 )

[wakka123]

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

[Neckara]

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?

[Ngork]

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 :

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
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 ...

[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 :

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

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

merci quand même bonne journée