Retour bizarre de InternetGetConnectedState

**Janitrix** · 14/10/2006, 22h50

Salut à tous. Depuis quelques temps, je recherche un moyen de connaitre le statut de la connection internet d'un ordinateur. J'ai finalement trouvé InternetGetConnectedState qui fait parti de l'api windows. La méthode me retourne TRUE lorsque je suis connecté mais lorsque je regarde la valeur du LPDword, elle ne correspond à rien. Elle me donne 0x56, ce qui ne correspond à rien (voir le lien msdn)... je comprend vraiment pas. Voiçi le code en question :

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
if(InternetGetConnectedState(&state, 0) != FALSE) {
            printf("Connection state : %x\n", state);
            switch(0) {
                case 0x40:
                    printf("Hello1\n");
                    break;
                case 0x02:
                    printf("Hello2\n");
                    break;
                case 0x01:
                    printf("Hello3\n");
                    break;
                case 0x08:
                    printf("Hello4\n");
                    break;
                case 0x20:
                    printf("Hello5\n");
                    break;
                case 0x04:
                    printf("Hello6\n");
                    break;
            }
        }

Ce code donc m'affiche tout le temps 56 (sois 0x56). Vous avez une idée du problème ? Merci.

**Trap D** · 15/10/2006, 00h06

Moi je testerais plutôt sous cette forme :

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
if (InternetGetConnectedState(&state, 0) != FALSE) {  
    printf("Connection state : %x\n", state); 
    if (0 & 0x40)
       printf("Hello1\n"); 
    if (o & 0x02)
        printf("Hello2\n"); 
    if (o & 0x01)
        printf("Hello3\n"); 
    if (o & 0x08)
        printf("Hello4\n"); 
    if (o & 0x20)
        printf("Hello5\n"); 
    if (o & 0x04)
        printf("Hello6\n"); 
}

Bine que je ne vois pas comment on peut faire 0x56 avec ces nombres.

**JeitEmgie** · 15/10/2006, 09h13

Envoyé par Trap D

Moi je testerais plutôt sous cette forme :

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
if (InternetGetConnectedState(&state, 0) != FALSE) {  
    printf("Connection state : %x\n", state); 
    if (0 & 0x40)
       printf("Hello1\n"); 
    if (o & 0x02)
        printf("Hello2\n"); 
    if (o & 0x01)
        printf("Hello3\n"); 
    if (o & 0x08)
        printf("Hello4\n"); 
    if (o & 0x20)
        printf("Hello5\n"); 
    if (o & 0x04)
        printf("Hello6\n"); 
}

Bine que je ne vois pas comment on peut faire 0x56 avec ces nombres.

if (state & 0x40)

etc.

**Janitrix** · 15/10/2006, 09h19

Le switch(0) était la juste pour le test. Je le remplacerai après. Désolé s'il y a eu confusion. Mon probleme en fait c'est que InternetGetConnectedState me sort un résultat qui ne correspond à rien et j'ai testé sur un autre ordi, il me sort 77

**Janitrix** · 15/10/2006, 14h54

J'ai un peu avancé... Voiçi le nouveau code :

Code C :

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
 
LPDWORD state;
 
    while(more) {
        InternetGetConnectedState((LPDWORD)&state, 0);
 
        printf("Connection state : %x\n", state);
 
        if(state == 0x04) {
 
        }else if(state == 0x02) {
 
        }else if(state == 0x01) {
 
        }else if(state == 0x08) {
 
        }else if(state == 0x20) {
 
        }else if(state == 0x04) {
 
        }
 
        Sleep(1000);
    }

Maintenant, il m'affiche 33eb0, ce qui est bien de l'hexadécimal(ouf) mais cela ne correspond encore une fois à rien. Sur MSDN, ils disent que les résultats peuvent être combinés, je ne comprend pas vraiment qu'est ce que je doit obtenir dans ma variable si c'est le cas ???
Et la codeblocks me donne 6 warnings pour les comparaisons :

warning: comparison between pointer and integer

Pourtant, 'state' n'est pas un pointer

? Merci.

**Médinoc** · 15/10/2006, 16h12

Pose-toi la question suivante:
"Que signifie 'LP' dans 'LPDWORD' ?"

Et ton cast (LPDWORD)&state me parait bizarre... Puisque du convertis un LPDWORD * en LPDWORD...

**Emmanuel Delahaye** · 15/10/2006, 16h14

Envoyé par Médinoc

Pose-toi la question suivante:
"Que signifie 'LP' dans 'LPDWORD' ?"

Si c'était écrit 'unsigned long *' ou 'ulong *', on se poserait moins de questions...

**Médinoc** · 15/10/2006, 16h17

En effet, mais il me semble que Windows garantit qu'un DWORD fait 32bits (ce qui n'est pas garanti par les long sur certaines plate-formes 64 bits).

D'une manière générale, il semblerait que l'API Windows essaie d'utiliser aussi peu que possible les types primitifs pour être sûr de pouvoir adapter à la plate-forme juste en modifiant le typedef.

Quand on sait qu'il y a Windows Mobile pour les PocketPC, c'est plutôt une bonne idée...

Donc, il aurait fallu un DWORD *

**Emmanuel Delahaye** · 15/10/2006, 16h35

Envoyé par Médinoc

En effet, mais il me semble que Windows garantit qu'un DWORD fait 32bits (ce qui n'est pas garanti par les long sur certaines plate-formes 64 bits).

D'une manière générale, il semblerait que l'API Windows essaie d'utiliser aussi peu que possible les types primitifs pour être sûr de pouvoir adapter à la plate-forme juste en modifiant le typedef.

Quand on sait qu'il y a Windows Mobile pour les PocketPC, c'est plutôt une bonne idée...

Donc, il aurait fallu un DWORD *

Les types de tailles fixes ne sont pas garantis par le langage. Un type WORD (16 bits non signés) n'existe pas sur une plateforme de type DSP Motorola/Freescale 56156.

D'ailleurs, en C99, les types de taille fixes (uint8_t etc.) sont définis 'sous reserve qu'il existent sur la plateforme considérée', sinon, ce sont des types 'at_least', ce qui ne change pas grand chose, car

char : at least 8 bits
short, int : at least 16 bits
long : at least 32 bits

mais ça on le savait depuis les origines du C...

On sait que depuis le début, Microsoft n'aime pas le langage C, d'ailleurs, ils ont fini par construire leur propre langage (C#), censé regler tous le problèmes de portabilité... en éliminant les plateformes 'impures !'.

**Janitrix** · 15/10/2006, 17h16

Merci de vos réponses. J'ai simplifié le code pour me concentrer sur la partie la plus importante.

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
 
//le main...
int more = 1;
LPDWORD *state;
    state = (LPDWORD *)malloc(sizeof(LPDWORD));
 
    while(more) {
        getInternetConnectionState(&state);
 
        printf("Connection state : %x\n", *state);
Sleep(1000);
    }
//fonction externe
int getInternetConnectionState(LPDWORD *state) {
    if(InternetGetConnectedState(*state, 0) != FALSE) {
        return 0;
    }
    return -1;
}

Maintenant, quand il me sort 56...

**Médinoc** · 15/10/2006, 17h27

Ouh, le bordel...
Voici le code corrigé:

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
//le main...
int more = 1;
DWORD state = 0;
while(more) {
	getInternetConnectionState(&state);
 
	printf("Connection state : %lx\n", state);
	Sleep(1000);
}
 
//fonction externe
//( on pourrait faire plus simple avec l'opérateur ?: )
int getInternetConnectionState(LPDWORD pState) {
	if(InternetGetConnectedState(pState, 0)) {
		return 0;
	}
	return -1;
}

C'est pourtant simple, non ?
Pourquoi tu nous fais des trucs super-compliqués avec de l'allocation dynamique et des pointeurs de pointeurs ? C'est un programme shadok ?

Emmanuel : Naturellement, Les types de l'API Windows ne sont disponibles que sur une plate-forme supportant Windows...

Ce n'est pas de la programmation C qu'on fait ic, c'est de la programmation système.

**Emmanuel Delahaye** · 15/10/2006, 17h38

Envoyé par Janitrix

Merci de vos réponses. J'ai simplifié le code pour me concentrer sur la partie la plus importante.

C'est hyper compliqué ! (Donc complètement faux)

Pourquoi tu ne fais pas un truc simple, complet et compilable 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
15
16
17
18
19
20
21
 
#include <windows.h>
#include <wininet.h>
#include <stdio.h>
 
int main (void)
{
   int more = 1;
   DWORD state;
 
   while (more)
   {
      if (InternetGetConnectedState(&state, 0) != FALSE)
      {
         printf("Connection state : %lx\n", (unsigned long)state);
      }
 
      Sleep(1000);
   }
   return 0;
}

(affiche 52).

**Janitrix** · 15/10/2006, 18h26

Merci pour vos réponses. En effet, vos codes sont beaucoups plus simples. C'est ma nature de toujours tout compliquer

. J'ai malgré tout un dernier doute et peut être vous pourrez me renseigner : pourquoi la fonction me renvoit-elle 56 (ou même pour Emmanuel Delahaye 52) ? Cela ne semble correspondre à rien dans la doc MSDN sur la fonction. De plus, 0x04 (un exemple de ce que la fonction pourrait renvoyer) c'est en hexadécimal non ? Alors pourquoi quand on demande d'afficher le résultat en hexa, 'printf' me met 56 et pas 0x56 ? Merci.

**Médinoc** · 15/10/2006, 18h33

pourquoi quand on demande d'afficher le résultat en hexa, 'printf' me met 56 et pas 0x56 ? Merci.

Parce que les auteurs du standard ne sont pas idiots, et nous permettent ainsi de mettre le préfixe que l'on veut, voire aucun...

D'ailleurs, ils permettent aussi de choisir entre les minuscules et les majuscules (ce qu'ils ne permettent, hélas, pas pour les pointeurs).

**Emmanuel Delahaye** · 15/10/2006, 18h42

Envoyé par Janitrix

Alors pourquoi quand on demande d'afficher le résultat en hexa, 'printf' me met 56 et pas 0x56 ? Merci.

Parce qu'il faut mettre le 0x (ou 0X) à la main.

Code :

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

         printf("Connection state : 0x%lx\n", (unsigned long)state);

Le format 'C' pour représenter les valeurs en héxadécimal n'a rien d'universel. J'en connais au moins 2 autres :

BASIC : #&AA55 (euh, j'ai un doute...)
ASM intel : AA55h ou AA55H

**Médinoc** · 15/10/2006, 18h45

Différents formats d'hexa que j'ai connu:

Assembleur 68000 X68K : $
Assembleur x86 TASM : Suffixe H
BASIC Locomotive: & ou &H
TI-Basic pour TI68k : 0h
Couleur RVB en HTML : #
Langage C : 0x

**Trap D** · 15/10/2006, 21h12

Pour en revenir au programme, il est dit dans la doc

This parameter may return a valid flag even when the function returns FALSE. This parameter can be one or more of the following values.

Donc il faut tester avec des &.
Chez moi, avec ce programme

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
#include <stdio.h>
#include <windows.h>
#include <Wininet.h>
 
int main(void)
{
  DWORD state;
  if (InternetGetConnectedState(&state, 0) != FALSE) 
  {  
    printf("Connection state : %x\n", state); 
    if (state & 0x40)
       printf("Hello1\n"); 
    if (state & 0x02)
        printf("Hello2\n"); 
    if (state & 0x01)
        printf("Hello3\n"); 
    if (state & 0x08)
        printf("Hello4\n"); 
    if (state & 0x20)
        printf("Hello5\n"); 
    if (state & 0x04)
        printf("Hello6\n"); 
  }  
   return 0;
}

j'obtiens

Connection state : 12
Hello2

C'est-à-dire en fait "INTERNET_CONNECTION_LAN " ce qui est correct.
Apparemment le 0x10 correspond à INTERNET_RAS_INSTALLED (Remote Access Service).

**Janitrix** · 15/10/2006, 21h21

Donc il faut tester avec des &.

Merci , ça marche ! Enfin... il trouve une correspondance. Mais pour comprendre mieux (sinon mon thread n'aurais servi à rien), que fait le '&' ?

Apparemment le 0x10 correspond à INTERNET_RAS_INSTALLED (Remote Access Service).

Où as tu trouvé ça, c'est pas sur MSDN... ?

**Emmanuel Delahaye** · 15/10/2006, 21h23

Envoyé par Janitrix

Mais pour comprendre mieux (sinon mon thread n'aurais servi à rien), que fait le '&' ?

C'est dans ton livre de C. C'est l'opérateur binaire ET.

Je te laisse réviser ton algèbre de Bool (Il y a quelques prérequis pour faire de la programmation...)