Problème insertion date

Version imprimable

12/05/2015, 11h33
mbenameur

Problème insertion date

Bonjour.

Je suis entrain de développer une application en java connectée à une bd Oracle8i via ojdbc14.
la date de mon pc est configurer à GMT +1 ainsi que celle du serveur de bd.
le problème que je rencontre est que lors d'une insertion d'une date (ex: sysdate), au niveau de la bd j’obtiens date - 1 heure (GMT).

la question est : est-ce qu'il y a une configuration spécifique pour régler ce problème.

Salutations et merci d'avance.
12/05/2015, 23h17
tchize_

code qui reproduit le problème STP.

Le serveur s'en fou de son GMT local, il va toujours stocker les timestamp dans un format indépendant de la timezone.
En fonction de la timezone de l'utilisateur, les timestamp s'afficheront donc différent pour une même donnée stockée dans la DB
Si tu as un soucis d'heure, c'est vraisemblablement que tu stocker un timestamp là où tu voulais une date => met à jour to schéma de table, stocke une date si c'est une date.

voilà mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 //encodeglb_dtime est une fonction qui encode un timestamp sous le format 'yyyy/mm/dd hh24:mi:ss'
private static final String SQL = "INSERT INTO matable(num, nom, glb_dtime) VALUES (?, ?, encodeglb_dtime( to_date( ?, 'yyyy/mm/dd hh24:mi:ss' ) ) )";
Connection con = null;
PreparedStatement pres = null;
try {
	Class.forName("oracle.jdbc.driver.OracleDriver");
	con = DriverManager.getConnection("jdbc:oracle:thin:@serveurdbip:1521:sid", "username","password");
	pres = con.prepareStatement(SQL);
	pres.setInt( 1, 999 );
	pres.setString( 2, "nom" );
	pres.setString( 3, getTimeStamp() );
	pres.executeUpdate();
} catch (SQLException e) {
	// bla bla
}
static String getTimeStamp(){
        //org.joda.time.DateTime
    	DateTime now = new DateTime();
    	Timestamp time = new Timestamp( now.getMillis() );
        DateFormat glbdtimeFormat = new SimpleDateFormat( "yyyy/MM/dd HH:mm:ss" );
        return glbdtimeFormat.format( time );
}

le problème est que quand j'affiche le timestamp j'obtiens la même heure que celle du système (client ou serveur puisqu'il sont synchronisés à GMT +1)

Code:

System.out.println( getTimeStamp() );

mais quand je fais appel à l'info insérée en PL/SQL j'obtiens le glb_dtime à h-1 (GMT)

Code:

1
2
3
 
-- décodeglb_dtime(glb_dtime) est la fonction inverse de encodeglb_dtime()
SELECT num, nom, décodeglb_dtime(glb_dtime) FROM matable WHERE num = 999

Pourquoi ?

POURQUOI?

Pourquoi récupérer un timestamp (par Joda en plus...), le convertir en String et le passer dans ce format à la base qui va utiliser une fonction PL pour... pour quoi d'ailleurs, en refaire une date?

Soit tu passes un objet Date (vu que tu veux juste la date du jour, et que au final tu veux stocker un objet date), soit dans ta requête tu mets un sysdate !
Et quand tu récupères tes infos, tu auras tout et sans problème vu que tu ne convertira pas trois fois une donnée pour revenir à son type d'origine !

Donc soit tu fais ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private static final String SQL = "INSERT INTO matable(num, nom, glb_dtime) VALUES (?, ?, sysdate ) )"; try (Connection con = DriverManager.getConnection("jdbc:oracle:thin:@serveurdbip:1521:sid", "username","password"); PreparedStatement pres = con.prepareStatement(SQL);){ pres = con.prepareStatement(SQL); pres.setInt( 1, 999 ); pres.setString( 2, "nom" ); pres.executeUpdate(); } catch (SQLException e) { // bla bla }
Soit tu fais ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private static final String SQL = "INSERT INTO matable(num, nom, glb_dtime) VALUES (?, ?, ?) )"; try (Connection con = DriverManager.getConnection("jdbc:oracle:thin:@serveurdbip:1521:sid", "username","password"); PreparedStatement pres = con.prepareStatement(SQL);){ pres.setInt( 1, 999 ); pres.setString( 2, "nom" ); pres.setDate( 3, new Date() ); pres.executeUpdate(); } catch (SQLException e) { // bla bla }
Accessoirement, si tu es en Java7, tu n'as plus besoin de faire un Class.forName pour charger le driver (le DriverManager sait se démerder tout seul maintenant), et il faut FERMER ses ressources
(je change le code dans ce sens)

(excusez mon ton énervé, mais c'est pas la première fois que je passe derrière du code ou quelqu'un a cru qu'il serait une bonne idée de convertir 12 fois des objets contenant des dates entre un format Date et un format String tout ça pour enregistrer une date au final... Code pour rien, générateur de bugs, compliqué de voir si ça a un sens, ...)

Citation:
Envoyé par eulbobo
Pourquoi ?

POURQUOI?

Pourquoi récupérer un timestamp (par Joda en plus...), le convertir en String et le passer dans ce format à la base qui va utiliser une fonction PL pour... pour quoi d'ailleurs, en refaire une date?

Soit tu passes un objet Date (vu que tu veux juste la date du jour, et que au final tu veux stocker un objet date), soit dans ta requête tu mets un sysdate !
Et quand tu récupères tes infos, tu auras tout et sans problème vu que tu ne convertira pas trois fois une donnée pour revenir à son type d'origine !

Donc soit tu fais ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private static final String SQL = "INSERT INTO matable(num, nom, glb_dtime) VALUES (?, ?, sysdate ) )"; try (Connection con = DriverManager.getConnection("jdbc:oracle:thin:@serveurdbip:1521:sid", "username","password"); PreparedStatement pres = con.prepareStatement(SQL);){ pres = con.prepareStatement(SQL); pres.setInt( 1, 999 ); pres.setString( 2, "nom" ); pres.executeUpdate(); } catch (SQLException e) { // bla bla }
Soit tu fais ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private static final String SQL = "INSERT INTO matable(num, nom, glb_dtime) VALUES (?, ?, ?) )"; try (Connection con = DriverManager.getConnection("jdbc:oracle:thin:@serveurdbip:1521:sid", "username","password"); PreparedStatement pres = con.prepareStatement(SQL);){ pres.setInt( 1, 999 ); pres.setString( 2, "nom" ); pres.setDate( 3, new Date() ); pres.executeUpdate(); } catch (SQLException e) { // bla bla }
Accessoirement, si tu es en Java7, tu n'as plus besoin de faire un Class.forName pour charger le driver (le DriverManager sait se démerder tout seul maintenant), et il faut FERMER ses ressources
(je change le code dans ce sens)

(excusez mon ton énervé, mais c'est pas la première fois que je passe derrière du code ou quelqu'un a cru qu'il serait une bonne idée de convertir 12 fois des objets contenant des dates entre un format Date et un format String tout ça pour enregistrer une date au final... Code pour rien, générateur de bugs, compliqué de voir si ça a un sens, ...)
Bonjour.
Votre excuse est accepté.
premièrement je ne suis pas un professionnel en java.
je travail avec le jodatime pour profiter de ses point fort (comparaison de date, ajout des jour, ...), ce que j'ai poster n'est qu'un extrait de mon code.
j'ai déjà essayé avec le sysdate mais dans la bd j'obtiens toujours heure -1, par contre si j’exécute cette requête directement sur pl/sql ça s’insère correctement (heure juste). c'est pour ça que j'ai employé jodatime, j'ajoute une heure de plus puis l’exécution de la requête me retourne heure juste :
Code:

1 2 3 4 5 6 7 // j'ajoute une heure a l'instant actuel. TimeStamp ts = new Timestamp( new DateTime().plusHours(1).getMillis() ); // ts reçoit alors un timestamp sous la forme : yyyy-mm-dd hh:mm:ss.fffffffff (heure actuelle +1 ) // la fonction encodeglb(Date arg) que j'ai sous oracle accepte comme argument une date sous format : yyyy/MM/dd HH:mm:ss DateFormat glbdtimeFormat = new SimpleDateFormat( "yyyy/MM/dd HH:mm:ss" ); String st = glbdtimeFormat.format( ts );
pour votre 2ème proposition

Code:

pres.setDate( 3, new Date() );

ça m'a déclenché l'erreur : java.sql.SQLException: ORA-01008: Toutes les variables ne sont pas liées

merci d'avance.

13/05/2015, 15h09
eulbobo

On peut enregistrer des champs TimeStamp ou Date directement en base avec JDBC, il ne FAUT PAS les convertir en String avant. Que ce soit en lecture ou en écriture.

Ensuite, le champ TimeStamp n'a d'intérêt que si tu veux conserver ou traiter avec les nanosecondes, d'après la conversion que tu fais, ce n'est pas le cas ! Utilise un type Date de bout en bout ! (en utilisant Joda pour la manipulation, mais Calendar est aussi efficace si tu veux juste ajouter/supprimer une heure. Joda permet d'avoir des fonctionnalités en plus si tu travailles sur les intervalles et les périodes, mais c'est intégré à Java8 avec la nouvelle API Date)

Dans les deux exemples, la requête a une erreur : il y a une parenthèse fermante en trop à la fin (erreur dans le copier/coller).

Essayes de manipuler des objets Date de bout en bout pour voir si tu as toujours le problème (et arrête le timestamp)
13/05/2015, 19h31
tchize_

Déjà dit, mais j'enfonce le clou. Avec des dates, tu oublie TO_DATE et TO_CHAR, ainsi que toute forme d'interaction avec le DB en utilisant des String. Utilise des preparedStatement avec setDate / setTimestamp /getDate / getTimestamp.

La DB elle fait sont travail proprement. Avec une date, elle stocke ce moment absolu dans sa base. Soit ce moment est un jour, soit c'est un timestamp, suivant la définition utilisée pour créer la table.

Que ce passe-t-il quand tu utilise toDate avec un String? Tu dis à la DB "Tiens, voilà une information partielle, débrouille toi pour deviner le reste".... La DB du coup elle fait avec les metainfo qu'elle a, comme les NLS_ qui sont définis dans la connexion. Dedans se trouve la timezone à utiliser pour lire la String. Timezone qui n'est pas nécessairement la même que celle que ton programme java a utilisé pour générer la String. Dis toi que deux string identiques ne représentent pas nécessairement le même timestamp. Deux Strings différentes peuvent représenter le même timestamp. Du coup tu comprendre que les Strings, c'est pas cool....

De la même manière, si tu lance sqlDevelopper et que tu fais un select * from taTable, dis toi que la string que tu aura a l'écran n'est qu'une conversion de la date dans la locale définie par le programme sqlDevelopper...

Ce qui est bien avec setDate / setTimestamp, c'est que le driver il cause avec la DB pour que toi et lui soyez dans le même référenciel, pour au final envoyer vers la DB un long dans un standard défini par a DB.

Bien entendu, tu sera tenté, une fois la date récupérée, d'utiliser SimpleDateFormat pour afficher dd/MM/yyyy HH:mm:ss à l'utilisateur. Encore une fois attention. Si t'es sûr maintenant de récupérer exactement la même date que ce que tu as envoyé, ça ne te garanti pas que l'utilisateur A aura le même affichage que l'utilisateur B. En effet, si A est en Angleterre et B à Moscou, il y aura quelques heures de décalage. Mais ce n'est pas un problème. En effet, si A appelle B au moment indiqué sur son écran, B décrochera son téléphone au moment indiqué sur son écran à lui et ils pourront parler.

Pour ça:

Code:

java.sql.SQLException: ORA-01008: Toutes les variables ne sont pas liées

tu dois avoir un ? de trop ou trop peu dans ta requete :)