Discussion :

[oranoutan]

Bonjour,

J'ai quelques soucis en voulant concaténer les 4 lignes d'un fichier texte "test.txt", contenant :

un
deux
trois
quatre

Voici le bout de code :

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{    
    FILE *f;
    char ligne[BUFSIZ];
    unsigned char i = 0;
    char **tab = NULL;
    char *str;
 
    // allocation
    tab = malloc(4 * sizeof *tab); // alloue 1 octet par ligne
    if (tab != NULL)
    {
        for (i = 0; i < 4; i++)
      	{
            tab[i] = malloc (10+1 * sizeof *tab[i]); // alloue 10 caractères + 1 caractère NULL
            if (tab[i] == NULL)
            {
      	        for (i = 0; i < 4; i++)
      	     	    free (tab[i]);
      	     	free (tab);
      	     	printf("problème de lors de l'allocation \n");
      	     	exit(EXIT_FAILURE);
            }
      	}
    }
    else 
    {
        free (tab);
    	printf("problème de lors de l'allocation \n");
    	exit(EXIT_FAILURE);
    }
 
    // lecture fichier
    f = fopen("test.txt", "r");
    if(f == NULL)
    	printf("erreur d'ouverture \n");
 
    i = 0;
    while (fgets(ligne, sizeof ligne, f) != NULL)
    {
    	strcpy(tab[i], ligne);
    	printf(tab[i]); // contrôle
    	i++;
    }
    tab[i] = NULL;
 
    // allocation
    str = malloc (2+4+5+6+1 * sizeof *str);
    if (str == NULL) 
    {
        free (str);
    	printf("problème de lors de l'allocation \n");
    	exit(EXIT_FAILURE);
    }
 
    //  concaténation des lignes du tableau
    i = 0;
    while (tab[i] != NULL)
    {
        str = strcat(str, tab[i]);
        i++;
    }
    printf(str); // contrôle
 
    // free() ...
 
    return EXIT_SUCCESS;
}

[Emmanuel Delahaye]

J'ai quelques soucis en voulant concaténer les 4 lignes d'un fichier texte "test.txt", contenant :

Tu veux obtenir quoi ?

"un\ndeux\ntrois\nquatre\n"
"undeuxtroisquatre"
"un deux trois quatre"
autre chose...

Il faut que ton problème soit mieux défini, sinon, on ne sait pas quoi coder.

[oranoutan]

"undeuxtroisquatre"

[diogene]

Quelques remarques :

Code :

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

tab = malloc(4 * sizeof *tab); // alloue 1 octet par ligne

Commentaire faux

Code :

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

1
2
      	        for (i = 0; i < 4; i++)
      	     	    free (tab[i]);

incorrect : en cas d'échec d'allocation pour une valeur donnée k, les allocations suivantes n'ont pas été faites et dans de cas tab[i] pour i>k est quelconque et le free va faire planter

Code :

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

1
2
3
4
 else 
    {
        free (tab);
    	printf("problème de lors de l'allocation \n");

free(tab) inutile : tab est NULL et aucune allocation n'a été faite

Code :

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

1
2
3
4
5
6
7
8
    i = 0;
    while (fgets(ligne, sizeof ligne, f) != NULL)
    {
    	strcpy(tab[i], ligne);
    	printf(tab[i]); // contrôle
    	i++;
    }
    tab[i] = NULL;

Plante sur les tab[i] si le fichier comporte 4 lignes ou plus. Le tableau tab ne comporte que 4 places.
Pour 4 lignes, tab[i]= NULL donne tab[4] = NULL, hors du tableau
Pour 5 lignes ou plus strcpy(tab[i], ligne) donne strcpy(tab[4], ligne), hors du tableau
Il faut cesser la boucle également en fonction du nombre de places dans le tableau

Code :

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

1
2
3
4
5
6
    if (str == NULL) 
    {
        free (str);
    	printf("problème de lors de l'allocation \n");
    	exit(EXIT_FAILURE);
    }

free(str) inutile, str == NULL

Code :

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

1
2
3
4
5
6
    i = 0;
    while (tab[i] != NULL)
    {
        str = strcat(str, tab[i]);
        i++;
    }

Concaténation incorrect : le contenu de str doit contenir au préalable une chaîne valide.
Mettre au départ une chaîne vide

Code :

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

1
2
3
    i = 0;
    str[0] = '\0';
    while (tab[i] != NULL)

[oranoutan]

Concaténation incorrect : le contenu de str doit contenir au préalable une chaîne valide.
Mettre au départ une chaîne vide

Code :

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

1
2
3
    i = 0;
    str[0] = '\0';
    while (tab[i] != NULL)

Oui, mais j'ai toujours l'impossibilité de concatèner les lignes ensemble.

[Emmanuel Delahaye]

Oui, mais j'ai toujours l'impossibilité de concatèner les lignes ensemble.

Poste la dernière version corrigé en fonction des remarques précédentes.

[Emmanuel Delahaye]

"undeuxtroisquatre"

Maintenant, tu dois écrire un algorithme qui décrit de manière précise le comportement attendu, et ce indépendamment des détails du langage.

[oranoutan]

???

Je souhaite juste concaténer les 4 lignes du fichier afin dobtenir 'undeuxtroisquatre" sur la sortie standard.

C'est ce que j'ai tenté de faire avec le code plus haut mais ça ne le fait pas.

[Emmanuel Delahaye]

???

Je souhaite juste concaténer les 4 lignes du fichier afin dobtenir 'undeuxtroisquatre" sur la sortie standard.

C'est ce que j'ai tenté de faire avec le code plus haut mais ça ne le fait pas.

Ce que je te propose, c'est de réfléchir, et non de coder au hasard. Donc tu prends une feuille blanche et un crayon (électroniques, si tu préfères), et tu décris le mécanisme précisément avec des phrases en français clair et simple.

Code :

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

1
2
3
4
5
6
 
Ouvrir le fichier
...
 
 
Fermer le fichier

Ce n'est qu'en apprenant à organiser sa pensée qu'on peut avancer en programmation. Certainement pas en fonçant dans le code sans savoir où on va.

"Réflechir avant d'agir"

[wiztricks]

???

Je souhaite juste concaténer les 4 lignes du fichier afin dobtenir 'undeuxtroisquatre" sur la sortie standard.

C'est ce que j'ai tenté de faire avec le code plus haut mais ça ne le fait pas.

d'accord mais:
- en entrée est-ce que cela aura toujours 4 lignes ou faut-il concaténer cela quelque soit le nombre de lignes?
- si la sortie est "standard", pourquoi ne pas se contenter d'afficher la ligne "lue": pas besoin de gérer un buffer "au milieu... il suffit? d'enlever la matérialisation de la fin de ligne: CR ou LF.

En fonction des réponses la solution pourra être plus compliquée mais aussi plus simple ;-)

- W