Discussion :

[Masstx]

Bonjour à ceux qui verront ce message,

je suis étudiant en licence 2 de mathématique-informatique et j'ai un exercice assez simple qui me bloque.
J'étais convaicue que ce que j'avais écrit étais correct mais lorsque je m'évalue il me met 0/100.
La consigne de l'exercice est la suivante :

"Définir la fonction createArray prenant en paramètre un entier capacity. La fonction allouera dynamiquement une variable de type dynIntArray ne contenant aucun élément mais pouvant stocker sans réallocation jusqu’à capacity entiers (autrement dit, le tableau tab devra contenir capacity cases). La fonction renverra l'adresse de la zone mémoire nouvellement allouée."

Voici mon code :

Code :

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

1
2
3
4
5
6
7
8
9
 
#ifndef _DYNINTARRAY
#define _DYNINTARRAY
 
typedef struct{
    int * tab;
    unsigned int capacity;
    unsigned int size;
} dynIntArray;

ma fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
dynIntArray* createArray(unsigned int capacity){
 
    if (capacity == 0){
      return NULL;
    }
    dynIntArray* dyn = malloc(sizeof(dynIntArray));
    if (dyn == NULL){
        fprintf(stderr,"Not enough memory");
        exit(EXIT_FAILURE);
    }
    dyn->tab = malloc(capacity * sizeof(int));
    return dyn ;
}

Je vous remercie d'avance de l'aide de ceux qui verront cette discussion

[foetus]

Le seul truc que je vois c'est que ta fonction fait 1 travail incomplet - indice : dynIntArray est 1 structure.

Tu as le type size_t pour 1 unsigned long/ int : size_t, documentation cplusplus.com en anglais

[gerald3d]

Bonjour.

Tu alloues une zone mémoire pour *Tab. Mais tu n'initialises pas capacity et size de la structure ! Cela te semble-t-il normal ?

[dalfab]

Bonjour,

Le problème le plus important t'a été doublement signalé.

J'ajouterais que les lignes 4 à 6 sont de trop. Tu considères qu'un tableau ne devrait pas avoir une capacité initiale nulle en refusant de créer le tableau. Pourtant ça n'est pas un problème et c'est même fréquent de débuter avec cette condition. C'est la fonction resizeArray() qui devra adapter cette capacité en fonction de la taille demandée.

[Sve@r]

Bonjour

En plus du tout ce qui a été dit, il reste les soucis suivants

• tu ne vérifies pas la réussite de l'allocation à dyn->tab (or si le premier malloc peut échouer, le second peut échouer tout aussi bien)
• si le malloc échoue tu quittes le programme. Si je me mets du point de vue appelant, j'aimerais bien que si la fonction que tu m'offres ne peut pas faire son job elle ne me plante pas mon code pour autant
  De fait, on ne quitte jamais le programme si la fonction échoue mais on lui fait retourner une valeur particulière pour que l'appelant puisse en avoir connassance et prendre une décision.