STORE PROCEDURE

lunedì 18 luglio 2011 - 14.38

Elenco Threads

Stanze Forum

Aggiungi ai Preferiti

Cerca nel forum

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 14:38

Ciao a tutti, sto facendo i store procedure, ma mi sono fermato con una piccola difficoltà che ho provato molte volte a capire di dove ho sbagliato:

ho creato add_column nel store procedure, però anche insert_log che serve per vedere i comandi eseguiti con la versione dell'applicazione e anche gli esiti.

il file INSERT_LOG

SET ANSI_NULLS ON

GO

SET QUOTED_IDENTIFIER ON

GO

-- =============================================

-- Author: Gabriele S.

-- Create date: 11/07/2011

-- Description:

-- =============================================

CREATE PROCEDURE DOCSADM.insert_log

-- Add the parameters for the stored procedure here

@nome_utente Nvarchar(200),

@comando_eseguito Nvarchar (200),

@versione_CD Nvarchar(200),

@esito Nvarchar(200)

AS

BEGIN

DECLARE @istruzione Nvarchar(2000)

DECLARE @data_eseguito Nvarchar (200)

set @data_eseguito = (select convert (varchar, getdate(), 113))

SET @istruzione = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL VALUES ('',

'+@data_eseguito+',

'+@comando_eseguito+',

'+@versione_CD+',

'+@esito+')'

execute sp_executesql @istruzione

END

GO

L'ERRORE RISPONDE CHE QUANDO FACCIO EXECUTE, RISULTA CHE NELLA DATE NON VA BENE.. DOVREI METTERE CONVERT 131? MA NON VOGLIO CHE METTA DD MMM AAAA HH:MM:SS:MM, MA VORREI DD/MM/AAAA HH:MM:SS H24

il file di ADD_COLUMN

SET ANSI_NULLS ON

GO

SET QUOTED_IDENTIFIER ON

GO

-- =============================================

-- Author: Gabriele S.

-- Create date: 11/07/2011

-- Description:

-- =============================================

ALTER PROCEDURE DOCSADM.add_column

-- Add the parameters for the stored procedure here

@versioneCD Nvarchar(200),

@nome_utente Nvarchar(200),

@nome_tabella Nvarchar(200),

@nome_colonna Nvarchar(200),

@tipo_dato Nvarchar(200),

@val_default Nvarchar(200),

@condizione_modifica_pregresso Nvarchar(200),

@condizione_check Nvarchar(200),

@RFU Nvarchar(200)

AS

BEGIN

if not exists(select * from syscolumns where name=@nome_colonna and id in

(select id from sysobjects where name=@nome_tabella and xtype='U'))

BEGIN

DECLARE @istruzione Nvarchar(2000)

--DECLARE @insert_log Nvarchar(2000)

--SET @insert_log = docasdm.insert_log

SET @istruzione = N'alter table [' + @nome_utente + '].

[' + @nome_tabella + '] ADD ' + @nome_colonna +'

'+ @tipo_dato +' '

execute sp_executesql @istruzione

execute sp_executesql insert_log @nome_utente,

NULL,

'Added column on' +@nome_tabella+,

@versione_CD,

'esito positivo'

end

SET NOCOUNT ON;

l'errore è Incorrect syntax near '@nome_utente'. è quello dove ce insert_log

-- Insert statements for procedure here

SELECT @versioneCD,

@nome_utente,

@nome_tabella,

@nome_colonna,

@tipo_dato,

@val_default,

@condizione_modifica_pregresso,

@condizione_check,

@RFU

END

GO

GRAZIE MILLE (UNO DEI MILLE GARIBALDINI) :D
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 14:45

>L'ERRORE RISPONDE CHE QUANDO FACCIO EXECUTE, RISULTA CHE NELLA
>DATE NON VA BENE.. DOVREI METTERE CONVERT 131? MA NON VOGLIO
>CHE METTA DD MMM AAAA HH:MM:SS:MM, MA VORREI DD/MM/AAAA HH:MM:SS
>H24

Ciao,

Le date sono definite come una coppia di numeri interi (uno per la parte data e uno per quella oraria).
I numeri come tali non hanno formato ma sarà opera delle applicazioni client applicare una formattazione in base alle regional options impostate su ogni client.

Par maggiori informazioni su questo data type leggi con molta attenzione questo articolo di Tibor:
http://www.karaszi.com/SQLServer/info_datetime.asp

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 15:02

ti ringrazio del link e ho già trovato.

Ma solo vorrei capire dove ho sbagliato, ad esempio nel insert_log, quando metto insert, mi risulta:

Unclosed quotation mark after the character string ',
20110718,
Add Column,
3.14.3,
NEGATIVO)'.

era quello nel @istruzione

invece add_column quando dovrei fare execute insert_log, non capisco dove ho sbagliato...
grazie ancora
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 15:05

>Ma solo vorrei capire dove ho sbagliato, ad esempio nel insert_log,
>quando metto insert, mi risulta:
>
>Unclosed quotation mark after the character string ',
> 20110718,
> Add Column,
> 3.14.3,
> NEGATIVO)'.
>
>era quello nel @istruzione
>
>invece add_column quando dovrei fare execute insert_log, non
>capisco dove ho sbagliato...

Ah boh

Ti posso solo dire che tutte le colonne stringa vanno racchiuse tra singoli apici (') e se sono UNICODE come le tue occorre specificare N'....' per evitare un cast implicito.
Tra l'altro, sei sicuro che ti servano effettivamente UNICODE? E per quale motivo tutti i parametri sono definiti nvarchar(200)?

>grazie ancora
Prego.

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 15:09

SET @istruzione = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL VALUES ('',
'+@data_eseguito+',
'+@comando_eseguito+',
'+@versione_CD+',
'+@esito+')'

mi è sembrato che si chiuso....

nvarchar, gli sviluppatori di visual studio mi hanno detto che devono essere con N, anche se io ho pensato che non servirebbe...

poi

BEGIN
DECLARE @istruzione Nvarchar(2000)
--DECLARE @insert_log Nvarchar(2000)

--SET @insert_log = docasdm.insert_log
SET @istruzione = N'alter table [' + @nome_utente + '].
[' + @nome_tabella + '] ADD ' + @nome_colonna +'
'+ @tipo_dato +' '
execute sp_executesql @istruzione
execute sp_executesql insert_log @nome_utente,
NULL,
'Added column on' +@nome_tabella+,
@versione_CD,
'esito positivo'

end

quello insert_log @nome_utente, è quello che è sottolineato per l'errore, MAH!
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 15:12

> SET @istruzione = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL
>VALUES ('',
> '+@data_eseguito+',
> '+@comando_eseguito+',
> '+@versione_CD+',
> '+@esito+')'
>
>mi è sembrato che si chiuso....

Cosa si è chiuso? Fai una PRINT @istruzione e vedrai con i tuoi occhi dove sta l'errore

>nvarchar, gli sviluppatori di visual studio mi hanno detto che
>devono essere con N, anche se io ho pensato che non servirebbe...
Da ex sviluppatore ti dico che "fai bene" a fidarti di loro (sono ironico, sia chiaro

).

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 15:24

infatti, hai ragione, ho usato print @istruzione e ho visto che non si vedono le apici.

ho usato

SET @istruzione = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL VALUES ('''',
'''+@data_eseguito+''',
'''+@comando_eseguito+''',
'''+@versione_CD+''',
'''+@esito+''')'

(madonna quanti apici!!

)

e invece quello execute insert_log nel add_column come funziona?

invece per gli sviluppatori, dovrei dire a loro che non conviene N?

----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 15:30

>e invece quello execute insert_log nel add_column come funziona?

Sostituisci:

execute sp_executesql insert_log...

con

EXEC DOCSADM.insert_log...

>invece per gli sviluppatori, dovrei dire a loro che non conviene
>N?

Leggi questo paragrafo dei BOL e quelli correlati:
http://msdn.microsoft.com/en-us/library/ms187828.aspx

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 15:39

ecco, ho visto il link, credo che nell'XML che hanno usato gli sviluppatore, avranno usato quello in italiano, ci provo a parlare con mio collega che usiamo gli scripts per gli sviluppatori.

Poi EXEC DOCSADM.insert_log

execute DOCSADM.insert_log @nome_utente,
NULL,
'Added column on' @nome_tabella,
@versione_CD,
'esito positivo'

E L'ERRORE DICE ERROR SYNTAX @NOME_TABELLA, ho già provato a mettere + @nome_tabella +, nemmeno non funziona

----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 15:42

>ecco, ho visto il link, credo che nell'XML che hanno usato gli
>sviluppatore, avranno usato quello in italiano, ci provo a parlare
>con mio collega che usiamo gli scripts per gli sviluppatori.
>
>Poi EXEC DOCSADM.insert_log
>
> execute DOCSADM.insert_log @nome_utente,
> NULL,
> 'Added column on' @nome_tabella,
> @versione_CD,
> 'esito positivo'
>
>
>E L'ERRORE DICE ERROR SYNTAX @NOME_TABELLA, ho già provato a
>mettere + @nome_tabella +, nemmeno non funziona

C'è un "+" di troppo:

EXEC DOCSADM.insert_log
      @nome_utente
    , NULL
    , 'Added column on' + @nome_tabella
    , @versione_CD
    , 'esito positivo';

Occhio che nella definizione della sp che hai postato sono previsti 4 parametri mentre in questa chiamata ne vedo 5...

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 15:56

è uguale anche se ho messo come hai scritto tu

EXEC DOCSADM.insert_log
@nome_utente
, NULL
, 'Added column on' + @nome_tabella
, @versione_CD
, 'esito positivo';

errore:
Incorrect syntax near '+'.

più in insert_log ci sono solo quattro perchè in date ho messo DECLARE del data_eseguito e per questo ho messo NULL

set @data_eseguito = (select convert (varchar, getdate(), 121))

poi ho chiesto al collega e mi ha risposto:

"perché li passiamo in input a execute_sp
quella SP di sistema, li vuole come nvarchar"
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 15:58

>è uguale anche se ho messo come hai scritto tu
>
>EXEC DOCSADM.insert_log
> @nome_utente
> , NULL
> , 'Added column on' + @nome_tabella
> , @versione_CD
> , 'esito positivo';
>
>errore:
>Incorrect syntax near '+'.
Allora concatena il valore e assegnalo ad una variabile prima di passarlo come parametro alla sp.

>più in insert_log ci sono solo quattro perchè in date ho messo
>DECLARE del data_eseguito e per questo ho messo NULL
>
>set @data_eseguito = (select convert (varchar, getdate(), 121))
Non ho capito.

>poi ho chiesto al collega e mi ha risposto:
>
>"perché li passiamo in input a execute_sp
>quella SP di sistema, li vuole come nvarchar"
E perché utilizzate il dynamic sql?

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 16:03

avevi scritto che nel insert_log c'erano solo 4:

@nome_utente Nvarchar(200),
@comando_eseguito Nvarchar (200),
@versione_CD Nvarchar(200),
@esito Nvarchar(200)

e invece nell'istruzione insert_log ci sono 5

SET @istruzione = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL VALUES ('''',
'''+@data_eseguito+''',
'''+@comando_eseguito+''',
'''+@versione_CD+''',
'''+@esito+''')'

perchè data_eseguito si trova dopo BEGIN e DECLARE data_eseguito

per questo durante l'exec insert_log ho messo NULL per far permettere di inserire la data_eseguito, non va bene?

poi non ho capito scusa per cosa significa concatenare quello + @nome_tabella.....

----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 16:05

>per questo durante l'exec insert_log ho messo NULL per far permettere
>di inserire la data_eseguito, non va bene?
No, se i parametri della sp sono 4, devi passare 4 valori.

>poi non ho capito scusa per cosa significa concatenare quello
>+ @nome_tabella.....

Definisci una variabile, valorizzala concatenando la stringa alla variabile e passala come parametro alla sp.

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 16:14

queste sp funzionano solo se usi dynamic sql
sennò come fa a creare una colonna senza sapere come si chiama? senza saperlo prima intendo?
è il motivo per cui nasce l'SQL dinamico....

poi per data_eseguito ho cambiato mettendo @data_eseguito sopra del begin

@nome_utente Nvarchar(200),
@data_eseguito NVARCHAR(200),
@comando_eseguito Nvarchar (200),
@versione_CD Nvarchar(200),
@esito Nvarchar(200)

DECLARE @istruzione Nvarchar(2000)
--DECLARE @data_eseguito Nvarchar (200)

SET IDENTITY_INSERT DOCSADM.DPA_LOG_INSTALL ON <--------------------
set @data_eseguito = (select convert (varchar, getdate(), 120))

però ce altro errore su IDENTITY_INSERT perchè ho ricevuto l'altro errore:
An explicit value for the identity column in table 'DOCSADM.DPA_LOG_INSTALL' can only be specified when a column list is used and IDENTITY_INSERT is ON.

per questo non ho mai ricevuto l'errore come questo.....
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

lbenaglia Profilo | Guru

5.625 messaggi | Data Invio: lun 18 lug 2011 - 16:16

>queste sp funzionano solo se usi dynamic sql
>sennò come fa a creare una colonna senza sapere come si chiama?
>senza saperlo prima intendo?
Ma che senso ha una sp che usa Dynamic SQL in questo modo?

>è il motivo per cui nasce l'SQL dinamico....
Si, vabbé...

Ciao!
--
Lorenzo Benaglia
Microsoft MVP - SQL Server
http://blogs.dotnethell.it/lorenzo/

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: lun 18 lug 2011 - 16:19

beh lascio perdere, è da 10 anni che lavorano in questo progetto e io sono entrato da poco, cioè da due mesi... vabbeh..

comunque sono più disperato per SP, sempre quello +@nome_tabella del add_column e identity_insert del install_log.

Se riesco a risolvare questi, così potrei fare molti SP che mi hanno chiesto
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: mer 20 lug 2011 - 11:54

Ho risolto in questo modo:

ALTER PROCEDURE [DOCSADM].[add_column]
-- Add the parameters for the stored procedure here
@versioneCD Nvarchar(200),
@nome_utente Nvarchar(200),
@nome_tabella Nvarchar(200),
@nome_colonna Nvarchar(200),
@tipo_dato Nvarchar(200),
@val_default Nvarchar(200),
@condizione_modifica_pregresso Nvarchar(200),
@condizione_check Nvarchar(200),
@RFU Nvarchar(200)
AS
BEGIN
if not exists(select * from syscolumns where name=@nome_colonna and id in
(select id from sysobjects where name=@nome_tabella and xtype='U'))
BEGIN
DECLARE @istruzione Nvarchar(2000)
DECLARE @insert_log Nvarchar(2000)
DECLARE @data_eseguito Nvarchar(2000)

set @data_eseguito = (select convert (varchar, getdate(), 120))

SET @istruzione = N'alter table [' + @nome_utente + '].
[' + @nome_tabella + '] ADD ' + @nome_colonna +'
'+ @tipo_dato +' '

SET @insert_log = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL VALUES (
'''+@data_eseguito+''',
''Added column ' +@nome_colonna+' on '+@nome_tabella+''',
'''+@versioneCD+''',
''esito positivo'')'

execute sp_executesql @istruzione
exec(@insert_log)
end
end

però una cosa che vorrei sapere come si fa: se l'insert non è andato bene (colonna già esistente, o tabella non esistente), vorrei inserire nel log (DPA_LOG_INSTALL) mettendo negativo....
Grazie
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

gsflash80 Profilo | Junior Member

95 messaggi | Data Invio: mer 20 lug 2011 - 12:15

ho risolto.

dopo exec(@insert_log)

else

begin
set @data_eseguito = (select convert (varchar, getdate(), 120))

SET @insert_log = N'INSERT INTO '+@nome_utente +'.DPA_LOG_INSTALL VALUES (
'''+@data_eseguito+''',
''Added column ' +@nome_colonna+' on '+@nome_tabella+''',
'''+@versioneCD+''',
''esito negativo'')'

exec(@insert_log)
end

grazie a tutti ;-)
----

..:: GSFLASH ::..
http://www.gsflash.it
http://blogs.dotnethell.it/gabrieleserpi/
MCTS - ACP - OCP - ECDL - ACSP
Ubuntu 11.04/OpenSuse 10.3
Vista Seven/MacOSX SnowLeopard

Torna su

Stanze Forum

Elenco Threads

Partecipa anche tu! Registrati!

Hai bisogno di aiuto ?
Perchè non ti registri subito?

Dopo esserti registrato potrai chiedere
aiuto sul nostro Forum oppure aiutare gli altri

Consulta le Stanze disponibili.

Registrati ora !