Discussion :

[corgato]

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 :

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
 
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 !

[superjaja]

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

Code :

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

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 :

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

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;
 
}

[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...

[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.

[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 :

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

bool TAccueil::connexion_auto()

Il faut mettre dans mon premier void :

Code :

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

connexion_auto(bool);

et dans mon .h :

Code :

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

bool connexion_auto(bool)

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

Merci pour ta réponse

[superjaja]

Si tu veux utiliser ce que je te propose tu fais ça:
Dans ton .h:

Code :

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

bool connexion_auto();

Dans ton .cpp:

Code :

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

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...

[corgato]

Ok c'est ce que j'ai fait

Super merci beaucoup !!