Avis sur ma classe FTP

Avis sur ma classe FTP - PHP - Programmation

Marsh Posté le 22-10-2008 à 11:26:44    

Bonjour à tous,
 
Bon, je n'ai aucun problème à signaler dans un de mes codes mais je veux juste votre avis.
Voilà je me lance dans la POO avec php et j'aimerai savoir si la classe FTP que je viens  
de créer (elle fonctionne) codée correctement.
 
Je vous rassure, elle n'est pas finie il y a encore des méthodes incomplètes, maintenant place au code:
 

Code :
  1. class FTP
  2. {
  3. //CONCERNE LE FTP
  4. private $ftp;
  5. private $login;
  6. private $password;
  7. private $path_serveur_upload;
  8. private $path_serveur_racine;
  9. private $path_serveur_courant;
  11. //CONCERNE L'UTILISATEUR
  12. private $nom_fichier;
  13. private $type_fichier;
  14. private $size_fichier;
  15. private $tmp_fichier;
  17. function __construct()
  18. {
  19.  $this->ftp="";
  20.  $this->login="";
  21.  $this->password="";
  22.  $this->path_serveur_upload="";
  23.  $this->path_serveur_racine="";
  24.  $this->path_serveur_courant=$this->path_serveur_racine;
  25.  $this->path_courant="";
  26.  $this->nom_fichier="";
  27.  $this->type_fichier="";
  28.  $this->size_fichier="";
  29.  $this->tmp_fichier="";
  30. }
  32. function __destruct(){}
  34. //ACCESSEURS
  36. public function GET_ftp()
  37. {
  38.  return $this->ftp;
  39. }
  41. public function GET_login()
  42. {
  43.  return $this->login;
  44. }
  46. public function GET_password()
  47. {
  48.  return $this->password;
  49. }
  51. public function GET_path_serveur_upload()
  52. {
  53.  return $this->path_serveur_upload;
  54. }
  56. public function GET_path_serveur_racine()
  57. {
  58.  return $this->path_serveur_racine;
  59. }
  61. public function GET_path_serveur_courant()
  62. {
  63.  return $this->path_serveur_courant;
  64. }
  66. public function GET_nom_fichier()
  67. {
  68.  return $this->nom_fichier;
  69. }
  71. public function GET_type_fichier()
  72. {
  73.  return $this->type_fichier;
  74. }
  76. public function GET_size_fichier()
  77. {
  78.  return $this->size_fichier;
  79. }
  81. public function GET_tmp_fichier()
  82. {
  83.  return $this->tmp_fichier;
  84. }
  86. //MUTATEURS
  88. public function SET_ftp($val)
  89. {
  90.  $this->ftp=$val;
  91. }
  93. public function SET_login($val)
  94. {
  95.  $this->login=$val;
  96. }
  98. public function SET_password($val)
  99. {
  100.  $this->password=$val;
  101. }
  103. public function SET_path_serveur_upload($val)
  104. {
  105.  $this->path_serveur_upload=$val;
  106. }
  108. public function SET_path_serveur_racine($val)
  109. {
  110.  $this->path_serveur_racine=$val;
  111.  $this->path_serveur_courant=$val;
  112. }
  114. public function SET_path_serveur_courant($val)
  115. {
  116.  $this->path_serveur_courant=$val;
  117. }
  119. public function SET_nom_fichier($val)
  120. {
  121.  $this->nom_fichier=$val;
  122. }
  124. public function SET_type_fichier($val)
  125. {
  126.  $this->type_fichier=$val;
  127. }
  129. public function SET_size_fichier($val)
  130. {
  131.  $this->size_fichier=$val;
  132. }
  134. public function SET_tmp_fichier($val)
  135. {
  136.  $this->tmp_fichier=$val;
  137. }
  139. //AUTRES METHODES
  141. protected function Connect()
  142. {
  143.  $conn_id=ftp_connect($this->ftp);
  145.  if (@ftp_login($conn_id, $this->login, $this->password))
  146.  {
  147.   return $conn_id;
  148.  }
  149.  else
  150.  {
  151.   return false;
  152.  }
  153. }
  155. protected function Close($ftp_stream)
  156. {
  157.  ftp_close($ftp_stream);
  158. }
  160. public function FileUpload()
  161. {
  162.  $ftp_stream=$this->Connect();
  164.  if(!$ftp_stream)
  165.  {
  166.   echo "La connexion a échouée";
  167.  }
  168.  else
  169.  {
  170.   ftp_put($ftp_stream,$this->path_serveur_upload,$this->tmp_fichier.$this->nom_fichier,FTP_ASCII);
  171.   $this->Close($ftp_stream);
  172.  }
  173. }
  175. public function FileDownload()
  176. {
  177. }
  179. public function ListRep()
  180. {
  181.  $ftp_stream=$this->Connect();
  183.  if(!$ftp_stream)
  184.  {
  185.   echo "La connexion a échouée";
  186.  }
  187.  else
  188.  {
  189.   $tab=ftp_nlist($ftp_stream,$this->GET_path_serveur_courant());
  191.   if(!$tab)
  192.   {
  193.    echo "Il n'y a aucun fichier";
  194.   }
  195.   else
  196.   {
  197.    foreach($tab as $value)
  198.    {
  199.     echo $value."<br />";
  200.    }
  201.    $this->Close($ftp_stream);
  202.   }
  203.  }
  204. }
  206. public function ParentRep()
  207. {
  208.  $path=$this->path_serveur_courant;
  209.  $path=explode("/",$path);
  210.  $nb_case=count($path)-2;
  212.  for($i=0;$i<$nb_case;$i++)
  213.  {
  214.   $npath.=$path[$i]."/";
  215.  }
  216.  $this->path_serveur_courant=$npath;
  217.  $this->ListRep();
  218. }
  220. public function DeleteFile($file)
  221. {
  222.  //ftp_delete
  223. }
  225. public function CreateDir($dirname)
  226. {
  227.  $ftp_stream=$this->Connect();
  229.  if(!$ftp_stream)
  230.  {
  231.   echo "La connexion a &eacute;chou&eacute;e";
  232.  }
  233.  else
  234.  {
  235.   error_reporting(0);
  236.   if(!ftp_mkdir($ftp_stream,$this->path_serveur_courant.$dirname))
  237.   {
  238.    echo "Le r&eacute;pertoire ".$dirname." existe d&eacute;j&agrave;<br />";
  239.   }
  240.   $this->Close($ftp_stream);
  241.  }
  242.  $this->ListRep();
  243. }
  245. public function RenameFile()
  246. {
  247.  //ftp_rename
  248. }
  250. public function DeleteDir($dirname)
  251. {
  252.  $ftp_stream=$this->Connect();
  254.  if(!$ftp_stream)
  255.  {
  256.   echo "La connexion a &eacute;chou&eacute;e";
  257.  }
  258.  else
  259.  {
  260.   ftp_rmdir($ftp_stream,$this->path_serveur_courant.$dirname);
  261.   $this->Close($ftp_stream);
  262.  }
  263.  $this->ListRep();
  264. }
  265. }


 
PS: je sais que tout les accesseurs et mutateurs ne sont pas forcément nécessaires ;)

Reply

Marsh Posté le 22-10-2008 à 11:26:44   

Reply

Marsh Posté le 22-10-2008 à 11:50:34    

J'aime pas trop ta "convention" de nommage, des _ pour les getters et setters et une synthaxe java pour le reste.
 
Je crois que les premières lettres des méthodes ne prennent pas de majuscule en Java.
 
Sinon peut être pourrais-tu donner un peu d'intelligence au constructeur et inclure la livraison de paramètres au niveau de l'instanciation.

Reply

Marsh Posté le 22-10-2008 à 12:16:04    

Ouais c'est prévu d'affecter directement les valeurs au constructeur pour une économie de ligne dans le programme principal

Reply

Marsh Posté le 22-10-2008 à 12:19:53    

C'est pas qu'une économie de ligne, c'est surtout qu'un constructeur est censé livrer un objet exploitable :)
 
Pour les erreurs, utilise peut-être des exceptions au lieu de simples echo qui ne permettent pas une gestion correcte.

Message cité 1 fois
Reply

Marsh Posté le 22-10-2008 à 12:31:11    

FlorentG a écrit :

C'est pas qu'une économie de ligne, c'est surtout qu'un constructeur est censé livrer un objet exploitable :)
 
Pour les erreurs, utilise peut-être des exceptions au lieu de simples echo qui ne permettent pas une gestion correcte.


+1, et il faudrait ajouter:

  • Des défauts corrects dans le constructeur, genre "~" en path local et "/" en path remote
  • connect n'a aucune raison d'être protected, il devrait être private
  • l'API pour l'upload est immonde, pourquoi ne pas filer juste un chemin de fichier local et en option un nom de fichier remote (par défaut, le fichier remote a le même nom que le fichier local et baste)
  • Une opération type "cd" serait bien plus utile qu'un ParentRep hardcodé (suffit de cd(".." ) et c'est équivalent)
  • C'est la première fois que je vois quelqu'un utiliser les code conv' C# en PHP [:pingouino]


---------------
Stick a parrot in a Call of Duty lobby, and you're gonna get a racist parrot. — Cody
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed