Error converting data type nvarchar to datetime

**eddyphan** · 19/11/2013, 11h50

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 :

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
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

**aieeeuuuuu** · 19/11/2013, 12h15

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]

**eddyphan** · 19/11/2013, 15h04

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 :

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

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)

Invité · 19/11/2013, 15h13

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.

**eddyphan** · 19/11/2013, 15h51

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 :

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

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

**aieeeuuuuu** · 19/11/2013, 16h10

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 :

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

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

**eddyphan** · 19/11/2013, 16h31

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 :

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

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

**SQLpro** · 19/11/2013, 17h57

(@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 +

**elsuket** · 19/11/2013, 22h56

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 :

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
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.

@++

**eddyphan** · 20/11/2013, 15h41

Grand Merci @elsuket pour les réponses et surtout pour les conseils.
Je vais tester et vous fais le point.

**eddyphan** · 21/11/2013, 13h39

Merci pour tout

Error converting data type nvarchar to datetime [2012]

MS SQL Server

Discussions similaires

Partager

Partager