Bonjour, avec ce bout de code:

	signed char* temp;
		temp = fgets(temp, 100, file);
		fseek(file, SEEK_CUR, SEEK_SET);

		signed char utfText[]="";
		
		while (!feof(file)) {
			
			utfText = strcat(utfText, fgets(temp, 100, file));
		}

mon compilateur affiche:

error: assignment to expression with array type
   43 |                         utfText = strcat(utfText, fgets(temp, 100, file));

      |                                 ^

Auriez vous une explication puisqu'il me semble que utfText est du bon type ?

Bonjour HenriHenri9,

En C on ne peut pas affecter le contenu d'un tableau avec l'opérateur =, si c'est ce que tu essayes de faire.

Tu n'utilises pas strcat correctement dans ce contexte.

strcat renvoie un pointeur sur char, son prototype est :

char * strcat ( char * destination, const char * source );

https://cplusplus.com/reference/cstring/strcat/

Si tu veux utiliser la valeur de retour de strcat, tu dois l'utiliser en tant que telle (un pointeur sur char). Cette valeur de retour est utile, par exemple, pour chaîner le résultat de strcat avec une autre commande attendant en paramètre un pointeur sur char (par exemple strcmp ou une autre fonction standard sous string.h).

Ici, pour faire ce que tu veux, comme ton "utfText" est un tableau de char, tu peux simplement écrire :

strcat(utfText, fgets(temp, 100, file));

car strcat concatène de toutes façons la chaîne passée en 2nd paramètre à la fin de la chaîne passée en 1er paramètre.

il faudra aussi que tu donnes à "utfText" une taille en le déclarant correctement et en l'initialisant correctement :

char utfText[256] = { '\0' };

Cela ne sert à rien de mettre "signed" devant le type "char", car le type char sans autre précisions est déjà signé.

-
Edité par Dlks 15 février 2023 à 19:48:38

Merci beaucoup pour votre réponse, désormais tout(concernant ma question) fonctionne.

Dlks a écrit:

il faudra aussi que tu donnes à "utfText" une taille en le déclarant correctement et en l'initialisant correctement :

char utfText[256] = { '\0' };

Cela ne sert à rien de mettre "signed" devant le type "char", car le type char sans autre précisions est déjà signé.

-
Edité par Dlks il y a environ 12 heures

Désolé d'être contrariant, mais

• le standard C dit explicitement que les char peuvent être signés par défaut ou pas, au choix de l'implémentation (1) https://files.lhmouse.com/standards/ISO%20C%20N2176.pdf  § 6.2.5, point 13 page 31.

•  the three types char,signed char, and unsigned char are collectively called the character types.
  The implementation shall define char to have the same range, representation, and behavior as either
  signed char or unsigned char .45

char tampon[256] = "";

-
Edité par michelbillaud 16 février 2023 à 9:30:22

Dlks a écrit:

Cela ne sert à rien de mettre "signed" devant le type "char", car le type char sans autre précisions est déjà signé

Pas du tout, c'est laissé à la norme.

Edit : grillé par michelbillaud

Concernant le code de l'OP il y a plusieurs choses qui ne vont pas :

• Utiliser strcat est souvent signe de risque.
• Utiliser une boucle basée sur feof est souvent une mauvaise pratique.
• Tu lis dans un pointeur non initialisé.
• Tu strcat dans un tableau de 1 caractère.

J'ai du mal à comprendre comment tu as pu en conclure « désormais tout fonctionne » ou alors ton code a changé depuis la dernière fois.

-
Edité par markand 16 février 2023 à 9:24:49

> J'ai du mal à comprendre comment tu as pu en conclure « désormais tout fonctionne » ou alors ton code a changé depuis la dernière fois.

En général, ça veut dire "j'ai bricolé mon truc qui a maintenant l'air de marcher, je suis bien incapable de dire pourquoi mais c'est pas grave, adieu"

-
Edité par michelbillaud 16 février 2023 à 9:36:21

@michelbillaud et @markand,

Au temps pour moi et merci de cette rectification concernant le type char :-)

L'utilisation du feof pour la boucle, c'est une relique du temps où Pascal  était le langage de prédilection pour l'enseignement de la programmation (1).   Avec l'idée de "on s'assure qu'il teste des trucs à lire, puis on lit". (2)

En C l'approche est différente, on tente de lire, et si ça n'a pas marché, hé bé voila, c'est que c'est fini.

while (fgets(temp, sizeof(temp), file) != NULL) {
    ....
}
    

(1) souvent les cours de programmation Pascal étaient eux-même des transpositions du cours Fortran qui précédait.   Comme les cours de C++ pompés sur C.

(2) tout comme : "on s'assure que la pile n'est pas vide, et on dépile le sommet", etc.  Ça part d'une bonne idée : un type abstrait avec des primitives dessus qui font une chose, avec des préconditions (on ne demande une lecture que si on a vérifié avant).

-
Edité par michelbillaud 16 février 2023 à 10:01:03

C'est surtout que le temps du strcat est proportionnel a la taille du résultat, donc en lisant les caractères 100 par 100, ça va coûter

• 100 (copies de caractères) pour le premier bloc
• 200 pour le second
• 300 pour le troisième
• 400 pour le quatrième
• etc

char texte[TAILLE_MAX_TEXTE] = "";

char *fin_de_texte = texte;

while (true) {
    char ligne[TAILLE_MAX_LIGNE];
    if (fgets(ligne, ....) ....) break;

    strcpy(fin_de_texte, ligne);
    fin_de_texte += strlen(ligne);
}

-
Edité par michelbillaud 16 février 2023 à 20:07:52