Skip to content

v1.0.0

Latest

Choose a tag to compare

@iSeaox iSeaox released this 08 Mar 10:51

Récapitulatif des problèmes restants


Problèmes identifiés dans le fichier essai.php (lignes 65 à 107) :

  1. Vous ne vérifiez pas si le résultat de la requête sur la table sensor est vide ou non, ce qui peut causer des erreurs dans ce cas. De plus, le fait que le programme plante et qu'il n'y ait pas de try-catch force la création d'alertes inutiles, basées sur des données qui n'existent pas.

    • Solution possible : Ajouter des if pour vérifier que les données ne sont pas absentes.
  2. Vous effectuez une requête sur tous les capteurs du bac et vous les ordonnez par ordre alphabétique suivant le type (clé non primaire). Mais si un capteur est ajouté, cela peut modifier l'ordre. Il serait préférable d'ordonner par rapport à la clé primaire, celle-ci étant unique.

  3. Pourquoi les données du luxmètre ne sont-elles pas affichées ?

  4. TODO : Ajouter la visualisation des images.

  5. TODO : Améliorer la mise en page, surtout dans l'onglet sommaire.

Problèmes identifiés dans le fichier essai.php (lignes 490 à 491) :

  • La ternaire est mal écrite :
const humidityData = <?= !empty($sensorsWithData) ? json_encode($sensorsWithData[0]['value']) : null; ?>;

Au lieu de :

const humidityData = <?= !empty($sensorsWithData) ? json_encode($sensorsWithData[0]['value']) : 'null'; ?>;

Problèmes identifiés dans le fichier profil.php :

Objectif :

  • Réduire le nombre de fichiers utilisés et harmoniser l'UI.

Solution possible : Utiliser AJAX (jQuery) pour POST sur la même page :

  1. TODO : Permettre de rapatrier le contenu de modifier_photo.php dans profil.php.
  2. Permettre de rapatrier le contenu de update_mode.php dans profil.php (ici, pas besoin d'AJAX, il suffit d'utiliser le même mécanisme que pour la modal du header).
<script>
  document.getElementById('mode').addEventListener('change', function() {
    document.getElementById('modeForm').submit();
  });
</script>

Devient :

<script>
  document.getElementById('mode').addEventListener('change', function() {
    const mode = this.value;
    window.location.href = `profil.php?mode=${mode}`;
  });
</script>

Problèmes identifiés dans le fichier recettes.php :

  1. Problèmes d'affichage pour le mode 'contraste'.