Discussion :

[BStylene]

Bonjour
J'essais de copier deux tableau dans un même tableau, j'ai fait une allocation de mémoire pour les trois tableaux
Voici mon code source

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
55
int main(int argc, char *argv[])
{
    int N = 0, M = 0, i = 0, j = 0;
    int *tableau_A = NULL, *tableau_B = NULL, *tableau_C = NULL, taille = 0;
    tableau_A = malloc(N*sizeof(int));
    tableau_B = malloc(M*sizeof(int));
    tableau_A[N] = 0, tableau_B[M] = 0;
    printf("Entrez la dimension du tableau A : ");
    scanf("%d", &N);
    printf("Entrez la dimension du tableau B : ");
    scanf("%d", &M);
    taille = N+M;
    tableau_C = malloc(taille*sizeof(int));
    tableau_C[taille] = 0;
    for(i = 0; i < N; i++)
    {
        printf("Entrez les valeurs du tableau A : ");
        scanf("%d", &tableau_A[i]);
    }
    for(i = 0; i < N; i++)
    {
        printf("\n%d\n", tableau_A[i]); // Vérification des valeurs entrées dans B
 
    }
    for(i = 0; i < M; i++)
    {
        printf("Entrez les valeurs du tableau B : ");
        scanf("%d", &tableau_B[i]);
    }
    for(i = 0; i < M; i++)
    {
        printf("\n%d\n", tableau_B[i]); // Vérification des valeurs entrées dans B
    }
    for(i = 0; i < taille; i++)
    {
        if(i < N)
        tableau_C[i] = tableau_A[i];
        else if(i >= N && i < taille)
        {
        tableau_C[j] = tableau_B[i];
        j++;
        }
    }
    printf("Les valeurs du tableau C sont :\n");
    for(i = 0; i < taille; i++)
    {
        printf("%d\n", tableau_C[i]);
    }
    free(tableau_A);
    free(tableau_B);
    free(tableau_C);
 
    return 0;
 
}

A La fin l'orsqu'il devrait afficher les valeurs du tableau C, Il m'affiche d'autres valeurs différentes de celles tapez au clavier, des valeurs qui me semblent être les adresses des valeurs de tableau A et tableau B.
Merci d'avance.

[pythéas]

Bonjour,

Cette ligne n'est pas bonne :

Code :

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

tableau_C[j] = tableau_B[i];

lire la position "i" de "tableau_B" n'a pas de sens (il faut lire ce tableau de "0" à sa taille - 1), pareil pour le "j" es-tu sûr qu'il est utile ou qu'il faut réécrire dans ton tableau résultat en repartant de la position "0" (et donc en écrasant ce que tu as déjà mis?

Déjà comme ça, ça devrait mieux marcher.

Code :

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

tableau_C[i] = tableau_B[i-N];

[BStylene]

Bonjour,

Cette ligne n'est pas bonne :

Code :

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

tableau_C[j] = tableau_B[i];

lire la position "i" de "tableau_B" n'a pas de sens (il faut lire ce tableau de "0" à sa taille - 1), pareil pour le "j" es-tu sûr qu'il est utile ou qu'il faut réécrire dans ton tableau résultat en repartant de la position "0" (et donc en écrasant ce que tu as déjà mis?

Déjà comme ça, ça devrait mieux marcher.

Code :

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

tableau_C[i] = tableau_B[i-N];

Je vois maintenant d'ou vient mon erreur
En utilisant le "j" dans mon code, On a ça

Code :

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

1
2
tableau_C[i] = tableau_B[j];
j++;

qui resout aussi le probléme
Merci

[Ngork]

Il y a plusieurs soucis dans ton code.

En premier lieu, tu tentes d'allouer la mémoire nécessaire à tes tableaux avant d'avoir obtenu la dimension de ces tableaux !

Puis tu essaies de mettre à zéro le contenu d'une case située après la mémoire allouée !

Il y a aussi un test inutile sur i dans la boucle de copie (mais je l'ai laissé parce qu'il ne pose pas de problème), ainsi qu'une inversion de i et j aboutissant à une recopie incorrecte de B dans C (ainsi que l'a signalé Pytheas pendant que j'écrivais ma réponse).

Voici ton code très légèrement corrigé et qui fonctionne tel quel :

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
int main()
{
  int N = 0, M = 0, i = 0, j = 0;
  int *tableau_A = NULL, *tableau_B = NULL, *tableau_C = NULL, taille = 0;
 
  printf("Entrez la dimension du tableau A : ");
  scanf("%d", &N);
 
  printf("Entrez la dimension du tableau B : ");
  scanf("%d", &M);
 
  tableau_A = malloc(N*sizeof(int));
  tableau_B = malloc(M*sizeof(int));
 
  taille = N+M;
  tableau_C = malloc(taille*sizeof(int));
 
  for(i = 0; i < N; i++)
    {
      printf("Entrez les valeurs du tableau A : ");
      scanf("%d", &tableau_A[i]);
    }
 
  for(i = 0; i < N; i++)
    {
      printf("\n%d\n", tableau_A[i]); // Vérification des valeurs entrées dans B
 
    }
 
  for(i = 0; i < M; i++)
    {
      printf("Entrez les valeurs du tableau B : ");
      scanf("%d", &tableau_B[i]);
    }
 
  for(i = 0; i < M; i++)
    {
      printf("\n%d\n", tableau_B[i]); // Vérification des valeurs entrées dans B
    }
 
  for(i = 0, j = 0; i < taille; i++)
    {
      if(i < N)
        tableau_C[i] = tableau_A[i];
      else if(i >= N && i < taille) // test inutile
        {
          tableau_C[i] = tableau_B[j];
          j++;
        }
    }
 
  printf("Les valeurs du tableau C sont :\n");
  for(i = 0; i < taille; i++)
    {
      printf("%d\n", tableau_C[i]);
    }
 
  free(tableau_A);
  free(tableau_B);
  free(tableau_C);
 
  return 0;
}

[BStylene]

Il y a plusieurs soucis dans ton code.

En premier lieu, tu tentes d'allouer la mémoire nécessaire à tes tableaux avant d'avoir obtenu la dimension de ces tableaux !

Puis tu essaies de mettre à zéro le contenu d'une case située après la mémoire allouée !

Il y a aussi un test inutile sur i dans la boucle de copie (mais je l'ai laissé parce qu'il ne pose pas de problème), ainsi qu'une inversion de i et j aboutissant à une recopie incorrecte de B dans C (ainsi que l'a signalé Pytheas pendant que j'écrivais ma réponse).

Voici ton code très légèrement corrigé et qui fonctionne tel quel :

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
 
int main()
{
  int N = 0, M = 0, i = 0, j = 0;
  int *tableau_A = NULL, *tableau_B = NULL, *tableau_C = NULL, taille = 0;
 
  printf("Entrez la dimension du tableau A : ");
  scanf("%d", &N);
 
  printf("Entrez la dimension du tableau B : ");
  scanf("%d", &M);
 
  tableau_A = malloc(N*sizeof(int));
  tableau_B = malloc(M*sizeof(int));
 
  taille = N+M;
  tableau_C = malloc(taille*sizeof(int));
 
  for(i = 0; i < N; i++)
    {
      printf("Entrez les valeurs du tableau A : ");
      scanf("%d", &tableau_A[i]);
    }
 
  for(i = 0; i < N; i++)
    {
      printf("\n%d\n", tableau_A[i]); // Vérification des valeurs entrées dans B
 
    }
 
  for(i = 0; i < M; i++)
    {
      printf("Entrez les valeurs du tableau B : ");
      scanf("%d", &tableau_B[i]);
    }
 
  for(i = 0; i < M; i++)
    {
      printf("\n%d\n", tableau_B[i]); // Vérification des valeurs entrées dans B
    }
 
  for(i = 0, j = 0; i < taille; i++)
    {
      if(i < N)
        tableau_C[i] = tableau_A[i];
      else if(i >= N && i < taille) // test inutile
        {
          tableau_C[i] = tableau_B[j];
          j++;
        }
    }
 
  printf("Les valeurs du tableau C sont :\n");
  for(i = 0; i < taille; i++)
    {
      printf("%d\n", tableau_C[i]);
    }
 
  free(tableau_A);
  free(tableau_B);
  free(tableau_C);
 
  return 0;
}

Merci, mais pourquoi avoir inclut string.h ?

[Ngork]

Merci, mais pourquoi avoir inclut string.h ?

C'est juste que tu n'avais pas mis les include dans ton code, alors j'ai mis mon en-tête minimale habituelle pour compiler, mais tu as raison, le string.h n'est pas nécessaire dans ce code.

Si les corrections qui te sont proposées te conviennent, c'est sympa de mettre un +1, et si cela résoud ton problème, n'oublie pas de marquer le sujet comme résolu !