Space Invaders

Hi, im trying to make a space invaders game and am stuck moving my gun. Ive put a loop in that moves the ship to the right when a button is pressed and then should stop when released. the stop part doesnt work. can anyone help! here is the code.

Private Sub CmdRight_MouseDown(ByVal Button As Integer, ByVal Shift As Integer, ByVal X As Single, ByVal Y As Single)

Do Until LblBullet.Left = 324
For i = 1 To 5000
DoEvents
Next i
Dim MyVar As Variant
MyVar = f
If MyVar = t Then
ImgGun.Left = ImgGun.Left + 1
LblBullet.Left = LblBullet.Left + 1
Else: Exit Do
End If
Loop

End Sub

Private Sub CmdRight_MouseUp(ByVal Button As Integer, ByVal Shift As Integer, ByVal X As Single, ByVal Y As Single)
MyVar = f
End Sub

have i just got my variable or if statements in the wrong place

sorry, in the mousedown sub MyVar is initially set to t, i set it to f to test if it was seeing the variable at all

and what is t

You don't declear t in your code, so what is it....

BTW..use [Highlight=VB] and [ /vbcode] tags whenposting code...

Couple of no-no's that pop right out:
1)

Quote:

---

Originally posted by davebat
For i = 1 To 5000
DoEvents
Next i

---

NEVER, I repeat, NEVER do this. A loop like this will run as quickly as the PC will let it, guaranteeing different results on different machines.

2)

Quote:

---

Originally posted by davebat
Dim MyVar As Variant
MyVar = f
If MyVar = t Then

---

ALWAYS, I repeat, ALWAYS use descriptive variable names. When you go back to look at code like this, you probably won't even remember what you were trying to do or what the variables are.

Are you using Option Explicit in all of your modules and forms? If not, DO IT NOW! :D

Dim Keys(0 to 255) as boolean
------------------------------------------
Sub Keydown (lalalalala)
Keys(keycode) = true
end sub
------------------------------------------
Sub keyUp(lallalala)
Keys(KeyCode) = false
End sub


So as long as you know which key uses which code you always know when one is down

thanks guys, Ive taken note of your advice and will change the code when i get in from work. what did you mean by open explicit? Im new to this you see.

From the MSDN:

Quote:

---

When Option Explicit appears in a module, you must explicitly declare all variables using the Dim, Private, Public, ReDim, or Static statements. If you attempt to use an undeclared variable name, an error occurs atcompile time.

If you don't use the Option Explicit statement, all undeclared variables are of Variant type unless the default type is otherwise specified with a Deftype statement.

---

On the filemenu click Tools, then Options and check the check box "Require Variable Delaration"....