Discussion :

[cedric_c]

Bonjour,

j'ai un problème d'allocation de matrice. J'aimerai créer un matrice de taille variable dont les dimensions sont demandées au début du programme.
Ensuite j'aimerai pouvoir écrire des valeurs dans cette matrice. Je fais un simple test mais ca ne m'indique pas la valeur que j'ai donné.

Voici mon code :

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 <math.h> 
 
int main(int argc, char *argv[])
{
 
    double** Aj = NULL;
        int n;
        int p;
        int i;
 
        // On demande le nombre d'amis à l'utilisateur
    printf("Combien de ligne ? ");
    scanf("%d", &n);
        printf("Combien de colonne ? ");
    scanf("%d", &p);
 
        Aj=(double**)malloc(sizeof(double*)*n);
        for ( i=0 ; i<n ; i++)
                {
                        Aj[i]=(double*)malloc(sizeof(double)*p);
                }
 
        Aj[1][1]=2;
        printf("%ld \n", Aj[1][1]);
 
    free(Aj);
 
  system("PAUSE");      
  return 0;
 
}

Si quelqu'un sait me dire ce qui ne va pas, ca m'aiderait beaucoup

[Melem]

L'allocation me semble bonne, mais pour la libération free(Aj) est insuffisant. Il faut d'abord avoir libéré tous les Aj[i] avant de libérer Aj. Pour l'affichage, on utilise %f pour afficher un double (%lf pour saisir) et non %ld qui sert à afficher un long int. Autres remarques :

- argc et argv ne sont pas utilisés, tu peux ne pas les déclarer.
- Le cast de malloc est inutile en C. malloc retourne un void * qui est compatible avec tous les pointeurs.
- malloc peut échouer, dans ce cas elle retourne NULL. Il manque les tests d'erreur.
- system("PAUSE") n'est pas portable.

[cedric_c]

OK c'était simplement %lf au lieu du %ld.

Un grand merci ainsi que pour tes remarques, je vais corriger tout ca.