Discussion :

[Yunchi]

Bonjour,

Voilà ce que j'essaye de faire :

Code :

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

1
2
char temp[200];
sprintf(temp,"abcdefg hijklmn opqrst  _-  . : !");

Est-ce normal qu'après un sprintf dans un string, si on affiche les 200 caractères de ce string par exemple, on ait des caractères bizzares à la fin ?

merci

[Médinoc]

Avec exactement ce code-là, je dirais que non.
Par contre, si tu lui passes une chaîne de format incorrecte ou si tu dépasses les 200 caractères, ce sera normal que ça foire.

PS: Pou rceux qui auraient oublié, je rappelle que le second argument de sprintf() est une chaîne de format. Ne pas confondre sprintf() et strcpy().

[Yunchi]

Quel serait le moyen pour initialiser une chaine de caractères déclarée de façon statique svp ?

Code :

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

1
2
char temp[200];
strcpy(temp,"\0");

marcherait ?

[Invité(e)]

Bonjour,

Quel serait le moyen pour initialiser une chaine de caractères déclarée de façon statique svp ?

Code :

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

1
2
char temp[200];
strcpy(temp,"\0");

marcherait ?

Code :

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

char temp[200] = "";

suffit

[Médinoc]

Et s'il ne s'agit pas d'une initialisation, alors on passe à ceci:

Code C :

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

strcpy(temp, "");

Qui correspond exactement au code suivant, plus simple:

Code C :

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

temp[0] = '\0';

[Yunchi]

ah ok merci

je ne pensais pas que c'était aussi simple :p

[Médinoc]

Disons que c'est très simple si les données ne sont pas sensibles, car elles sont toujours en mémoire.
Si tu voulais effacer tout le contenu de la chaîne (plutôt que simplement la tronquer), il faudrait que tu mette à zéro tout le buffer.

Cela peut aussi se montrer problématique si ta chaîne n'est pas systématiquement terminée par un caractère nul (par exemple, pour certains cas de concaténations successives), car avec un buffer non-initialisé, cela peut potentiellement mener à une lecture qui déborderait du buffer.

[Yunchi]

Euh pas tout compris pour l'histoire du buffer ...

sinon, si on a une chaine de caractères dynamique, on doit l'initialiser comme ça ? :

Code :

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

char *temp = NULL;

[Médinoc]

Tu l'initialises à NULL ou à une adresse valide. Dans les deux cas, il ne faut pas laisser le pointeur non-initialisé.

[Yunchi]

ok merci, j'essaye de changer mon code qui était statique en quelque chose de plus flexible mais j'ai quelques petits soucis : /

par exemple là j'ai une erreur
(pas de problème avec chemin_txt, en statique ça marche)

Code :

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

1
2
3
4
5
6
7
8
9
FILE* fp;
char *temp = NULL;
 
if (!(fp= fopen(chemin_txt,"r"))) printf("Can't open the file (%s)!\n",chemin_txt);
if (fp = fopen(chemin_txt,"r"))
{
    fgets(temp,strlen(temp),fp);
    fclose(fp);    
}

Le problème doit venir de strlen(temp) car avec une constante ça marchait : /

[Yunchi]

Euh j'en profite pour poser une autre question :

Quand on déclare un tableau de int, est-il possible de tous les initialiser à 0 sans passer par une boucle ?

Code :

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

1
2
int tableau[5];
for (i=0;i<5;i++) tableau[i] = 0;

[Médinoc]

Seulement lors de la déclaration:

Code C :

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

int tableau[5] = {0};

Ce code initialise le premier entier explicitement à zéro, les quatre autres implicitement.

[rilou]

Seulement lors de la déclaration:

Code C :

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

int tableau[5] = {0};

Ce code initialise le premier entier explicitement à zéro, les quatre autres implicitement.

Attention !

J'ajouterais que si on tente (par exmeple) :

Code C :

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

int tableau[5] = {1};

Ne pas croire que ça initialise tout a 1 !!! Cela va initialiser la premiere 'case' a 1 et les autre a 0 !

[gl]

ok merci, j'essaye de changer mon code qui était statique en quelque chose de plus flexible mais j'ai quelques petits soucis : /

par exemple là j'ai une erreur
(pas de problème avec chemin_txt, en statique ça marche)

Code :

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

1
2
3
4
5
6
7
8
9
FILE* fp;
char *temp = NULL;
 
if (!(fp= fopen(chemin_txt,"r"))) printf("Can't open the file (%s)!\n",chemin_txt);
if (fp = fopen(chemin_txt,"r"))
{
    fgets(temp,strlen(temp),fp);
    fclose(fp);    
}

Le problème doit venir de strlen(temp) car avec une constante ça marchait : /

Pourquoi essaies-tu d'ouvrir deux fois le fichier ?
Dans l'exemple ci-dessus, temp ne contient pas l'adresse d'un buffer valide, tu ne peux donc pas l'utiliser pour lire quelquechose, il faut soit utiliser un tableau statique soit allouer dynamiquement le buffer.
En outre strlen() n'est la fonction à utiliser ici, tu veux avoir la taille du buffer et non le nombre de caractère de la chaîne éventuellement contenue dans le buffer avant la lecture.

[Yunchi]

Merci pour vos réponses.

En fait la première ouverture c'est pour indiquer s'il y a une erreur si le fichier n'existe pas ou n'est pas ouvrable

[Invité(e)]

En fait la première ouverture c'est pour indiquer s'il y a une erreur si le fichier n'existe pas ou n'est pas ouvrable

Mais après la première ouverture, si elle réussi, le fichier est ouvert...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
FILE* fp;
char temp[200];
 
fp = fopen(chemin_txt,"r")
 
if(fp != NULL) {
    fgets(temp, sizeof temp, fp);
    fclose(fp);   
} else {
    fprintf(stderr, "Can't open the file (%s)!\n",chemin_txt); 
}

Si on tiens absolument à faire un allocation dynamique :

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
FILE* fp;
/* EDIT : pourquoi se compliquer la vie avec malloc et memset quand calloc est là ?*/
char * temp = calloc(200, sizeof *temp);
if(NULL != temp) {
    fp = fopen(chemin_txt,"r")
    if(fp != NULL) {
        fgets(temp, 200, fp);
        fclose(fp);   
        /* EDIT: le free est mal placé */
        /* free(temp); */
    } else {
        fprintf(stderr, "Can't open the file (%s)!\n",chemin_txt); 
    }
    /* EDIT: la bonne place est ici */
    free(temp);
} else {
    fprintf(stderr, "error malloc");
}