search

ModjoInc / nomophobie-questionnaire

0 stars 0 forks source link

Feedback #1

Open pixeline opened 7 years ago

pixeline commented 7 years ago

hello Habib,

voici mon feedback.

Respect des consignes

code

  1. c'est un détail, mais en général, on utilise echosans parenthèses. Tu peux aussi carrément écrire ainsi <?= $variable ?> ce qui correspond à <?php echo $variable; ?> C'est pas catholique, mais les créateurs de php ont sans doute voulu nous éviter les ampoules au bout des doigts. ;)
  2. il me semble que si je désactive le js, le formulaire ne fonctionnera plus. Donc tu ne fais pas de l'amélioration progressive. Tu veux vraiment que Tim Berner-Lee vienne sonner chez toi en plein milieu de la nuit avec ses copains des Hell's Angels ?
  3. ton "submit" est en dehors de ta balise <form> cela ne fonctionnera pas. https://github.com/ModjoInc/projet-8-nomophobie/blob/6e7b149bc806ddc32aed532e62037bfa18f7508f/index.php#L58

conclusion

Du coup, je suis mitigé: tu sembles maitriser les conditions, boucles et array, mais les points 2 et 3 ci dessus, ainsi qu'un tel non-respect des consignes me posent question et me font penser plus à un accident qu'à de réelles lacunes. on dirait du code fait trop vite. C'est bâclé et je t'avoue que je suis un peu déçu parce que je sais que tu es capable de réaliser cet exercice beaucoup mieux. Si c'est le cas, je voudrais que tu augmentes ton niveau d'exigences et je dirais "pas grave, mais fais mieux la prochaine fois". Si par contre tu as de vraies difficultés sur certains points, n'hésite pas à venir vers moi et les autres coachs pour t'aider à les élucider, sans plus attendre, tout de suite.

On en reparle à mon retour lundi si tu veux.

Alexandre.

ModjoInc commented 7 years ago

Merci beaucoup pour le retour Alexandre Oui effectivement c bâclé parce que la deadline était plus qu'explosée comme tu dis et résulte d'un relâchement la semaine passée. J'ai passé beaucoup de temps sur les composants design pendant les vacances et pas sur celui-là...je crois avoir fait ça en deux heures...donc tes observations sont 'en plein dans le mille'. Je commence à corriger de suite.

pixeline commented 7 years ago

ok, no hard feeling, hein. Ping moi quand tu veux du feedback sur la version la plus parfaite dont tu es capable. 💯

ModjoInc commented 7 years ago

Bien sûr que non Alex...c'est pile poil la méthode pour booster ma productivité :) g ajouté un project pour résoudre les différents problèmes

pixeline commented 7 years ago

top. attention que "progressive enhancement" = faire fonctionner sans JS (pas php, il n'est pas méchant, lui. C'est gentil les éléphants).