MuPuF
Messages postés536Date d'inscriptionmercredi 27 avril 2005StatutMembreDernière intervention22 août 2008
-
13 avril 2007 à 21:09
Jonat95
Messages postés5Date d'inscriptionmardi 30 octobre 2007StatutMembreDernière intervention 9 novembre 2008
-
14 juil. 2008 à 15:40
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
Je ne vais pas appliqué tous vos conseil notament les booleens tanpi pour l'optimisation^^ et l'utilisation de ReadFile WriteFile de l'API WIN32 ^^
Et je ne vais pas faire évoluer le programme car je veux qu'il y est le moins de ligne pour qu'il soit facilement et rapidement intégrable dans de vrai projet.
J'ai rajouté les #define pour une meilleur comprehension du code comme tu me l'as dit tularis.
Ok je me suis trompé MuPuF mais bon j'ai voulu faire un code tout simple qui marche et rapidement reutilisable car il y a les source
Sinon comment vérifier la bonne allocation de la mémoire stp ?
A+++++
PS: je connais pas le c++ donc je connais pas cout, cin, cerr... etj'utilise Microsoft Visual C++ Express :D
Et je ne connais rien au socket encore lol j'ai tout copié collé d'un exemple d'un site, je sais juste utiliser les fonctions send et recv et avec ca je me suis débroullé pour coder mes 2 fonctions d'upload et de download lool :)
tularis
Messages postés21Date d'inscriptionmardi 8 mars 2005StatutMembreDernière intervention12 juillet 2009 13 avril 2007 à 21:32
c'est pas mal du tous,
petite suggestion : tu devrai utiliser des defines en debut de fonction pour l'ip, le port, le nom de fichier, comme sa, si tu veu changer des valeur aucun probleme, et mieux si tu veux ameliorer ton code, les variable sont prete a etre utilisés, ...
Dans tes condition tu utilise des characteres, utilise plutot des booleens ou des entiers ... (et oui l'optimisation :p) mois d'espace memoire, plus dans les normes de prog.
Tres interessant le systeme de gestion brut des fichiers :) ...
Tu utilise turbo C comme logiciel de developpement?
PS: Tous le monde utilise les printf et les scanf O_o, je pensais qu'avec c++ tous le monde evoluerai vers cout, cin, cerr malgrès que la gestion des flux ne soit peut etre plus une contrainte q'une evolution (l'utilisation avec des flush en entrée et en sortie pour cout et cin et cerr rend le code .. plus clair pour moi ...
et puis plus rapide a prog ... sans compter la faciliter d'utilisation, et l'integration des string ...)
MuPuF
Messages postés536Date d'inscriptionmercredi 27 avril 2005StatutMembreDernière intervention22 août 2008 13 avril 2007 à 21:09
Bon, pour le code :
- int SizeTransfertPaquet=1024
char *paquet=(char*) malloc(SizeTransfertPaquet); <- Tu ne vérifies pas la bonne allocation de la mémoire. Le mieux aurait été
#define SizeTransfertPaquet 1024
char paquet[SizeTransfertPaquet]; <- Le static est la meilleure solution içi
- Pour le test de fin de transfert, tu fais confiance à la valeur lu dans le premier paquet, tu devrais tester si elle est supérieure à 0 (erreur de conversion par atol et transférer un fichier de 0 octets) ça pourrait éviter des erreurs bizarres que tu risques d'avoir si tu continues là dedans.
- A utiliser des apis autant les utiliser jusqu'au bout et l'utiliser pour la lecture dans les fichiers.
- Un petite doc trés courte sur le protocole au dessus des fonctions aurait été sympa ;-)
Vila, bonne chance pour ton projet si tu continues là dedans, et la prochaine fois, cherche plus avant d'affirmer des choses, il n'y a pas qu'UNE source de transfert de fichier sur cppfrance ;-)
14 juil. 2008 à 15:40
18 avril 2007 à 20:07
18 avril 2007 à 08:51
17 avril 2007 à 19:57
Je ne vais pas appliqué tous vos conseil notament les booleens tanpi pour l'optimisation^^ et l'utilisation de ReadFile WriteFile de l'API WIN32 ^^
Et je ne vais pas faire évoluer le programme car je veux qu'il y est le moins de ligne pour qu'il soit facilement et rapidement intégrable dans de vrai projet.
J'ai rajouté les #define pour une meilleur comprehension du code comme tu me l'as dit tularis.
Ok je me suis trompé MuPuF mais bon j'ai voulu faire un code tout simple qui marche et rapidement reutilisable car il y a les source
Sinon comment vérifier la bonne allocation de la mémoire stp ?
A+++++
PS: je connais pas le c++ donc je connais pas cout, cin, cerr... etj'utilise Microsoft Visual C++ Express :D
Et je ne connais rien au socket encore lol j'ai tout copié collé d'un exemple d'un site, je sais juste utiliser les fonctions send et recv et avec ca je me suis débroullé pour coder mes 2 fonctions d'upload et de download lool :)
13 avril 2007 à 21:32
petite suggestion : tu devrai utiliser des defines en debut de fonction pour l'ip, le port, le nom de fichier, comme sa, si tu veu changer des valeur aucun probleme, et mieux si tu veux ameliorer ton code, les variable sont prete a etre utilisés, ...
Dans tes condition tu utilise des characteres, utilise plutot des booleens ou des entiers ... (et oui l'optimisation :p) mois d'espace memoire, plus dans les normes de prog.
Tres interessant le systeme de gestion brut des fichiers :) ...
Tu utilise turbo C comme logiciel de developpement?
PS: Tous le monde utilise les printf et les scanf O_o, je pensais qu'avec c++ tous le monde evoluerai vers cout, cin, cerr malgrès que la gestion des flux ne soit peut etre plus une contrainte q'une evolution (l'utilisation avec des flush en entrée et en sortie pour cout et cin et cerr rend le code .. plus clair pour moi ...
et puis plus rapide a prog ... sans compter la faciliter d'utilisation, et l'integration des string ...)
13 avril 2007 à 21:09
Bon, pour le code :
- int SizeTransfertPaquet=1024
char *paquet=(char*) malloc(SizeTransfertPaquet); <- Tu ne vérifies pas la bonne allocation de la mémoire. Le mieux aurait été
#define SizeTransfertPaquet 1024
char paquet[SizeTransfertPaquet]; <- Le static est la meilleure solution içi
- Pour le test de fin de transfert, tu fais confiance à la valeur lu dans le premier paquet, tu devrais tester si elle est supérieure à 0 (erreur de conversion par atol et transférer un fichier de 0 octets) ça pourrait éviter des erreurs bizarres que tu risques d'avoir si tu continues là dedans.
- A utiliser des apis autant les utiliser jusqu'au bout et l'utiliser pour la lecture dans les fichiers.
- Un petite doc trés courte sur le protocole au dessus des fonctions aurait été sympa ;-)
Vila, bonne chance pour ton projet si tu continues là dedans, et la prochaine fois, cherche plus avant d'affirmer des choses, il n'y a pas qu'UNE source de transfert de fichier sur cppfrance ;-)