Discussion :

[saturn1]

Bonjour je débute en oo et j'ai fais le début d'une classe news.
Mais vous pouvez me dire ce qui ne va pas svp

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
22
23
24
25
26
27
28
29
30
 
<?php
class News {
		private $_titre;
		private $_auteur;
		private $_contenu;
		private $_date;
	public function __construct($id) {
		if($id != NULL) {
			$QS_news = mysql_query("SELECT * FROM news WHERE id='".$id."'")or exit(mysql_error());
			$D_news = mysql_fetch_assoc($QS_news);
			$this->_titre = $D_news['titre'];
			$this->_auteur = $D_news['auteur'];
			$this->_contenu = $D_news['contenu'];
			$this->_date = $D_news['date'];
		}
	}
}
class NewsManager {
	public function getAllNews() {
		$news = array();
		$QS_all_news = mysql_query("SELECT id FROM news ORDER BY id DESC");
		while($D_news = mysql_fetch_assoc($QS_all_news)) {
			$news[] = new News($D_news['id']);
		}
	}
}
 
 
?>

Voilà étant donner que je suis noob merci de ne pas me dire qu'il y a plein de truc qui ne vont pas.
Mais je préfère que vous me dîtes un seul défaut et que vous l'expliquiez correctement.

MERCI

[s.n.a.f.u]

La première remarque que je ferais est de faire un fichier pour chaque déclaration de classe, le nom du fichier étant le même que celui de la classe.
Cela te simplifiera grandement la maintenance.

[grunk]

Je rajouterais :
1- Tu fait une requete pour récupérer toutes les ID de tes news , mais ensuite tu instancie ta classe news qui récupère les infos d'une news.
Donc tu fait 2x la même chose.

Pourquoi ne pas modifier ton instanciation de news pour lui passer dirrectement tous les paramètres voulu ?
Dans ton cas si tu as 200 news à afficher tu fais 201 requetes au lieu d'une seule :/

2- Ton news manager n'as pas de constructeur donc ta méthode getAllNews pourrait (devrait ?) devenir static

[trotters213]

Code :

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

mysql_query("SELECT * FROM news WHERE id='".$id."'")

Ne fais jamais ça si tu ne contrôles pas le contenu de $id car il y a des possibilités d'injection SQL (même si mysql_query n'exécute qu'une seule requête) : c'est une très mauvaise habitude

Sinon, hormis le fait que je ne vois pas à quoi sert cette fonction (je suppose que tu ne l'as pas terminée), tu peux la déclarer "static" si elle l'est :

Code :

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

1
2
3
4
5
6
7
public function getAllNews() {
		$news = array();
		$QS_all_news = mysql_query("SELECT id FROM news ORDER BY id DESC");
		while($D_news = mysql_fetch_assoc($QS_all_news)) {
			$news[] = new News($D_news['id']);
		}
	}

[saturn1]

Ok merci !!
Sinon j'ai une question subsidiaire.
Imaginons que j'ai un site complet donc supposé avec pas mal de classe.
Est-ce que c'est utile de créer une classe
InputUser
avec comme méthode vérifier tout ce que rentre l'utilisateur!

Merci

[s.n.a.f.u]

Ok merci !!
Sinon j'ai une question subsidiaire.
Imaginons que j'ai un site complet donc supposé avec pas mal de classe.
Est-ce que c'est utile de créer une classe
InputUser
avec comme méthode vérifier tout ce que rentre l'utilisateur!

Merci

Il faut toujours vérifier les entrées utilisateur, et un bon programmeur fait du code réutilisable.

Après, en classe, en fonction ou autre, c'est un choix personnel. Si ton site est tout objet, c'est une bonne idée.