Some donts
Don't hardcode variables. If you have an array always refer to it's length as the proper array.length() or getLength() not sure. If you think you may need something more than one place definitely give it a variable, makes life a lot easier.
Don't needlessly use up block id's - don't...