PHP

Avis sur ma classe FTP

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Avis sur ma classe FTP

Metabolicman

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 :

class FTP
{
//CONCERNE LE FTP
private $ftp;
private $login;
private $password;
private $path_serveur_upload;
private $path_serveur_racine;
private $path_serveur_courant;
//CONCERNE L'UTILISATEUR
private $nom_fichier;
private $type_fichier;
private $size_fichier;
private $tmp_fichier;
function __construct()
{
$this->ftp="";
$this->login="";
$this->password="";
$this->path_serveur_upload="";
$this->path_serveur_racine="";
$this->path_serveur_courant=$this->path_serveur_racine;
$this->path_courant="";
$this->nom_fichier="";
$this->type_fichier="";
$this->size_fichier="";
$this->tmp_fichier="";
}
function __destruct(){}
//ACCESSEURS
public function GET_ftp()
{
return $this->ftp;
}
public function GET_login()
{
return $this->login;
}
public function GET_password()
{
return $this->password;
}
public function GET_path_serveur_upload()
{
return $this->path_serveur_upload;
}
public function GET_path_serveur_racine()
{
return $this->path_serveur_racine;
}
public function GET_path_serveur_courant()
{
return $this->path_serveur_courant;
}
public function GET_nom_fichier()
{
return $this->nom_fichier;
}
public function GET_type_fichier()
{
return $this->type_fichier;
}
public function GET_size_fichier()
{
return $this->size_fichier;
}
public function GET_tmp_fichier()
{
return $this->tmp_fichier;
}
//MUTATEURS
public function SET_ftp($val)
{
$this->ftp=$val;
}
public function SET_login($val)
{
$this->login=$val;
}
public function SET_password($val)
{
$this->password=$val;
}
public function SET_path_serveur_upload($val)
{
$this->path_serveur_upload=$val;
}
public function SET_path_serveur_racine($val)
{
$this->path_serveur_racine=$val;
$this->path_serveur_courant=$val;
}
public function SET_path_serveur_courant($val)
{
$this->path_serveur_courant=$val;
}
public function SET_nom_fichier($val)
{
$this->nom_fichier=$val;
}
public function SET_type_fichier($val)
{
$this->type_fichier=$val;
}
public function SET_size_fichier($val)
{
$this->size_fichier=$val;
}
public function SET_tmp_fichier($val)
{
$this->tmp_fichier=$val;
}
//AUTRES METHODES
protected function Connect()
{
$conn_id=ftp_connect($this->ftp);
if (@ftp_login($conn_id, $this->login, $this->password))
{
return $conn_id;
}
else
{
return false;
}
}
protected function Close($ftp_stream)
{
ftp_close($ftp_stream);
}
public function FileUpload()
{
$ftp_stream=$this->Connect();
if(!$ftp_stream)
{
echo "La connexion a échouée";
}
else
{
ftp_put($ftp_stream,$this->path_serveur_upload,$this->tmp_fichier.$this->nom_fichier,FTP_ASCII);
$this->Close($ftp_stream);
}
}
public function FileDownload()
{
}
public function ListRep()
{
$ftp_stream=$this->Connect();
if(!$ftp_stream)
{
echo "La connexion a échouée";
}
else
{
$tab=ftp_nlist($ftp_stream,$this->GET_path_serveur_courant());
if(!$tab)
{
echo "Il n'y a aucun fichier";
}
else
{
foreach($tab as $value)
{
echo $value."<br />";
}
$this->Close($ftp_stream);
}
}
}
public function ParentRep()
{
$path=$this->path_serveur_courant;
$path=explode("/",$path);
$nb_case=count($path)-2;
for($i=0;$i<$nb_case;$i++)
{
$npath.=$path[$i]."/";
}
$this->path_serveur_courant=$npath;
$this->ListRep();
}
public function DeleteFile($file)
{
//ftp_delete
}
public function CreateDir($dirname)
{
$ftp_stream=$this->Connect();
if(!$ftp_stream)
{
echo "La connexion a échouée";
}
else
{
error_reporting(0);
if(!ftp_mkdir($ftp_stream,$this->path_serveur_courant.$dirname))
{
echo "Le répertoire ".$dirname." existe déjà<br />";
}
$this->Close($ftp_stream);
}
$this->ListRep();
}
public function RenameFile()
{
//ftp_rename
}
public function DeleteDir($dirname)
{
$ftp_stream=$this->Connect();
if(!$ftp_stream)
{
echo "La connexion a échouée";
}
else
{
ftp_rmdir($ftp_stream,$this->path_serveur_courant.$dirname);
$this->Close($ftp_stream);
}
$this->ListRep();
}
}

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

Publicité

yellu

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.

Metabolicman

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

FlorentG

Unité de Masse

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

masklinn

í dag viðrar vel til loftárása

FlorentG a écrit :

+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

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

FORUM HardWare.fr

Programmation

PHP

Avis sur ma classe FTP

Sujets relatifs
Connaitre le nom d'une classe enfant (sans l'instancier)	std::map associer des index et une classe, comparison function
utiliser dans une methode toString() celle d'une autre classe	Question classe session
[VB .NET] Comment utiliser une classe écrite en C#	mon programme ne trouve pas la classe
votre avis m'interesse <?=	[SGBD] Avis sur modélisation
pb applet et classe java	Une requete mySQL qui ne classe pas les résultats
Plus de sujets relatifs à : Avis sur ma classe FTP

Page générée en 0.118 secondes