Discussion :

[orezator]

Bon(jour,soir), je débute en C et je souhaiterais via la console récupérer la chaine que l'utilisateur ecrit (moi) pour la passer en argument de fopen. Malheureusement je pense qu'il y a quelque chose de fondamental qui m'échappe, parce que ça foire en beauté, et que je m'y casse les dents depuis plusieures heures. J'ai fais énormément de recherches active, histoire de ne pas encombrer les forums mais je n'ai rien trouvé qui réponde à ma question (peut être est ce que je la formule mal)
bref, je suis ouvert à toute nouvelle voie (en langage C) de mon future sauveur.

Voici mon code foireux

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
#include <stdio.h>
#include <stdlib.h>
 
 
int main(void)
{
	char chain[50];
	FILE *fichier = NULL;
	printf("Entrez le nom de fichier contenant la formule a analyser\n");
	fgets(chain, sizeof(chain), stdin);
	printf("le nom du fichier est : %s", chain);
	fichier = fopen(chain, "w+");
	if(fichier != NULL)
	{	
		fclose(fichier);
		exit(EXIT_SUCCESS);
	}
	else
	{
		fprintf(stderr, "Impossible de lire le fichier %s en lecture\n", chain);
		exit(EXIT_FAILURE);
	}
}

P.S. je réclame votre indulgence c'est la première fois que j'utilise un forum en écriture. A noter également que la robustesse du code est nulle mais ce qui m'importe c'est de pouvoir faire la liaison entre "chain et fopen", je m'occupperai des tests après.

D'avance merci pour le futur graal.

[LittleWhite]

Bonjour,

Petite remarque , le forum permet d'afficher du code clairement grace aux balises [ C O D E ] et [ / C O D E ] ( sans les espaces ), petit boutton '#' en haut quand tu ecrit un message. Si tu met ton code entre les balises ils sera coloré pour qu'on puisse mieux s'y repéré.

Tu n'as pas vraiment précisé ton problème, donc je ne sais pas trop ce qui te pose problème, donc je ne sais pas trop quoi répondre.

Juste que cela semble marcher ( au moins chez moi O_o ), et que je lui donne un nom et il va me crée un fichier si celui n'existe pas.
Par contre la chaine de caractère récupéré à un caractère qui pose problème , donc il faut bien délimité la fin de la chaine avec ceci :

Code :

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

1
2
 
chain[strlen(chain+1)] = '\0';

juste après le printf

( Et il te faudra ajouter #include <string.h> en haut pour que la fonction strlen() soit trouvable )

En espérant t'aider,
N'hésite pas à demander si tu n'as pas compris ou si j'ai pas répondu au problème ( que je ne connais pas vraiment )

[Obsidian]

Malheureusement je pense qu'il y a quelque chose de fondamental qui m'échappe, parce que ça foire en beauté, et que je m'y casse les dents depuis plusieures heures.

Juste que cela semble marcher ( au moins chez moi O_o ), et que je lui donne un nom et il va me crée un fichier si celui n'existe pas.
Par contre la chaine de caractère récupéré à un caractère qui pose problème , donc il faut bien délimité la fin de la chaine avec ceci :

Le code est presque parfait. La clé du mystère se trouve dans la man page de fgets() :

fgets() lit au plus size - 1 caractères depuis stream et les place dans le buffer pointé par s. La lecture s'arrête après EOF ou un retour-chariot. Si un retour-chariot (newline) est lu, il est placé dans le buffer. Un caractère nul '\0' est placé à la fin de la ligne.

Donc, le retour à la ligne est acquis en même temps que le reste et fait partie intégrante du nom de fichier. Certains systèmes n'aiment pas du tout cela. Il faut aller le retirer à la main.

[orezator]

Merci LittleWithe pour le conseil sur la lisibilité, je n'y manquerai pas la prochaine fois. Eh oui c'est vrai que je n'ai pas donné les feedbacks du code compilé (parce qu'il compilait le bougre), mais ce joyeux OS de windows me donnait un rapport d'erreur. Je préfère linux mais, j'aime aussi travailler en musique, et alsa n'est pas top avec mon laptop .

Et donc pour conclure en phase prétest merci à toi Obsidian, car je pense qu'effectivement c'est ce retour chariot qui fait pèter une case à windows (c'est le cas de le dire), et une à moi aussi par la même occasion .

Je m'en vais donc de ce pas implémenter une fonction qui boucle pour shooter ce "\0". Je vous tiens au courant de l'issue, quelle qu'elle soit.

[tonton fred]

Salut,

Je m'en vais donc de ce pas implémenter une fonction qui boucle pour shooter ce "\0".

Petit probleme d'interpretation: ce n'est pas \0 qu'il faut shooter (sinon ta chaine n'est plus valide) mais \n

[Médinoc]

Regarde sur le forum et cherche la fonction fclean(), qui fait tout le travail nécessaire: Soit elle tronque la ligne (et vide l'excédant du buffer d'entrée) si on a saisi plus de 48 caractères, soit elle supprime le \n.

[orezator]

Bonsoir, désolé d'avoir tardé pour ceux qui m'ont aimablement aidé, mais le problème étant résolu je me suis plongé dans mon code pour ... en trouver d'autres.
C'est ça qui est génial en programmation c'est que le niveau des problèmes rencontrés croissent en même temps que le niveau du programmeur :p.
Mais heureusement toujours avec comme constante la cohérence logique chère à nos ordinateurs.

Vive la vie et merci.

[Obsidian]

toujours avec comme constante la cohérence logique chère à nos ordinateurs.

Certaines fois, on se demande, quand même !