Discussion :

[riddickk]

Bonjour,

Je voudrais savoir si il existe des outils ou m�me des options dans des compilateurs capables de d�tecter des cast utilis� en langage C dans du code C++.
J'utilise un analyseur statique de code puissant mais il ne parvient pas � detecter ce genre de cast. Je joins le code suivant qui est archi-faux mais qui compile bien :

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
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
 
 
#include <stdio.h>
 
class TestA {
public:
 
    virtual void Print ( void ) {
 
        printf("TestA [%08X].\n",this);
    }
 
    static void Test ( TestA* Class ) {
 
        if (Class)
            Class->Print();
        else
            printf("TestA : NULL passed.\n");
    }
} ;
 
class TestB {
public:
 
    virtual void FunctionDummy ( void ) {
 
        printf("CRASH + CORE !!!! \n");
    }
 
    virtual void Test_CastC ( void ) {
        // This should show an error of invalid cast/ useless /dangerous
        // Devrait montrer une erreur de cast invalide / inutile / dangereux.
        // Normalement, crash ici.
        TestA::Test((TestA*)this);
    }
 
/*  // Don't compile
    // Ne compile pas.
    virtual void Test_SCastCPP ( void ) {
 
        TestA::Test(static_cast<TestA*>(this));
    }
*/
 
    virtual void Test_DCastCPP ( void ) {
        // don't crash for an execution if the class TestA checks NULL values
        // Ne plante pas à l'exécution si la classe TestA vérifie les valeurs NULL.
        TestA::Test(dynamic_cast<TestA*>(this));
    }
} ;
 
int main(int argc, char* argv[]) {
 
    TestA A ;
    TestB B ;
    int i = 0 ;
 
    A.Print();
    A.Test(&A);
    A.Test(NULL);                  // Major crash in the execution => the dynamic_cast in TestA doesn't work (dead code) 
//    A.Test((TestA*)(&i));     // Crash sévère à l'exécution => le dynamic_cast dans TestA ne sert à rien (code mort).
    B.Test_CastC();
//    B.Test_SCastCPP();
    B.Test_DCastCPP();
 
	return 0;
}

Si quelqu'un a une id�e sur la question ?

Merci

[riddickk]

J'ai trouv� une option dans gcc, j'utilise la version 4.4.0. En activant l'option -Wold-style-cast pour un programme C++, il d�tecte l'utilisation de cast utilis� en langage C dans du code C++.
J'utilise la commande:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
 
gcc -c -x c++ -Wold-style-cast monfichier.cpp .

Avec cette commande voici ce que j'obtiens:

Code : Console

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
gcc -c -x c++ -Wold-style-cast Test_ClassCast.cpp
 
Test_ClassCast.cpp: In member function 'virtual void TestB::Test_CastC()':
Test_ClassCast.cpp:32: warning: use of old-style cast

[Davidbrcz]

Je voudrais savoir si il existe des outils ou m�me des options dans des compilateurs capables de d�tecter des cast utilis� en langage C dans du code C++.

Une reponse universelle a cette question (hors option du compilateur), c'est grep et/ou des regex

[Mac LAK]

Une reponse universelle a cette question (hors option du compilateur), c'est grep et/ou des regex

S�r, un pattern "parenth�se - identificateur - parenth�se" en C/C++, �a va remonter les casts C...

Et quasiment tous les appels de fonction � un seul param�tre, les variables de macros, les arguments des "switch" ou "while" si ce sont des variables qui sont test�es, etc...

[Davidbrcz]

Sauf que ces cast se font apres un signe =, une virgule ou une parenthese (jen oublie p-e, mais je pense pas). Et ca permet de limiter les faux positifs

[Mac LAK]

Sauf que ces cast se font apres un signe =, une virgule ou une parenthese (jen oublie p-e, mais je pense pas). Et ca permet de limiter les faux positifs

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

(int)var = (var2) + (type_perso_num)var3

Le premier cast n'est pas dans ce cas de figure.
Le deuxi�me sera un faux positif.
Le troisi�me ne sera pas d�tect� non plus.

Bref, c'est loin d'�tre aussi simple qu'un test de regexp... Il faut quasiment arriver au niveau de la compilation (apr�s analyse lexicale et une bonne partie d'analyse s�mantique) pour tous les d�tecter, et c'est largement hors du cadre d'une "simple" regexp.