Discussion :

[laurentSc]

Bonjour,

je récupère des dates dans ce format : MMM-DD, YYYY (MMM in letters). Je souhaite les enregistrer dans une BDD MySQL, donc d'abord, il faut les mettre au format YYYY-MM-DD HH:MM:SS. Je fais donc une fonction PHP pour les convertir, mais erreur dont je ne trouve pas la cause :

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
22
23
 function convert_date($olddate) {
    //to convert month from letters to figures
        function convert_month_letters_to_figures($Month_Letter) {
            switch($Month_Letter) {
                case 'Jan' :$month=1; break;
                case 'Feb' :$month=2; break;
                case 'Mar' :$month=3; break;
                case 'Apr' :$month=4; break;
                case 'May' :$month=5; break;
                case 'Jun' :$month=6; break;
                case 'Jul' :$month=7; break;
                case 'Aug' :$month=8; break;
                case 'Sep' :$month=9; break;
                case 'Oct' :$month=10; break;
                case 'Nov' :$month=11; break;
                case 'Dec' :$month=12; break;
            }
            return($month);
        }
 
        list($mmdd,$year)=explode(',',$olddate);
...
}

Notice: Undefined offset: 1 in C:\wamp64\www\ticket\classes\Ticket.php on line 21

Merci de m'aider

[ma5t3r]

Salut,

En considérant que la chaîne passée en paramètre est au bon format.
A toi de traiter les éventuelles erreurs possibles.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
    function get_date_sql($date = 'Jan-11, 2019') {
        $months_text = array ('', 'Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec');
        $first = explode(',', $date);
        $mj = explode('-', $first[0]);
 
        $my_date = new DateTime();
        $month = array_search($mj[0], $months_text);
        $my_date->setDate($first[1], $month, $mj[1]);
 
        return $my_date->format('Y-m-d H:i:s');
    }

[laurentSc]

Merci pour ta réponse. Ton code est plus concis que le mien et semble répondre à mon besoin. Mais en voulant l'utiliser, j'ai ce message d'erreur :

Fatal error: Uncaught Error: Class 'classes\Ticket\DateTime' not found in C:\wamp64\www\ticket\classes\Ticket.php on line 168

Y a quelque chose à faire pour utiliser la classe DateTime ?

[rawsrc]

salut

soit tu remets new DateTime() sur l'espace de nom global : new \DateTime(),
soit tu déclares ta dépendance dans l'en-tête avec use DateTime; et comme ça tu pourras faire directement new DateTime()

[rawsrc]

sincèrement, depuis le temps laurentSc, tu pourrais quand même faire un effort mon vieux (surtout pour ce qui est des dates, PHP fournit tout ce qu'il faut nativement) :

Code :

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

1
2
$date = DateTime::createFromFormat('M-d, Y', 'Jan-11, 2019');
echo $date->format('Y-m-d'); // 2019-01-11

Faut juste aller potasser la doc

[laurentSc]

Ca paraît simple et pourtant

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
function get_date_sql($date = 'Jan-11, 2019') {
        $months_text = array ('', 'Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec');
        $first = explode(',', $date);
        $mj = explode('-', $first[0]);
 
        //$my_date = new DateTime();
        $my_date = DateTime::createFromFormat('M-d, Y', $date);
//echo $my_date->format('Y-m-d'); // 2019-01-11
        $month = array_search($mj[0], $months_text);
        $my_date->setDate($first[1], $month, $mj[1]);
 
        return $my_date->format('Y-m-d H:i:s');
    }

donne

Fatal error: Uncaught Error: Call to a member function setDate() on bool in C:\wamp64\www\ticket\classes\Ticket.php on line 10

[rawsrc]

tu as déclaré ta dépendance ?
sinon comme ça :

Code :

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

$my_date = \DateTime::createFromFormat('M-d, Y', $date);

et vire moi tout le code inutile, cela t'évitera l'erreur avec setDate()

[ma5t3r]

OOPS !
Pourquoi faire simple quand on peut faire compliqué ?

C'est le format MMM-DD, YYYY du départ qui m'a induit en erreur.
Pourtant, je me suis posé la question

[laurentSc]

Si virer le code inutile, c'est juste enlever les lignes commentées, donc comme ça :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
function get_date_sql($date = 'Jan-11, 2019') {
        $months_text = array ('', 'Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec');
        $first = explode(',', $date);
        $mj = explode('-', $first[0]);
 
        $my_date = DateTime::createFromFormat('M-d, Y', $date);
        $month = array_search($mj[0], $months_text);
        $my_date->setDate($first[1], $month, $mj[1]);
 
        return $my_date->format('Y-m-d H:i:s');
    }

ça n'enlève pas le message d'erreur.

Et au fait, j'ai bien au début de la page, la dépendance : use DateTime;

[rawsrc]

vire ta fonction entièrement et essaie juste avec ça :

Code :

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

1
2
3
$d = 'Jan-01, 2019';
$date = \DateTime::createFromFormat('M-d, Y', $d);
echo $date->format('Y-m-d');

pour cette transformation tu n'as pas besoin à vrai dire d'une fonction, la classe DateTime suffit amplement
Fonction nettoyée :

Code :

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

1
2
3
4
function get_date_sql($date = 'Jan-11, 2019') {
    $my_date = \DateTime::createFromFormat('M-d, Y', $date); 
    return $my_date->format('Y-m-d H:i:s');
}

[laurentSc]

J'ai quand même gardé la fonction car elle est appelée plus loin dans le code, donc ça m'a paru plus simple.

La voici :

Code php :

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

1
2
3
4
5
function get_date_sql($date = 'Jan-11, 2019') {
        $date = \DateTime::createFromFormat('M-d, Y', $date);
        var_dump($date);
        return $date->format('Y-m-d');
    }

et le var_dump donne

C:\wamp64\www\ticket\classes\Ticket.php:164:boolean false

Pourquoi c'est un booléen ?

[rawsrc]

parce que DateTime renvoie false s'il échoue. Cela veut dire qu'il n'est pas arrivé à parser la chaîne.
En tout cas ton bout de code fonctionne impec sur mon serveur de dev.

Code :

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

1
2
3
4
5
6
function get_date_sql($date = 'Jan-11, 2019') {
    $date = \DateTime::createFromFormat('M-d, Y', $date);
    var_dump($date);
    return $date->format('Y-m-d');
}
echo get_date_sql();

me renvoie :

object(DateTime)[1]
  public 'date' => string '2019-01-11 20:53:45.000000' (length=26)
  public 'timezone_type' => int 3
  public 'timezone' => string 'UTC' (length=3)

2019-01-11

[Celira]

sincèrement, depuis le temps laurentSc, tu pourrais quand même faire un effort mon vieux (surtout pour ce qui est des dates, PHP fournit tout ce qu'il faut nativement) :

Code :

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

1
2
$date = DateTime::createFromFormat('M-d, Y', 'Jan-11, 2019');
echo $date->format('Y-m-d'); // 2019-01-11

Faut juste aller potasser la doc

Attention, il me semble que ça ne fonctionne qu'avec les mois en anglais. Si il veut les abbréviations en français, il va falloir le faire à la main.

[laurentSc]

Tout compte fait, c'est pas la fonction qui est en cause : le format entrant des dates n'était pas celui attendu ! C'est bizarre car dans le fichier csv d'où il vient, c'est bien ce format-là (M d, Y)(y a pas de tiret finalement) mais à l'entrée de la fonction, il est devenu M d Y. Je vois pas pourquoi mais avec

Code php :

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

1
2
3
4
5
6
 function get_date_sql($date = 'Jan-11, 2019') {
        $date1 = \DateTime::createFromFormat('M d, Y', $date);
        if ($date1==false) {$date1=\DateTime::createFromFormat('M d Y', $date);}
         if ($date1!=false) return( $date1->format('Y-m-d'));
         else return(false);
    }

plus de souci.

[ma5t3r]

Tu nous a bien cassé les pieds avec ton format d'entrée MMM-DD, YYYY pour t'apercevoir que tu induisais tout le monde en erreur.

Tout ça pour un simple formatage de date. Je suis plié (sans ironie)

[rawsrc]

ça se voit que tu ne connais pas encore laurentSc...


@laurentSc
sois gentil et vire moi la date par défaut de ta fonction

[laurentSc]

Faut pas le prendre mal, car même si je me suis trompé sur le format d'entrée de la fonction, la discussion m'a permis d'avoir un code qui répond à mon besoin, et bien plus concis que si je l'avais fait sans votre aide.

PS : j'ai retiré la valeur par défaut de la fonction.