[Gestion d'erreur] comment remplacer un goto ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [Gestion d'erreur] comment remplacer un goto ?

dd_isamu

Bonjour,

Je me balade assez souvent sur le site pour trouver des solutions à mes divers problèmes de programmation (surtout en C++ en fait) Mais j'avoue que là, bien que j'ai trouvé 2 ou 3 pistes, je n'arrive pas à faire ce que je voudrais et devrais faire... :cry: j'ai une super gestion d'erreur, sauf que celle-ci passe par un "goto", ce qui, dans ma section, nous est interdit du à la rupture de séquence... du coup j'ai essayé de remplacer cette gestion d'erreur par une fonction, mais, je sais pas pourquoi, j'arrive pas à trouver comment faire, ça change ma logique, ou ça appelle des fonctions qui ne me viennent pas à l'esprit... j'en sais rien. Si vous avez des idées elles sont les bienvenues...
Voici mon extrait de code le plus simple (j'ai une bonne vingtaine de sources dans le même genre si ça vous intéresse , il faut que je les corriges tous...).

Code :

#include <stdio.h>
#include <NIDAQmx.h>
#define DAQmxErrChk(functionCall) if( DAQmxFailed(error=(functionCall)) ) goto Error; else
int main(void)
{
int32 error=0;
TaskHandle taskHandle=0;
int32 read;
float64 data[10];
char errBuff[2048]={'\0'};
int i=0;
/*********************************************/
// DAQmx Configure Code
/*********************************************/
DAQmxErrChk (DAQmxCreateTask("",&taskHandle));
DAQmxErrChk (DAQmxCreateAIVoltageChan(taskHandle,"Dev3/ai1","",DAQmx_Val_Cfg_Default,-10.0,10.0,DAQmx_Val_Volts,NULL));
DAQmxErrChk (DAQmxCfgSampClkTiming(taskHandle,"",10,DAQmx_Val_Rising,DAQmx_Val_FiniteSamps,10));
/*********************************************/
// DAQmx Start Code
/*********************************************/
DAQmxErrChk (DAQmxStartTask(taskHandle));
/*********************************************/
// DAQmx Read Code
/*********************************************/
DAQmxErrChk (DAQmxReadAnalogF64(taskHandle,10,10.0,DAQmx_Val_GroupByChannel,data,10,&read,NULL));
printf("Acquired %d points\n",read);
for(i=0;i<10;i++)
{
printf("TEST %f points\n", data[i]);
};
Error:
if( DAQmxFailed(error) )
DAQmxGetExtendedErrorInfo(errBuff,2048);
if( taskHandle!=0 ) {
/*********************************************/
// DAQmx Stop Code
/*********************************************/
DAQmxStopTask(taskHandle);
DAQmxClearTask(taskHandle);
}
if( DAQmxFailed(error) )
printf("DAQmx Error: %s\n",errBuff);
printf("End of program, press Enter key to quit\n" );
getchar();
return 0;
}

Donc vous l'aurez compris, à chaque ligne je test ce que me renvoie la fonction, et je trouverai idiot de recopier entièrement tout mon code de gestion d'erreur à chaque ligne...
Merci d'avance !

Message édité par gilou le 02-06-2011 à 18:47:08

Publicité

tpierron

Ben faudra faire un paquet de refactoring, et je doute que tu puisses automatiser ça. Je ferais un truc du genre :

Code :

#include <stdio.h>
#include <NIDAQmx.h>
 
#define DAQmxErrChk(functionCall) DAQmxFailed(error=(functionCall))
 
int main(void)
{
    int32      error=0;
    TaskHandle taskHandle=0;
    int32      read;
    float64    data[10];
    char       errBuff[2048]={'\0'};
    int        i = 0;
 
    /*********************************************/
    // DAQmx Configure Code
    /*********************************************/
    if (! DAQmxErrChk(DAQmxCreateTask("",&taskHandle)) &&
        ! DAQmxErrChk(DAQmxCreateAIVoltageChan(taskHandle,"Dev3/ai1","",DAQmx_Val_Cfg_Default,-10.0,10.0,DAQmx_Val_Volts,NULL)) &&
        ! DAQmxErrChk(DAQmxCfgSampClkTiming(taskHandle,"",10,DAQmx_Val_Rising,DAQmx_Val_FiniteSamps,10)))
    {
 
        /*********************************************/
        // DAQmx Start Code
        // DAQmx Read Code
        /*********************************************/
        if (! DAQmxErrChk(DAQmxStartTask(taskHandle)) &&
            ! DAQmxErrChk(DAQmxReadAnalogF64(taskHandle,10,10.0,DAQmx_Val_GroupByChannel,data,10,&read,NULL)))
    {
            printf("Acquired %d points\n",read);
 
            for (i=0;i<10;i++)
                printf("TEST %f points\n", data[i]);
        }
    }
 
    if (DAQmxFailed(error))
        DAQmxGetExtendedErrorInfo(errBuff,2048);
    if (taskHandle)  {
        /*********************************************/
        // DAQmx Stop Code
        /*********************************************/
        DAQmxStopTask(taskHandle);
        DAQmxClearTask(taskHandle);
    }
    if (DAQmxFailed(error))
        printf("DAQmx Error: %s\n",errBuff);
 
    printf("End of program, press Enter key to quit\n" );
    getchar();
    return 0;
}

Le truc, c'est de regrouper les initialisations/allocations qui nécessite un nettoyage explicite dans une instruction if. Si ça foire, tu nettoies dans le bloc contenant, puis en cascade ensuite. De cette manière, à la fin du traitement normal, ça va nettoyer tout comme il faut.

Ou alors faire comme tu as fais: si ça foire à un endroit, tu te redirige le code au plus vite dans une partie qui va tout nettoyer.

FORUM HardWare.fr

Programmation

[Gestion d'erreur] comment remplacer un goto ?

Sujets relatifs
erreur de requete mysql -mysql_fetch_array(): supplied argument is not	erreur methode java
Dream cs5: remplacer images style diaporama(edit:résolu)	Url Rewriting et erreur 403
Erreur code java String[][]	script AS3 pour remplacer l'anime jouée par une image dans une DIV
gestion de variables	erreur de syntaxe, probleme langage C
remplacer un texte dans un lien dans la bdd	Gestion d'une combinaison de touches dans un userform
Plus de sujets relatifs à : [Gestion d'erreur] comment remplacer un goto ?

Page générée en 0.076 secondes