[C] Problème dans mon code

Ca y est ca marche! en fait je n'avais pas bien saisie l'utilité des sizeof de ces fonctions, voici mon code final qui marche :

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[])
{
    FILE *fichier = NULL, *cpyFichier = NULL ;
    char  buffer[16900];
    fichier = fopen("calculatrice.exe", "rb+");
        if(fichier == NULL)
           exit(-1); 
    cpyFichier = fopen("test.exe","wb+");
        if(cpyFichier == NULL)
           exit(-1); 
     
              if(fread(buffer, sizeof(buffer), sizeof(char), fichier) != 0);
              fwrite(buffer, sizeof(buffer), sizeof(char), cpyFichier);
    
fclose(fichier);
fclose(cpyFichier);
  system("PAUSE");	
  return 0;
}

fiddy, dubcek et mype merci de m'avoir aider je n'y serai pas arriver sans vous :), je passe le sujet en résolu.

Re,
cpyfichier est ouvert en lecture et non écriture. Ouvre le avec le droit "w" plutôt que "r".
De plus c'est mieux de faire :
fopen(fichier,...);
if(fichier ==NULL) exit(-1);
fopen(cpyfichier,...);
if(cpyfichier==NULL) exit(-1);

Sinon t'auras des messages d'erreur, en voulant fermer le fichier par exemple dans ton cas.

Tu as toujours une erreur dans fread et fwrite avec les sizeof.
sizeof(int) renvoie 4 et sizeof(char) renvoie 1, donc tu ne vas pas copier une grande chaîne.
Peut-être que tu voulais faire sizeof(char), sizeof(buffer)/sizeof(char)

je poste mon nouveau code :

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char *argv[])
{
    FILE *fichier = NULL, *cpyFichier = NULL ;
    char  buffer[5000];
    int continuer = 4;
    fichier = fopen("calculatrice.exe", "rb+"); //ouverture de calculatrice.exe
    cpyFichier = fopen("test.exe","rb+"); //ouverture d'un .exe vide (fichier .txt renommé en .exe)
    if(fichier && cpyFichier != NULL)
     {
              if(fread(buffer, sizeof(int), sizeof(char), fichier) != 0); //copie le contenu du fichier dans le buffer
              fwrite(buffer, sizeof(int), sizeof(char), cpyFichier); //copie le buffer (donc le fichier calculatrice .exe) dans le .exe vide
     }
    
fclose(fichier);
fclose(cpyFichier);
  system("PAUSE");	
  return 0;
}

il devrait marcher mais je ne comprends pas pourquoi il ne marche pas :(

ah oui j'avais mal compris la réponse de dubcek donc après plusieurs essai j'ai réussi à ouvrir un executable, copier son contenu dans le buffer et enregistrer celui ci dans un fichier .txt que j'avais au préalable renommé en .exe, mais voila lorsque je lance ce dernier rien ne se passe (la console disparait en un clin d'oeil) je pense que ca vient du fait que j'ai écris dans un fichier texte malgré l'avoir renommer en .exe
j'aimerai savoir si mon problème vient bien de là et comment faire pour y remédier.

j'espère avoir été clair, faite le moi savoir si ca n'est pas le cas.

fread suffit puisqu'il va stocker les elements lu dans buffer pas besoin de metre buffer = fread

ah d'accord donc si je veux lire les éléments du fichier (et pas le nombre qu'il y en a) quel fonction je doit utiliser ?

hello
fread retourne le nombres d'éléments lus donc buffer = fread .. est incorrect

Prends en compte les remarques que je t'ai fait dans le post précédent. Poste la correction apportée à ton code, et fais un copier coller du message d'erreur car là tu ne me donnes pas le numéro de la ligne.

ah oui dsl, il me dit : incompatible types in assigment

EDIT : l'erreur se trouve à la ligne 13

Pourrais-tu donner le message que retourne ton compilateur, sinon c'est dur d'aider ^^.
Sinon il y a des erreurs dans ton code.
cpyFichier n'est pas initialisé avec fopen.
sizeof(continuer) va retourner 4, et non 1.
Cdt

merci fiddy je viens d'allouer le tableau mais ca ne marche toujours pas :(

Salut,
Tu dois allouer le tableau buffer ;)
Cdt