Existance d'un fichier, d'une apps.

Version imprimable

Voir 40 message(s) de cette discussion en une page

Voila mon probleme est tout simple, je veux savoir si une application d'on je connaitrais l'@ existe.
J'ai essayer ceci, sans grand sucses...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 void main() { char NOM_FICHIER[5][BUFSIZ],dossier_actuel[BUFSIZ]="\0",dossier_temp[BUFSIZ]="\0"; FILE *P_FICHIER; /* pointeur sur FILE */ GetCurrentDirectory( 260 ,Dossier_actuel); sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe"); printf("%s ",dossier_temp); P_FICHIER = fopen(dossier_temp,"r"); if ( P_FICHIER != NULL ) { fclose(P_FICHIER); printf("1 : Boum.exe"); //... } //... }
Mon "printf()"affiche le bon chemin, le fichier existe mais la condition n'est pas validée.
Une Idée ?

06/08/2006, 16h20
David.Schris

Citation:

Envoyé par miron

Une Idée ?

Deux :
- main() doit retourner un int (rien à voir avec ta question) ;
- ajouter un "#include <errno.h>" et un "extern int errno;" au début de ton code, puis faire un " perror("erreur"); " quand l'ouverture échoue (sans rien faire entre l'échec et l'appel à perror() !) et nous donner le message affiché par perror().
06/08/2006, 16h30
miron

Citation:

Envoyé par David.Schris

Deux :
- main() doit retourner un int (rien à voir avec ta question) ;

C'étais pour faire un programme pour vous qui soit compilable.
Moi depuis un certain temps je mets tj int.

Citation:

- ajouter un "#include <errno.h>" et un "extern int errno;" au début de ton code, puis faire un " perror("erreur"); " quand l'ouverture échoue (sans rien faire entre l'échec et l'appel à perror() !) et nous donner le message affiché par perror().

Citation:

Envoyé par perror()

Erreur: Invalid argument
06/08/2006, 16h35
_kal_
A tout hasard, est-ce normal que dossier_actuel ait une majuscule à la ligne :
Code:

1 2 GetCurrentDirectory( 260 ,Dossier_actuel);
Le C est sensible à la casse
06/08/2006, 16h43
miron
Citation:
Envoyé par _kal_

A tout hasard, est-ce normal que dossier_actuel ait une majuscule à la ligne :

Code:

1 2 GetCurrentDirectory( 260 ,Dossier_actuel);

Le C est sensible à la casse
Je sais, et je pensse de plus que c'est fort possible que tu ait raison, je vais verifier.
Merci.
PS: Ca ne resout qd meme pas mon probleme!

VERIFICATION
Apres verif en fait non car Dossier_actuel est un pointeur sur dossier_actuel, donc
Dossier_actuel = @ de la premiere case du tableau dossier_actuel[].
Et
dossier_actuel = @ de la premiere case du tableau dossier_actuel[] aussi.
D'où cela ne change rien.
06/08/2006, 16h51
_kal_

Citation:

Envoyé par miron

Je sais, et je pensse de plus que c'est fort possible que tu ait raison, je vais verifier.
Merci.
PS: Ca ne resout qd meme pas mon probleme!

Je ne fait pas de programmation Windows, donc je ne connais pas la fonction GetCurrectDirectory. Est ce que celle-ci retourne une valeur ? Si oui, tu pourrai peut-être la tester avant d'ouvrir ton fichier.
06/08/2006, 16h52
miron

Citation:

Envoyé par _kal_

Je ne fait pas de programmation Windows, donc je ne connais pas la fonction GetCurrectDirectory. Est ce que celle-ci retourne une valeur ? Si oui, tu pourrai peut-être la tester avant d'ouvrir ton fichier.

Oui il me semble quelle retourne une valeur, mais peu importe puisquelle marche.
Comme je lai dit le chemin d'acces au fichier est le bon.
06/08/2006, 16h57
David.Schris

Citation:

Envoyé par miron

C'étais pour faire un programme pour vous qui soit compilable.

Mauvaise excuse : un main() retournant un int ne rend pas un programme non compilable (mais un main() retournant un int sans "return"...). Ce n'est pas pour t'embêter, c'est pour que tu prennes de bonnes habitudes.

Citation:

Envoyé par miron

Citation:

Envoyé par perror()

Erreur: Invalid argument

Citation:

Envoyé par man errno

EINVAL Argument invalide.

Citation:

Envoyé par man fopen

ERREURS
EINVAL Le mode fourni à fopen, fdopen, ou freopen était invalide.

Mmmh... Je reste perplexe...

Qu'affiche exactement (fais un copier/coller) ton " printf("%s ",dossier_temp); " ? (pense aussi à mettre la sortie de ton programme quand tu as un rpoblème, cela peut aider)

Est ce que tu initialises errno à 0 avant l'instruction :

Code:

1
2
 
P_FICHIER = fopen(dossier_temp,"r");

Personnellement, je ferai :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
int main()
{
  char NOM_FICHIER[5][BUFSIZ],dossier_actuel[BUFSIZ]="\0",dossier_temp[BUFSIZ]="\0";
  FILE *P_FICHIER;      /* pointeur sur FILE */
  GetCurrentDirectory( 260 ,Dossier_actuel);
  sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
  printf("%s ",dossier_temp);
  errno = 0;
  P_FICHIER = fopen(dossier_temp,"r");
  if (P_FICHIER == NULL)
  {
     perror(dossier_temp);
     exit(EXIT_FAILURE);
  }
  if ( P_FICHIER != NULL ) 
   { fclose(P_FICHIER);
      printf("1 : Boum.exe");
        //...   }
  //...
 
  return EXIT_SUCCESS;
}

06/08/2006, 17h02
miron

Citation:

Envoyé par David.Schris

Mauvaise excuse : un main() retournant un int ne rend pas un programme non compilable (mais un main() retournant un int sans "return"...). Ce n'est pas pour t'embêter, c'est pour que tu prennes de bonnes habitudes.
Mmmh... Je reste perplexe...
Qu'affiche exactement (fais un copier/coller) ton " printf("%s ",dossier_temp); " ? (pense aussi à mettre la sortie de ton programme quand tu as un rpoblème, cela peut aider)

Ces dernieres lignes mon fait comme un electrochoc, et j'ai penssasi dou pouvais provenir le probleme !!
Ce n'est pas

Code:

sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

Mais

Code:

sprintf(dossier_temp,"%s%s",dossier_actuel,"\\Boum\\Boum.exe");

Quil fallais mettre!!!
Et la le code d'erreur est le bon:

Citation:

No such file or directory

(car pour essais je l'ai suprimé)
Donc un grand :merci:
06/08/2006, 17h05
_kal_
Il y a un truc aussi qui m'intrigues, c'est l'utilisation de sprintf avec dossier_temp qui est de taille BUFSIZE :
Code:

1 2 sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
dossier_temp est un tableau de char de même taille que dossier_actuel. Or, tu va stocker dans ce tableau une chaine dont on ne connais pas la taille. Utilise plutot snprintf, qui est plus sur :
Code:

1 2 snprintf(dossier_temp, sizeof dossier_temp, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
06/08/2006, 17h09
miron
Citation:
Envoyé par _kal_

Il y a un truc aussi qui m'intrigues, c'est l'utilisation de sprintf avec dossier_temp qui est de taille BUFSIZE :

Code:

1 2 sprintf(dossier_temp,"\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");

dossier_temp est un tableau de char de même taille que dossier_actuel. Or, tu va stocker dans ce tableau une chaine dont on ne connais pas la taille. Utilise plutot snprintf, qui est plus sur :

Code:

1 2 sprintf(dossier_temp, sizeof dossier_temp, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
8O
sprintf ou snprintf.???
06/08/2006, 17h12
_kal_
Citation:

Envoyé par miron

8O
sprintf ou snprintf.???

snprintf pardon, le 'n' a sauté au passage. D'ailleurs, ca m'étonne que ton programme ait reussi l'instruction du sprintf sans broncher.

Aussi, ici j'ai utilisé sizeof dossier_temp dans l'expression avec snprintf, car ton tableau est de type char. Tu peux tres bien mettre :
Code:

1 2 snprintf(dossier_temp, BUFSIZ, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
Ou encore :
Code:

1 2 snprintf(dossier_temp, sizeof dossier_temp / sizeof dossier_temp[0], "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
Si ton tableau avait été d'un autre type que char, il n'aurai surtout pas fallu utiliser sizeof(dossier_temp) tout seul.

EDIT : suppression du E à BUFSIZE, ça risquerai d'en troubler quelque uns ;)
06/08/2006, 17h14
David.Schris

Citation:

Envoyé par miron

8O
sprintf ou snprintf.???

snprintf()

Citation:
Envoyé par _kal_
Code:

1 2 snprintf(dossier_temp, BUFSIZE, "\"%s%s\"",dossier_actuel,"\\Boum\\Boum.exe");
Pourquoi BUFSIZE et pas BUFSIZ

06/08/2006, 17h41
_kal_

Citation:

Envoyé par miron

Pourquoi BUFSIZE et pas BUFSIZ

Effectivement, BUFSIZ. D'ailleurs, c'est une macro que tu définie toi même ?
06/08/2006, 18h01
miron

Citation:

Envoyé par _kal_

Effectivement, BUFSIZ. D'ailleurs, c'est une macro que tu définie toi même ?

Euuu
Arrette tu va trouver une faille dans mon programme qui fonctionne tres bien d'ailleur...
06/08/2006, 18h09
_kal_
Citation:

Envoyé par miron

Euuu
Arrette tu va trouver une faille dans mon programme qui fonctionne tres bien d'ailleur...

C'est mon objectif. Un programme peut sembler bien fonctionner à première vue, et disposer de trou de sécurité dangereux.
Un conseil : evite d'utiliser les fonctions de copie qui n'établissent aucun contrôle sur le nombre d'élement copié.

Exemple :
- sprintf -> snprintf
- strcpy -> strncpy
- gets -> fgets
Cela t'evitera des mals de têtes à l'avenir ;)
06/08/2006, 19h52
miron
Citation:
Envoyé par _kal_

C'est mon objectif. Un programme peut sembler bien fonctionner à première vue, et disposer de trou de sécurité dangereux.
Un conseil : evite d'utiliser les fonctions de copie qui n'établissent aucun contrôle sur le nombre d'élement copié.

Exemple :

sprintf -> snprintf
strcpy -> strncpy
gets -> fgets

Cela t'evitera des mals de têtes à l'avenir ;)
En fait sprintf m'à étais conseillé.
06/08/2006, 19h54
_kal_

Citation:

Envoyé par miron

En fait sprintf mas aiai conseillé.

Pas comprendre :aie:

Voir 40 message(s) de cette discussion en une page

Fuseau horaire GMT +2. Il est actuellement 05h40.