créer une bar de menu

Version imprimable

Voir 40 message(s) de cette discussion en une page

23/06/2008, 17h05
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:

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.
23/06/2008, 17h07
IrmatDen

Citation:

Envoyé par Gulish

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.
23/06/2008, 17h15
guiyomh

Tu est un dieu IrmatDen :ave:

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 :ave:)

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

je me punirait plus tard (:sm:)
23/06/2008, 17h20
yan

Citation:

Envoyé par IrmatDen

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

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.

Citation:

Envoyé par Mongaulois

Code:

menubar = new menuBar()

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

:roll:
moi je dit que c'était pas loin :P
23/06/2008, 17h32
guiyomh

Citation:

Envoyé par IrmatDen

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 ?
23/06/2008, 17h55
IrmatDen

Citation:

Envoyé par Mongaulois

:roll:
moi je dit que c'était pas loin :P

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) :P

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 :/
23/06/2008, 20h10
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 ?

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

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:

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 :D (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.

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:

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 ?

24/06/2008, 12h04
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:

menuFichier->addAction(MesActions->actionForRole(MySoftActions::AR_FileNew));
24/06/2008, 12h20
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. :king:

Resalut IrmatDen

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

Code:

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 :

Citation:

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::PhpWriterActions(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 :aie:

24/06/2008, 14h00
IrmatDen

J'ai raté une étoile. C'est QObject *parent et non QObject parent.
24/06/2008, 14h41
guiyomh

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

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

Voir 40 message(s) de cette discussion en une page