Discussion :

[Watilin]

Bonsoir,
je m'arrache les cheveux depuis trois jours sur un problème assez bête : quand je lis un fichier sur l'entrée standard, dès qu'il dépasse le millier d'octets, il est tronqué.
Je voudrais pouvoir utiliser les redirections avec mon programme, par exemple :

Code :

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

mon_programme < gros_fichier.txt

mais visiblement je ne m'y prends pas de la bonne manière.

Voici un code simpliste avec lequel je peux reproduire le problème :

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
// affiche la taille d'un fichier
#include<stdio.h>
#define BUFFER_SIZE 1024
 
char buffer[BUFFER_SIZE];
int main() {
   int n;
   unsigned long long int i = 0;
   while (n = fread(buffer, sizeof(char), BUFFER_SIZE, stdin)) {
      i += n;
   }
   printf("%llu\n", i);
 
   return 0;
}

Pour les fichiers de moins de 1 Ko, ça m'affiche la bonne taille. Ça marche aussi avec un fichier un peu plus gros que mon buffer, 1197 octets. Mais pour les fichiers plus gros, j'obtiens des valeurs toujours inférieures à 1024.
J'ai essayé aussi avec fgetc, même résultat. En revanche, pas de problème quand j'ouvre le fichier directement avec fopen, mais alors je ne peux pas utiliser les redirections…

Si ça peut aider, je suis sous Windows. J'ai essayé sans succès sous cmd et Powershell – mais j'ai l'intuition que ça ne dépend pas du système. C'est agaçant, je suis sûr que je passe à côté d'un truc évident… J'ai besoin de votre aide !

[Melem]

Je vois déjà deux problèmes :

- La valeur initiale de i doit être égale à 0 et non -1. Avec ton code actuel il manquera toujours au moins 1 octet à la taille affichée. En plus i est censé être non signé ...

- stdin est toujours ouvert par défaut en mode texte, alors qu'un fichier doit être ouvert en mode binaire pour qu'on puisse obtenir correctement sa taille. En clair, ce code n'a aucune chance de fonctionner.

A part ça, il faut utiliser %llu pour afficher un long long unsigned int et mettre return 0 si tu veux faire plus joli.

[Watilin]

Ok, je corrige pour la postérité. C'est un code censé reproduire le problème, autant qu'il le fasse correctement '^^

Pour le -1 je me suis dit, peu importe qu'il soit unsigned ou pas puisque c'est représenté par quelque chose comme FFFFFFFF, et dans tous les cas, si on rajoute 1 ça fait 0.
Et pourquoi -1 ? Je l'avais mis pour tester fgetc, j'ai juste oublié de le remettre à 0…

Mais avec tout ça, on parle de la forme et pas du fond

[matafan]

Ce code fonctionne correctement sous linux, donc je pense que Melem est dans le vrai quand il parle de lire en mode binaire.

Tu pourrais commencer par tester feof() pour voir si effectivement tu es à la fin du fichier, ou si tu es sorti en erreur.

[Melem]

Mais avec tout ça, on parle de la forme et pas du fond

On parle bien du fond parce que i doit être initialisé à 0 pour espérer avoir un résultat correct et parce que je te dis qu'il faut ouvrir le fichier en mode binaire donc tu dois utiliser fopen, autrement ça ne marchera jamais.

[Watilin]

Est-ce que ça signifie que le C ne permet pas, sous Windows, de traiter des gros fichiers en utilisant les redirections ?

Pour situer un peu plus les choses, j'aimerais pouvoir utiliser mon programme dans une chaîne avec le pipe :

Code :

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

prog1 | prog2 | prog3 | etc.

[Melem]

La taille du fichier n'a rien à voir. Le problème c'est que stdin est toujours ouvert par défaut en mode texte, ce qui ne permet pas de calculer correctement la taille d'un fichier (3ème fois que je le répète ). Sous Unix, il n'y a pas de différence entre un fichier texte et binaire mais sous les autres systèmes, cette distinction existe la plupart du temps.

Pour situer un peu plus les choses, j'aimerais pouvoir utiliser mon programme dans une chaîne avec le pipe :

Code :

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

prog1 | prog2 | prog3 | etc.

Alors fais get_file_size input.txt | prog2 | prog3 | etc. au lieu de get_file_size < input.txt, comme ça tu pourras utiliser fopen ...