Discussion :

[0v3rb1t]

Bonjour.

Je viens de créer un programme qui est censé dupliquer une image (.JPG) qui aura été glissée sur l'exécutable de ce programme.
Le problème est que la copie n'a pas lieu.
Toute aide serait la bienvenue.

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
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
        int bit = 0;
        FILE *fichier_source = NULL, *fichier_copie = NULL;
 
        if(argc < 2)
        {
            puts("Faites glisser le fichier image sur l'executable.");
            getchar();
            return 0;
        }
 
 
        fichier_source = fopen(argv[1], "rb");
 
        if(fichier_source==NULL)
        {
            fprintf(stderr, "erreur lors de l'ouverture du fichier source") ;
            exit(EXIT_FAILURE);
        }
 
 
        fichier_copie = fopen("fichier_copie.jpg", "wb");
 
    if(fichier_copie==NULL)
        {
            fprintf(stderr, "erreur lors de l'ouverture du fichier copie") ;
            fclose(fichier_source);
            exit(EXIT_FAILURE);
        }
 
        while(bit != EOF)
        {
            bit = fgetc(fichier_source);
            fputc(bit, fichier_copie);
        }
 
        fclose(fichier_source);
        fclose(fichier_copie);
 
        return 0;
}

[souviron34]

bit est initialisé à 0..

Si tu fais :

Code :

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

1
2
3
4
 
        bit = 1 ;
        while(bit != EOF)
        {

je pense que ça devrait passer...

[0v3rb1t]

Le fait d'initialiser bit à 1 ne change rien.

C'est peut-être les espaces dans les noms des dossiers?

[souviron34]

Le fait d'initialiser bit à 1 ne change rien.

C'est peut-être les espaces dans les noms des dossiers?

??

Comment est ton argv[1] ??

[josse95]

Je ne suis pas sûr que tu pourras voir tes messages d'erreur, vu la façon dont tu les affiches. Ajoute une saisie clavier avant chaque exit pour être sûr de les voir. Cela t'aidera peut-être à cerner le problème.

Autre chose: recopier un fichier octet par octet, ce n'est peut-être pas ce qui se fait de plus performant !

[0v3rb1t]

J'ai rajouté des couples printf() et getchar() avant les exit() et le return de la fonction main:

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
{
        if(fichier_source==NULL)
        {
            printf("source");
            getchar();  
            fprintf(stderr, "erreur lors de l'ouverture du fichier source") ;
            exit(EXIT_FAILURE);
        }
 
        .....
 
        if(fichier_copie==NULL)
        {
            printf("copie");
            getchar();  
            fprintf(stderr, "erreur lors de l'ouverture du fichier source") ;
            exit(EXIT_FAILURE);
        }
 
        .....
 
        printf("bon deroulement");
        getchar();
        return 0;
}

"bon deroulement" s'affiche toujours quand je fais glisser une image sur l'exécutable mais l'image n'est pas copiée.

[josse95]

tu ne spécifies pas le path du fichier destination. Es-tu sûr qu'il n'est pas créé dans un autre répertoire que celui auquel tu t'attends ?

[souviron34]

si ton fichier source a des espaces dans le nom et que tu ne l'as pas mis entre guillemets, argv[1] ne contiendra qu'un parti du nom de fichier.

tu peux d'autre part remplacer tes :

Code :

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

1
2
3
 
            printf("source");
            getchar();

par

Code :

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

1
2
 
fprintf ( stderr, "source");

d'autre part, pour être certain de ce que je fais, je ferais :

Code :

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

1
2
3
4
5
6
7
 
        bit = fgetc(fichier_source);
        while(bit != EOF)
        {
            fputc(bit, fichier_copie);
            bit = fgetc(fichier_source);
        }

et vraisemblablement en debug une fois je ferais :

Code :

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

1
2
3
4
5
6
7
8
 
        bit = fgetc(fichier_source);
        while(bit != EOF)
        {
fprintf ( stderr, "b%d",bit);
            fputc(bit, fichier_copie);
            bit = fgetc(fichier_source);
        }

là tu vas bien voir si déjà tu y passes ou non.

et si tu y passes, c'est que josse à raison... et que ça n'écrit pas là ou tu cherches..

[0v3rb1t]

Je viens de spécifier le chemin absolu pour le fichier copie et tout ce passe bien, je m'attendais à ce que le programme créé un fichier dans son répertoire...étrange.

[0v3rb1t]

Personne ne peut me dire pourquoi le programme ne créé pas le fichier copie dans son répertoire?
Pourquoi faut-il spécifier le chemin absolu?
Ca m'aiderai vraiment.

[souviron34]

c'est pas "dans son répertoire" mais dans le répertoire dans lequel tu lances le programme.

Comme tu n'écrit pas de chemin du tout, il écrit par défaut là où il se trouve.. Logique...