A lot of small fixes

2019-10-13 20:58:13 +02:00 · 2019-10-13 20:58:13 +02:00 · 0200c2d240
commit 0200c2d240
parent 79a558a715
7 changed files with 19 additions and 56 deletions
--- a/README.md
+++ b/README.md
@ -15,7 +15,7 @@ Le projet est disponible sur [function.divlo.fr](https://function.divlo.fr/).
 ## La liste des Fonctions Principales :   
 | Nom | Description | Paramètre(s) |
 | -- | -- | -- |
-| **weatherRequest()** | Affiche la météo et l'heure local selon la ville grâce à l'API : [openweathermap.org](https://openweathermap.org/). | Aucun paramètre |
+| **weatherRequest()** | Affiche la météo et l'heure locale selon la ville grâce à l'API : [openweathermap.org](https://openweathermap.org/). | Aucun paramètre |
 | **randomNumber(min, max)** | Génère un nombre aléatoire entre un minimum inclus et un maximum inclus. | - min : Nombre Minimum - max : Nombre Maximum |
 | **calculateAge(birthDate)** | Calcule l'âge de quelqu'un selon la date de naissance. | - birthDate : date de naissance au format (dd/mm/yyyy). |
 | **convertTemperature(degree,unit)** | Convertit des Degré Celsius en Degré Fahrenheit et l'inverse aussi. | - degre : Le nombre que vous voulez convertir - unite : l'unité du nombre que vous voulez convertir (°C ou °F)	|
--- a/php/config.php
+++ b/php/config.php
@ -40,7 +40,7 @@ switch ($currentpage) {
        break;
    case '/views/function-views/weatherRequest.php':
        $title = 'Météo';
-        $description = "Affiche la météo et l'heure local selon la ville.";
+        $description = "Affiche la météo et l'heure locale selon la ville.";
        $image = 'https://function.divlo.fr/img/function-image/weatherRequest.png';
        break;
    case '/views/function-views/filterStudents.php':
--- a/scripts/fonctions_annexes.js
+++ b/scripts/fonctions_annexes.js
@ -2,12 +2,7 @@

 // Vérifie si une valeur est vide
 function isEmptyValue(value) {
-    if (value === '' || value === null || value === undefined) {
-        return true
-    } 
-    else {
-        return false
-    }
+    return value === '' || value === null || value === undefined;
 }

 // Formate les nombres avec des espaces (ex : 76120 = 76 120)
@ -92,23 +87,15 @@ function realDateTime(id)
    return true;
 }

-// Récupére le décalage en secondes à partir de l'heure UTC grâce à l'API
+// Récupére le décalage en secondes à partir de l'heure UTC grâce à l'API openweathermap.org
 function timeZone(json) {
-    if(json.name === 'Moscou') // Il faut ajouter + 1h de décallage à Moscou
-    {
-        timeZoneValue = (json.timezone / 60 / 60) + 1;
-    }
-    else
-    {
-        timeZoneValue = json.timezone / 60 / 60;
-    }
+    timeZoneValue = json.timezone / 60 / 60;
    let timeZoneStr = timeZoneValue.toString();
    return dateTimeUTC(timeZoneStr); 
 }

 // Vérifie si une date est valide par rapport à la date d'aujourd'hui 
 function isValidDate(s) {
-
    // Date à vérifier 
    let toVerifyDate = s.split('/');
    let splitedToVerifyDate = toVerifyDate[2] + '-' + (parseInt(toVerifyDate[1]) + 1) + '-' + toVerifyDate[0];
--- a/scripts/fonctions_principales.js
+++ b/scripts/fonctions_principales.js
@ -35,13 +35,12 @@ function randomNumber(min, max) {

 // Calcule l'âge de quelqu'un selon la date de naissance
 function calculateAge(birthDateEntered) {
-
    // Les variables de la fonction
-    let birthDateDay = parseInt(birthDateEntered[0] + birthDateEntered[1]);
-    let birthDateMonth = parseInt((birthDateEntered[3] + birthDateEntered[4]) - 1);
-    let birthDateYear = parseInt(birthDateEntered[6] + birthDateEntered[7] + birthDateEntered[8] + birthDateEntered[9]);
+    let birthDateDay = parseInt(birthDateEntered.substring(0, 2));
+    let birthDateMonth = parseInt((birthDateEntered.substring(3, 5)) - 1);
+    let birthDateYear = parseInt(birthDateEntered.substring(6, 10));
    dateTimeUTC('0');
-    day = parseInt(day)
+    day = parseInt(day);
    month = parseInt(month - 1);
    year = parseInt(year);

@ -51,16 +50,14 @@ function calculateAge(birthDateEntered) {
    // Calcule l'âge - Moment.js
    let ageYears = dateNow.diff(birthDate, 'year');
    birthDate.add(ageYears, 'years');
-    
    let ageMonths = dateNow.diff(birthDate, 'months');
    birthDate.add(ageMonths, 'months');
-    
    let ageDays = dateNow.diff(birthDate, 'days');

    let isValidDateFunction = isValidDate(birthDateDay + '/' + birthDateMonth + '/' + birthDateYear); 

    // Vérifie si la valeur entrée correspond à une date de naissance valide
-    if(isValidDateFunction === true)
+    if(isValidDateFunction === true && !isNaN(ageDays))
    {
        ageYears = formatNumberResult(ageYears);
        // Si c'est ton anniversaire aujourd'hui
@ -76,7 +73,7 @@ function calculateAge(birthDateEntered) {
    else 
    {
        return messageError;
-    }
+    }  
 }

 // Convertis des °C en °F et l'inverse aussi
--- a/scripts/main.js
+++ b/scripts/main.js
@ -21,7 +21,6 @@ $(function () {
    {
        let minEntered = $('#minValue').val();
        let maxEntered = $('#maxValue').val(); 
-
        if(isEmptyValue(minEntered) || isEmptyValue(maxEntered))
        {
            $('.results').html(emptyMessageError);
@ -41,14 +40,7 @@ $(function () {
    {
        let birthDateEntered = $('#birthDateValue').val();
        let result = calculateAge(birthDateEntered);
-        if(result === messageError)
-        {
-            $('.results').html(messageError);
-        }
-        else 
-        {
-            $('.results').html(result);
-        }
+        $('.results').html(result);
    });

    $("#submitConvertTemperature").click(function() 
@ -56,7 +48,6 @@ $(function () {
        let temperatureValue = $('#temperatureValue').val();
        let degree = parseFloat(temperatureValue.slice(0, temperatureValue.length - 2));
        let unit = temperatureValue.slice(temperatureValue.length - 2);
-
        if(isEmptyValue(temperatureValue))
        {
            $('.results').html(emptyMessageError);
@ -80,7 +71,6 @@ $(function () {
        let firstValue = $('#firstValue').val();
        let unitFirstValue = $("#firstValueUnit option:selected").text();
        let secondValue = $("#secondValue option:selected").text();
-
        if(isEmptyValue(firstValue) || isEmptyValue(secondValue))
        {
            $('.results').html(emptyMessageError);
@ -89,14 +79,7 @@ $(function () {
        {
            firstValue = parseFloat(firstValue.replace(/\s/g,''));
            let result = convertDistance(firstValue, unitFirstValue, secondValue);
-            if(result === messageError)
-            {
-                $('.results').html(messageError);
-            }
-            else 
-            {
-                $('.results').html(result);
-            }
+            $('.results').html(result);
        }
    });

@ -104,7 +87,6 @@ $(function () {
    {
        let nameEntered = $('#nameEntered').val();
        let filteredLetter = $("#filteredLetter").val();
-
        if(isEmptyValue(nameEntered) || isEmptyValue(filteredLetter))
        {
            $('.results').html(emptyMessageError);
@ -150,9 +132,6 @@ $(function () {
        if(isEmptyValue(value) || isNaN(parseFloat(value)))
        {
            $('.results').html(emptyMessageError);
-            $("#value, #submitConvertCurrency").click(function() {
-                document.location.replace("../function-views/convertCurrency.php");
-            });
        }
        else 
        {
@ -211,7 +190,6 @@ $(function () {
    {
        let numbersValue = $('#numbersArabic').val();
        let convertNumberType = $("#convertNumberType option:selected").text();
-
        if(isEmptyValue(numbersValue)) {
            $('.results').html(emptyMessageError);
        }
@ -245,8 +223,9 @@ $(function () {
    $("#numberToTest").bind("keyup change", function() 
    {
        let numbersValue = $('#numberToTest').val();
-        if (!isNaN(parseInt(numbersValue))) { 
-            let result = armstrongNumber(parseFloat(numbersValue.replace(/\s/g,''))); 
+        numbersValue = parseInt(numbersValue.replace(/\s/g,''));
+        if (!isNaN(numbersValue) && numbersValue >= 0) { 
+            let result = armstrongNumber(numbersValue); 
            $('.results').html(result);
        }
        else {
@ -257,7 +236,6 @@ $(function () {
    $("#submitHeapAlgorithm").click(function() 
    {
        let value = $('#value').val();
-
        if(isEmptyValue(value))
        {
            $('.results').html(emptyMessageError);
--- a/views/function-list.php
+++ b/views/function-list.php
@ -15,7 +15,7 @@
            <div class="text-center pb-5">
                <h2 class="function-list-title"><a href="./function-views/weatherRequest.php">Météo</a></h2>
                <a href="./function-views/weatherRequest.php"><img class="function-list-image" src="/img/function-image/weatherRequest.png" alt="Météo"></a>
-                <p class="function-list-description">Affiche la météo et l'heure local selon la ville.</p>
+                <p class="function-list-description">Affiche la météo et l'heure locale selon la ville.</p>
            </div>  
        </div>
        <div class="col-sm-12 col-md-6 pb-4">
--- a/views/function-views/weatherRequest.php
+++ b/views/function-views/weatherRequest.php
@ -9,7 +9,8 @@
  <h1><span class="important"><?php echo $title?></span> :</h1>
  <p class="pt-3 text-center"><?php echo $description?> <br> 
  Pour une meilleure précision, je recommande de préciser le pays de la ville. <br>
-  Exemple : Paris, FR
+  La météo est récupérée grâce à l'API <a href="https://openweathermap.org/" target="_blank">openweathermap.org</a>. <br> Il faut rentré le nom anglais de la ville (si il est différent qu'en français). <br>
+  Exemples : Paris, FR - London, UK - Moscow, RU - etc.
  </p>
  <div class="text-center">
    <img class="function-image" src="/img/function-image/weatherRequest.png" alt="Météo">