Discussion :

[moman]

Bonjour,
Je suis entrain de coder un serveur et j'ai un petit problème lorsque le client me demande la liste des fichiers disponible car je n'arrive à afficher que la dernière ligne du fichier(qui contient la liste de tout les fichiers).
Voici la partie de mon code concerné :

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
 
else if (strcmp (data, "LIST") == 0)
               { 
		FILE* fichier = NULL;
		char text[50];
		system("ls > shell.txt");
    		fichier = fopen("shell.txt", "r");
		 if (fichier != NULL)
    			{
				while (fgets(text, sizeof text, fichier) != NULL);
				{
					char text0[100] = "FICH ";
					strcat(text0, text);
	         			send (p_cli->sock, text0, strlen (text0), 0);
					fflush (stdout);
	               		}
			fclose(fichier);
			}
		char const response5[] = "ENDL\n";
                send (p_cli->sock, response5, strlen (response5), 0);
		}

Je pense que l'erreur vient lorsque j’envoie une chaîne, je pense quelle ne part pas tant que la boucle n'est pas fini.

Merci d'avance pour votre aide.

[Trademark]

Salut,

C'est une erreur de copier coller je présume... Tu as oublié d'enlever le ";" à la fin de ta boucle while... Du coup il boucle sur tout le fichier et quand ça renvois NULL, alors seulement tu affiches quelque chose, qui est la dernière ligne du fichier donc...

Mais puisque j'ai lu ton code voila d'autres remarques:

• Tu utilises system(...), tu ne devrais pas passer par cette fonction et plutôt directement utiliser scandir. De plus, que se passe t'il si le client demande un dossier avec GETF ?
• Tu fais un fflush(stdout) qui ne sert strictement à rien. Tu demandes au système de vider le buffer clavier sur l'écran..
• char text0[100] = "FICH ";, pourquoi 100 ? alors que tu es sur que le tampon d'avant ne fera qu'au maximum 50...
• Tu ne vérifies pas les codes de retour de send et compagnie... De plus, send peut ne pas envoyer tout les caractères et retourner.

[moman]

Merci pour votre réponse, j'ai corrigé tous ça.
Pour la demande de fichier avec GETF , j'ai encore quelque problème avec ma file mais je demanderais de l'aide plus tard si nécessaire.