Discussion :

[genetin]

Bonjour,

Je voudrais créer une classe héritée de QDate, en ajoutant l'information "Matin / Après-midi".

Code :

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

1
2
3
4
5
6
7
 
class DateEvenement : public QDate
{
public:
    void RecalculeDate ();
    bool matin;
};

La fonction RecalculeDate() doit recalculer la nouvelle date de l’événement, mais je ne sais pas comment modifier les valeurs (jour, mois, année) héritées de QDate. J'ai essayé ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
void DateEvenement::RecalculeDate ()
{
   QDate dateTmp;
   dateTmp = ..... /* calcule la nouvelle date */
   if (.....) /* si la date doit être modifiée */
   {
      setDate (dateTmp);
      matin = ..... /* calcule si c'est le matin ou l'après-midi */
   }
}

Mais QDate::setDate() n'accepte pas une QDate comme argument. Je pourrais bien sûr utiliser la forme QDate::setDate(jour,mois, année) en décomposant dateTmp, mais je ne trouve pas ça très efficace ni très joli !

Avez-vous une meilleure solution ?
Merci d'avance pour votre aide.

[LittleWhite]

Bonjour,

Je ne comprends pas pourquoi vous voulez hériter de QDate (autrement dit, je n'en vois pas l'intérêt).
Sinon, vous pouvez utiliser les fonctions add* ou encore, setDate(myDate.year(), myDate.month(), myDate.day());.

[dalfab]

Bonjour,

Tout d'abord, on peut se poser la question : est-ce une bonne idée d'hériter de QDate? Cela n'est viable que si toutes les fonctions de QDate ne changent pas de sens une fois devenue une DateEvenement. Il te faut donc potentiellement les reprendre toutes. Par exemple, si on appelle QDate::setDate() alors DateEvenement::matin doit être mis à jour. Ça n'est pas possible car QDate::setDate() n'est pas virtual et qu'on ne peut pas lui préciser si c'est le matin. Ça donc pas une bonne idée.

On devrait pouvoir hériter de QDateTime, ici elle va naturellement "savoir" si c'est le matin, il suffit alors d'y ajouter une fonction setDate façon QDate avec un paramètre matin supplémentaire, et une fonction indiquant si c'est le matin (si heure < 12). La DateEvenement n'a alors plus de champ matin, car il est toujours calculable. Est-ce une bonne idée? Peut-être que non, mais ça ça peut marcher.

Autre possibilité, la DateEvenement n'est pas une QDate mais contient une QDate. Là plus rien n'est interdit. On fait ce que l'on veut c'est un tout nouvel objet avec ses membres et ses fonctions. Cette inclusion peut être gérée sous la forme d'un "héritage privé" ou d'une inclusion simple c'est de la tambouille interne.

Revenons à ton code. Qu'est que le fonction RecalculeDate() veut faire? On a une date (dans DQate) à la demi-journée prêt (dans matin), on on en fait quoi? Je ne comprend pas ce que l'on peut faire avec juste ces infos.
Je ne vois pas ce qu'il pourrait y avoir dans tes ......, la seule ligne que je peux te fournir est ta ligne 8:

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
class DateEvenement : private QDate {  // Ça n'est plus un héritage
public:
    DateEvenement(int y,int m,int d,bool am) : QDate(y,m,d), matin(am) {}
    void  setDate(int y,int m,int d,bool am) { QDate::setDate(y,m,d); matin=am; }
    void  RecalculeDate();  // !?
    bool  est_ce_matin()const { return matin; }
    int  year()const { return QDate::year(); }
    int  month()const { return QDate::month(); }
    int  day()const { return QDate::day(); }
    ... ... ...
private:
    bool  matin;
};
... ... ...
        *static_cast<QDate*>(this) = dateTmp;  // pour accéder au égal de QDate
        // ou bien
        QDate::setDate(dateTmp.year(),dateTmp.month(),dateTmp.day());

[genetin]

Merci beaucoup Dalfab d'avoir passer du temps pour donner une réponse aussi détaillée.
Je comprends un peu mieux l'héritage, et ce n'est en effet pas bien adapté à mon cas. Je vais plutôt intégrer une QDate dans ma classe.

[dourouc05]

Je comprends un peu mieux l'héritage, et ce n'est en effet pas bien adapté à mon cas.

En général, l'héritage n'est pas une bonne solution, sauf quand tout est prévu pour ça (comme les widgets ou les interfaces abstraites). Tente toujours de passer par de la composition, c'est souvent une meilleure pratique (même si l'héritage se justifie dans certains cas, y compris l'héritage multiple).