Discussion :

[totofe49]

Bonjour à tous,
je voulais savoir si je pouvais vous soumettre un fichier de classe que j'ai écrit pour avoir votre avis ? (Tout en étant indulgent, je début dans les classes)

Merci

[totofe49]

Alors attention voici la bête !
merci

[stealth35]

dans l'ensemble c'est pas trop mal, attention a l'indentation par contre, et met tes public/private sur tes fonctions, mettre le type hinting, et pour les commentaire format docbook

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
/**
 * Fonction qui écrit des champs
 * $datas['type'], $datas['label'], $datas['name'], $datas['db'], $datas['class'
 * 
 * @param array $datas
 * @return string
 */
public function ecrireChamp(array $datas)

pour gérer tes verifes a utilise filter_var (au moins pour l'email)

pour la date tu peux utiliser stototime ou parse_date


par contre faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$_POST[''.$datas['name'].'']

ca sert a rien
fais juste

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$_POST[$datas['name']]

[totofe49]

merci pour toutes ces infos.
Juste 1 petite questions :

- hinting ?

[totofe49]

e j'oubliai une dernière petite question :

est-ce qu'il faut que j'utilise la fonction __construct dans ma classe
et pourquoi ?

Merci

[stealth35]

e j'oubliai une dernière petite question :

est-ce qu'il faut que j'utilise la fonction __construct dans ma classe
et pourquoi ?

Merci

c'est pas obligatoire, tu met cette fonction uniquement si tu t'en sert

merci pour toutes ces infos.
Juste 1 petite questions :

- hinting ?

http://php.net/manual/fr/language.oop5.typehinting.php

ca vérifie le type ca marche uniquement avec les type array, object (et les class)

function test(array $array)

si tu fais : test('blabla')
va y avoir une erreur pusique 'blabla' c'est pas du stype array

[totofe49]

je pense qu'avec tout ça je vais me débrouiller.
Merci beaucoup et peut-être à plus tard avec une autre class ?