TABLEAU AVEC ALLOCATION DYNAMIQUE

Commentaire de jonathanmcdougall le 03/03/2003 15:00:41

>#include<iostream.h>

Déprécié, préferer <iostream>

# include <iostream>
using namespace std;

>void saisie(int,int);
>void affich(int,int);

Correct, mais il serait meilleur de spécifier le nom des variables, pour une meilleure lisibilité.

>int *palloc = new int;
>int taille;

Variables globales : à éviter.  D'ailleurs, tu n'en as pas besoin.

>void main()

int main()

main() retourne *toujours* un int.

>{
>    cout<<"Donner la taille du tableau:";
>    cin>>taille;

>    saisie(taille,*palloc);
>    affich(taille,*palloc);

pourquoi passes-tu les tableaux si tu ne t'en sers pas?  De plus, l'argument s'appelle 'adr' ou 'adre', pour 'adresse' je suppose, et tu lui passes la valeur de palloc[0] (en lui passant *palloc).  De plus, si 'adr' ou 'adre' doivent contenir une adresse, ils devraient être de type int*, non?

>    delete[] palloc;
>}

Quelques questions : que se passerait-il en cas d'erreur dans saisie() ou affich()?  Par exemple, si saisie() lance une exception?

Ensuite, que se passerait-il si l'utilisateur entrait un nombre négatif?  Si 'cin' plantait? Si 'new' manquait de mémoire?

Question de faire aller les méninges un peu :)

>void saisie(int tail,int adr)
>{

Comme j'ai dis plus haut, tu ne te sers pas de 'adr'.  Il serait plus avantageux de mettre tes variables globales locales au main() et de les passer en argument dans ces deux fonctions.

>    int i;
>    palloc=new int[taille];
>    for(i=0;i<taille;i++)

Deux choses : il est mieux de définir 'i' dans le for puisque 1) cela améliore la visibilité et 2) cela permet de restreindre la durée de vie de 'i' à la boucle.  Deuxièmement, préférer ++i à i++ puisque ce dernier requiert une copie et, donc, plus de temps d'exécution.

>    {
>        cout<<"saisissez un nombre:";
>        cin>>*(palloc+i);

Mauvais.  Pourquoi ne pas utiliser palloc[i]?

>    }
>}

>void affich(int taile, int adre)

Même chose concernant 'adre'.

>{
>    int j;
>    for(j=0;j<taille;j++)

Même chose concernant la définition et l'incrémentation de 'j'.

>    {
>        cout<<"tab["<<j<<"]="<<*(palloc+j)<<",
";

Mauvais aussi, préférer palloc[j].

>    }
>}

C'est bon, la seul source de problèmes est l'usage de variables globales. Là où l'on fait

int variable_globale;

void f()
{
std::cout << variable_globale;
}

int main()
{
variable_globale = 20;
f();
}

il faut préférer

void f(int variable_locale)
{
std::cout << variable_locale;
}

int main()
{
int variable_locale = 20;
f(variable_locale);
}

Mais la structure est très intéressante et très efficiente : une fonction, une tâche.  Pour ce court programme, c'est très bien, mais il serait encore mieux d'ajouter un

void* allouer(int* adr, int taille);

qui s'occupera du new et en retournera l'adresse.  Ainsi, il sera plus facile d'ajouter, par exemple, des tests sur la taille, la création d'objets...

Finalement, il serait bon d'avoir un peu plus de constance dans les noms : 'taile' et 'tail', 'adr' et 'adre'.

Amuses-toi bien!

Jonathan Mcdougall
Montréal, Québec
mcdougalljonathan@hotmail.com
http://www.multimania.com/utopiasoftware

Commentaire de leneuf22 le 04/03/2003 02:20:45

    int *palloc = new int;


Pourquoi tu l'alloues alors que tu vas l'allouer juste après ?
vu que tu fais 2 fois new sur le même pointeur, ya 4 octets qui se sont enfuis...

Commentaire de Fabi1 le 05/03/2003 21:45:06

en ce qui concerne les noms de variables e et  
>cin>>*(adr+i);
>   cout<<"tab["<<j<<"]="<<*(adre+j)<<", ";

c est une consigne qu on m a donné!!!


merci pour le reste!!