L'allocation mémoire ne marche pas bien

**geeka** · 20/01/2016, 22h29

Bonjour,
J'aimerais allouer mon fichier dans une matrice. J'ai essayé de faire ça mais le résulta est que ça m'affiche à partir de la 2ème ligne du fichier et après ça bug.
Pourquoi ?
Voici le 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
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
/* Fichiers d'en-tête */
#include <stdlib.h>    // Génération nbres aléatoires, allocation dynamique mémoire
#include <stdio.h>    //  Gestion erreurs et E/S (scanf, printf)
#include <math.h>	 //   Fonctions mathématiques
#include <string.h>
 
#define TAILLE_MAX  4700
 
// 1ère étape : Chargement du fichier IRIS DATA ==> pas OK
int main()
{
    /* Descripteur fichier */
    FILE* fichier = NULL;
 
    /* Ouverture fichier */
    fichier = fopen("iris_data.txt", "r+");
 
    if (fichier != NULL)
    {
        char chaine[TAILLE_MAX] = ""; // Chaîne vide de taille TAILLE_MAX
 
        while(fgets(chaine, TAILLE_MAX, fichier)!= NULL)  // On lit maximum TAILLE_MAX caractères du fichier, on stocke le tout dans "chaine"
        {
            // On peut lire et écrire dans le fichier
            printf("Le fichier IRIS DATA est ouvert \n");
 
            /*----------------- Etape 2 : Allocation mémoire--------------------*/
            /* Allocation mémoire d'un tableau de 100 valeurs flottantes */
            double** vect;
            if  ( ( vect =malloc(100*sizeof(**vect) ))== NULL)
            {
                printf("Il n'y a pas assez de mémoire !\n ");
                exit(EXIT_FAILURE);
            }
            printf("La memoire est allouee! \n");
            //exit(EXIT_SUCCESS);
 
 
 
            // indice
            int i=0;
            for (i=0; i<=149; i++)
            {
 
                printf("vecteur numero %d \n",i);
                printf("----------------------------------------\n");
 
                fscanf(fichier, "%lf ,%lf ,%lf ,%lf", &vect[i][0], &vect[i][1], &vect[i][2], &vect[i][3]);
                printf("Le vecteur de donnees %d est : %f,%f,%f,%f \n",i, vect[i][0], vect[i][1], vect[i][2], vect[i][3]);
            }
        }
    }
    else
    {
        // On affiche un message d'erreur si on veut
        printf("Impossible d'ouvrir le fichier iris_data.txt");
    }
 
    return 0;
}

Merci

**Matt_Houston** · 20/01/2016, 22h53

Code :

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

1
2
double** vect;
vect = malloc(100 * sizeof(**vect));

Tu as un niveau d'indirection de trop, double * suffit. D'autre part, écris plutôt sizeof(double) dans l'appel à malloc c'est plus clair.

**geeka** · 20/01/2016, 23h11

Merci pour votre réponse.
si je mets seulement double*, j'ai une erreur sur la ligne où j'ai vect[i][0], ça doit être une matrice.
sizeof(double) marche bien, merci!

Je sais pourquoi ça n'affiche pas la première ligne.
En fait, j'ai utilisé fscanf au lieu de sscanf vu que j'ai utilisé avant la fonction fgets. Du coup, j'ai la première ligne puis encore la première ligne qui s'est répétée puis ça bug

**kolodz** · 20/01/2016, 23h48

Si c'est une matrice, celle-ci devrait être déclaré entant que tel.

Code :

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

double [100][100]vect;

Comme le présente la FAQ C :
Comment déclarer et utiliser un tableau « à plusieurs dimensions » ?

Cordialement,
Patrick Kolodziejczyk.

**geeka** · 21/01/2016, 00h07

Merci kolotz. En fait, la matrice doit être mise dans un pointeur.

**kolodz** · 21/01/2016, 01h26

Cela ne change rien sur le principe de la déclaration.

**foetus** · 21/01/2016, 01h42

Envoyé par kolodz

Cela ne change rien sur le principe de la déclaration.

Mais sur l'initialisation si

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
double** vect;
// Here 100 == numbers of rows
vect = (double**) malloc(100 * sizeof(double*));
 
if (vect == NULL) { /*error*/ }
 
for (int i = 0; i < 100; ++i) {
    // Here 100 == numbers of columns
    vect[i] = (double*) malloc(100 * sizeof(double));
 
    if (vect[i] == NULL) { /*error*/ }
}
 
 
// Release vect
for (int i = 0; i < 100; ++i) { free(vect[i]); vect[i] = NULL;  }
 
free(vect);
vect = NULL;

Édit: Merci Bktero

, je me suis laissé influencer par kolodz et sa déclaration statique

**Bktero** · 21/01/2016, 08h02

Code :

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

1
2
3
double vect[100][100]; // or double** vect;
// Here 100 == numbers of rows
vect = (double**) malloc(100 * sizeof(double*));

Pardon ?

EDIT : le code a été corrigé dans le message précédent. Notez que la seconde ligne génère une erreur de compilation.

**Matt_Houston** · 21/01/2016, 10h17

Kaboom !

Envoyé par geeka

Merci pour votre réponse.
si je mets seulement double*, j'ai une erreur sur la ligne où j'ai vect[i][0], ça doit être une matrice.

La représentation mémoire t'est imposée ? Parce qu'un tableau à une seule dimension suffit pour stocker une matrice, hein.

**kolodz** · 21/01/2016, 10h45

Bonjour,

Je ne comprends pas bien votre besoin de déclarer en dynamique une matrice...

Depuis quand le fait de récupérer le pointeur d'une variable nécessite une déclaration dynamique ?
Pourquoi vous ne faites pas une déclaration statique propre, puis une récupération du pointeur avec &vect ?
Pour ma part, votre déclaration double** ne peut qu'amener des problèmes. Sans parler du fait qu'une indirection ne change pas le type final de la structure. Dans ce cas, c'est un double. Certes, c'est le pointeur du pointeur d'un double, mais cela n'en fait pas une matrice...

Cordialement,
Patrick Kolodziejczyk.

**Médinoc** · 21/01/2016, 11h17

Finalement, la question importante, c'est: Les dimensions sont-elles connues à la compilation?

Car dans le cas contraire, il faut alors un tableau dynamique de double à deux dimensions (exemple).

**geeka** · 21/01/2016, 11h42

Mon besoin c'est d'allouer les valeurs d'un fichier dans une matrice, si je n'utilise pas de pointeurs je n'aurais pas assez de mémoire pour tout afficher à la fois. Quelle serait la solution ?
Les dimensions sont bien connues à l'avance 150 lignes et 5 colonnes, que je dois formater.

**Bktero** · 21/01/2016, 11h47

si je n'utilise pas de pointeurs je n'aurais pas assez de mémoire pour tout afficher à la fois.

Pourquoi dis-tu cela ?

**Médinoc** · 21/01/2016, 11h51

Ah, problème de pile contre tas. Qu'à cela ne tienne, on peut avoir une matrice de taille statique sur le tas.

Un des moyens les plus simples étant de mettre ça dans une structure:

Code C :

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

1
2
3
4
5
6
7
8
struct maMatrice
{
	double valeurs[150][5];
};
 
...
 
struct maMatrice * pMat = malloc(sizeof *pMat);

**geeka** · 21/01/2016, 15h08

Merci
@Bktero : car j'ai testé et les données sont coupées à l'affichage, toutefois, je peux afficher ces vecteurs un à un, mais j'ai besoin de les afficher en même temps.

@Médinoc:
Comment je pourrais utiliser avec votre code, lequels des pMat, maMatrice ou valeurs peut remplacer vect ?

Code :

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

1
2
3
4
5
6
7
8
9
for (i=0; i<=149; i++)
            {
 
                printf("vecteur numero %d \n",i);
                printf("----------------------------------------\n");
 
                fscanf(fichier, "%lf ,%lf ,%lf ,%lf", &vect[i][0], &vect[i][1], &vect[i][2], &vect[i][3]);
                printf("Le vecteur de donnees %d est : %f,%f,%f,%f \n",i, vect[i][0], vect[i][1], vect[i][2], vect[i][3]);
            }

**Médinoc** · 21/01/2016, 16h38

Il s'agit de pMat->valeurs.

L'allocation mémoire ne marche pas bien

C

Discussions similaires

Partager

Partager