Discussion :

[pompy]

Bonjour
Je suis en train de coder un jeu de dames en C++
Le menu de base s'affiche, j'ai bien défini les zones cliquables (en tout cas, quand je teste avec un cout ça marche).
Mais quand je clique par exemple sur Start, normalement je devrais avoir une nouvelle partie, et j'ai un message d'erreur du type:

Code :

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

length < sizeInTChars

(je vous ai mis l'image en PJ)


Pièce jointe 168791
Voici la partie où j'utilise la fonction _itoa_s

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
void Game::DisplayInformation()
{
       //
       // Scoring
       char buffer[250];
       char t1[10];
       char t2[10];
       char t3[10];
       _itoa_s (TotalWhite,t1,10); //the number given by TotalWhite into a chain of caracters
       _itoa_s (TotalBlack,t2,10);
 
       strcpy_s(buffer,"White : ");
       strcat_s(buffer,t1);
 
    Text(10, HEIGHT+10, screen, buffer);
 
       strcpy_s(buffer,"Black : ");
       strcat_s(buffer,t2);
 
    Text(10, HEIGHT+30, screen, buffer);
 
       //Round: white or black?
       strcpy_s(buffer,"Round : ");
       strcat_s(buffer,(round == White ? "White" : "Black"));
 
    Text(WIDTH-320, HEIGHT+30, screen, buffer);
 
       //Chronometer
       if(round != Empty)
       {
             time (&end_t);
             int dif = (int)difftime (end,start);
             _itoa_s (dif,t3,10);
             strcpy_s(buffer,"Time : ");
             strcat_s(buffer,t3);
             strcat_s(buffer," s");
       }
       else
       {
             strcpy_s(buffer,"Time : ");
       }
 
    Text(WIDTH-160, HEIGHT+10, screen, buffer);
 
      //Display the type of the opponent
       bool vsCPU = getAgainstCPU();
       strcpy_s(buffer,"Opponent : ");
       strcat_s(buffer,(vsCPU == true ? "CPU" : "Player"));
    Text(WIDTH-160, HEIGHT+30, screen, buffer);
 
       //Winner?
       CheckEnd();
}

Est-ce que vous pouvez m'aider s'il vous plaît?
Merci

[Bousk]

Si tu faisais vraiment du C++ tu n'aurais pas ce code.
Tu as char t1[10]; et _itoa_s (TotalWhite,t1,11); déjà c'est un peu bizarre non ?
Et il me semblait que _iota_s prenait 4 paramètres.
Et on ne sait rien de TotalWhite ni de la valeur que tu espères.

[pompy]

Bonjour

Oui petit souci avec le 10, 11 j'ai corrigé cela merci
TotalWhite est un entier qui compte le nombre de pions noirs pris par les Blancs
Je souhaite afficher cet entier
Ah oui je viens de regarder mon fichier xtoa il me manque un paramètre

[fenkys]

Bonjour,

Ton code présente deux graves problèmes à mon sens :
- le premier, déja signalé et que ce n'est pas du C++, mais du C.
- le second est que les fonctions que tu utilises ne sont pas standard et ton programme n'est pas portable.
L'excuse de MS pour conseiller l'usage de strcpy_s à la place de strcpy est bidon, puisque la fonction strncpy fait exactement la même chose, mais en étant portable.

Mais en utilisant les fonctionnalités du C++ ce genre de problème ne se pose plus.

Cordialement.

[Médinoc]

L'excuse de MS pour conseiller l'usage de strcpy_s à la place de strcpy est bidon, puisque la fonction strncpy fait exactement la même chose, mais en étant portable.

C'est faux (strncpy n'insère pas de caractère nul si la chaîne est tronquée, pour des raisons historiques), mais ça reste une excuse bidon et inapproprié en C++.