Mejorar metodo!!

[kye_z]

buenas,

despues de conseguir conectar a la base de datos, i que me haga el insert de lo que kiero correctamente i comprovando todo lo necesario.

me gustaria que me ayudaseis a terminar el codigo del metodo.
Os pego el codigo del metodo i luego os explico que es lo ke kiero..



var
values : String;
begin
if Trim(nomClub.Text) <> '' then
begin
sqlCrearClub.Sql.Clear;
sqlCrearClub.Sql.Add(' INSERT INTO Club (nom_club');
values:= ') VALUES ("'+ nomClub.Text;
if Trim(direccioClub.Text) <> '' then
begin
sqlCrearClub.Sql.add(',direccio');
values:= values+ '","' + direccioClub.Text;
end;

if Trim(mailClub.Text) <> '' then
begin
sqlCrearClub.Sql.add(',mail_club');
values:= values+ '","' + mailClub.Text;
end;
sqlCrearClub.SQL.Add(values + '")') ;
// aqui es conecta a la bdd i executa la SQL
conexio.Connected:=True;
sqlCrearClub.ExecSQL;
conexio.Connected:=False;
end
else
ShowMessage('Es obligatori omplir el cam Nom del club!!');
EXIT;
end;
end.


bueno, el metodo me funciona exactamente i hace lo que quiero pero mi pregunta es... creeis que seria mejor hacerlo de otra manera, en vez de utilizar tantos if no seque, if nosekuanto... Quedaria mejor si lo iciera con case?!.. no se i kualkier critica tb la aceptare k ahun soy novatillo como supongo habeis visto en otros de mis posts... ejjeej :D (para ser mas esactos es el primer metodo que me funciona jajajaja)

i otra pregunta que me queda en mente, como puedo hacer para que la conexion que he hecho (conexio.conected=true) se deskonecte de la base de datos?!...

pos eso.. muchas gracias de antemano i espero ser un buen PadaWan y aprender lo antes possible... jejejeje

[Mario1980]

No te enojes pero quiero decirte que te acostumbres a poner un solo tema por hilo, yo antes lo hacia tambien y me lo repitieron varias veces :D.
Con lo que decis de un case creo que no te funcionaria porque el case lo que hace creo que al encontrar una condicion verdadera se mete a ejecutar esa opcion y las otras las pasa de largo, asi que olvidate de eso.
Para desconectar la base de datos podrias ir al onclose del form y poner el valor en false.( nunca probe eso fijate si no funciona decime).
Una pregunta, si los valores que ingresas son blancos a excepcion del primero se ejecuta igual el sql?.
Saludos.

[kalimero]

Hola

¿Por que desconectas cada vez que ejecutas el Insert?

Saludos