You should learn to use the debugger as soon as possible. Rather than guessing what your code is doing, It is time to see your code executing and ensuring that it does what you expect.
The debugger allow you to follow the execution line by line, inspect variables and you will see that there is a point where it stop doing what you expect.
Debugger - Wikipedia, the free encyclopedia[
^]
Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[
^]
In first, put a breakpoint on very first line executed and check that variables meet your expectations. Then executed line by line.
Weird, your code contain very basic mistakes on things that you should have mastered long before turning to threads.