ADO, select sur un champ vide, plante mon appli

Bonjour a tous !


J'utilise ADO en mfc, et je viens de remarquer que lorsqu'une requete SELECT "pointe" sur un champ vide (non renseigné), cela part en exception au niveau du traitement de mon recordset ( que j'ai mis dans un try catch )

Ca viendrais aparamment du mon cast en t_variant vers un CString.

Je souhaitais savoir si il y a un moyen d'éviter se plantage en c++, ou mieux, directement dans la base de donnée ( ACCESS 2003 )

Merci a tous et bonne journée

si tu pouvais coller la partie du code qui plante ça serait utile ...
 Tu peux pas tester le retour de ta requete et ne pas effectuer le cast si aucun resultat ?

++

Voici mon code, ba en fait, l'interpretation de ma requete est déja dans un try catch !



 m_strRequete="SELECT STATUT FROM UTILISATEUR WHERE ID_CARTE=3128";
 _RecordsetPtr m_pRecordSet=m_pDialog->m_MaBase.ExecuteSql(m_strRequete); // J'execute ma requete, je stocke le recordset.

 ///////////////////////////////////////////////////////////////////////////////////////////
 //////// JE TRAITE MON RECORDSET AFIN DE SAVOIR LE STATUT DE L'UTILISATEUR ////////////////
 ///////////////////////////////////////////////////////////////////////////////////////////

 TESTHR(m_pRecordSet.CreateInstance(__uuidof(Recordset)));//On crée un recordset vide:

 try
 {
  _variant_t var;// variable pour récupérer des données du recordset
  m_pRecordSet->PutRefActiveConnection( m_pDialog->m_MaBase.GetPtrConnection()); // Type de connection
  // GetPtrConnection me sert à récupérer le pointeur sur ma connection
  m_pRecordSet->Open((LPCTSTR)m_strRequete, vtMissing, adOpenDynamic,adLockBatchOptimistic, -1); // Requete SQL
  m_pRecordSet->MoveFirst(); // Initialisation du recordset
  
  ///////////////////////////////////////////////////////////////////////////////////
  /////   Je récupère le statut de l'utilisateur et le stocke dans m_nStatut   //////
  ///////////////////////////////////////////////////////////////////////////////////

  while (!m_pRecordSet->EndOfFile) // Tant que la fin du recordset n'est pas atteinte
  {
   var=m_pRecordSet->Fields->GetItem((long)0)->Value;               // Récupération de la valeure du recordset
   CString strStatut=(LPCTSTR)_bstr_t(var);                                   // On CASTE la valeur vers un CString
   m_pRecordSet->MoveNext();                                                     // lecture de l'index suivant du recordset.

   if (strStatut=="M") // Le statut de l'utilisateur est "Maintenance"
    m_nStatut=0;
   
   if (strStatut=="P")// Le statut de l'utilisateur est "Professeur"
    m_nStatut=1;
   
   if (strStatut=="E")// Le statut de l'utilisateur est "Eleve"
    m_nStatut=2;

  }

 }
 catch(_com_error) // Si erreur durant l'interpretation du recordset
 {
  MessageBox(0,"Erreur au niveau du recordset, impossible de connaitre le statut","ERREUR",MB_OK);
 }

m_pRecordSet->Open((LPCTSTR)m_strRequete, vtMissing, adOpenDynamic,adLockBatchOptimistic, -1); // Requete SQL
m_pRecordSet->MoveFirst();  NON, badaboum si recordset est vide.
Le recordset est toujours à l'ouverture sur le 1er enreg s'il existe. 

CString strStatut=(LPCTSTR)_bstr_t(var); NON NON

#define isNotNull(var) (int) (var.vt != VT_NULL)
CString strStatut;
if(isNotNull(var)) strStatut=(LPCTSTR)_bstr_t(var);
else strStatut = ""; // PAR EXEMPLE

ciao...
BruNews, MVP VC++

Salut,
Comme signalé par BruNews, l'appel de certaines méthodes est interdit si le recordset est vide. Pour savoir si le recordset est vide ou non on fait comme ceci:
ADO_LONGPTR count= m_pRecordSet->RecordCount;
if(count)
{
    // travailler avec le recordset
}
else
{
// faire autre chose
}

Pour avoir une description de l'erreur de l'exception dans le messagebox tu peux faire:
 catch(_com_error &erreur)
 {
  MessageBox(0,(LPCSTR)erreur.Description(),"ERREUR",MB_OK);
// ajouter code ici
 }
Il est préférable d'ajouter du code à ce catch pour  éviter le plantage du programme.

Merci a tous pour vos réponses !
Je vais donc tout d'abord enlever mon "MoveFirst"

Pour le :

#define isNotNull(var) (int) (var.vt != VT_NULL)
CString strStatut;
if(isNotNull(var)) strStatut=(LPCTSTR)_bstr_t(var);
else strStatut = ""; // PAR EXEMPLE

L'idée est géniale, mais mes variable ne sont jamais les meme ( strStatut) dc, je pense que je vais essayer avec

ADO_LONGPTR count= m_pRecordSet->RecordCount;
if(count)
{
    // travailler avec le recordset
}
else
{
// faire autre chose
}

Merci a tous, je vais tester cela, et espère que cela ira !
Sinon, pas moyen d'en Acces de forcer a mettre les champ vide a une valeure ( NULL, ou 0, ou " " ) ?

Attention que lire RecordCount pourrait s'avérer couteux.
Un recordset vide est signalé par BOF && EOF positionnés.

Bien entendu Access prend des valeurs par défaut sur les champs.

ciao...
BruNews, MVP VC++

A, donc je pourrais faire un teste avec BOF && EOF avant de traiter mon recordset?

c'est ce que tu fais dans:
while (!m_pRecordSet->EndOfFile)

Il te suffit donc d'un test BOF avant la boucle.

ciao...
BruNews, MVP VC++

Brunews > J'aimerais bien savoir pourquoi lire RecordCount pourrait être couteux. Je l'utilise très souvent sans jamais rien remarquer. Merci.