Closed NolanEvard closed 3 years ago
NicolasGlassey
commented
3 years ago Bonsoir Nolan,
Je vous propose de tester si le fichier existe avant d'essayer de l'ouvrir. Evitant ainsi de réaliser un fopen sur un fichier inexistant.
Le résultat sur l'IHM est celui-là :
Et voilà le commit: https://github.com/NicolasGlassey/PROJWEB-INFOWEB/commit/5e7b34a236978d79d60b13bf82eaaa6d530aa464
On regarde demain ensemble si besoin.
Bonne soirée !
NolanEvard
commented
3 years ago Bonjour,
Je suis conscient que le fichier n'existe pas, c'était dans le but de gérer cette exception que j'ai fait ce test. Mais il est vrai que c'est une exception qui est assez peu probable et donc qui ne mérite certainement pas que l'on y passe trop de temps.
Merci pour le temps passé pour cette issue et meilleures salutations.
Nolan Evard , étudiant CPNV filière informatique, Sainte-Croix SI-MI2b @.***
On May 26, 2021 23:51, NicolasGlassey @.***> wrote:
Bonsoir Nolan,
Je vous propose de tester si le fichier existe avant d'essayer de l'ouvrir. Evitant ainsi de réaliser un fopen sur un fichier inexistant.
Le résultat sur l'IHM est celui-là :
[image]https://user-images.githubusercontent.com/5616312/119735668-f9bbf900-be7c-11eb-9f81-78d08d6a283a.png
Et voilà le commit: 5e7b34ahttps://github.com/NicolasGlassey/PROJWEB-INFOWEB/commit/5e7b34a236978d79d60b13bf82eaaa6d530aa464
On regarde demain ensemble si besoin.
Bonne soirée !
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/NicolasGlassey/PROJWEB-INFOWEB/issues/3#issuecomment-849145092, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOXJ63JEG2U2IFCU5VO5XULTPVUPFANCNFSM45SIYX2Q.
NolanEvard
commented
3 years ago Veuillez m'excuser j'ai mal lu votre mail précédent. Je vais faire les corrections nécessaires pour éviter le problème.
Meilleures salutations.
Nolan Evard , étudiant CPNV filière informatique, Sainte-Croix SI-MI2b @.***
On May 27, 2021 08:04, EVARD Nolan @.***> wrote: Bonjour,
Je suis conscient que le fichier n'existe pas, c'était dans le but de gérer cette exception que j'ai fait ce test. Mais il est vrai que c'est une exception qui est assez peu probable et donc qui ne mérite certainement pas que l'on y passe trop de temps.
Merci pour le temps passé pour cette issue et meilleures salutations.
Nolan Evard , étudiant CPNV filière informatique, Sainte-Croix SI-MI2b @.***
On May 26, 2021 23:51, NicolasGlassey @.***> wrote:
Bonsoir Nolan,
Je vous propose de tester si le fichier existe avant d'essayer de l'ouvrir. Evitant ainsi de réaliser un fopen sur un fichier inexistant.
Le résultat sur l'IHM est celui-là :
[image]https://user-images.githubusercontent.com/5616312/119735668-f9bbf900-be7c-11eb-9f81-78d08d6a283a.png
Et voilà le commit: 5e7b34ahttps://github.com/NicolasGlassey/PROJWEB-INFOWEB/commit/5e7b34a236978d79d60b13bf82eaaa6d530aa464
On regarde demain ensemble si besoin.
Bonne soirée !
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/NicolasGlassey/PROJWEB-INFOWEB/issues/3#issuecomment-849145092, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOXJ63JEG2U2IFCU5VO5XULTPVUPFANCNFSM45SIYX2Q.
J'ai terminé de modifier le système d'erreurs cependant il y a un petit bémol. Dans le fichier jsonManager.php je récupère le résultat du fopen chargé d'ouvrir le fichier JSON. Cependant lorsque ce fopen échoue à ouvrir le fichier JSON un WARNING s'affiche quand même sur la page de boutique.
J'ai donc dû ajouter quelques lignes de code pour contrer cela mais je ne suis pas sûr que ce soit optimal. Pourriez vous jeter un coup d'œil aux lignes 29-31 du fichier JsonManager.php.