Oh,

Nao fala assim do VB.NET não Pedro :P

O VB.NET não é mau. Um codigo mau escrito pode ser em qualquer linguagem, desde as mais poderosas até aquelas menos utilizadas e ineficientes como o C# :P

Tou a brincar Pedro, mas realmente este codigo é de fazer qualquer um chorar, ou rir :D

Abs,

Boas Pedro,

Nao chateou nao Pedro. Pode ficar tranquilo.

Abs,

Epah não me levem a mal, mas eu assim que olhei para esse código em VB .NET pensei logo: "Mas que diabo!?" :-)

Ja que ninguem mordeu a isca, ca estao os erros que eu apanhei:

1- Arrays em VB.NET sao defenidas pelo limite, logo a array defenida tem dois elementos, um dos quais nunca e' preenchido

2- O connection Close ta fora de um Finally, logo se um excepcao for atirada, a ligacao potencialmente fica aberta

3- Nao há necessidade de fazer Dispose e Close, pois no caso de Connections eles fazem a mesma coisa ( se bem me recordo Dispose chama Close).

4- A variavel Retorno e' defenida como um Integer, mas a funcao tem o retorno defenido como Boolean. Ha' coercao de tipos ali, e nao tenho bem a certeza se o resultado obtido e' o desejado.

5- Ver o codigo:
Catch ex As Exception
Throw ex
E' das coisas que mais me irrita em "tratamento de erros" em .NET ( pior mesmo seria perder a excepcao). Se nao vais adicionar qualquer informacao a excepcao para que tar a apanhala e a atirar ela de novo.
Se queres fazer algum tratamento, como logar, podes voltar a atirar sem estragar o stackstrace fazendo só Throw

Escapou-me alguma coisa ( sem as piadas de utilizacao tar-se a utilizar VB.NET, pois sinceramente isto so demonstra falta de conhecimento de como o CLR funciona)

Só complementando:

"se bem me recordo Dispose chama Close"
>> Está com a memória fraca hein João :P (abaixo):

"The Dispose method calls Close, and returns the SqlConnection to the connection pool."

Fonte: http://weblogs.pontonetpt.com/jpc/posts/414.aspx :P

Humz... não queria chatear ninguém com a piada do VB.NET, peço desculpa se fui inconveniente. Seja como for eu não gosto é da sintaxe do VB.NET, tal como não gostaria de ver código em COBOL.NET.

Isso não demonstra que não se saiba como funciona o CLR, demonstra que cada um tem as suas preferências, daí haver várias hipóteses de linguagens quando queremos programar para .NET. :-)

Epah não me levem a mal, mas eu assim que olhei para esse código em VB .NET pensei logo: "Mas que diabo!?" :-)

Ja que ninguem mordeu a isca, ca estao os erros que eu apanhei:

1- Arrays em VB.NET sao defenidas pelo limite, logo a array defenida tem dois elementos, um dos quais nunca e' preenchido

2- O connection Close ta fora de um Finally, logo se um excepcao for atirada, a ligacao potencialmente fica aberta

3- Nao há necessidade de fazer Dispose e Close, pois no caso de Connections eles fazem a mesma coisa ( se bem me recordo Dispose chama Close).

4- A variavel Retorno e' defenida como um Integer, mas a funcao tem o retorno defenido como Boolean. Ha' coercao de tipos ali, e nao tenho bem a certeza se o resultado obtido e' o desejado.

5- Ver o codigo:
Catch ex As Exception
Throw ex
E' das coisas que mais me irrita em "tratamento de erros" em .NET ( pior mesmo seria perder a excepcao). Se nao vais adicionar qualquer informacao a excepcao para que tar a apanhala e a atirar ela de novo.
Se queres fazer algum tratamento, como logar, podes voltar a atirar sem estragar o stackstrace fazendo só Throw

Escapou-me alguma coisa ( sem as piadas de utilizacao tar-se a utilizar VB.NET, pois sinceramente isto so demonstra falta de conhecimento de como o CLR funciona)

Só complementando:

"se bem me recordo Dispose chama Close"
>> Está com a memória fraca hein João :P (abaixo):

"The Dispose method calls Close, and returns the SqlConnection to the connection pool."

Fonte: http://weblogs.pontonetpt.com/jpc/posts/414.aspx :P

Humz... não queria chatear ninguém com a piada do VB.NET, peço desculpa se fui inconveniente. Seja como for eu não gosto é da sintaxe do VB.NET, tal como não gostaria de ver código em COBOL.NET.

Isso não demonstra que não se saiba como funciona o CLR, demonstra que cada um tem as suas preferências, daí haver várias hipóteses de linguagens quando queremos programar para .NET. :-)