Skip to content

Clean separation#2

Open
rocambille wants to merge 2 commits intoLucBlocker01:masterfrom
rocambille:clean-separation
Open

Clean separation#2
rocambille wants to merge 2 commits intoLucBlocker01:masterfrom
rocambille:clean-separation

Conversation

@rocambille
Copy link
Copy Markdown

La commande php -S localhost:8000 sert la totalité de ton projet. Certaines parties servent au fonctionnement interne et ne devraient pas être accessibles par un client. Par exemple, l'URL http://localhost:8000/src/classes/Quiz.php est accessible, alors que ce n'est pas une page : c'est une classe utilisée dans le processus de génération de quiz, et elle n'a pas de sens appelée toute seule.

Tu as mis en place un dossier public et un dossier src : les choses accessibles publiquement pour un client doivent aller dans le dossier public et les fichiers sources qui servent au fonctionnement de ton projet doivent aller dans le dossier src.

Tu peux alors servir uniquement ton dossier public avec l'option -t :

php -S localhost:8000 -t public

Et l'URL http://localhost:8000/src/classes/Quiz.php n'existe plus ;)

Aussi, évite les valeurs en dur comme http://localhost:8000/src/quizCreator.php : tu peux utiliser une URL relative à la racine du nom de domaine /quizCreator.php. De cette façon, ton code reste valable qu'il tourne en local (sur localhost:8000 ou sur un autre port) ou qu'il tourne sur un verveur en ligne (lucblocker01.com par exemple)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant