Discussion :

[alassanediakite]

Salut
Je cherche à lister les disques ou le contenu d'un répertoire selon que le paramètre reçu.
Voici le code qui me permet de lister les disques
un ensemble de codes pris sur le net puis assemblés Je suis donc ouvert à toute amélioration.

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
 
#include <tchar.h>
#include <stdio.h>
#include <windows.h>
void disquepret(TCHAR *d)
{
	int r;
	__int64 avlblspc, ttlspc, frspc;
 
r=GetDiskFreeSpaceEx(d, (PULARGE_INTEGER)&avlblspc, (PULARGE_INTEGER)&ttlspc, (PULARGE_INTEGER)&frspc);
if(r!=0){printf(" %I64dGO disponible", (ttlspc-frspc)/1000000000);}else{printf(" %s","Non disponible");}
}
 
void listerdisque()
{
    TCHAR lpBuffer[500], * p;
 
    GetLogicalDriveStrings(sizeof(lpBuffer), lpBuffer);
 
    for(p = lpBuffer; *p != '\0'; p += 4)
	{
		if(GetDriveType(p)==2)
		{printf("%s","Disque amovible");}
		if(GetDriveType(p)==3)
		{printf("%s","Disque local");}
		if(GetDriveType(p)==3||GetDriveType(p)==2)
        {
			disquepret(p);
			printf(" %s\n",p);
		}
	}
 
    return ;
}
 
void explorerdossier()
{
    HANDLE hEnt;
    WIN32_FIND_DATA ent;
 
    if ((hEnt = FindFirstFile(_T("*.*"), &ent)) != INVALID_HANDLE_VALUE)
    {
        do
            printf("%s\n", ent.cFileName);
        while (FindNextFile(hEnt, &ent));
 
        FindClose(hEnt);
    } 
    return;
}

Je voudrais gérer les erreurs générées par GetDiskFreeSpaceEx. Je pensais que "if(r!=0)..." suffisait.
Merci d'avance.
@+

[alassanediakite]

Salut
La solution (trouvée ici)

Code :

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

1
2
3
SetErrorMode(SEM_FAILCRITICALERRORS);
r=GetDiskFreeSpaceEx(d, (PULARGE_INTEGER)&avlblspc, (PULARGE_INTEGER)&ttlspc, (PULARGE_INTEGER)&frspc);
SetErrorMode(0);

@+

[Médinoc]

Bonjour,
J'ai quelques remarques:

1. Pourquoi caster les pointeurs au lieu d'utiliser des ULARGE_INTEGER pour de vrai?
2. d devrait être un pointeur const. p aussi, d'ailleurs.
3. GetLogicalDriveStrings() attend une taille en TCHARs, pas en bytes. Tu devrais donc utiliser la macro ARRAYSIZE() (définie en C comme #define ARRAYSIZE(x) (sizeof(x)/sizeof((x)[0]))) plutôt que l'opérateur sizeof seul.
4. Je ne suis pas sûr que p+=4 soit garanti marcher, pour passer à la chaîne suivante. Par contre, p += (_tcslen(p)+1) devrait marcher.
5. GetLogicalDriveStrings() retourne une valeur; il faut faire un test sur cette valeur, pour vérifier qu'elle est comprise entre 1 et ARRAYSIZE(lpBuffer)-1
6. lpBuffer est un tableau, non un pointeur. Si tu tiens à utiliser la notation hongroise selon les conventions de Microsoft, tu devrais l'appeler rgBuffer ("rg" étant l'abbréviation de "range", je pense) ou peut-être szzBuffer (chaîne terminée par deux caractères nuls).
7. La structure WIN32_FIND_DATA contient une chaîne de TCHARs: Pour l'afficher, tu devrais utiliser
8. _tprintf(_T("%s\n"), ent.cFileName);

Autrement, le code a l'air à peu près correct.

[alassanediakite]

Salut
Médinoc , pouvez-vous proposer un code avec vos remarques.
VS2010 me crie dessus quand j'essaye d’intégrer vos remarques. Peut-être des "include" manquants.
@+

[Médinoc]

Ceci compile sans problème et ne caus epas non plus d'erreurs d'édition de liens.
Je n'ai pas tenté de l'exécuter.

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
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
#include <windows.h>
#include <stdio.h>
#include <tchar.h> /*À inclure APRÈS le reste*/
 
/*Juste au cas où, mais normalement dans Visual c'est déjà défini*/
#ifndef ARRAYSIZE
#define ARRAYSIZE(x) (sizeof(x)/sizeof((x)[0]))
#endif
 
void disquePret(LPCTSTR diskName)/*Note: 'LPCTSTR' = 'const TCHAR *' */
{
	BOOL r;
	ULARGE_INTEGER availSpc = {0}, totalSpc = {0}, freeSpc = {0};
 
	r=GetDiskFreeSpaceEx(diskName, &availSpc, &totalSpc, &freeSpc);
	if(r) {printf(" %I64dGiO disponible", (freeSpc.QuadPart)/(2ull << 30)); }
	else { printf(" %s","Non disponible"); }
}
 
void listerDisques()
{
	TCHAR buffer[500] = _T("");
	TCHAR const* p;
 
	DWORD length = GetLogicalDriveStrings(ARRAYSIZE(buffer), buffer);
	if(length==0 || length >= ARRAYSIZE(buffer))
		printf("%s", "Echec de GetLogicalDriveStrings().");
 
	for(p = buffer; *p != _T('\0'); p += _tcslen(p)+1)
	{
		UINT driveType = GetDriveType(p); /*Pas la peinde de le rappeler 4 fois*/
		if(driveType==2)
		{ printf("%s","Disque amovible"); }
		if(driveType==3)
		{ printf("%s","Disque local"); }
		if(GetDriveType(p)==3 || GetDriveType(p)==2)
		{
			disquePret(p);
			_tprintf(_T(" %s\n"), p);
		}
	}
 
	return ;
}
 
void explorerdossier()
{
	HANDLE hEnt;
	WIN32_FIND_DATA ent = {0};
 
	if ((hEnt = FindFirstFile(_T("*.*"), &ent)) != INVALID_HANDLE_VALUE)
	{
		do
		{ _tprintf(_T("%s\n"), ent.cFileName); }
		while (FindNextFile(hEnt, &ent));
 
		FindClose(hEnt);
	}
	return;
}