Veo algunos inconvenientes con tu declaración de función:
- Debes declarar las variables que vas a utilizar
- Los nombres de variables deben iniciar con el símbolo
@
- La sintaxis para retornar valores de un select en variables no es la que estás utilizando, sino es
select @variable = campo/valor/etc
- La correcta indentación no ayuda a las computadoras, pero si a los humanos, a entender el código y su flujo.
- La semántica de las clausulas
on
del join
y where
son:
where
sirve para filtrar los resultados de la consulta
on
define la relación que hay entre las tablas en el join
Digo esto, porque filtras en la clausula on
y, aunque esto produce los resultados correctos, me parece confuso y puede confundir a otras personas.
Integrando estas recomendaciones, tu código luciría algo como esto:
create function dbo.ventasxciudad_dada (@ciudad_dada char(3))
returns varchar(MAX)
as
begin
declare @ciudad varchar(100);
declare @valor_ventas money;
select @ciudad = c.nom_ciudad
, @valor_ventas = sum(f.val_factura)
from CIUDAD c
inner join CLIENTE cl ON c.cod_ciudad = cl.cod_ciudad
inner join FACTURA f ON cl.cod_cliente = f.cod_cliente
where c.cod_ciudad = @ciudad_dada
group by c.nom_ciudad
RETURN(@ciudad + '---->' + cast(@valor_ventas as varchar(30)))
end
go
Dicho todo esto, el error que te está dando el SQL Server no tiene que ver con ninguno de los puntos anteriores... y estoy suponiendo aquí, ya que el único código que muestras es el de la creación de la función, que esto es parte de un script que tiene más sentencias SQL.
Con SQL Server, se utiliza la sentencia go
para indicar el final de una sección lógica de un script, que se ejecuta inmediatamente se encuentra la sentencia. Esta sección lógica es a lo que se refiere con un batch
el error.
Las sentencias create function
y create procedure
deben ser las primeras (hasta donde recuerdo, no necesariamente las únicas de un batch. Por tanto, si tienes un script con otras cosas, coloca una línea con la palabra go
antes de create function
, así:
create table....
alter table...
create index...
go
create function....
go
create procedure...
go
BEGIN
sin su correspondienteEND
. En segundo lugar, estás haciendo unSELECT
que tiene unGROUP BY
, con lo cual es factible que entregue como resultado varias filas (y tu función es escalar). En tercer lugar, no estás asignando ese resultado a ninguna variable, y simplemente pasando eso alRETURN
, lo que también está maloINTO
que no funciona de esa forma en SQL Server