Discussion :

[alaparra]

bonjour ,

je suis en train de faire un ptit prog qui me permetrai de recup des info sur les differents process tournant sur une machine et pour recuperer le temps pendant lequel le process a tourner jutilise GetProcessTimes que jai trouver sur la msdn (http://msdn.microsoft.com/library/default.asp?url=/library/en-us/dllproc/base/getprocesstimes.asp)

mais jai un souci
voila la partie de mon code ou je cherche le temp :

Code :

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

1
2
3
4
5
6
7
8
9
 
 
    hProcess = OpenProcess(  PROCESS_QUERY_INFORMATION |
                                    PROCESS_VM_READ,
                                    FALSE, processID );
    if (NULL == hProcess)
       return ""; 
if(GetProcessTimes(hProcess,lpCreationTime,lpExitTime,lpKernelTime,lpUserTime)!=0){}
      cout<<"time::>"<<lpUserTime->dwLowDateTime;

pour l'erreur ca me dit que je ne peut pas ecrire a ladresse memoire 0xcccccccc


si qqun a deja utiliser cette fonction et vois ou jai oublier kkechose merci davance

[Aurelien.Regat-Barrel]

Pointeur non initialisé sûrement... Tu dois donner l'adresse de strcutures à remplir et non des adresses qui pointent nul part:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
FILETIME CreationTime;
FILETIME ExitTime;
FILETIME KernelTime;
FILETIME UserTime;
 
GetProcessTimes(
    hProcess,
    &CreationTime,
    &ExitTime,
    &KernelTime,
    &UserTime
);

[alaparra]

c'est surement une question conne mais comment j'initialise mes pointeurs je leur donne pas de valeur moi c'est des valeurs que je recupere en sortie de la fonctions non ?

[Laurent Gomila]

Comme l'a dit et illustré Aurélien, tu dois passer des adresses de structures et non des pointeurs déclarés à l'arrache. Regarde son exemple.

[alaparra]

enfait l'initialisation je l'ai deja faite voila le code de ma fonction :

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
 
void GetTemp(DWORD processID)
{
 
  HANDLE hProcess;
  LPFILETIME lpCreationTime=NULL;
  LPFILETIME lpExitTime=NULL;
  LPFILETIME lpKernelTime=NULL;
  LPFILETIME lpUserTime=NULL;
 
 
	 hProcess = OpenProcess(  PROCESS_QUERY_INFORMATION |
                                    PROCESS_VM_READ,
                                    FALSE, processID );
 
  if (NULL != hProcess)
  {
  	if (GetProcessTimes(hProcess,lpCreationTime,lpExitTime,lpKernelTime,lpUserTime)!=0)
			cout<<"time ::>"<<lpUserTime->dwLowDateTime;
 
 
  }
  CloseHandle(hProcess);
}

en fait mon erreur arrive des que mon hprocess est different de 0 c'est à dire que des que la fonction GetProcessTimes est lancer mon executable plante.

merci de votre aide parceque la je comprend pas :p

je suis pas tres douer en programmation c++ je commence et je galere un peu avec ces fonctions que je trouve sur la msdn pour la gestion des process

[Aurelien.Regat-Barrel]

Tu n'as pas tout compris je crois...
Ca plante si hProcess n'est pas nul car tu as mis un test si différent de nul... Tes pointeurs sont initialisés à NULL, c'est à dire à rien.
Tes pointeurs ils servent à donner l'adresse d'une structure à remplir, ils doivent donc pointer vers une structure que GetProcessTimes va remplir.
Toi tu pointes vers NULL = rien, le vide intersidéral en quelque sorte.
pointeur sur le type X doit être initialisé avec l'adresse d'une variable du type X. Dans ton cas, ben le pointeur ne sert à rien en fait, il te faut juste passer l'adresse des structures comme je te le montre dans mon exemple.
Un petit conseil de lecture:
http://chgi.developpez.com/pointeur/

[alaparra]

c'est bon merci en fait me suffisai juste de faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
HANDLE hProcess;
  PFILETIME tempCreation;
  PFILETIME tempExit;
  PFILETIME tempKernel;
  PFILETIME tempUser;
 
  tempCreation= new FILETIME;
  tempExit = new FILETIME;
  tempKernel = new FILETIME;
  tempUser = new FILTIME;

merci de votre aide

[Laurent Gomila]

Non... Tu te compliques la vie pour rien.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
FILETIME CreationTime;
FILETIME ExitTime;
FILETIME KernelTime;
FILETIME UserTime;
 
GetProcessTimes(
    hProcess,
    &CreationTime,
    &ExitTime,
    &KernelTime,
    &UserTime
);

[Aurelien.Regat-Barrel]

Tu te compliques, et tu crées des fuites de mémoire.

[alaparra]

non ca marche pas ca en fait il me faisai chier car il maddressai mal mon pointeur et essayer decrir a une adresse ou javai pas les droits decriture si je faisai pas les new

[Laurent Gomila]

Je crois que tu as mal lu. Il y a une différence entre

Code :

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

1
2
LPFILETIME CreationTime; // équivalent à LIFETIME* CreationTime
GetProcessTimes(..., CreationTime, ...);

et

Code :

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

1
2
FILETIME CreationTime;
GetProcessTimes(..., &CreationTime, ...);

Dans le 1er cas on envoie un pointeur non initialisé à la fonction, dans le second non (puisqu'on lui envoie l'adresse d'une variable).

[Aurelien.Regat-Barrel]

il maddressai mal mon pointeur et essayer decrir a une adresse ou javai pas les droits decriture si je faisai pas les new

C'est ça qu'on appelle un pointeur non initialisé. Il pointe n'importe où. Ce qui est différent d'un pointeur initialisé à NULL.
Toi tu voulais

Code :

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

1
2
 
LPFILETIME lpCreationTime;

C'est équivalent à :

Code :

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

1
2
 
FILETIME * lpCreationTime;

le "LP" devant FILETIME signifie "Long Pointer" = pointeur sur FILETIME
Pour l'initialiser:

Code :

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

1
2
3
4
 
LPFILETIME lpCreationTime = new FILETIME;
...
delete lpCreationTime;

la même chose sans allocation dynamique mais en allouant sur la pile:

Code :

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

1
2
3
 
FILETIME CreationTime;
LPFILETIME lpCreationTime = &CreationTime;

et encore en plus simple utiliser directement &CreationTime là ou on attend un LPFILETIME...

[alaparra]

ha ok bah merci pour ton aide precieuse :p

et en fait je suis degouter la fonction fait pas ce que jattend d'elle mais bon jaurai appri kkes truc ca le fait