Discussion :

[Gulish]

Salut,

Deja, je pense qu'il serait bon que les noms de tes classes commencent par une Majuscule. MaClasse maVariable. Ca évite les confusions, et avec certains langages/compilateurs, c'est nécessaire. Autant prendre de bonnes habitudes.

Ensuite, as tu essayé d'utiliser le même type entre la déclaration de ton pointeur, et l'instanciation de ton objet ?

Code :

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

1
2
3
4
5
menuBar * mainbar = new menuBar();
 
au lieu de : 
 
QMenuBar * mainbar = new menuBar();

En lisant ton message de compilation, on voit bien que ton constructeur menuBar renvoir un "int *" que tu associes à un "QMenuBar *". As tu essayé de caster ?

Il y a plein de choses que je ne comprends pas dans ton code, mais ça vient peut être de moi. Par contre j'ai l'impression que tu fais parfois une confusion entre un QMenu et un QMenuBar.
En effet, dans ton menuBar.cpp, à ce que je comprends, tu définis 3 QMenus, mais je ne vois nul part la création d'une QMenuBar dans laquelle tu as ajouté les 3 QMenus que tu as crées.

Je te conseille d'analyser l'exemple du MDI donné par Qt.

Bonne chance,

G.

Sinon, il est souvent plus malin de déclarer toutes ses variables et pointeurs dans le header (.h) d'une classe.

[IrmatDen]

Sinon, il est souvent plus malin de déclarer toutes ses variables et pointeurs dans le header (.h) d'une classe.

Euh non, surtout pas. Il y a des cas où il faut, et d'autres ou ce n'est pas nécessaire.

[guiyomh]

Tu est un dieu IrmatDen

j'ai toujours le chic pour prendre les nom de variable réservé !
Du coup je l'ai renommer ma class en PhpwriterMenuBar comme tu me conseillait avec le nom de l'appli ! (encore une fois )

et effectivement j'ai le problème pour les actions et je suis toute ouïe a ton ecoute.

je me punirait plus tard ()

[yan]

Râââh, mais quel idiot de pas y avoir pensé avant...

menuBar() est une fonction membre de QMainWindow. Donc forcément ça pouvait pas compiler... Change donc le nom de ta classe. Après ça y'aura un paquet d'erreurs sur les actions, mais c'est normal. Je vais expliquer le raisonnement auquel je pense un peu plus tard, j'ai pas le temps tout de suite.

Code :

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

menubar = new menuBar()

tu as nommé ta variable comme ta class... pas bien

moi je dit que c'était pas loin

[guiyomh]

Euh non, surtout pas. Il y a des cas où il faut, et d'autres ou ce n'est pas nécessaire.

Peut tu expliquer les cas ou il faut et ceux ou il faut pas ?

[IrmatDen]

moi je dit que c'était pas loin

Rien à voir, et la variable a un 'b' minuscule, c'est légal (si on enlève la fonction menuBar() de QMainWindow bien sûr)

Pour les variables locales/variables d'instance, ben, y'a pas trop de règles, c'est selon le besoin quoi... Il faut réduire la portée d'une variable au minimum qui fasse marcher l'appli pour pas bloater la déclaration d'une classe avec toutes les variables qui sont propres à chaque méthode, mais c'est un peu l'évidence même quoi :/

[guiyomh]

Oki !

sinon est ce que tu as un peu de temps pour un exemple de la gestion des actions dans une classe le tout gérer avec des ENUM ?

[IrmatDen]

Tu fais bien de le rappeller, j'avais un peu zappé

Première suggestion: utiliser des enum.
* avantages : typage (assez) fort, vérifiable à la compilation
* inconvénients : structure qui oblige à un peu de redondance (maintenance d'un type enum en fait)

Code cpp :

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
class MySoftActions: public QObject
{
public:
    enum ActionRole
    {
        AR_FileNew,
        AR_FileOpen
        //...
    };
 
public:
    MySoftActions(QObject parent = 0)
    : QObject(parent)
    {
        QAction *action;
 
        action = new QAction(QIcon(":/images/new.png"), tr("&New"), this);
        action->setShortcut(tr("Ctrl+N"));
        action->setStatusTip(tr("Create a new file"));
        actions[AR_FileNew] = action;
 
        action = new QAction(QIcon(":/images/open.png"), tr("&Open"), this);
        action->setShortcut(tr("Ctrl+O"));
        action->setStatusTip(tr("Open a file"));
        actions[AR_FileOpen] = action;
 
        // ...
    }
 
    QAction* actionForRole(ActionRole role)
    {
        ActionMap::iterator ai = actions.find(role);
        return (ai != actions.end() ? ai.value() : 0);
    }
 
private:
    typedef QMap<ActionRole, QAction*> ActionMap;
    ActionMap actions;
};

Ca, c'est la structure de base nécessaire, après tu peux l'agrémenter de petite features genre le classement d'action par leur "catégorie" (Fichier, Edition etc....) voire la recherche d'une action selon ses champs textes (nom ou status tip); mais dans longtemps, quand t'auras un soft suffisamment complexe pour avoir besoin de ça (genre 3DS, Excel et tout ces poids lourds)

Pour la solution à base de string, c'est sensiblement pareil. Tu supprimes l'enum, tu changes la clé de la map pour une string, et c'est parti. Il n'y a pas de vérifs à la compilation possible par contre. Si tu réfléchis le code suffisamment, tu peux du coup ouvrir ton soft à des plugins plus facilement.

[guiyomh]

Oki je vois le truc, par contre je comprend pas bien le QMap !

Et comment faire pour utiliser ces ENUM dans une fonction de class par exemple :

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
 
#include "ma_class_action.h"
//[...]
PhpWriterMenuBar::PhpWriterMenuBar()
{
    ma_class_action *MesActions = ma_class_action();
}
void PhpWriterMenuBar::buildMenuFichier()
{
    menuFichier = new QMenu(this);
    //menuFichier = addMenu(trUtf8("&Fichier"));
    menuFichier->setTitle(trUtf8("&Fichier"));
    menuFichier->addAction(MesActions::AR_FileNew);
    menuFichier->addAction(MesActions::AR_FileOpen);
    menuFichier->addSeparator();
    //...
    addAction(menuFichier->menuAction());
}

Est ce correcte ?

[IrmatDen]

1. QMap est équivalent à std::map. C'est un conteneur associatif qui prend une clé à laquelle est associée une et une seule valeur pour faire court.

2.a. faut que les actions soient membres de classes
2.b. la présence d'une méthode renvoyant un QAction* ne t'a pas intrigué?

Code :

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

menuFichier->addAction(MesActions->actionForRole(MySoftActions::AR_FileNew));

[guiyomh]

oki je te remercie encore pour tes explications.

pour répondre au 2.b, => j'avais été distrait par le QMap et je n'avais pas vus la méthode.
Mais maintenant que tu le dis, ça parait évidant.

En tout cas merci ça m'a beaucoup aider.

[guiyomh]

Resalut IrmatDen

j'ai essayer de reprendre ton exemple en le modifiant légèrement :
j'ai fai tun fichier PhpWriterActions.h :

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
#ifndef PHPWRITERACTIONS_H_INCLUDED
#define PHPWRITERACTIONS_H_INCLUDED
 
 
#include <QObject>
#include <QAction>
#include <QMap>
#include <QKeySequence>
#include <QIcon>
 
class PhpWriterActions : public QObject
{
public:
    enum ActionRole
    {
        AR_FileNew,
        AR_FileOpen
    };
 
public:
    PhpWriterActions(QObject parent = 0): QObject(parent)
    {
        QAction *action;
 
        //Nouveau fichier
        action = new QAction(QIcon(":/tool/icon/crystal/filenew.png"),trUtf8("&Nouveau","Nouveau fichier"),this);
        action->setStatusTip(trUtf8("Créer un nouveau fichier"));
        action->setShortcut(QKeySequence(Qt::CTRL + Qt::Key_N));
        actions[AR_FileNew] = action;
 
        //Ouvrir un fichier
        action = new QAction(QIcon(":/tool/icon/crystal/fileopen.png"),trUtf8("&Ouvrir","Ouvrir un fichier"),this);
        action->setStatusTip(trUtf8("Ouvre un fichier"));
        action->setShortcut(QKeySequence(Qt::CTRL + Qt::Key_O));
        actions[AR_FileOpen] = action;
 
    };
 
    QAction* actionForRole(ActionRole role)
    {
        ActionMap::iterator ai = actions.find(role);
        return (ai != actions.end() ? ai.value() : 0);
    };
 
private:
    typedef QMap<ActionRole, QAction*> ActionMap;
    ActionMap actions;
};
 
#endif // PHPWRITERACTIONS_H_INCLUDED

mais l'ors de la compilation j'ai un petit problème :

PhpWriterActions.h:21: erreur: default argument for parameter of type «QObject» has type «int»
/usr/local/Trolltech/Qt-4.4.0/include/QtCore/qobject.h: In constructor «PhpWriterActions:hpWriterActions(QObject)»:
/usr/local/Trolltech/Qt-4.4.0/include/QtCore/qobject.h:295: erreur: «QObject::QObject(const QObject&)» is private
PhpWriterActions.h:21: erreur: à l'intérieur du contexte
make: *** [mainWindow.o] Erreur 1

ps : je sais je suis une buse

[IrmatDen]

J'ai raté une étoile. C'est QObject *parent et non QObject parent.

[guiyomh]

ok ça marche je te remercie une fois de plus (pour aujourd'hui).

Grâce a toi j'ai beaucoup progressé. Merci