A.exe a cesse de fonctionner

 <>▼ 

main()

int

v2

char

char

v1

char

char

char

v2

char

v2

v2

strcat

strcpy

• l7 :

  v1

  fait 7 caractères. Or tu stockes dans la longueur de

  v1

  (7 caractères) +

  v2

  soit 8 caractères et donc ça déborde.
• l5 :

  v1

  devrait en toute rigueur être un

  const char *

  , ce qui veut dire une chaîne non modifiable. Cela signifie que si tu veux concaténer

  v1

  et

  v2

  . Cela signifie qu'il faut créer un buffer suffisamment long (au moins 8 caractères à ce stade), dans lequel tu recopies

  v1

  puis auquel tu concatènes

  v2

  .
• l5 :

  v1

  est une chaîne de caractère constante (donc

  const char *

  ) or tu déclares cette variable comme un

  char

  .
• l8 : tu recopies une chaîne qui a priori fait la longueur de v (= la longueur de

  v1

  et

  v2

  ) dans var2, qui ne fait que 2 caractères !
• l9 : même erreur
• l12 : cet appel rend ton programme spécifique à Windows. C'est un peu dommage et inutile si ton programme est lancé depuis un terminal (e.g. celui de dev cpp).
• l13 : cette ligne pourrait être incorporé dans le 3e membre de la boucle

  for

  .

(mando@silk) (~) $ gcc toto.c
toto.c: In function ‘main’:
toto.c:5:47: warning: initialization of ‘char’ from ‘char *’ makes integer from pointer without a cast [-Wint-conversion]
    5 |     char var1[7] = "COLOR ", var2[2] ,v ,v1 = "0", v2, cmnd[8];
      |                                               ^~~
toto.c:6:12: warning: assignment to ‘char’ from ‘char *’ makes integer from pointer without a cast [-Wint-conversion]
    6 |     for(v2 = "0"; v2 < "f"; v2) {
      |            ^
toto.c:6:22: warning: comparison between pointer and integer
    6 |     for(v2 = "0"; v2 < "f"; v2) {
      |                      ^
toto.c:7:20: warning: passing argument 1 of ‘strcat’ makes pointer from integer without a cast [-Wint-conversion]
    7 |         v = strcat(v1, v2);
      |                    ^~
      |                    |
      |                    char
In file included from toto.c:2:
/usr/include/string.h:133:39: note: expected ‘char * restrict’ but argument is of type ‘char’
  133 | extern char *strcat (char *__restrict __dest, const char *__restrict __src)
      |                      ~~~~~~~~~~~~~~~~~^~~~~~
toto.c:7:24: warning: passing argument 2 of ‘strcat’ makes pointer from integer without a cast [-Wint-conversion]
    7 |         v = strcat(v1, v2);
      |                        ^~
      |                        |
      |                        char
In file included from toto.c:2:
/usr/include/string.h:133:70: note: expected ‘const char * restrict’ but argument is of type ‘char’
  133 | extern char *strcat (char *__restrict __dest, const char *__restrict __src)
      |                                               ~~~~~~~~~~~~~~~~~~~~~~~^~~~~
toto.c:7:11: warning: assignment to ‘char’ from ‘char *’ makes integer from pointer without a cast [-Wint-conversion]
    7 |         v = strcat(v1, v2);
      |           ^
toto.c:8:22: warning: passing argument 2 of ‘strcpy’ makes pointer from integer without a cast [-Wint-conversion]
    8 |         strcpy(var2, v);
      |                      ^
      |                      |
      |                      char
In file included from toto.c:2:
/usr/include/string.h:125:70: note: expected ‘const char * restrict’ but argument is of type ‘char’
  125 | extern char *strcpy (char *__restrict __dest, const char *__restrict __src)
      |                                               ~~~~~~~~~~~~~~~~~~~~~~~^~~~~
toto.c:10:9: warning: implicit declaration of function ‘system’ [-Wimplicit-function-declaration]
   10 |         system(cmnd);
      |

#include <stdio.h>

int main(){
    const char * var1 = "COLOR";
    const char v1 = '0';
    char cmnd[255];
    for (char c = '0'; c <= 'f'; c++) {
        snprintf(cmnd, sizeof(cmnd), "%s %c%c", var1, v1, c);
        printf("%s", cmnd);
        printf("\nBONJOUR\n");
    }   
    return 0;
}

• La boucle

  for

  énumère les caractères selon la table ASCII (voir

  man ascii

  ) et donc il n'y a pas que les caractères hexadécimaux, et donc utiliser un

  char

  est inadéquat.
• Beaucoup de variables sont inutiles, et surchargent inutilement le programme (

  var1

  ,

  v1

  ).

#include <stdio.h>

int main(){
    char buffer[255];
    for (unsigned i = 0; i < 16; i++) {
        snprintf(buffer, sizeof(buffer), "COLOR 0%x", i);
        printf("%s", buffer);
        printf("\nBONJOUR\n");
    }   
    return 0;
}

system(color)