Well to start with you have no object named ListBox1.

Secondly, if your rows are going to change over time, you need to use dynamic ranges. Lots of exam[ples of those on the board.