Member Avatar for zyaday

Hi everyone,
I'm trying to write this code.
Whenever a user types a letter that matches what is already displayed, it should clean up the list box..the problem here is it is not detecting the keyDown event. What is wrong?
Thanks for the help.

namespace TypingGame
{
    public partial class Form1 : Form
    {

        Status status = new Status();
        Random random = new Random();

        public Form1()
        {
            InitializeComponent();
                    this.KeyDown +=new KeyEventHandler(Form1_KeyDown);
            }

        private void statusStrip1_ItemClicked(object sender, ToolStripItemClickedEventArgs e)
        {

        }

        private void timer1_Tick(object sender, EventArgs e)
        {
            listBox1.Items.Add((Keys) this.random.Next(65, 90));
            if (listBox1.Items.Count > 7)
            {
                listBox1.Items.Clear();
                listBox1.Items.Add("Game Over!");
                timer1.Stop();
            }

        }



        private void Form1_KeyDown(object sender, KeyEventArgs e)
        {

            if (listBox1.Items.Contains(e.KeyCode))
            {
                status.update(true);
                listBox1.Items.Remove(e.KeyCode);
                listBox1.Refresh();
            }
            else
                status.update(false);

            this.CorrectLabel.Text = "Correct: " + status.Correct;
            this.missedLabel.Text = "Missed: " + status.Missed;
            this.accuracyLabel.Text = "Accuracy: " + status.Accuracy;
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            timer1.Start();
        }

        private void listBox1_SelectedIndexChanged(object sender, EventArgs e)
        {

        }

        private void Form1_KeyPress(object sender, KeyPressEventArgs e)
        {

        }
    }
}
  • 2 Contributors
  • 1 Reply
  • 129 Views
  • 7 Hours Discussion Span
  • Latest Post Latest Post by Momerath

Dani AI

Generated

Problem reported by : a Form.KeyDown handler was added but never fires while the ListBox has focus. correctly pointed out the root cause — keyboard messages go to the control that has focus, and a focused ListBox will handle key input itself, so the form does not see those events by default.

Make the form receive keyboard events before the focused control by enabling Form.KeyPreview. Put this into the form constructor (after InitializeComponent) or set it in the designer:

this.KeyPreview = true;

With KeyPreview true the form's KeyDown/KeyPress handlers run first, so the existing Form1_KeyDown will be invoked even when the ListBox has focus. Microsoft docs: Form.KeyPreview documentation.

Other useful options and checks:

  • Attach the handler to listBox1.KeyDown instead if the logic should only run when the listbox is focused.
  • Use KeyPress (e.KeyChar) when comparing character input; KeyDown gives Keys values.
  • Verify the types stored in listBox1.Items: Items.Contains(e.KeyCode) only succeeds if items were added as Keys (not as strings). If items are strings, compare with e.KeyCode.ToString() or use e.KeyChar.
  • For application-wide capture, consider an IMessageFilter or overriding ProcessCmdKey, but those are more advanced.

Basic debugging: set a breakpoint in the key handler to confirm whether it runs, and check that the handler subscription and the form's KeyPreview state are as intended.

Member Avatar for Momerath

You've attached the keydown event to the form, is that what you wanted to do? If you are in the listbox when you press the key, it will handle the keypress and not pass it to the form.

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.