Void avec juste une condition

Version imprimable

Bonjour tout le monde !

Je viens vous voir, parce que j'aimerais avoir un petite aide par rapport a ce que j'ai fait...

Je voulais faire lorsque que mon programme et lancé il va automatiquement faire un vérification si une valeur dans mon fichier *.ini et true ou false
si c'est true alors je vais faire ma requête.

Mais je sais pas si c'est bien de faire comme j'ai fait...
Je demande juste confirmation qu'il n'y a pas de problème svp :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 TAccueil::TAccueil(QWidget *parent) : QMainWindow(parent), ui(new Ui::TAccueil) { ... connexion_auto(); } void TAccueil::connexion_auto() { QSettings connexion("programme.ini", QSettings::IniFormat); if(connexion.value("Config/connexion_auto").toBool()) { QMessageBox::information(0, tr("Connexion Automatique"), tr("Vous êtes connecté.")); } }
Donc je vais directement dans le void connexion_auto quand je lance mon programme et si connexion_auto = true alors j'exécute mon action sinon rien...

C'est le sinon rien qui me dérange... Je sais pas si c'est bien de faire ça...

Merci !

Perso cela ne me choque pas après tu peux peut être faire comme ça:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
void TAccueil::connexion_auto()
{
    QSettings connexion("programme.ini", QSettings::IniFormat);
 
    if(!connexion.value("Config/connexion_auto").toBool())
     return;
 
        QString PSEUDO = connexion.value("Config/pseudo", "Login Incorrect").toString();
        QString MDP = connexion.value("Config/mdp", "Mot De Passe Incorrect").toString();
 
        QMessageBox::information(0, tr("Connexion Automatique"), tr("Vous êtes connecté."));
 
}

C'est peut etre plus propre de faire ça sinon:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
bool TAccueil::connexion_auto()
{
    QSettings connexion("programme.ini", QSettings::IniFormat);
 
    if(!connexion.value("Config/connexion_auto").toBool())
     return false;
 
        QString PSEUDO = connexion.value("Config/pseudo", "Login Incorrect").toString();
        QString MDP = connexion.value("Config/mdp", "Mot De Passe Incorrect").toString();
 
        QMessageBox::information(0, tr("Connexion Automatique"), tr("Vous êtes connecté."));
 
return true;
 
}

01/07/2009, 11h00
corgato

Enfaite c'est juste pour savoir si c'est correct de coder comme ça.
De rentrer dans un void et de rien faire si c'est false par exemple...
01/07/2009, 11h05
superjaja

J'ai édité mon post. C'est une histoire de gout et de savoir ce que tu fais. En général c'est bien d'avoir un retour pour pouvoir contrôler ce qui se passe savoir si on est en connexion auto ou pas dans ton cas par exemple. Après peut être que d'autres personnes auront surement un autre avis.
01/07/2009, 11h13
corgato

Cool merci je vais utiliser ta méthode, je la trouve plus propre que la mienne vu qu'il y a un retour.

Une dernière question quand tu fais :

Code:

bool TAccueil::connexion_auto()

Il faut mettre dans mon premier void :

Code:

connexion_auto(bool);

et dans mon .h :

Code:

bool connexion_auto(bool)

Ou pas obligé de mettre des bool dans les parenthèses ?

Merci pour ta réponse ;)
01/07/2009, 11h36
superjaja
Si tu veux utiliser ce que je te propose tu fais ça:
Dans ton .h:

Code:

bool connexion_auto();

Dans ton .cpp:
Code:

1 2 3 4 bool TAccueil::connexion_auto() { //Code a mettre }
Pourquoi le bool en paramètre ? Cela peux peut être, être utile mais seul toi peux le savoir cela depdend de ce que tu veux faire...
01/07/2009, 11h42
corgato

Ok c'est ce que j'ai fait :)

Super merci beaucoup !!