UN PETIT CALENDRIER SYMPA PHP/JAVASCRIPT/MYSQL

Commentaire de kiki2sirom le 20/04/2006 11:01:08

bon et bien beaucoup de choses qui malheureusement ne vont pas.
je ne pourrais pas tout mettre, n'ayant pas le temps, mais en vrac quelques remarques :

* la connexion à la base ne doit pas se faire dans chaque php, sachant en plus quetu as un fichier mydb.php avec les connexions à la base. faire un include_once après dans chaque php et ce sera beaucoup plus facile à gérer.

* il faut systématiquement écrire <?php à la place de <?
on trouve les 2
à ne plus faire <?= mais <?php echo

* c'est écrit partout dans code source : préférer les ' aux ", pour optimiser

* calculer sur chaque PHP également les jours ou les mois : NON, faire une fonction et c'est réglé

* echo"<option>08</option>";  
echo"<option>09</option>";
echo"<option>10</option>";
echo"<option>11</option>";
echo"<option>12</option>";
echo"<option>13</option>";
echo"<option>14</option>";
echo"<option>15</option>";
echo"<option>16</option>";
echo"<option>17</option>";
echo"<option>18</option>";
echo"<option>19</option>";
echo"<option>20</option>";
echo"<option>21</option>";
echo"<option>22</option>";
echo"<option>23</option>";
echo"<option>24</option>";

heu, une boucle ce serait pas mieux ?? si bien sûr

* <SCRIPT language="javascript"> à remplacer par <script type="text/javascript">
* dans ton fichier Semaine.php, tu as une fonction javascript Fermer() que tu n'utilises pas et même qui ne sert à rien, de plus met ton javascript dans la balise head, le css également
il est préférable même de gérer le CSS dans un fichier séparé, appelé après sur chaque page

* et puis les popup lundi, popup mardi.... c'est LARGEMENT optimisable, heureusement qu'il n'y a que 7 jours dans la semaine.

* insert.php :echo "<script langauge=\"jacascript\">"; ahhhhhhhhhhhhhhhhhhhhhhhhhhhh !!!! relis toi un peu !!

* les fichiers insert.php et update.php peuvent être regroupés en un seul...

* identify.php : @ $name=$HTTP_POST_VARS['name'];
@ $password =$HTTP_POST_VARS['password'];
quelle horreur !!!!!!!!

* choisit echo, plus rapide que print, en tout cas pas les 2

* tous les commentaires, tu aurais pu les virer quand même

* tu as un fichier CSS externe, biennnnnnn, mais il faut l'utiliser dans tes PHP sinon à quoi ça sert que DUCROS.....

bon je n'ai même pas testé l'appli, juste regarder les fichiers php

Généralement : pense aux boucles, ton prog prendra 10 fois moins de place et pense également aux fonctions PHP
(je n'ai pas tout relevé, loin de là)
je ne note pas, essaye de faire qq modifs constructives et là je pourrais peut-être mettre une note ?!?!

on sent bien aussi que tous les php ne sont pas de toi, (peut-être aucun d'ailleurs ????) ;) tu me confirmes ??

kiki

Commentaire de Garno le 21/04/2006 03:36:04

EUPHREUX je pense que tu a mal compris le sens du commentaire de kiki2sirom. En fait sa sert tout simplement à aider celui qui à poster la source à progresser dans son niveau de programmation et je crois que c'est l'esprit même de la communauté codes-sources.

Un programme (ou une source) qui marche n'est pas nécessairement un bon programme.

Commentaire de MyXiLo le 21/04/2006 12:55:18

De la même manière un commentaire qui pointe des erreurs n'est pas nécessairement un commentaire aimable. Celui ci paraissait un brin autoritaire, mais c'est une question de sensibilité probablement...