FORUM STYLE PHPBB ET AGENDA PERSONNALISÉ

Commentaire de Skwal le 13/05/2004 13:13:50

Joli boulot pour un début de forum ;)
Seul problème c'est qu'il n'y a aucune identification pour l'admin. N'importe qui peut donc aller sur admin.php et modifier ce qu'il veut :/

Bon courage pour la suite :)

Commentaire de scoder le 25/05/2004 10:41:51

Merci ;)

Skwal-> effectivement je n'ai pas fait de test sur la page admin mais je vais rapidement corriger ça

expresso630-> j'ai vu que certains fichier avaient été renommés en majuscule et je ne sais pas pourquoi... j'ai utilisé scriptedit pour coder ça mais étant donné que je l'ai fait sur plusieurs ordi dont les configs sont totalement différentes (c'est un projet pour mon bts info gestion), les noms de certains fichiers ont été changés...

Je devrai prochainement faire des mises à jour pour ajouter des modérateurs, des thèmes et si je suis vraiment motivé du BBCode :S

Commentaire de scoder le 01/06/2004 09:39:13

Merci astrolab ;)

bizarre que le calendrier ne marchait pas chez toi... :S
enfin si t'as pu le faire fonctionner c cool :)

merci pour la note ça fait plaisir

Commentaire de scoder le 01/11/2004 20:48:56

ouep merci :)

d'ailleurs je vais d'ici peu mettre la source à jour, j'ai pas eu bcp de temps mais je vais vite attaquer ça

Commentaire de scoder le 02/05/2005 18:32:11

salut stephgphy,

merci ça fait plaisir si ça plaît :D
en ce qui concerne le calendrier, à la base il doit s'afficher dès la connexion d'un membre, et s'ouvrir en popup. mais c'est un peu lourd et j'ai apporté quelques modifications à mon code de sorte à ouvrir le calendrier via un lien, ce qui est plus comode.
j'uploaderai le code mis à jour avant la fin de la semaine, dès que j'en aurai le temps ;=)

Commentaire de monoceros01 le 09/11/2005 04:27:42

Bon... Je me suis "amusé" à lire ta source. Et j'avoue avoir eu quelques surprises...
Je vais te dire ce que j'en pense :

En général, je commence par lire une source par sa base de données que j'ai aisément trouvée (forum_BDD.txt). Puis, je commence les choses dans l'ordre logique, c'est-à-dire, en théorie, index.php.
Et là, malheure de malheure! Pas de index.php... C'est déplorable, mais passons.
Je me rabat alors sur le beau install.php.
J'ai alors parcouru tout ton script en comprenant assez vite ma foie, que le gros ACCUEIL.php était ce qui, à mon sens, aurait dû s'appeler index.php.

J'ai alors relevé plusieurs mauvais points.

Tout d'abord, commençons soft par le HTML. Le tiens est particulièrement moche :
  - Tu ne mets pas de Doctype
  - Tu mets des balises en majuscule (chose inutile et même déconseillée pour des raison de compression de données et parce que ce n'est après tout pas très beau)
  - Tu utilises des balises (<font>,...) et des attributs (bgcolor, align,...) obsolètes. Maintenant on a les CSS ;)
  - Tu ne ferme pas tes balises <br />, <hr />, <img /> et <input />
  - dans admin_fac.php tu vas même jusqu'à mettre des attributs entre ' et non entre "

Ensuite, ton système de fichiers :
  - je te déconseille personnellement de t'amuser a mettre certains fichiers ou dossiers en majuscules et d'autre en minuscules. Je pense qu'il est plus aisé de ne travailler qu'avec des fichiers écrits en minuscules. Ça évite les erreurs.
  - voir un Thumbs.db dans un dossier (IMAGES) je trouve ça pas terrible.
  - les images bmp sont d'un format non compressé. Ça serait bien sur internet de diminuer ne serait-ce qu'un peu le poid d'une page.

Maintenant entrons dans le vif du sujet avec le PHP :
  - Globalement je te reprocherais de méler traitement php et affichage html. Ça rend le code particulièrement difficile à lire et à entretenir. Je préconise d'utiliser des variables et de tout afficher avec un seul echo.
  - Tu utilises des stripslashes et addslashes sans d'abord vérifier si le paramètre de configuration magic_quotes_gpc est à 1 (ce qui voudrait dire que les variables renvoyé par des formulaire sont échappées)
  - Tu ouvres tes tags php avec <? au lieu de <?php qui est préférable.
  - Tu utilises à plusieurs reprise une boucle while pour afficher les résultats d'une requête qui n'en retournera jamais plus d'un. (accueil.php ligne 97, forum.php ligne 37, post.php ligne 28, profilem.php ligne 13, titre.php ligne 69)
  - Tu utilises souvent dans ton admin un javascript de redirection. Mais si l'utilisateur désactive son javascript, il aura accès à toutes les fonctionnalités de l'admin!!! Il est donc préférable d'utiliser header('Location: url ');
  - Tu es souvent inconstant sur la récupération des données envoyées par méthode POST ou GET. En effet, tu ne vérifie pas toujours si elles sont définis (isset()).
  - Tu as mis un saut de ligne avant les tags d'ouverture dans le fichier connec.php et temp.php
  - Tu ne limite pas le nombre total de caractères des champs de tes formulaires, alors que tu le fait dans la base de données... Il faudrait empêcher les dépassements.
  - Ton code n'est pas beaucoup commenté.

Regardons certains fichiers plus en détail :
install.php
  - Il serait bon d'utiliser les insertions mutiples au lieu de faire plein de requête d'insertions.
  - les mots de passe des membres ne sont pas cryptés!!!! C'est très mauvais, ça veut dire que tu peux connaître le mot de passe de tout tes membres. C'est limite de l'atteinte à la vie privée pour moi (bon ok je pousse un peu, mais tu vois l'esprit)
  - je verrais plutôt un DATETIME au lieu d'un DATE pour les post (l'horaire du post peut être utile =) )

accueil.php
  - tu mets des if($mid == 0) et if($mid == 1) un peu de partout. Il serait mieux de tous les regrouper selon moi.

calend.php
  - à quoi sert de vérifier la valeur de $jeu, puisque tu lui donne 1 quelques lignes plus tôt?

calendrier.php
  - ligne 94 à 99 : il suffit d'utiliser date("t"); pour avoir le nombre de jours du mois.
  - je trouve que tu te prend beaucoup la tête pour pas grand chose. Personnellement j'aurais fait  :
<?php
include('connec.php');
# Récupération du numéro du premier jour du mois
$first_day = date("N", mktime(0,0,0,$nmois,1,$an));
# Récupération du nombre total de jour dans le mois
$nb_day = date("t");

# Création de la matrice du calendrier
# Un calendrier peut avoir au maxium 42 cases
# (Si le premier jour commence à la dernière case de la première ligne et que le mois à plus de 30 jours,
# ça fait un total de 6 lignes. Donc 6×7... )
for($i=1; $i<=42 $i++)
{
$day_of_month = $i+1-$first_day;
$matrix = ($day_of_month < 1 || $day_of_month > $nb_day) ? $day_of_month : "&nbsp;";
}

# Récupération de la liste des événements et création d'un tableau (Array) de la forme :
# $event['2005-11-07'] = '<br />Anniversaire de Paulette<br />Je me suis gratté la tête</br />C\'est la fête' ;
$query = 'SELECT ev1, ev2, ev3, date FROM calendrier WHERE num_membre=".$mid." ORDER BY date ";
$result = mysql_query($query);
while($data = mysql_fetch_array($result))
{
$event[$data['date']] = ($data['ev1'] !== '') ? '<br />'.$data['ev1'] : '' ;
$event[$data['date']].= ($data['ev2'] !== '') ? '<br />'.$data['ev1'] : '' ;
$event[$data['date']].= ($data['ev3'] !== '') ? '<br />'.$data['ev1'] : '' ;
}

# Création du calendrier
$calendrier = '<table><tr><th>Lundi</th><th>Mardi</th><th>Mercredi</th><th>Jeudi</th><th>Vendredi</th><th>Samedi</th><th>Dimanche</th></tr>';
while(list($key,$val) = each($matrix) )
{
# Début d'une nouvelle ligne si $key = 1,8,15,22 ou 29
$calendrier.= ($key%7 === 1) ? '<tr>' : '' ;
# Affichage du numéro du jour, et de(s) événement(s)
$calendrier.= '<td>'.$val.$event[$an.'-'.$mois.'-'.$val].'</td>';
# Fin d'une ligne si $key = 7, 14, 21 ou 28
$calendrier.= ($key%7 === 0) ? '</tr>' : '' ;
}
$calendrier = '</table>';

# Affichage du Calendrier
echo($calendrier);
?>

exec.php
- ou est l'intérêt
- tu ne l'a même pas protégé contre les non-admins

modif.php
- tu peux faire toutes tes updates en une seule requête.

posted.php
- ligne 31 et 32 : pour enregistrer la date du jour dans un base de données MySQL, il y a la fonciton NOW() ce qui donne :
$query2="insert into post (Msg, Nb_membre, Nb_titre, Date) VALUES ('".$msg."','".$mid."','".$titre."',NOW());";

Enfin, parlons un peu de la structure de ta base de données
Tout d'abord ton système d'agenda :
Je pense qu'il aurait été plus judicieux de faire une table de ce type :
table calendrier :
  - num (PRIMARY KEY)
  - num_membre
  - evenement
  - date
De cette sorte, un membre peut avoir autant d'événements par jours qu'il le souhaite. Et ça évite d'avoir des champs qui ne servent à rien lorsque les membres n'ont pas trois évenements pile à enregistrer.

Sinon je t'avoue être légèrement perplexe sur ta table titre. Mais vu l'heure qu'il est, mes neuronnes n'ont plus le courage d'analyser les requêtes dans lesquelles elle intervient. Mais je pense qu'elle est justifiable :)

En espérant n'avoir pas dit trop de bêtises à cette heure tardive =_=
Bonne nuit!