Discussion :

[Somato]

Bonjour,

Lors d'une requete SQL je récupère un QVariant invalide, voici mon code:

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
void DatabaseClass::initNewCodeP(FicheProprietaire *fichepro)
{
     QString S;
 
 
     QSqlQuery q;
     q.prepare("SELECT MAX(codeP) FROM Proprietaire");
     q.exec();
 
     if (q.lastError().isValid()) 
     {
        qDebug() << q.lastError();
     } 
     else 
     {
        qDebug() << "requete correcte";
        QVariant v = q.value (0);
        if (v.type() == QVariant::Invalid)
        {qDebug() << "mauvais type";}
        qDebug() << S.setNum(q.value(0).toInt());
        fichepro->setCodeP(S);
     }
}

voici les debugs générés:

requete correcte
QSqlQuery::value: not positioned on a valid record
mauvais type
QSqlQuery::value: not positioned on a valid record
"0"

Ma requête SQL me semble correcte pourtant, donc tout me laisse penser que c'est la méthode value de QSqlQuery qui m'embête, et je n'ai pas trop compris le fonctionnement de cette méthode dans la doc...

Merci d'avance de votre aide!

[IrmatDen]

Salut,

Comme le dit le message d'erreur, tu n'es pas positionné sur un enregistrement valide. Il faut toujours faire q.first() et ensuite itérer sur les records en appelant q.next().

[Somato]

Salut,

Je reposte ici car j'ai encore un problème de compréhension à ce sujet.

Ma syntaxe doit être fausse:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    QSqlQuery q;
    q.prepare("SELECT COUNT(codeP) FROM Proprietaire WHERE codeP = :codeP");
    q.bindValue(":codeP", v.value(0));
    if(q.lastError().isValid())
       qDebug(qPrintable(q.lastError().text()));
    if (!q.exec())
       qDebug() << "codeP n'existe pas...";
    else
       q.first();
       while (q.next())
       {qDebug()<<"bool si ligne existante ou non:";
       qDebug() <<q.value(0).toInt();}

[IrmatDen]

Et le problème est... ?

[Somato]

Que mon variant reste invalide avec le code précédent.
Je pense que j'utilise mal next(), mais j'ai vu un exemple ou il est utilisé de la sorte, j'ai donc suivit cet exemple... (j'ai aussi essayé de l'utiliser différemment, sans itération par exemple, mais toujours rien)

Le but est que je récupère la valeur de mon SELECT COUNT


Merci d'avance

PS: j'aimerai aussi savoir pour tout autre chose:
Si j'ai plusieurs fois ":codeP" dans ma requete SQL,
bindValue(":codeP", 1) va t-il mettre tout mes ":codeP" à 1 ou le fait-il pour un seul?

[IrmatDen]

Oups, mea culpa, désolé de t'avoir induit en erreur! Le first est inutile

Exemple tiré d'un de mes codes:

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
	QVector<boost::shared_ptr<Product> > productSet;
 
	QString queryStr("SELECT id FROM product WHERE sourceId=:sid");
	QSqlQuery query;
	query.prepare(queryStr);
	query.bindValue(":sid", s->getId());
 
	if(!query.exec())
	{
		error = QObject::tr("Can't execute statement: %1").arg(query.executedQuery());
		return productSet;
	}
 
	while(query.next())
	{
		productSet<< boost::shared_ptr<Product>(new productSet(query.value(0).toULongLong(), s));
	}
 
	return productSet;

[IrmatDen]

PS: j'aimerai aussi savoir pour tout autre chose:
Si j'ai plusieurs fois ":codeP" dans ma requete SQL,
bindValue(":codeP", 1) va t-il mettre tout mes ":codeP" à 1 ou le fait-il pour un seul?

Aucune idée; j'aurais tendance à dire que oui. Mais tu penses à quoi comme requête pour utiliser ça?

[Somato]

A une requete du genre: je teste si codeP = 4 et si ce n'est pas le cas je mets codeP = 4

j'appelle donc deux fois BindValue("codeP", 4) dans ma requete.

[IrmatDen]

Je vois pas trop l'intérêt (je vois surtout le danger ), mais soit. Essaie et tu verras; je ne peux pas te certifier que c'est bon bien que je le suppose (l'appeler une fois ou 2 ou 3 ou... ne devrait rien changer à mon avis).

[Somato]

L'intérêt est le suivant:
je test si codeP == 4 dans la base existe, s'il n'existe pas, je fais une requête INSERT pour sauvegarder mes données.
Si le test de codeP == 4 est concluant et retourne true, je fais une requête UPDATE pour sauvegarder mes données.

Etant donné que mon IF... THEN ... ELSE ne peut pas fonctionner avec SQLite

Mais du coup, ce sera différent: je fais le test si codeP == 4 dans un première requête avec un premier bindValue, et la sauvegarde de la data se fait dans une seconde requête.

En tout cas, j'arrive bien à récupérer la valeur de mon QVariant maintenant, merci!

[IrmatDen]

Ah, ça c'est plus précis comme explication; tu as cette option:
* faire de codeP une valeur sans doublon dans la table
* faire un INSERT et utiliser ON CONFLICT pour faire un UPDATE au lieu de l'INSERT si le conflit d'unicité est détecté.

Par contre, je ne sais pas comment ça va s'utiliser et je ne pourrais t'aider beacoup plus :/
Mais à lire la doc, ça n'a pas l'air compliqué
INSERT => http://www.sqlite.org/lang_insert.html
clause ON CONFLICT => http://www.sqlite.org/lang_conflict.html

[Somato]

Désolé, le problème est maintenant réglé donc je n'ai pas eu le réflexe de passer plus tôt.

Donc, codeP est déjà une valeur sans doublon puisque c'est la primary key de ma table.

Ensuite, voici du coup ce que j'ai fais:

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
    QSqlQuery q;
    q.prepare("SELECT COUNT(codeP) FROM Proprietaire WHERE codeP = :codeP");
    q.bindValue(":codeP", v.value(0));
 
    if (!q.exec())
       qDebug(qPrintable(q.lastError().text()));
    else
       while (q.next())
       {
             if(q.value(0).toInt() == 0) //traitement des données si codeP inexistant dans database
             {
                  QSqlQuery q2;
                  q2.prepare("INSERT INTO "//suite du code avec bindValue, etc...
              }
              else // traitement des données si codeP existe déjà dans la base (donc si INSERT déjà effectué)
              {
                   QSqlQuery q2; 
                   q2.prepare("UPDATE " //suite du code avec bindValue, etc...
              }

Maintenant, c'est vrai que la clause "ON CONFLICT" m'intéresse beaucoup!
Je l'étudie et la garde de coté si j'en ai l'utilité dans la suite de mon code.
Je garde mon code actuel, pour le moment il ne me pose pas de bug apparent

[IrmatDen]

Je garde mon code actuel, pour le moment il ne me pose pas de bug apparent

Et alors? La propreté du code, c'est de la réduction de risque de bug, et une localisation accélérée lorsqu'il se présente (Je le sais pour avoir fait l'erreur de ne pas garder un code propre dans les coups de bourre => ça se paye plus tard, cher.)

[Somato]

J'ai pas trop été "éduqué" pour identifier du code propre ou non...
Je voulais surtout garder du code 'que je connais', car j'ai l'impression qu'une erreur SQL est plus dur à identifier qu'une erreur de programme où des qDebug peuvent un peu plus nous aiguiller sur le problème.

Mais donc, tu ne trouve pas mon code, ici présent, propre?
Ton avis m'intéresse, et s'il faut que je préfère une requête SQL bourrée de ligne de fonction dans un string plutôt qu'un code C++, alors je le ferrais

[IrmatDen]

En effet, c'est plus dur à comprendre s'il y a une erreur SQL.
Le code serait plus propre avec la clause on conflict car il serait beaucoup plus concis, et plus rapide car tu n'as pas de check à faire avant

[Somato]

Ok merci,
je verrais pour faire ça se soir alors, en aparté.