Discussion :

[bndd24]

oui je me disais bien aussi
non t’inquiète je ne suis pas très fort je te l'accorde mais j'utilise quand même le bon format hehe

[ABCIWEB]

Je viens d'éditer ma précédente réponse, y'avait effectivement un petit malentendu de ma part

[bndd24]

EDIT : ah ok, je viens de comprendre que c'est dans ta requête de sélection que tu obtiens le format français.

exact

[ABCIWEB]

Oui ce qui m'a dérouté c'est que tu formatais le résultat au format français dans ta requête de sélection pour ensuite utiliser les fonctions date de php et ça n'était pas logique.

...après si tu n'utilises cette date que pour afficher l'année (si tu n'as pas besoin d'exploiter cette date autrement ensuite avec php) tu pourrais faire l'équivalent de ce que tu as fais avec php, directement dans ta requête avec les fonctions dateTime de mysql (je dis pas que ce sera exactement la même syntaxe). Sur l'ensemble du script cela devait être un peu plus performant, mais bon là on est dans le domaine de l'optimisation "ultime" Eventuellement cela peut te faire un bon exercice car on a souvent besoin d'utiliser les fonctions date de mysql pour faire du tri.

[bndd24]

Oui ce qui m'a dérouté c'est que tu formatais le résultat au format français dans ta requête de sélection pour ensuite utiliser les fonctions date de php et ça n'était pas logique.

...après si tu n'utilises cette date que pour afficher l'année (si tu n'as pas besoin d'exploiter cette date autrement ensuite avec php) tu pourrais faire l'équivalent de ce que tu as fais avec php, directement dans ta requête avec les fonctions dateTime de mysql (je dis pas que ce sera exactement la même syntaxe). Sur l'ensemble du script cela devait être un peu plus performant, mais bon là on est dans le domaine de l'optimisation "ultime" Eventuellement cela peut te faire un bon exercice car on a souvent besoin d'utiliser les fonctions date de mysql pour faire du tri.

que pour afficher l'année ? non j'affiche que l'age..

voici mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
function age($date_naissance)
{
    $diff = (new \DateTime())->diff(new \DateTime($date_naissance));
    return $diff->y;
}

Pour une date de naissance de 1987-02-28 il affiche à l'écran 28 ans
Pour une date de naissance de 1987-02-22 il affiche à l'écran 29 ans

donc parfait.

par contre je ne vois pas comment optimiser ce code (qui est déjà pas mal compressé)

pour les exercices sur les dates oulaa t’inquiète pas, j'ai de quoi faire, regarde :
gerer-l-affichage-d-date-vers-format-plus-confortable-visuellement/

[ABCIWEB]

Non ce code ne peut plus être optimisé, enfin je pense pas.

Ce que je voulais dire c'est que plutôt que de le faire en php tu pourrais certainement le faire dans ta requête de sélection sql (il y a aussi datediff dans les fonctions sql). Mais bon t'embête pas pour ça. Cela aurait de l'intérêt si cela pouvait t'éviter un gros traitement côté php, mais ici ce n'est pas le cas.

[rawsrc]

Salut,

en une ligne (profite de le possibilité de déréférencer) sur une opération aussi simple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
function age($date_naissance)
{
    return (new \DateTime())->diff(new \DateTime($date_naissance))->y;
}

[bndd24]

Je viens de faire la mise à jour ! eh hop une ligne en moins

[abderrahmane23]

Salut,

j'opterais pour ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
if ((new \DateTime())->diff(new \DateTime($_POST['date_naissance']))->format('%y') < 18)
{
    $errors['date_naissance'] = "Vous devez être majeur pour vous inscrire";    
}

pouvez-vous expliquer le code s'il vous plaît ? je veux l'utiliser mais j'ai besoin de le comprendre d'abord ...

[rawsrc]

Salut,

si la différence entre la date d'aujourd'hui et la date de naissance est strictement inférieure à 18 ans alors message d'alerte.