Copier deux tableaux dans un même tableau

Version imprimable

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:

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.

05/02/2013, 17h07
pythéas

Bonjour,

Cette ligne n'est pas bonne :

Code:

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:

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

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 ! 8O

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

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:

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

07/02/2013, 10h57
BStylene
Citation:

Envoyé par pythéas

Bonjour,

Cette ligne n'est pas bonne :

Code:

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:

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:

1 2 tableau_C[i] = tableau_B[j]; j++;
qui resout aussi le probléme
Merci

Citation:

Envoyé par Ngork

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

07/02/2013, 14h35
Ngork

Citation:

Envoyé par BStylene

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

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 ! :ccool: