Error Sumando Columnas SQL

[martini002]

Saludos amigos, donde quiera que se encuentren.

En el siguiente codigo, podran darse cuenta que funciona
como un ciclo infinito, hasta que el usuario decida detener
el programa, basicamente lo que hacemos es agregar en una tabla temporal
algunos datos para visualizarlos en un grid, luego vaciamos esos datos
en nuevas tablas para el registro de facturas, y luego limpia la tabla temporal
para poder crear nuevas facturas.

El problema consiste en lo siguiente:
agregamos los articulos con el boton1 y luego cuando tengamos
los que queramos, entonces despachamos la orden con el boton2
la cuestion es que cuando vuelo a agregar articulos,
se tranca el proceso. Justo cuando esta sumando, en esta linea:

Código Delphi [-]
ZQuery1.SQL.Text:=  'Select Sum(Precio) as Suma From Factura';

La primera vez el lo hace bien, todo correcto,
pero la segunda vez ya no pasa de alli.

Hechenle un vistazo al codigo:

Código Delphi [-]
procedure TForm1.Button1Click(Sender: TObject);
begin
  if Edit1.Text<>'' then
    begin
      ZQueryBD.Close;
      ZQueryBD.SQL.Text:= 'Select Codigo From Inventario Where Codigo = '+Edit1.Text;
      ZQueryBD.Open;
      if ZQueryBD.IsEmpty then //Controlar el Codigo del producto
                               //Verifica si es correcto o no entrara a las otras condiciones.
        Begin
          ShowMessage('El Codigo del Producto es Invalido o NO esta Registrado');
        End
      Else
        Begin
          if StrToInt(Edit2.Text)>=1 then
            begin
              DataSourceBD.Enabled:= True;
              ZQueryBD.Close;
              ZQueryBD.SQL.Text:= 'Insert Into Factura (Articulo, Cantidad, Precio) Select Articulo, '+Edit2.Text+', Precio*'+Edit2.Text+' From Inventario Where Codigo = '+Edit1.Text;
              ZQueryBD.ExecSQL;
              ZQueryBD.Close;
              ZQueryBD.SQL.Text:= 'Select * From Factura';
              ZQueryBD.Open;
              ZQuery1.Close;
              ShowMessage('La suma de los precios');
              //Aqui se tranca
              ZQuery1.SQL.Text:=  'Select Sum(Precio) as Suma From Factura';
              ShowMessage('agregado');
              //Justo aqui abajo, no abre el query
              ZQuery1.Open;
              ShowMessage('sumados');
              Edit3.Text:= ZQuery1.FieldByName('Suma').AsString+' BsF';
              Monto:= ZQuery1.FieldByName('Suma').AsString;
            end
          Else
            begin
              ShowMessage('La Cantidad debe ser Mayor o Igual que 1');
            end;
        End;
    end
  Else
    begin
      ShowMessage('Ingrese un Codigo');
    end;
end;


procedure TForm1.Button2Click(Sender: TObject);
Var
  Codigo:Integer;
begin
  DataSourceBD.Enabled:= False;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Insert Into RFac (Monto, Fecha, Hora) Values ('+Monto+', Date(), Time())';
  ZQueryBD.ExecSQL;
  ZQuery1.Close;
  ZQuery1.SQL.Text:= 'Select Max(CodigoFactura) as CodigoFactura From RFac';
  ZQuery1.Open;
  Codigo:= ZQuery1.FieldByName('CodigoFactura').AsInteger;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Create Table F_'+IntToStr(Codigo)+' (NumArticulo Integer Not Null Primary Key AutoIncrement, Articulo Char(48), Cantidad Integer, Precio Decimal)';
  ZQueryBD.ExecSQL;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Insert Into F_'+IntToStr(Codigo)+' (Articulo, Cantidad, Precio) Select Articulo, Cantidad, Precio From Factura';
  ZQueryBD.ExecSQL;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Insert Into F_'+IntToStr(Codigo)+' (Precio) Values ('+Monto+')';
  ZQueryBD.ExecSQL;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Delete From Factura';
  ZQueryBD.ExecSQL;
  Monto:='';
  Edit3.Text:='0.00 BsF';
  Edit1.Text:='';
end;

Gracias siempre por su tiempo y su apoyo

[[Caral]]

Hola
Cuando vuelves a agregar productos, antes o despues de eliminar los datos de la tabla temporal ?.
Saludos

[ElMug]

Tal vez tienes que mandar un "commit" despues de cada transaccion, creo que al final de cada ciclo.

Yo no uso Zeos, pero checa a ver si se aplica el concepto de concluir una transaccion antes de empezar otra.

[martini002]

Despues de darle click al segundo boton,
luego de agregar los nuevos registros y crear la tabla nueva
con los valores de la nueva factura;
despues de borrar la tabla temporal.

Pienso que podria haber un problema
entre el ZQuery1 del primer boton
y el ZQuery1 del segundo, he intentado con otro ZQuery
pero me da otro error :/

[[Caral]]

Hola
Para mi gusto esta algo desordenado. no entiendo bien el concepto.
Hay variables que son globales, no me parece.
No se que BD usas, eso complica algo mas el asunto.
No entiendo como el articulo y el precio estan en el mismo edit ?

Código Delphi [-]
Select Articulo, '+Edit2.Text+', Precio*'+Edit2.Text+' From Inventario

Bueno, no entiendo como te complicas la vida amigo.
Saludos

[martini002]

Bueno estoy usando SQLite,

Esto basicamente, multiplica la Cantidad por el Precio y me da un subtotal
el Edit2 corresponde a la Cantidad, Estoy usando una variable global "Monto"
porque necesito ese dato en el otro Procedimiento.

Código Delphi [-]
Select Articulo, '+Edit2.Text+', Precio*'+Edit2.Text+' From Inventario

Pues sigo estancado aqui, he intentado de todo pero hasta ahora
no he tenido nada de avance :/
si hay un poco de desorden, pero los momentos solo quiero que funcione
ya despues con mas calma cuando este modulo este listo, entonces
comienzo a agregarle los comentarios, y algunas condiciones de control,
detallitos pues

[[Caral]]

Hola
Sigo sin entender.
Cuando se hace una factura esta tiene un numero UNICO.
Se hace sobre la tabla directa y si se requiere eliminar un item se elimina sin problemas.
Cuando usas una tabla temporal no necesitas eliminar los datos ya que te podrian servir de algo, pero lo veo innecesario.
Lo que no veo es ningun numero que identifique la factura.
No vi el signo de multiplicar, por eso me parecio raro, aun asi creo que estas complicandote la vida sin necesidad.
Saludos

[Casimiro Notevi]

Empezado por martini002

Pues sigo estancado aqui, he intentado de todo pero hasta ahora
no he tenido nada de avance :/

Amigo, te aconsejo que primero leas este libro, luego lo verás todo de otra forma.

[martini002]

Empezado por Casimiro Notevi

Amigo, te aconsejo que primero leas este libro, luego lo verás todo de otra forma.

Casimiro te agradezco mucho tu consejo,
pero no puedo leerme todo ese libro para poder resolver este problemita,
te agradeceria aun mas, si me dijeras mas o menos que partes me recomiendas,
a mi me gusta ir aprendiendo de acuerdo a las necesidades,
yo tengo la Cara Oculta de Delphi 6, no se si tambien sea conveniente,
ademas tengo a mano el manual de SQL que lo he venido leyendo
a medida que estoy manejando estas BD, pues soy nuevo con esto.

Edito, No habia leido tu comentario Caral.

Mira, el codigo de la factura se guarda,
uso una tabla para almacenar los registros de:
NumeroFactura y MontoFactura
para que se haga mas facil para mi la lectura de este.

uso la tabla temporal porque necesito mostrarla en un DBGRid,
y alli se editara que es lo que queremos agregar y eliminar o modificar,
una vez tengamos eso listo, entonces procedemos a guardar y registrar todo

Gracias

Saludos

[[Caral]]

Hola
Busca un commit.
ZQueryBD.commit.
Me avisas si esta.
saludos

[martini002]

Empezado por Caral

Hola
Busca un commit.
ZQueryBD.commit.
Me avisas si esta.
saludos

No creo que hay ZQueryBD.commit;
pero si logre encontrar un ZQueryBD.CommitUpdates;
:/

[[Caral]]

Hola
Intenta esto es esta linea:

Código Delphi [-]
 ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Delete From Factura';
  ZQueryBD.ExecSQL;
  ZQueryBD.CommitUpdates;

Saludos

[martini002]

Bueno,
probe esto que me dices, pero me da un error,
dice que no puede ejecutar esto: "ZQueryBD.CommitUpdates;"
si esta cerrado, entonces lo coloque asi:

Código Delphi [-]
  ZQueryBD.SQL.Text:= 'Delete From Factura';
  ZQueryBD.ExecSQL;
  ZQueryBD.Open;
  ZQueryBD.CommitUpdates;

Y continua pues, pero me sigue dando el error incial,
no puedo agregar otra factura.

[[Caral]]

Hola
Pensando............
Saludos
PD: No te desesperes, estamos en esto.

[martini002]

Te lo agradezco, yo he estado
pensando en esto mucho mucho,
inclusive, en cambiar la logica
y crearlo de otra forma mas eficiente.

[[Caral]]

Hola
A ver por partes que hacemos.
Yo diria que no necesitas dos consultas para la suma, cambiaria por esto:

Código Delphi [-]
              ZQueryBD.Close;
              ZQueryBD.SQL.Text:= 'Select Articulo, Cantidad, Precio, Sum(Precio) as Suma  From Factura' +
                                  ' Group By Articulo, Cantidad, Precio' ;
              ZQueryBD.Open;
            //  ZQuery1.Close;
            //  ShowMessage('La suma de los precios');
              //Aqui se tranca
            //  ZQuery1.SQL.Text:=  'Select Sum(Precio) as Suma From Factura';
              ShowMessage('agregado');
              //Justo aqui abajo, no abre el query
            //  ZQuery1.Open;
              ShowMessage('sumados');
              Edit3.Text:= ZQueryBD.FieldByName('Suma').AsString+' BsF';
              Monto:= ZQueryBD.FieldByName('Suma').AsString;
            end

Empecemos por aqui, me dices si te sirve o si te gusta.
Saludos

[martini002]

Me gusto mucho esa solucion,
porque acortas bastante el codigo,
ya no presenta el error cuando agregaba otro producto,
pero ahora me presenta un error en el segundo boton,
se tranca en la linea que esta descrita abajo,
y entonces no crea las tablas ni borra la temporal tampoco.
creo que podemos aplicar la misma tecnica que arriba,
intentare a ver como me va

mira aqui:

Código Delphi [-]
procedure TForm1.Button2Click(Sender: TObject);
Var
  Codigo:String;
begin
  DataSourceBD.Enabled:= False;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Insert Into RFac (Monto, Fecha, Hora) Values ('+Monto+', Date(), Time())';
  ZQueryBD.ExecSQL;
  ZQuery1.Close;
  ZQuery1.SQL.Clear;
  ZQuery1.SQL.Text:= 'Select Max(CodigoFactura) as CodigoFactura From RFac';
  ZQuery1.Open;
  //Justo aqui Abajo.
  Codigo:= ZQuery1.FieldByName('CodigoFactura').AsString;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Create Table F_'+Codigo+' (NumArticulo Integer Not Null Primary Key AutoIncrement, Articulo Char(48), Cantidad Integer, Precio Decimal)';
  ZQueryBD.ExecSQL;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Insert Into F_'+Codigo+' (Articulo, Cantidad, Precio) Select Articulo, Cantidad, Precio From Factura';
  ZQueryBD.ExecSQL;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Insert Into F_'+Codigo+' (Precio) Values ('+Monto+')';
  ZQueryBD.ExecSQL;
  ZQueryBD.Close;
  ZQueryBD.SQL.Text:= 'Delete From Factura';
  ZQueryBD.ExecSQL;
  Monto:='';
  Edit3.Text:='0.00 BsF';
  Edit1.Text:='';
end;

[[Caral]]

Hola
La verdad estoy algo perdido.
Me gustaria saber:
1- que campos tiene la tabla factura.
2- que campos tiene la tabla RFac
3- estas creando una tabla por cada numero de factura ????
Saludos

[martini002]

estas creando una tabla por cada numero de factura ????

hahaha Si :S estoy seguro que es lo menos eficiente,
pero hasta ahorita no he logrado dar con un formato
"amigable" para lograr almacenar todos los detalles de cada factura en una sola tabla,

Vamos con los campos:

Estos son los DDL que me da el SQLite Personal Expert

Para Tabla Factura

CREATE TABLE [Factura] (
  [Articulo] CHAR(48), 
  [Cantidad] INT, 
  [Precio] DECIMAL);

Para RFac

CREATE TABLE [RFac] (
  [CodigoFactura] INTEGER NOT NULL PRIMARY KEY, 
  [Monto] DECIMAL, 
  [Fecha] DATE, 
  [Hora] TIME);

Gracias

[[Caral]]

Hola
Lo modificamos del todo ?
Saludos