Error converting data type nvarchar to datetime

Version imprimable

Salut
J'ai un gros soucis avec le type datetime2 de sql server 2012.
J'ai une requête qui marche très bien avec sql server 2008 mais quand je l'exécute en sql server 2012, ça me génère une erreur :

"Msg 8114, Level 16, State 5, Procedure ETAT_RAPPORT_VENTE_PERIODE_EVD, Line 0
Error converting data type nvarchar to datetime."

Voic la requête sql server :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
USE [MSELLER]
GO
/****** Object:  StoredProcedure [dbo].[ETAT_RAPPORT_VENTE_PERIODE_EVD]    Script Date: 19/11/2013 11:28:18 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
 
  ALTER PROCEDURE [dbo].[ETAT_RAPPORT_VENTE_PERIODE_EVD]
  @IdDepot int,
  @dateA datetime,
  @dateB datetime
 
  AS
  SET NOCOUNT ON
 
  SELECT [ID_SIM],[Id_Caissier], sum([MTTREGLEMENT])as Montant,
  --(SELECT [NOM]+ ' ' +[Prenoms] as LBLPERSONNE FROM [dbo].[PERSONNE] WHERE [dbo].[PERSONNE].[ID_AUTO] = [dbo].[REGLEMENT].[ID_SIM]) as Client,
  (SELECT LBLSOCIETET FROM SOCIETE) as lblSociete,
--ECT [NOM]+ ' ' +[Prenoms] as LBLPERSONNE  FROM [dbo].[PEPRESENTANT_ADMIN] WHERE [dbo].[PEPRESENTANT_ADMIN].[ID_AUTO] =  (SELECT Id_Caissier FROM [dbo].[REGLEMENT] WHERE [dbo].[REGLEMENT].[ID_AUTO] = (SELECT ID_AUTO FROM [dbo].[REGLEMENT])))as Caissier,
 (SELECT [NOM]+ ' ' +[Prenoms] as LBLPERSONNE FROM [dbo].[PERSONNE] WHERE [dbo].[PERSONNE].[ID_AUTO] = (SELECT ID_PERSONNE 
   FROM [dbo].[SIM] WHERE [dbo].[SIM].[ID_AUTO] = [dbo].[REGLEMENT].[ID_SIM])) as Client,
  (SELECT @dateA) as DateA,
  (SELECT @dateB) as DateB,
 (SELECT [NOM]+ ' ' +[Prenoms]  FROM [dbo].[PEPRESENTANT_ADMIN]  WHERE [dbo].[PEPRESENTANT_ADMIN].[ID_AUTO] =  @IdDepot) as Caissier
 
  FROM [dbo].[REGLEMENT]
  WHERE [dbo].[REGLEMENT].[DTEOP] BETWEEN @dateA and (@dateB + 1) and [dbo].[REGLEMENT].[Id_Caissier] = @IdDepot GROUP BY [dbo].[REGLEMENT].[ID_SIM],[dbo].[REGLEMENT].[Id_Caissier]

Merci de m'aider SVP

19/11/2013, 12h15
aieeeuuuuu

Bonjour,

Pouvez vous nous donner la commande avec laquelle vous appelez la Procédure.

Pouvez vous également nous indiquer le type de la colonne [dbo].[REGLEMENT].[DTEOP]

Citation:

Envoyé par aieeeuuuuu

Bonjour,

Pouvez vous nous donner la commande avec laquelle vous appelez la Procédure.

Pouvez vous également nous indiquer le type de la colonne [dbo].[REGLEMENT].[DTEOP]

Merci beaucoup

TABLE REGLEMENT
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 ID_AUTO int Unchecked REF_REGLEMENT varchar(15) Unchecked DTEOP datetime2(7) Checked ID_OPENCAISSE int Checked ID_SIM int Unchecked MTTREGLEMENT decimal(18, 3) Checked IS_VALIDER bit Checked IS_COMPTABILISER bit Checked IS_ARCHIVER bit Checked Id_Caissier int Checked is_annuler bit Checked Unchecked
Le type de DTEOP est datetime2(7)

19/11/2013, 15h13
Invité

C'est ça qui ne passe plus : @dateB + 1.
Il faut faire ça plus littéralement genre dateadd(day, + 1, @dateB )

Entre nous, il serait temps que tu apprennes à faire des jointures comme du monde avec INNER JOIN au lieu de pondre ce genre de requête immonde.

Citation:

Envoyé par 7gyY9w1ZY6ySRgPeaefZ

C'est ça qui ne passe plus : @dateB + 1.
Il faut faire ça plus littéralement genre dateadd(day, + 1, @dateB )

Entre nous, il serait temps que tu apprennes à faire des jointures comme du monde avec INNER JOIN au lieu de pondre ce genre de requête immonde.

Merci beaucoup
Mais quand j'enlève le (+1) ça me dire la même chose.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 USE [MSELLER] GO DECLARE @return_value int EXEC @return_value = [dbo].[ETAT_RAPPORT_VENTE_PERIODE_EVD] @IdDepot = 3, @dateA = N'01/10/2010', @dateB = N'31/12/2013' SELECT 'Return Value' = @return_value GO

19/11/2013, 16h10
aieeeuuuuu
Votre format de date doit différer entre les deux serveurs.

Vous pouvez faire un SET DATEFORMAT dmy,

Mais vous devriez plutot utiliser le format iso suivant pour spécifier vos datetime en entrèe de procédure stockée :
Code:

1 2 3 4 5 EXEC @return_value = [dbo].[ETAT_RAPPORT_VENTE_PERIODE_EVD] @IdDepot = 3, @dateA = N'20101001', @dateB = N'20131231'
Ainsi, plus de souci quel que soit le format de date paramétré pour l'instance sur laquelle cette commande s’exécute
19/11/2013, 16h31
eddyphan
Citation:
Envoyé par aieeeuuuuu

Votre format de date doit différer entre les deux serveurs.

Vous pouvez faire un SET DATEFORMAT dmy,

Mais vous devriez plutot utiliser le format iso suivant pour spécifier vos datetime en entrèe de procédure stockée :

Code:

1 2 3 4 5 EXEC @return_value = [dbo].[ETAT_RAPPORT_VENTE_PERIODE_EVD] @IdDepot = 3, @dateA = N'20101001', @dateB = N'20131231'
Merci
ça marche avec ce format sur le serveur. Mais mon inquiétude est comment faire pour mettre les valeurs de mon DatePTimePicker (C#) sous ce format?
Ainsi, plus de souci quel que soit le format de date paramétré pour l'instance sur laquelle cette commande s’exécute
19/11/2013, 17h57
SQLpro

(@dateB + 1)
N'est plus autorisé avec un type DATETIME2.
Ajouter un entier à une date était une facilité imbécile de la version DATETIME !
Vous devez utilisez la fonction DATEADD.

A +

Bonjour,

Pour étoffer ce que disait 7gyY9w1ZY6ySRgPeaefZ, on peut établir un standard de codage et le suivre pour éviter d'écrire des requêtes illisibles, mais probablement aussi plus performantes.

En suivant le standard de codage auquel je suis habitué, j'obtiens :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 ALTER PROCEDURE dbo.ETAT_RAPPORT_VENTE_PERIODE_EVD @IdDepot int , @dateA datetime , @dateB datetime AS BEGIN SET NOCOUNT ON SELECT ID_SIM , Id_Caissier , SUM(R.MTTREGLEMENT) AS Montant , SOC.LBLSOCIETET AS lblSociete , P.NOM + ' ' + P.Prenoms AS Client , @dateA AS DateA , @dateB AS DateB , C.Caissier FROM dbo.REGLEMENT AS R INNER JOIN dbo.SOCIETE AS SOC ON R.??? = SOC.??? INNER JOIN dbo.PERSONNE AS P ON P.??? = R.??? INNER JOIN dbo.SIM AS S ON S.ID_AUTO = R.ID_SIM CROSS JOIN ( SELECT NOM + ' ' + Prenoms FROM dbo.REPRESENTANT_ADMIN WHERE ID_AUTO = @IdDepot ) AS C (Caissier) WHERE R.DTEOP BETWEEN @dateA AND DATEADD(day, 1, @dateB) AND dbo.REGLEMENT.Id_Caissier = @IdDepot GROUP BY R.ID_SIM, R.Id_Caissier END
Je ne dis pas que vous devez adopter ce standard de codage : c'est celui sur lequel les développeurs de mon entreprise et moi nous sommes accordés. Peut-être est-il temps d'en établir un avec votre équipe ;)

Ainsi dans votre requête, on voit tout de suite qu'il y a de nombreux SELECT inutiles, qu'il manque des noms de tables ou leurs alias, et surtout qu'il manque des prédicats de jointure. Souvenez-vous aussi que vous pouvez aliaser le nom des tables, ce qui simplifie l'écriture des requêtes. Vous pouvez établir un trigramme par table, dans le nom de la table, comme SQLPro le fait dans ses exercices.

Enfin les crochets sont totalement inutiles dans votre cas, puisque vous respectez les conventions de nommages de la norme SQL (pas d'espaces, pas de caractères de ponctuation, cabalistiques et autres bizarreries ;))

Vous noterez qu'il est probable que vous puissiez récupérer le nom du caissier séparément de la requête principale, ce qui devrait simplifier votre plan d'exécution : un produit cartésien (CROSS JOIN ou comme vous l'avez exprimé) est généralement peu performant.

@++ ;)

20/11/2013, 15h41
eddyphan

Grand Merci @elsuket pour les réponses et surtout pour les conseils.
Je vais tester et vous fais le point.
21/11/2013, 13h39
eddyphan

Merci pour tout