"La communauté codes-sources à un esprit didacticiel, et non critique."= > ou alors critique dans le sens positif du terme : critique en vue d'une amélioration... dans ton commentaire, tu as bien expliqué comment améliorer son programme, alors tu as bien fait.
bon, sinon, pour cette source
global register est à mettre à on pour tester, ce qui est extrèmement dangereux coté sécurité...
plutot que d'utiliser une suite de echo, mieux vaut utiliser un seul echo.
echo "<center> </center>";
center est dépressié en xhtml
<td align =center >
align aussi, et on met des " autour des attributs.
t'as un peu de POO en javascript, mais je n'ai pas bien compris à quoi elle sert, tu pourrais expliquer ?
moi je ne te conseille pas la boucle, mais te demande pourquoi tu passes par du php ?
on ne met pas une balise style dans le body, on la met dans le head...
voilà l'une des erreurs que j'ai quand j'exécute ton fichier :
Notice: Use of undefined constant Avril - assumed 'Avril' in /var/www/html/Calendar/index.php on line 23
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...
tout à fait euhreux, je ne donnais que des conseils pour essayer d'aider lamine11 à mieux programmer. La communauté codes-sources à un esprit didacticiel, et non critique.
as-tu regarder les fichiers ? connais-tu le PHP ? dans ce cas, tu ne pourras que faire les mêmes remarques que moi, sinon tu n'as rien à dire au sujet de mon commentaire.
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.
Bonjour à tous!
kiki2sirom : WOW!, n'oubli pas qu'il s'agit d'une source du niveau Débutant. au moins tu as donné des conseils.
Bravo lamine11, d'arrivé a faire dequoi de fonctionnel et qui réponds a tes attentes personnelle ca prends de la patience. Alors ne t'inquiete pas! ce monde est rempli d'idiot! qui n'ont qu'une seul idée en tete! TE FAIRE CHIER! (genre kiki2sirom)
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é
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...
* 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 ??
Pourquoi ne pas faire évoluer cette version au fur et à mesure des Bugs découvert ? plutot que de la laisser en l'état !
Le développeur initial pourrait mieux voir ses erreurs et comprendre comment on peut optimiser son code.
Dommage l'idée était bonne....
- Messages postés
-
12303
- Date d'inscription
- mardi 10 février 2004
- Statut
- Modérateur
- Dernière intervention
- 30 juillet 2012
38bon, sinon, pour cette source
global register est à mettre à on pour tester, ce qui est extrèmement dangereux coté sécurité...
plutot que d'utiliser une suite de echo, mieux vaut utiliser un seul echo.
echo "<center> </center>";
center est dépressié en xhtml
<td align =center >
align aussi, et on met des " autour des attributs.
t'as un peu de POO en javascript, mais je n'ai pas bien compris à quoi elle sert, tu pourrais expliquer ?
<select name="valMiF">
<?php
echo"<option>00</option>";
echo"<option>05</option>";
echo"<option>10</option>";
echo"<option>15</option>";
echo"<option>20</option>";
echo"<option>25</option>";
echo"<option>30</option>";
echo"<option>35</option>";
echo"<option>40</option>";
echo"<option>45</option>";
echo"<option>50</option>";
echo"<option>55</option>";
?>
</select>
moi je ne te conseille pas la boucle, mais te demande pourquoi tu passes par du php ?
on ne met pas une balise style dans le body, on la met dans le head...
voilà l'une des erreurs que j'ai quand j'exécute ton fichier :
Notice: Use of undefined constant Avril - assumed 'Avril' in /var/www/html/Calendar/index.php on line 23
as-tu regarder les fichiers ? connais-tu le PHP ? dans ce cas, tu ne pourras que faire les mêmes remarques que moi, sinon tu n'as rien à dire au sujet de mon commentaire.
Merci Garno de ton soutien ;)
Cordialement