Retour de fonction de type DateTime

Bonsoir,

voici une fonction qui retourne une variable de type DateTime, sauf que ce retour n'est pas accepté. Pourquoi ? Merci.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

static public function SQLFormatDate($date): DateTime { if (UtilFct::contains("Earlier",$date)) $date='Dec-18'; $formats = ['d/m/Y H:i','M d, Y', 'M d Y', 'M,d', 'M-d']; foreach ($formats as $f) { if (($val=\DateTime::createFromFormat($f, $date)) !== false) { return ((DateTime)$val); } } return (DateTime)false; }

---

Citation:

---

Parse error: syntax error, unexpected '$val' (T_VARIABLE), expecting ';' in C:\wamp64\www\ticket2\Classes\Olivier\Validator.php on line 58

---

Salut,

mais c'est quoi ces notations ?

Code:

---

1 2	return ((DateTime)$val); return (DateTime)false;

---

Tu fais quoi là ? Je lis tes codes et je suis dubitatif...
Réfléchis un peu : comment veux-tu convertir false en DateTime ?
Pareil pour $var, tu ne peux pas caster avec une classe : une classe produit une instance donc tu dois l'instancier avec new.
D'ailleurs, $val est déjà une instance de la classe DateTime, donc ton code est bancal.
Il faut que tu essaies de suivre le déroulé du code et pas écrire des lignes comme ça au hasard, tu ne vas pas y arriver.

Oui, j'ai tellement pas l'habitude de la POO que j'ai fait n'importe quoi. C'est mieux comme ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

static public function SQLFormatDate($date) { if (UtilFct::contains("Earlier",$date)) $date='Dec-18'; $formats = ['d/m/Y H:i','M d, Y', 'M d Y', 'M,d', 'M-d']; foreach ($formats as $f) { if (($val=\DateTime::createFromFormat($f, $date)) !== false) { return ($val->format('Y-m-d')); } } return false; }

---

encore une chose : perds l'habitude de mettre des () au return.
return ($val->format('Y-m-d')); ⇒ return $val->format('Y-m-d');. Généralement les () sont réservés à la gestion des priorités dans une opération complexe ou à l'évaluation boolean d'une expression.

Bonjour,

Je rappelle : fais SIMPLE.

Code:

---

1 2 3

if (($val=\DateTime::createFromFormat($f, $date)) !== false) { return ($val->format('Y-m-d')); }

---

s'écrit PLUS LISIBLEMENT *:

Code:

---

1 2 3 4

$val = \DateTime::createFromFormat($f, $date); if ($val !== false) { return $val->format('Y-m-d'); }

---

* Donc, plus facile à DÉBOGUER.

Merci pour vos conseils.