Conversion de date

**moimp** · 27/11/2017, 12h37

Bonjour à tous,
Je crée deux fonctions de conversion de dates. La première fonctionne mais pas la seconde qui provoque une erreur à la ligne 7:

Parse error: syntax error, unexpected 'createFromFormat' (T_STRING), expecting variable (T_VARIABLE) or '$' in ...

Comment faire?

Code php :

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

1
2
3
4
5
6
7
8
9
function convertDateFromSQLToEur($sql) {
	$date = new DateTime($sql);
	return $date->format('d/m/Y');
}
convertDateFromEurToSQL('27/11/2017');
function convertDateFromEurToSQL($eur) {
	$date = new DateTime::createFromFormat('d/m/Y');
	var_dump($date->format('Y-m-d'));
}

**Celira** · 27/11/2017, 12h46

Si tu utilises la fonction createFromFormat, tu n'utilises pas le constructeur, donc pas de "new", Et si tu regardes bien, tu as oublié d'utiliser la variable $eur dans ton appel

**moimp** · 27/11/2017, 15h43

Merci pour le new. Pour le $eur, c'est normal, je m'étais arrêté sur l'erreur. Tout est maintenant OK.

**moimp** · 11/12/2017, 11h03

Bonjour,

J'avais cru mon problème résolu mais je m'aperçois que ma fonction convertDateFromEurToSQL() n'accepte pas toutes les formes courtes ou longues sur 1, 2 ou 4 chiffres.

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function convertDateFromSQLToEur($sql, $emptyValue) {
	if (empty($sql)) return $emptyValue;
	if ($sql === '0000-00-00') return $emptyValue;
	$date = new DateTime($sql);
	return $date->format('d/m/Y');
}
 
var_dump(convertDateFromEurToSQL('1/1/1991', ''));
function convertDateFromEurToSQL($eur, $emptyValue, $zone='Europe/Paris') {
	if (empty($eur)) return $emptyValue;
	$eur = str_replace(['.', '-', ' '], '/', $eur);
	if (! date_create($eur)) return false; // Si la date n'existe pas
	//$date = DateTime::createFromFormat('d/m/Y', $eur, new DateTimeZone($zone));
	$date = DateTime::createFromFormat('j/n/y', $eur, new DateTimeZone($zone));
	if ($date === false) return false;
	return $date->format('Y-m-d');
}

**Celira** · 11/12/2017, 11h54

Si tu utilises createFromFormat, il faut que tu précises le format. Donc si il y a plusieurs formats possibles, ça risque de poser un problème.
Tu as deux options : soit lister les formats possibles et les tester les uns après les autres, soit forcer le format au niveau de la récupération des données. D'où vient ta donnée date ? d'un formulaire ?

**moimp** · 11/12/2017, 14h06

S'agissant d'une fonction, les données peuvent provenir de n'importe où - généralement effectivement d'un formulaire.
J'avais envisagé de tester tous les cas de figures mais c'est lourd et je ne serai jamais sûr de ne pas en oublier.
J'ai pensé utiliser un preg_replace() pour forcer le format à JJ/MM/AAAA mais je n'arrive pas à créer le masque correctement.

**Celira** · 11/12/2017, 15h14

Tout dépend de ce que tu dois gérer. Si les formats sont proches, par exemple 01/01/2017 et 12/3/17, ça reste simple à gérer. Par contre, si tu dois gérer des différences comme 5/8/17 et 2017-12-23, là ça va devenir coton...

**moimp** · 11/12/2017, 18h33

J'avance. Je gère les dates sous la forme J/M/AA où J et M peuvent être à 1 ou 2 chiffres et AA à 2 ou 4 chiffres. Voici mon code. Il semble fonctionner avec des tests sur les 28 et 30 février. Je le trouve très lourd comparativement à la fonction réciproque en #1.

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
var_dump(convertDateFromEurToSQL('30/02/71', ''));
function convertDateFromEurToSQL($eur, $emptyValue, $zone='Europe/Paris') {
	if (empty($eur)) return $emptyValue;
	$eur = str_replace(['.', '-', ' '], '/', $eur);
	$segments = explode('/', $eur);
	if (strlen($segments[2]) <= 2) {
		$siecle = (40 < (int)$segments[2] and (int)$segments[2] <= 99) ? '19': '20';
		$annee = $siecle.sprintf("%'.02d", $segments[2]);
	}
	elseif (strlen($segments[2]) === 4) {
		$annee = $segments[2];
	}
	else {
		return false;
	}
	$eur = sprintf("%'.02d/%'.02d/", $segments[0], $segments[1]) . $annee;
	if (!checkdate((int)$segments[1], (int)$segments[0], (int)$segments[2])) return false; // retourne false pour 29/02/1971
	$date = DateTime::createFromFormat('d/m/Y', $eur, new DateTimeZone($zone)); // retourne 01/03/1971 pour 29/02/1971
	if ($date === false) return false;
	return $date->format('Y-m-d');
}

**Celira** · 11/12/2017, 18h57

Hum... On peut essayer de deviner le format à partir de la chaine et l'appliquer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
function convertDateFromEurToSQL($eur, $emptyValue, $zone='Europe/Paris') {
	if (empty($eur)) return $emptyValue;
	$eur = str_replace(['.', '-', ' '], '/', $eur);
	$segments = explode('/', $eur);
    if (count($segments) != 3) return false;
    // day
    $formats['day'] = (strlen($segments[0]) == 1)?'j':'d';
    // month
    $formats['month'] = (strlen($segments[1]) == 1)?'n':'m';
    // year : 
    $formats['month'] = (strlen($segments[2] == 2)?'y':'Y';
	$format = implode('/', $formats);
	$date = DateTime::createFromFormat($format, $eur, new DateTimeZone($zone));
	if ($date === false) return false;
	return $date->format('Y-m-d');
}

**rawsrc** · 11/12/2017, 19h43

Salut,

ou beaucoup plus simplement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function convertDateFromEurToSQL($p)
{
    [$d, $m, $y] = explode('/', $p);
    return (checkdate($m, $d, $y))
               ? (new DateTime("{$y}-{$m}-{$d}"))->format('Y-m-d')
               : '0000-00-00';
}
 
// petit code de test
echo convertDateFromEurToSQL('1/1/76'), '<br>';
echo convertDateFromEurToSQL('1/1/1976'), '<br>';
echo convertDateFromEurToSQL('28/2/01'), '<br>';
echo convertDateFromEurToSQL('28/2'), '<br>';
echo convertDateFromEurToSQL('29/2/01'), '<br>';

EDIT :
Et si tu veux avoir la possibilité d'utiliser n'importe quel séparateur entre les jours/mois/année ; il te faut ruser comme ça :

Code :

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

1
2
3
4
5
6
7
8
[$d, $m, $y] = preg_split('/[^\d]/u', $p);
 
// petit code de test
echo convertDateFromEurToSQL('1/1/76'), '<br>';
echo convertDateFromEurToSQL('1-1-1976'), '<br>';
echo convertDateFromEurToSQL('28-2/01'), '<br>';
echo convertDateFromEurToSQL('28/2'), '<br>';
echo convertDateFromEurToSQL('29/2/01'), '<br>';

**moimp** · 12/12/2017, 15h38

rawsrc, ta solution est séduisante par sa simplicité, mais elle pose quelques problèmes:

Le test '28/2' ne crée que 2 dimensions, il faut donc gérer l'erreur, d'autant que cette écriture n'a pas de sens (de quelle année parle t-on?)
L'écriture [$d, $m, $y] = explode('/', $p) n'est pas reconnue par ma version de php (7.0) et doit être remplacée par list($d, $m, $y) = explode('/', $p)
Ta fonction ne gère pas le cas des chaînes vides mais ça, j'en fait mon affaire.
Dans le cas présent, je ne vois pas à quoi servent les accolades.

Merci pour le petit plus du EDIT (utile par exemple pour les allemands)
Voici la fonction finale à laquelle je suis arrivé:

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
/*
// petit code de test pour la fonction suivante
var_dump(convertDateFromEurToSQL('1/1/76', NULL));
var_dump(convertDateFromEurToSQL('1/1/1976', NULL));
var_dump(convertDateFromEurToSQL('28/2/01', NULL));
var_dump(convertDateFromEurToSQL('28/2', NULL));
var_dump(convertDateFromEurToSQL('29/2/01', NULL));
var_dump(convertDateFromEurToSQL('', NULL));
*/
function convertDateFromEurToSQL($eur, $emptyValue) {
	if (empty($eur)) return $emptyValue;						// Si la chaîne est vide, je retourne la chaîne par défaut
	$pattern = '#[^\d]#u';
	if (count(preg_split($pattern, $eur)) !== 3) return false;	// Si la chaîne est incomplète, je signale une erreur
	list($d, $m, $y) = preg_split($pattern, $eur);
	if (!isset($d, $m, $y)) return false;
	return (checkdate($m, $d, $y)) ?
		(new DateTime("{$y}-{$m}-{$d}"))->format('Y-m-d'):
		false;													// Si la date n'existe pas, je signale une erreur
}

Invité · 12/12/2017, 17h51

Bonjour,

au lieu de vouloir gérer tout et n'importe quoi, il faut IMPOSER un format de date au moment de la saisie !

Pour ça, on dispose de plusieurs techniques :

l'indiquer textuellement (!) : "date de la forme jj/mm/yyyy" (ou yyyy-mm-dd)
l'attribut pattern de <input>
datePicker (jQuery UI ou autre)
faire une pré-vérification du format en JavaScript (regex,...)
faire une vérification du format en PHP (regex,...)
...

Pourquoi pas le "dix neuf thermidor de l'an de grâce 1789", pendant qu'on y est !

**rawsrc** · 12/12/2017, 22h10

Salut,

c'est pas mal ton code mais il y a moyen de l'améliorer :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
function convertDateFromEurToSQL($eur, $emptyValue)
{
    if (empty($eur))
        return $emptyValue;                        // Si la chaîne est vide, je retourne la chaîne par défaut
 
    $parts = preg_split('#[^\d]#u', $eur);
    if (count($parts) !== 3)
        return false;
 
    list($d, $m, $y) = $parts;
 
    return (checkdate($m, $d, $y)) ?
        (new DateTime("{$y}-{$m}-{$d}"))->format('Y-m-d'):
        false;                                                    // Si la date n'existe pas, je signale une erreur
}

**moimp** · 13/12/2017, 09h49

Jreaux62, merci pour ce rappel. J'ignorais complètement l'existence de l'attribut pattern et n'avais même pas l'idée que ça puisse exister. Je ne l'ai pas encore tester mais ça me paraît très intéressant.
Merci rawsrc, sauf que ce n'est pas 'mon' code mais le résultat de tous vos conseils.
Je marque le sujet comme résolu.

**Celira** · 13/12/2017, 12h24

Je ne pense jamais à l'attribut pattern non plus. HTML5 n'est pas encore un réflexe, bien qu'il soit supporté par la grande majorité des navigateurs maintenant. A utiliser en combinaison avec placeholder pour indiquer à l’utilisateur ce qu'il doit écrire.
Pour vérification du support par le navigateur : https://html5test.com/compare/featur...aceholder.html (en gros, ça va marcher sauf si tu dois supporter IE 9-)

**moimp** · 13/12/2017, 12h34

Envoyé par jreaux62

on dispose de plusieurs techniques :

l'indiquer textuellement (!) : "date de la forme jj/mm/yyyy" (ou yyyy-mm-dd)
l'attribut pattern de <input>
datePicker (jQuery UI ou autre)
faire une pré-vérification du format en JavaScript (regex,...)
faire une vérification du format en PHP (regex,...)
...

Point 1: C'était déjà fait.
Point 2: Ceci me parait une excellente solution mais tuto à l'appui, je n'arrive pas à l'utiliser et j'ai posté une question ici car elle me paraît être un sujet différent.
Point 3: J'utilise déjà.
Point 4: Ceci me parait faire double emploi avec l'attribut pattern
Point 5: C'est l'un des rôle de la fonction objet de cette discussion

Invité · 13/12/2017, 12h43

Point 5: C'est l'un des rôle de la fonction objet de cette discussion

Non. Tu fais une fonction de conversion.

Moi, je parle de "Gestion d'erreur" (avant tout traitement : enregistrement, conversion,...)

**Celira** · 13/12/2017, 12h43

Tu n'as pas obligé d'appliquer toutes les options de la liste de jreaux

La base c'est :
- imposer le format en entrée : pattern est une solution parmi d'autres. Si tu utilises un DatePicker, tu peux verrouiller le champ associé pour forcer l'utilisateur à utiliser le DatePicker (qui du coup écrit la date au bon format automatiquement)
- vérifier le format côté serveur et jeter une erreur si le format est incorrect.

Pourquoi pas le "dix neuf thermidor de l'an de grâce 1789"

En fait, le calendrier républicain modifie aussi les années. Sauf que comme il commence en 1792, ce serait le 19 thermidor de l'an -3

Et nous sommes aujourd'hui le 23 frimaire de l'An 226

**moimp** · 16/12/2017, 12h08

Envoyé par jreaux62

Non. Tu fais une fonction de conversion.

Moi, je parle de "Gestion d'erreur" (avant tout traitement : enregistrement, conversion,...)

Je suis bien d'accord mais je veux vérifier que la date est correcte et existe avant de la convertir, même si ce n'est pas une bonne pratique.

**moimp** · 16/12/2017, 12h14

Envoyé par Celira

Tu n'as pas obligé d'appliquer toutes les options de la liste de jreaux

La base c'est :
- imposer le format en entrée : pattern est une solution parmi d'autres. Si tu utilises un DatePicker, tu peux verrouiller le champ associé pour forcer l'utilisateur à utiliser le DatePicker (qui du coup écrit la date au bon format automatiquement)
- vérifier le format côté serveur et jeter une erreur si le format est incorrect.

Oui, j'avais bien compris.

Envoyé par Celira

En fait, le calendrier républicain modifie aussi les années. Sauf que comme il commence en 1792, ce serait le 19 thermidor de l'an -3

Et nous sommes aujourd'hui le 23 frimaire de l'An 226

Merci pour ton humour. Il y a déjà quelques années dans une application de généalogie, j'avais fait des fonctions de conversion de et vers le calendrier républicain mais ce n'était pas parfait et posait des problèmes avec les années bissextiles.

Conversion de date

Langage PHP

Discussions similaires

Partager

Partager