Discussion :

[JokerAs]

Bonjour, j'essaie depuis ce matin d'extraire des données de configuration d'un fichier texte.
Ce fichier comprend pour tester des identifiants à une base de donnée.

J'ai donc

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
 
	FILE* fichier = NULL;
	fichier = fopen("config.cnf", "r+");
 
	char chaine[23] ;
	char *token;
	int i = 0;
	int j = 0;
 
	char *server;
	char *user;
	char *password;
	char *database;
	int caractereActuel = 0;
 
	if (fichier != NULL){
 
		//fseek(fichier, 1, SEEK_CUR);
 
        do{
            caractereActuel = fgetc(fichier); // On lit le caractère
			if(caractereActuel == '='){
			i++;
				fseek(fichier, 0, SEEK_CUR);
				fgets(chaine, 20, fichier);
				switch(i){
					case 0:
						server = chaine;
					break;
					case 1:
						user = chaine;
					break;
					case 2:
						password = chaine;
					break;
					case 3: 
						database = chaine;
					break;
					default:
					break;
				}
			}
        } while (caractereActuel != EOF); // On continue tant que fgetc n'a pas retourné EOF (fin de fichier)
 
		printf("%s <br />", server);
		printf("%s<br />", user);
		printf("%s<br />", password);
		printf("%s<br />", database);
 
 
		fclose(fichier); 
    }

J'ai essayé de 100 manière différents, a chaque fois c'est le foutoir.
Mon fichier ressemble a
ip=localhost
user=user
pass=pass
database=database

Merci pour votre aide

[foetus]

Il faut faire un automate ... comme la machine à café
Je te laisse chercher avec ton moteur de recherche WEB préféré, mais en gros:

Si on élimine les espaces et que l'on garde que des caractères minuscules et les fins de lignes:
On attend soit un i soit un p soit un u soit un d (à moins que les attributs soient dans un ordre précis)

Si c'est i, on va attendre successivement un p, un = (sinon erreur) et ensuite jusqu'à la fin de ligne on prend tous les caractères -> on a l'ip
Si c'est p, on va attendre successivement un a, un s, un s, un = (sinon erreur) et ensuite jusqu'à la fin de ligne on prend tous les caractères -> on a le mot de passe

Et ainsi de suite

Édit: un automate c'est une machine à état.
Mais dans ton cas, la programmation objet n'est pas nécessaire.
Il faut faire des switch dans des switch

[ternel]

pas forcément switch switch.
le fonctionnement de base serait:

tant qu'on peut lire une ligne {
    s'il n'y a pas un unique '=', sauter la ligne (avec un message?)
    découper chaque ligne autour du '='
    retirer les espaces extérieurs de chaque morceau. (trim...)
    si le premier morceau correspond à une des clés de définition, extraire la valeur du second morceau.
}

à toi de coder les fonctions utiles, et de récupérer sur le net des codes pour trim et split.

L'automate caractère à caractère n'est pas toujours une bonne idée

[foetus]

L'automate caractère à caractère n'est pas toujours une bonne idée

Au moins avec l'automate caractère par caractère, on peut mettre un couple (clef, valeur) sur plusieurs lignes (mais le mieux pour cela c'est d'avoir un caractère fin de ligne)

Tu oublies de dire (comme moi), qu'en partant du début d'une ligne, il faut mémoriser la position du premier caractère "lettre ou éventuellement chiffre" (CL) et du dernier CL avant l'égal pour le découpage.
Et de même après l'égal

Et enfin tu oublies de dire qu'avec ta méthode , il faudra faire des strcmp en "try and retry", avec éventuellement une sélection sur les premiers caractères

Ça me semble un peu brutal comme méthode pour parser un fichier aussi simplement formaté ...
Et puis imagine la tête du mec qui reprend le code après

Sinon l'algorithme de leternel est à peu près ce que j'aurais fait.

ou

Justement je me demande si tu peux faire le switch sur le caractère courant avant le switch sur l'état, pour compacter l'automate


Et éventuellement, c'est peut-être du code vite-fait pour dépanner: donc la tronche qu'il a on s'en fiche

[the Hound]

Il faut faire un automate ... [...]
Édit: un automate c'est une machine à état.
Mais dans ton cas, la programmation objet n'est pas nécessaire.
Il faut faire des switch dans des switch

Ça me semble un peu brutal comme méthode pour parser un fichier aussi simplement formaté ...
Et puis imagine la tête du mec qui reprend le code après

Sinon l'algorithme de leternel est à peu près ce que j'aurais fait.

[ternel]

J'aurai bien donné un peu de code, mais je ne connais que la solution C++ (std::getline et istringstream)
En C, je ne sais plus comment l'écrire, mais j'imagine qu'avec fgets et les fonctions de recherche dans une chaine, on peut s'en sortir.

[JokerAs]

Super, merci.

je vais me pencher sur l'algo de leternel

A plus tard pour les nouvelles,

[hartyshow]

Salut,

A moins d'être en auto-formation, pourquoi refaire la roue !

Il existe un snippets C qui contient un code complet et documenté pour ce genre de problème !

Aller, je mache le travail de recherche snip-c.zip

@+ et bon dev

[ternel]

Sauf que visiblement, c'est une question de formation.
Parce que ce problème est assez simple pour qu'un pratiquant du C professionnel connaisse la réponse.
(je ne suis qu'amateur en C, mais j'ai une certaine habitude du C++, et les algos sont les mêmes)

Comme en plus notre règlement nous interdit de donner des réponses toutes faites, c'est dommage.

@JokerAs:
N'hésite surtout pas à revenir nous montrer ce que tu as fait, pour qu'on te donne nos avis.