Discussion :

[burndev]

Bonjour,

Pour l'énoncé je ne dois pas utilisé la class vector

Je possède un pointeur de tableau d'objet de 4 éléments au départ, qui doit être redimensionné en fonction de la lecture d'une db.

j'ai une fonction membre Add qui me permet d'incrément une variable membre de l'objet a chaque passage de boucle(ou lecture de db).

la classe

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
 
class Utilisateur
{
    private:
    int _numutilisateur;
    int _compteur;
    string _nom;
 
    public:
    Utilisateur();
    ~Utilisateur(){}
    Utilisateur operator++();
 
    void setNom(const string value){_nom = value;}
    void setNumUtilisateur(int value){_numutilisateur = value;}
    void Add(){++_compteur;}
 
    int Count()const {return _compteur;}
    int GetNumUtilisateur()const{return _numutilisateur;}
 
    string GetNom()const{return _nom;}
 
};
 
Utilisateur::Utilisateur():_compteur(0){}
 
Utilisateur Utilisateur::operator++()
{
    //++_compteur;
    Utilisateur user;
    user.Add();
    return user;
}

le traitement

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
 
int main(){
 
    int TailleAllocation = 4, ElementsUtilise = 0;
    int ElementsMaximunAutorise = TailleAllocation;
 
    //Représente une table de base de données
    string elements[7][2] = {{"1","Nom 1"},{"2","Nom 2"},{"3","Nom 3"},{"4","Nom 4"},{"5","Nom 5"},{"6","Nom 6"},{"7","Nom 7"}};
 
    //Declare un pointeur de tableau d'objet de classe utilisateur
    Utilisateur *pUsers = new Utilisateur[TailleAllocation];
 
    //Nombre elements dimension 1
    int dim1 = sizeof(elements) / sizeof(*elements);
 
    //Nombre elements dimension 2
    //int dim2 = sizeof(*elements) / sizeof(*(elements[0]));
 
    //Lecture de la table de base données pour le test
    for(int i =0; i < dim1; i++)
    {
        int index = ElementsUtilise++;
 
        pUsers[index].setNumUtilisateur(toInt(elements[i][0]));
        pUsers[index].setNom(elements[i][1]);
 
        pUsers->Add();
 
        //Utilise la surcharge d'operateur
        //++pUsers;
 
        if(ElementsUtilise == ElementsMaximunAutorise)
        {
            //Declare un pointeur de tableau d'objet utilisateur
            Utilisateur *pTempo = new Utilisateur[ElementsMaximunAutorise+TailleAllocation];
 
            //Effectue une copie
            for(int IndiceCopie = 0; IndiceCopie < ElementsMaximunAutorise; IndiceCopie++)
            {
                pTempo[IndiceCopie] = pUsers[IndiceCopie];
            }
 
            delete[] pUsers; //Supprime le pointeur de tableau
            pUsers = pTempo;
            ElementsMaximunAutorise+=TailleAllocation;  //Nouvelle Taille
        }
    }
 
    //Affiche les données
    for(int i = 0; i < pUsers->Count(); i++)
    {
        cout << "[" << pUsers[i].GetNumUtilisateur() << "] [" << pUsers[i].GetNom() << "] " << &pUsers[i] << endl;
    }
 
    return 0;
}

jusqu ici tout cela fonctionne normalement.
Mais lorsque je cherche a surchargé un opérateur++ pour incrémenté, rien a faire cela ne fonctionne pas.

merci

[Luc Hermitte]

Pour réinventer le roue ?
Les vecteurs sont faits pour toi.

[Invité]

Pour réinventer le roue ?
Les vecteurs sont faits pour toi.

J'aurais voulu envoyer un MP à Luc Lermith pour lui expliquer mon profond désaccord avec sa réponse, mais sa boite de messages est pleine.
Donc, je confirme mon désaccord. Mon argumentation est à votre disposition. Tant pis si ça me vaut une exclusion, mais au moins j'aurais dit ce que je pensais.
Bonne soirée, cordialement.

[Davidbrcz]

J'aurais voulu envoyer un MP à Luc Lermith pour lui expliquer mon profond désaccord avec sa réponse, mais sa boite de messages est pleine.
Donc, je confirme mon désaccord. Mon argumentation est à votre disposition. Tant pis si ça me vaut une exclusion, mais au moins j'aurais dit ce que je pensais.
Bonne soirée, cordialement.

Tu peux aussi la rendre publique, ton avis nous intéresse. Les forums sont fait pour s'exprimer et accessoirement progresser.

[Flob90]

@Luc: Pour un vraie programme je suis d'accord, mais c'est peut-etre un exercice pédagogique. Dans ce cas, il DOIT peut-etre réinventer la roue.

@Pierre Dolez: Tu pourrais développez un peu ? Parce que réinventer la roue tout le temps me semble contre-productif au plus haut point.

[Luc Hermitte]

+1 à flo et David.
Effectivement, quand il s'agit d'un aspect pédagogique cela se tient. C'est un exercice intéressant et à faire au moins une fois. Mais cela ne reste que cela : un exercice.
Par défaut, j'assume toujours le cas "non exo", un peu par provocation, j'avoue. ^^'

[Après, je pense sincèrement qu'il est aberrant de reinventer la roue en production, surtout quand elle est standard et disponible, parce que: - nous ne sommes pas payés pour, cela sera toujours plus inefficace (j'ai eu l'occasion de réduire une exécution de plus de 24h (mes doigts finirent par glisser sur un kill -9) à 20 minutes à cause de ce fichu NIH qui avait foiré l'écriture de resize()), cela sera toujours moins robuste. Il reste des cas rares où l'on n'a pas le choix.]

<HS>
Quant à ma BAL, oui elle est pleine. J'ai fini par laisser tomber de la vider de choses que je tiens à garder. En général je renvoie un MP pour inciter à me contacter directement quand je reconnais le pseudo comme celui d'un habitué, ou de poster sur le forum quand c'est à teneur technique. Ici cela me parait suffisamment à teneur technique.
Les raisons sont multiples :
- d'autres que moi peuvent répondre, apporter des arguments en désaccord, ou juste une autre vision tandis que je serai offline.
- les forums ne doivent pas être en écriture seulement. Un réponse à un problème doit pouvoir profiter à ceux qui se souviennent de la netiquette : chercher une réponse avant de poser une question.
D'où ma signature.

PS: j'espère bien que ton opinion ne te vaudra aucun exclusion. Cela serait bien le comble.
PPS: Luc suffit amplement, sinon c'est "Hermitte" quand on y tient. Merci.
</HS>

[Luc Hermitte]

Pour l'OP, vu qu'il semble s'agir d'un exo.
Il y a trop de choses non réunies ensemble pour que j'ai la force de plonger dans le code. Je t'invite vivement à crée une classe à la vector.
- Elle fournira un accès [] à ses éléments,
- Elle saura combien d'éléments elle contient
- Son destructeur libèrera la mémoire,
- Elle supportera une fonction pour ajouter des nouveaux éléments. Cette fonction redimensionnera le tableau interne au besoin -- c'est à dire si tu distingue capacité et taille.

[burndev]

bonsoir,

Je confirme, cela est effectivement un exercice. Mais c'est pas grave.
La question général de l'exercice est développé un tableau d'objet de classe (utilisateur avec les membres fournis) unidimensionnel pouvant être redimensionné. l'exemple doit être alloué sur la pile et sur le tas et sans l'utilisation de vector,list et map.

j'ai pris la liberté de simulé l'extraction d'une db pour l'exercice. Cette exercice doit nous mener a la réalisation d'une class Tableau ultérieurement.

Pour ma part coté pile c'est ok. Mais coté tas lorsque je passe avec les pointeurs je ne comprend pas pourquoi la surcharge d'opérateur ne fonctionne pas.

merci

[Luc Hermitte]

o_O
Incrémenter un utilisateur n'a aucun sens.
Ajouter un utilisateur à une liste (ou une autre), oui.

[burndev]

peuuuuf, sacrée perte de temps à joué sur les mots !

donc l'incrémentation est uniquement pour comptés mes utilisateurs puisque mon tableau est redimensionnée et toujours dans l'optique de la réalisation de cette class tableau.

pourquoi compté les utilisateurs, je sais pas, y a cas demandé a l'intervenant du cours.

[Luc Hermitte]

Ce n'est pas aux utilisateurs de se compter. C'est à la liste qui les englobe d'en connaitre la quantité. JE ne joue pas sur les mots. Je distribue les responsabilités. C'est ce qui fait la différence entre un truc simple et qui marche et un truc compliqué que personne ne comprend comment ça marche.

A moins que ton prof t'ai explicitement demander d'utiliser la surcharge d'opérateur pour cela, oublie.