[VB.NET]-Problema nella classe Login

[Scorpio2010]

Ciao a tutti, sto facendo il mio primo programma in VB e ho un paio di problemi con la classe per fare il Login. Ho fatto una finestra madre che contiene una finestrella per fare il login e ho riscontrato i due seguenti problemi:
1)Il pulsante accedi non mi fa nulla (probabilmente perchè non riesco a prendere i valori delle due textbox);

2)In fase di compilazione mi dà i due seguenti errori: 'Text' non è un membro di 'String' e 'TextLength' non è un membro di 'String'

vi posto il codice:

Private Sub Entra(ByVal sender As System.Object, ByVal e As System.Windows.Forms.KeyPressEventArgs)

Dim Username As String
Dim Password As String

Username = "SELECT Username FROM Utenti WHERE" & username
Password = "SELECT Password FROM Utenti WHERE" & password

If Trim(Username.Text).Length = 0 Then
MsgBox("Inserire l'username per accedere.", vbCritical, "ERRORE")
Exit Sub
End If

If Trim(Password.Text).Length = 0 Then
MsgBox("Inserire la password per accedere.", vbCritical, "ERRORE")
Exit Sub
End If

If Username.TextLength > 0 And Password.TextLength > 0 Then
accedi.Enabled = True
Else
accedi.Enabled = False
End If

End Sub

P.s.: come DB ho usato Access

[Red Storm]

Hai provato ad aggirare l'ostacolo?

...
if Username = "" then

...

if Password = "" then

...

[Scorpio2010]

Ci ho già provato così e se è vero che mi toglie gli errori è pure vero che non mi mosta le MsgBox

[Red Storm]

... l'ho detto perchè le variabili sono dichiarate come string: in questo caso il sistema si aspetta un valore tra doppi apici! Quindi sostituendo le condizioni da = 0 a = "" dovrebbe funzionare anche il MessageBox...

[MItaly]

In ogni caso le query sono sbagliate; così com'è adesso la clausola where è incompleta e l'uso di due query separate è errato, sarà semmai una roba del tipo:

"SELECT Username, Password FROM Utenti WHERE Username='" & username & ''" AND Password='" & password & "'"

Tuttavia se usi questa roba comunque la tua applicazione rimane estremamente vulnerabile ad attacchi di tipo SQL injection. Dovresti usare delle query parametriche.

[Scorpio2010]

Per quanto riguarda le MsgBox non mi funzionano.

La stringa SQL mi da errore di sintassi e MItaly che intendi per: "rimane estremamente vulnerabile ad attacchi di tipo SQL injection. Dovresti usare delle query parametriche."?

[MItaly]

SQL injection
Query parametriche

[gibra]

Originariamente inviato da MItaly
SQL injection
Query parametriche

E aggiungo (giusto giusto per rincarare la dose )

Una piaga chiamata SQL Injection
http://msdn.microsoft.com/it-it/library/cc185099.aspx

Tecniche: SQL Injection
http://sicurezza.html.it/articoli/le...sql-injection/

[Jeremy75]

E volendo infierire ..... mi verrebbe da dire:
Ma il codice che hai scritto lo hai inventato? E' un nuovo linguaggio di programmazione?

Se dentro la variabile UserName c'è scritto

Username = "SELECT Username FROM Utenti WHERE" & username

per quanto sbagliato possa essere .... non sarà mai vera la condizione:

If Trim(Username.Text).Length = 0 Then

ti sembra?

Ciao

[Scorpio2010]

Supponiamo che io abbia corretto gli errori di cui parla Jeremy75, rimane il fatto che non mi funzionano nè le msgbox e nè le Console.Write() o .WriteLine() di cui sotto:

Console.Write("testo")
Console.WriteLine("testo")
MsgBox("testo", MsgBoxStyle.Critical, "ERRORE")
MsgBox("testo", vbCritical, "ERRORE")

Per "non mi funzionano" intendo che nella console non mi scrive niente e non mi compare nessuna finestrella di alert. Qualche idea?