Discussion :

[Tartar Ukid]

Bonjour,

Je bute actuellement sur une erreur SQL avec un TQuery.

Je veux en faites modifier le parametre d'une requetes en fonction de la sélection d'un radioButon.

j'ai mis dans la methode BeforeOpen() de mon TQuery:

Code :

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

1
2
3
4
       if(Form1->SortbyStatus->Checked)
                QueryDevices->Params->ParamByName("colonne")->AsString = "OverallStatus";
       else
                QueryDevices->Params->ParamByName("colonne")->AsString = "Name";

mais g une erreur me disant que le nom de colonne n'existe pas
voiçi ma requete SQL:

Code :

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

1
2
3
4
select OverallStatus,Name,DeviceKey 
from devices(NOLOCK) 
WHERE OverallStatus>=1 AND OverallStatus<=5 
order by :colonne DESC

Trouvé vous une erreur??

Tarik

[pilpagouna]

Salut Tarik,

Essaye de mettre ta condition if...else dans le même bloc que ta requête sql..

tiens moi au courant..

Linda

[Superstivix]

Tu pourrais nous en dire plus, comme par exemple, lequel des deux bouts de code arrive en premier dans ton code?

[pilpagouna]

Essaye de faire un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
MaQuery->SQL->Clear();
MaQuery->SQL->Text="... ORDER BY :COLONNE DESC";
MaQuery->Prepare();
if(Form1->...->Checked)
{
     MaQuery->ParamByName("COLONNE")->AsString=TonTruc;
}
else
{
     MaQuery->ParamByName("COLONNE")->AsString=TonAutreTruc;
}
 
MaQuery->Open();

Et dans ce cas, tu n'as pas besoin de méthode MaQueryBeforeOpen,

C'est ce que je fais et ça marche bien..

Linda

[Tartar Ukid]

ok pour plus de précision, j'ai en fait mis ma requete SQL dans la propriete du composant TQuery (directement).

Et pour ce qui est de l'execution la methode beforeOpen du TQuery passe avant l'execution de la requete.

je pensais faire la requete statiquement avec les composants builder mais ca plante beaucoup.

merçi

Tarik

[Tartar Ukid]

j'ai essayé la solution de Linda qui est de mettre tout dans une methode:

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
Refresh les données du DBGrid
DataModules->QueryDevices->SQL->Clear();
DataModules->QueryDevices->SQL->Text="select OverallStatus,Name,DeviceKey from devices(NOLOCK)WHERE OverallStatus>=1 AND OverallStatus<=5 order by :colonne DESC";
DataModules->QueryDevices->Prepare();
 
if(Form1->SortbyStatus->Checked)
{
     DataModules->QueryDevices->ParamByName("colonne")->AsString="OverallStatus";
}
else
{
     DataModules->QueryDevices->ParamByName("colonne")->AsString="Name";
}
 
 DataModules->QueryDevices->Open();

Mais toujours sans succes a mon avis ça ne vient pas de l'execution mais plutot de la syntaxe du query.


y a til un spécialist BD? La requete serait elle fausse??

Tarik

[pilpagouna]

j'aurai peut être une autre solution..désolée si ça marche pas..
Pour éviter d'avoir un paramètre, tu peux faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
DataModules->QueryDevices->SQL->Clear();
if(Form1->SortbyStatus->Checked) 
{ 
 DataModules->QueryDevices->SQL->Text="select  OverallStatus,Name,DeviceKey from devices(NOLOCK)WHERE OverallStatus>=1 AND OverallStatus<=5 order by OverAllStatus DESC"; 
 
} 
else 
{ 
 DataModules->QueryDevices->SQL->Text="select  OverallStatus,Name,DeviceKey from devices(NOLOCK)WHERE OverallStatus>=1 AND OverallStatus<=5 order by Name DESC"; 
} 
DataModules->QueryDevices->Prepare(); 
 DataModules->QueryDevices->Open();

ça pourra au moins te dire si ça vient du paramètre ou pas..

Linda

[Tartar Ukid]

Bon apparement c'est le parametre qui n'est pas bon.

J'accepte ta solution

merçi encore Linda.

Tarik