Mini CMS
cs_foxmaster
Messages postés
38
Date d'inscription
mercredi 9 février 2005
Statut
Membre
Dernière intervention
23 septembre 2013
-
23 sept. 2013 à 00:27
doorgets Messages postés 9 Date d'inscription mercredi 7 août 2013 Statut Membre Dernière intervention 10 décembre 2013 - 27 sept. 2013 à 15:47
doorgets Messages postés 9 Date d'inscription mercredi 7 août 2013 Statut Membre Dernière intervention 10 décembre 2013 - 27 sept. 2013 à 15:47
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/100132-mini-cms
https://codes-sources.commentcamarche.net/source/100132-mini-cms
27 sept. 2013 à 15:47
votre CMS n'est n'est pas mauvais, sachant que vous utiliser des algo en mode procédurale, et surtout vous n'utilisez pas de model MVC Complet.
Je pense que vous devez prendre en compte les progrès de php pour la suite de votre projet, voici un exemple de cms trés petit et trés simple que j'ai écris : http://codes-sources.commentcamarche.net/source/100061-doorgets-cms-open-source
Merci pour le partage !!! c'est ça qui est important !
23 sept. 2013 à 12:32
23 sept. 2013 à 12:31
Une petite broutille pour commencer :
dans articles .php tu as, à la ligne 58 :
je pense que ca serai plutot :
il me semble (je ne peux pas tester) qu'en php si la variable est dans la chaîne il l'interprète mais pas certain (mais du coup on se rend moins compte, avec les chartes graphiques, que l'on insère une variable à la chaîne) ... ce qui m'a surtout fait bizarre c'est le découpage au niveau du saut de ligne ^^
ensuite dans connexion.php tu pourrais épurer un peu tes tests de manière à ordonner tes tests. C'est plutot pour la compréhension, mais aussi vis à vis de la taille de tes pages, le nombre de tests, etc. j'ai l'impression que ca risque d'être redondant dans ta manière de coder.
tu écris :
<?php if($_GET['login'] and !$_SESSION['logedon']) { echo '<div id="logon">'; } if($_GET['login'] and $_SESSION['logedon']) { echo '<div id="logedon">'; } if($_GET['login'] or $_SESSION['logedon']) { ... } if($_GET['login']){echo '</div>';} ?>il serai mieux d'écrire :
<?php if($_GET['login'] or $_SESSION['logedon']) { if(!$_SESSION['logedon']) { echo '<div id="logon">'; echo 'Login :<br /><br />Mot de passe :<div id="logpwd"><form method="post" action="/">'; echo '<input name="login" type="text" id="login" size="12" /></br><input name="pass" type="password" id="pass" size="12"><br /><input name="submit" type="submit" value="Connecter" class="bouton">'; echo '</form></div>'; } else { echo '<div id="logedon">'; echo $_SESSION["nom"].' '.$_SESSION["prenom"];if($_SESSION['admin']){echo ' (Administrateur)';}echo '<br>'; echo '<input name="administration" type="button" class="bouton" onClick="change_admlocation('Administration');" value="Administration"><br>'; echo '<input name="deconnect" type="button" class="bouton" onClick="deconnect();" value="Déconnecter">'; echo '<a href="javascript:void(0);" rel="nofollow" style="text-decoration:none;border:0px;" onclick="connect();" title="Connexion"><img src="icones/adminfemme.png" width="20" height="20" alt="Connexion" /></a>'; } echo '</div>'; } ?>je ne suis pas certain d'avoir bien fait tous mes copy paste mais normalement ca doit être bon. Le but ici n'étant pas de "corriger" le code, car je pense qu'il fonctionne très bien, mais éviter des tests redondants (ca ne va pas faire gagner des secondes lors des appels, car les traitements sont rapids, mais éviter d'effectuer trop de test). J'ai mis la page de connexion en exemple, mais si je le vois dans d'autres pages je ne le noterai pas (je te laisse le soins de oui ou non prendre en compte cela ^^) .
En tout cas bon dev !
naga
23 sept. 2013 à 11:28
ah oui tient j'avais pourtant mis un ZIP, je l'ai rajouté...
Ben j'ai regardé les CMS et je trouve que c'est pour la plupart des usines à gaz, là l'idée est vraiment de le simplifer au max et le rendre + intuitif
C'est aussi un travail pour m'améliorer personnellement
d'ou cette demande de commentaire...
23 sept. 2013 à 08:00
Le contenu est pas mal et à du te demander pas mal de taff, mais le truc c'est que des cms fonctionnels, il y en a déjà pas mal. Donc la question est : qu'est ce qui différencie le tiens de l'existant ? (peu être intégrer une gestion du référencement ? je ne suis pas les nouveautés des cms mais je ne crois pas en avoir vu).
Sinon je n'ai pas trouvé de quoi voir la source donc je ne donnerai pas de commentaires sur le code ^^
bon dev.
naga