CLABEL : UN CSTATIC AMÉLIORÉ

Commentaire de magic_Nono le 15/07/2004 10:33:40

conseil de prog:
évite les goto... (surtout si il y en a un seul)

OnClicked() modifié & commenté devient :


void CLabel::OnClicked()
{
/// -----------------------------------------------------
/// ---------------- CLabel::OnClicked() ----------------
/// -----------------------------------------------------
/// -----  Objectif : clic de l'utilisateur
/// -----  Auteur(s) : Grégory Allouche 10/07/04
/// -----  PreCond : /
/// -----  PostCond : /
/// -----  Etat : 1 (-1<0<1<2)
/// -----  MaJ 14/07/04 : suppression du goto & commentaires formalisés
/// -----------------------------------------------------
/// -----  Var Utilisées de la classe (1) : m_strUrl
/// -----  Var Internes à la fonction (3) : ShellExecute ,err ,hKey
/// -----  Var Utilisées par adresse (2) : buff ,valName
/// -----  Var In  (1) : m_strUrl
/// -----  Var In  Globales (1) : err
/// -----  Var In  Globales Constantes (3) : ERROR_SUCCESS ,HKEY_CLASSES_ROOT ,SW_NORMAL

///algo : SI une URL est spécifiée
if(m_strUrl.GetLength())
{
///algo : Recherche du navigateur par défaut
HKEY hKey;
///algo : SI on arrive à lire la base de registres,
if( RegOpenKey(HKEY_CLASSES_ROOT,"http\\shell\\open\\command", &hKey) == ERROR_SUCCESS)
{
DWORD size=512;
unsigned char buff[512];
char valName[512];
DWORD type;
if( RegEnumValue(hKey, 0, valName, &size, NULL, &type, buff, &size) == ERROR_SUCCESS )
{
CString strCommandLine=buff;
CString strExe;
CString strArgs;
strCommandLine.TrimLeft();
strCommandLine.TrimRight();
///algo : separe l'EXE des paramètres
if(strCommandLine.Left(1) =="\"")
{
int nPos=strCommandLine.Find('"', 1);
strExe=strCommandLine.Mid(1, nPos - 1);
strArgs=strCommandLine.Mid(nPos+1);
strArgs.TrimLeft();
}
else
{
strCommandLine+=" ";
int nPos=strCommandLine.Find(' ');
strExe=strCommandLine.Left(nPos);
strArgs=strCommandLine.Mid(nPos+1);
}
///algo : SI %1 est utilisé
if(strArgs.Find("%1") != -1)
///algo : le remplace par l'url
strArgs.Replace("%1", m_strUrl);
else ///algo : SINON
///algo : ajoute l'url aux paramètres existants
strArgs+=" \""+m_strUrl+"\"";
///algo : lance le navigateur par défaut
ShellExecute(NULL,"open", strExe, strArgs, NULL, SW_NORMAL);
}
}
else///algo : SINON
///algo : utilise ShellExecute en desespoir de cause
ShellExecute(NULL,"open", m_strUrl,"", NULL, SW_NORMAL);

}
return;
}

Commentaire de garslouche le 15/07/2004 15:17:37

Pour les goto, je sais qu'il est coutume de dire qu'il faut s'en passer mais c'est unisuement conventionnel...
Je m'explique:
1) Que fait le compilateur au final ? Des JMP ! C'est-à-dire des goto
2) Il me semble que le code que j'ai fourni ici est plus lisible que des if imbriqués

En fait de manière générale je suis pour éviter les goto à moins qu'on y gagne en lisibilité (et ça arrive plus souvent qu'on ne le crois)


Pour SetMail, c'est-une bonne idée. J'essaierai de l'intégrer ce WE. J'y ajouterai peut-être quelques paramètres supplémentaires (par exemple subject).


Merci pour ces commentaires