poblème avec FileSearch et wildcard (*)

Version imprimable

Bonjour,

j'essaie de tester l'existence de tout fichier correspondant au format suivant iview*.exe dans un répertoire d'un réseau.

J'obtiens en retour une chaine vide.
Quand je précise le nom exact d'un fichier, j'obtiens bien en retour le fullname de l'objet recherché.

Comment puis-je faire pour simuler un espèce de dir et récupérer le premier fullname de l'objet qui correspond à ma recherche ?

Voici mon code

Ceci fonctionne :
Code:

1 2 3 4 Setup_Converter = "iview428_setup.exe"; Setup_Path = "Y:\\99_Varions\\Free_Software\\Irfanview\\"; asFileName = FileSearch(Setup_Converter, Setup_Path); // Setup Exists ? if (asFileName.IsEmpty())
mais pas ceci :

Code:

Setup_Converter = "iview*.exe";

Merci d'avance pour vos lumières :roll:

NB : ce n'est pas un problème de CurrentDrive. J'ai fait le test avec et sans ChgCurrentDrive.

Voilà,

il fallait utiliser FindFirst.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
	  // look for Any Converter Setup Program
	  Setup_Converter       =  "iview*.exe";
	  Setup_Path            =  "Y:\\99_Various\\Free_Software\\Irfanview\\";
 
 
	  if (FindFirst(Setup_Path+Setup_Converter, faAnyFile, Infos_Files))
		{
		  return false;
		}
		else
		{                                          // Setup Irfanview
		   asFileName = Infos_Files.Name;          // Get Name from First find
		   asFileName = Setup_Path + asFileName;   // Add Full path

19/05/2011, 14h41
ShaiLeTroll
Question visibilité, je trouve plus logique de mettre d'abord le code en cas de bon fonctionnement d'une fonction, même si le retour est 0 comme NO_ERROR ou ERROR_SUCCESS déclaré dans win.h qui est le cas pour FindFirst
Code:

1 2 3 4 5 6 SI TEST ALORS ça c´est bien passé SINON ouch ! erreur !
Code:

1 2 3 4 5 6 7 if (FindFirst(...) == ERROR_SUCCESS) { asFileName = Infos_Files.Name; ... } else return false;
bcp utilise " ! " au lieu de " == 0 "
En tant Delphiste, cela me surprend un peu, mais je m'habitue !
19/05/2011, 14h51
Godzestla

Citation:

Question visibilité, je trouve plus logique de mettre d'abord le code en cas de bon fonctionnement d'une fonction, même si le retour est 0 comme NO_ERROR ou ERROR_SUCCESS déclaré dans win.h qui est le cas pour FindFirst

J'approuve et j'adapte en conséquence.
L'inconvénient que j'y vois c'est que si le code est long, le else se perd un peu dans la masse et devient alors moins lisible, tandis qu'avec le teste inversé, on a toujours une vue sur les 2 cas.

Mais c'est très relatif !!
20/05/2011, 12h05
ShaiLeTroll

Après, il y a les extrémistes de la fonction de 25 lignes maximum !
J'ai rarement des boucles ou des imbrications qui dépasse une page écran
En Delphi, je fais des sous-fonctions, cela permet d'avoir un code principal de la fonction très court et lisible

Une règle important à mon avis, c'est la séparation du code !
Il faut éviter de mélanger les choses, rien est plus horrible qu'un code qui mélange un peu tout et n'importe quoi !

Ton code peut-être réutilisable, une fonction FileExistsMask par exemple, dans une unité GZFileUtils, perso, j'ai fait mes fonctions GetFileDir, GetDirDir... qui remplisse une TStringList, le code est certe un peu plus long car découper, utilise des objets intermédiaires donc plus lent, mais mieux vaut un code maintenable qui répond au besoin du client, qu'un code fouilli qui certe répond au besoin mais dont l'évolution sera pénible et soumise au bug !

Excellente remarque, d'ailleurs je tend à cela, mais tu te doutes qu'en tant que total débutant total en C++, cela fait un monde à assimiler pour moi.

Mais, je ne suis pas mécontent de mon autodidactisme assisté. J'arrive a récupérer un fichier PCX, à le convertir avec un soft externe (irfanview) en .JPG, tout en ayant vérifié l'existence de irfanview et installé si pas présent, puis à afficher mon Jpg généré.

Semaine rude, mais avec une grande progression.

Ce que je ne sais pas faire c'est justement externaliser mes fonctions pour les rendre accessibles à d'autres codes que je ferai plus tard.

Par exemple, si je veux prendre ceci et le mettre en dehors de mon code actuel, je fais comment ? :

Code:

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
//---------------------------------------------------------------------------
//# bool TForm1::ExecuteWait(Prog , Parms, Voir)
//   Execute an application and Wait until completion
//   "Voir" Allows display or not
//---------------------------------------------------------------------------
bool __fastcall  TForm1::ExecuteWait(String Programme,String  parametres, bool voir)
{
  SHELLEXECUTEINFO execinfo;      // Structure ShellExecute(Ex) info
 
  //typedef struct _SHELLEXECUTEINFO {
  //	DWORD cbSize;
  //	ULONG fMask;
  //	HWND hwnd;
  //	LPCTSTR lpVerb;
  //	LPCTSTR lpFile;
  //	LPCTSTR lpParameters;
  //	LPCTSTR lpDirectory;
  //	int nShow;
  //	HINSTANCE hInstApp;
  //	LPVOID lpIDList;
  //	LPCTSTR lpClass;
  //	HKEY hkeyClass;
  //	DWORD dwHotKey;
  //	union {
  //		HANDLE hIcon;
  //		HANDLE hMonitor;
  //	} DUMMYUNIONNAME;
  //	HANDLE hProcess;
  //} SHELLEXECUTEINFO, *LPSHELLEXECUTEINFO;
 String action;                                 // action to run
 
  action =  "open";
  memset(&execinfo, 0, sizeof(execinfo));
  execinfo.lpFile = Programme.c_str();          // programme
  execinfo.cbSize = sizeof(execinfo);
  execinfo.lpVerb = action.c_str();             // Action = Open
  execinfo.fMask = SEE_MASK_NOCLOSEPROCESS;
  execinfo.nShow = SW_SHOWDEFAULT;
  execinfo.lpParameters = parametres.c_str();   // Parms
  if ((voir)){                                  // Show Process or not
	 execinfo.nShow = SW_SHOW;
	}
	else {
	 execinfo.nShow = SW_HIDE;
  }
  if (ShellExecuteEx(&execinfo)) {
		WaitForSingleObject(execinfo.hProcess, INFINITE);
		return TRUE;
	  }
	  else {
		return FALSE;
   }
 
}

20/05/2011, 14h01
ShaiLeTroll

Pour externaliser
Tu peux faire une simple série de fonction sans classe (pas de :: devant le nom de la fonction)

Dans ton code, la classe c'est TForm1, l'étape suivante serait de créer tes propres classes en ayant que des includes de bcb mais aucune du projet lui même pour éviter les dépendances !
Ainsi une même unité pourrait servir dans plusieurs projets !
20/05/2011, 14h31
Godzestla

Merci,

je comprend le principe, mais pas dans la pratique.

Mettons que j'ai 10 fonctions se rapportant à 2 nature différentes.
Donc je fais 2 classes.

Je prend la classe 1, avec par exemple 4 fonctions.

Je met tou cela où et comment ?
Dans un nouveau unit avec son .h ?
Dans quel répertoire ?

Je les inclus par un simple include ?
Comment le compilateur les trouve - t 'il ?

Désole si les questions sont basiques, mais je ne vois pas trop bien. :aie:
20/05/2011, 14h57
ShaiLeTroll

C'est cela, un thème par classe, pour "ranger" chaque chose qui se ressemble
Attention, la classe cela peut-être pénible, faut l'instancier ou faire du static, le procédural peut avoir du bon parfois pour encapsuler des fonctions comme FindFirst et les API Windows en général ...

En Delphi, c'est plus simple, un seul fichier, le .pas, suffit d'ajouter un repertoire dans les chemins de bibliothèque, et ça compile

Pour le répertoire, tu peux te faire un dossier genre "commun" ou tu regroupes tes unités communes à plusieurs projets
Si tu fais une classe, le .h contient les protoypes, le .cpp contient l'implémentation !

Pour l'inclusion, en C++Builder, j'ai une peu de mal par rapport à Delphi, include c'est pour le ".h" mais je ne sais pas spécifier du chemin pour cpp (surtout pour qu'il les compils) alors je les ajoutes au projet tout simplement !
Tu peux aussi fournir le fichier obj (au-lieu du cpp) et le .h, ainsi c'est gérer à la liaison et non à la compilation

Faudrait voir en ajoutant dans "Outils > Options > Options d'environnement > Options C++ > Chemins et répertoires"
Chemin d'inclusion = .h
Chemin de bibliothèque = .???
Chemin à parcourir = .cpp ???

Tu as aussi Chemins et répertoires dans les Options du Projet