Discussion :

[TheHitman]

Bonjour, je dois effectuer des tests sur un programme en C, j' ai un petit probleme sur la partie de code suivante. (Je tiens a preciser que ce n'est
pas moi qui ai code le programme, mon travail se resume au test )

Code :

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

1
2
3
4
5
6
7
8
9
 
    ...
    if(access(argv[1],R_OK)==0) /*check file available for reading*/
        {
          in = fopen(argv[1], "r");  /* open file for reading */
          if(in==NULL) printf("can't open file %s\n", argv[1]);
        }
        else printf("no file %s, ", argv[1]);
    ...

Donc, si mon parametre argv est un fichier qui n'existe pas, le programme m'affiche bien "no file blabla"
Par contre, est ce que c'est possible d'avoir un cas de test dans lequel la fonction access me retourne 0
mais la fonction fopen retourne null ?

Ce que j'ai reussi a faire, c'est inserer un getchar() comme suit :

Code :

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

1
2
3
4
5
6
7
 
    ...
    if(access(argv[1],R_OK)==0) /*check file available for reading*/
        {
          getchar();
          in = fopen(argv[1], "r");  /* open file for reading */
    ...

quand le programme attend sur le getchar(), j' efface mon fichier et donc j'ai mon cas de test. Mais en fait, je dois reussir a tester sans modifier le code.
La solution proposee doit fonctionner sur Linux

Merci à tous

PS: Desole clavier a la noix QWERTY

[ram-0000]

Lancer en background un programme qui créée le fichier et qui l'efface le plus vite possible (priorité élevée).

Ce n'est pas très déterministe et cela ne fait pas très "test unitaire" mais avec un peu de persévérance et de chance (pour toi) et de malchance pour le développeur, tu devrais avoir des cas ou tu reproduis ce que tu essayes de mettre en évidence.

[TheHitman]

En effet, je pourrai essayer... mais est ce que l'on peut dire catégoriquement si access dit OK alors fopen doit dire OK ?

[ram-0000]

Non puisqu'il a des cas ou le fichier peut être effacé entre le access() et le fopen().

Les 2 instructions ne sont pas atomiques ou alors, il faudrait poser un lock sur le file system (je ne sais pas si c'est possible) pour être sûr que le file system n'est pas modifié entre ces instructions. Mais bonjour le problème si tu tues le process pendant que le lock est posé et que cela ne libère pas le lock, il te restera alors que le bouton reset.

Je crois que cela s'appelle une "race condition" ce type de problème.

[matafan]

Pour ton test la seule façon systématique serait à mon avis d'exécuter le programme dans un débugger. Tu met un point d'arrêt sur le fopen, tu supprimes le fichier, puis tu reprends l'exécution. Le problème c'est que piloter le débugger depuis un script, c'est pas forcément facile.

[TheHitman]

Merci pour vos reponses, j'en deduis donc que dans le cadre d'une execution "normale", il y a seulement un faible risque que access soit ok mais fopen soit nok

[ram-0000]

Faible risque, peut être mais risque quand même.

Au fait, pourqui faire 2 tests ? Pourquoi ne pas supprimer l'appel à access() et utiliser seulement le fopen(). Ainsi, tu seras atomique.

[matafan]

En fait le truc c'est que le access() ne sert strictement à rien dans ce cas, puisque tout ce que fait access() est refait dans le fopen(). Donc effectivement, si j'étais toi je ne chercherai pas à tester ça.

[TheHitman]

Le but de l'exercice est justement de faire des "tests" . Donc oui access ne sert à rien ... c'est limite de la redondance si on fait fopen derrière. Mais malheureusement je ne peux pas toucher au code. Donc je déclarerais que la ligne de code ne sera probablement jamais exécuté.

Code :

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

1
2
 
if(in==NULL) printf("can't open file %s\n", argv[1]);

A la limite, c'est le but des tests, de trouver ce genre de ligne "inutile".

[droggo]

Jai,

Cette ligne n'est pas inutile, puisque fopen peut échouer (en plus d'un éventuel effacement du fichier entre access et fopen, le fichier peut parfaitement être interdit d'accès car verrouillé par un autre prog, par exemple).

[Emmanuel Delahaye]

Code :

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

1
2
3
 
    ...
    if(access(argv[1],R_OK)==0) /*check file available for reading*/

Pas de access() en C standard.

[TheHitman]

hmm c'est déjà plus intéressant ... dans le cas d'un fichier verrouillé par un autre prog je crois qu' access dis direct NOK non ? et l'éventuel effacement du fichier entre access et fopen est ce que j'essaye de reproduire ... mais comment ? on tourne en rond

EDIT : pas d'access en C standard ... je ne peux pas quand même demander au prof de changer le sujet non ? ,je suis persuadé que c'est fait exprés pour que l'on puisse pas tester cette branche.

[Emmanuel Delahaye]

EDIT : pas d'access en C standard ... je ne peux pas quand même demander au prof de changer le sujet non ? ,je suis persuadé que c'est fait exprés pour que l'on puisse pas tester cette branche.

Ca dépend. C'est un cours de C ou un cours de programmation système ?

[TheHitman]

Pire ... un cours de Génie Logiciel ... et d'ailleurs je suis content d'avoir
fait du C parce qu'au moins je remarque qu'il y a des problèmes à certains endroits (le programme est beaucoup plus long ... et il y a quelques cas de dépassements tableaux qui passent à l'exécution sans trop de problème, du
shiftage de bit farfelue, etc )