Non-comprehensive list of things you could address.
Some developers omit curly braces around loop and branching blocks if there is only one line-- if(blah) true; else false; versus if(blah) { true; } else { false; }. This should be brought up, and explicitly discouraged.
Usage of the ternary operator ?: -- when (or is it...) is it acceptable to use?
Usage of _ character in names -- when (or is it..) is it acceptable to use?
Usage of System.debug( ... ) statements. They should never be left in production code, as they can cost 45ms each time they're called, and that's assuming they don't do anything complex. They should be discouraged.
When should function variables be declared? Near the top of the function or nearer the use of the function?
Constructor versus inline initialization of member variables. Likewise for static members.
Non-comprehensive list of things you could address.
Some developers omit curly braces around loop and branching blocks if there is only one line-- if(blah) true; else false; versus if(blah) { true; } else { false; }. This should be brought up, and explicitly discouraged.
Usage of the ternary operator ?: -- when (or is it...) is it acceptable to use?
Usage of _ character in names -- when (or is it..) is it acceptable to use?
Usage of System.debug( ... ) statements. They should never be left in production code, as they can cost 45ms each time they're called, and that's assuming they don't do anything complex. They should be discouraged.
When should function variables be declared? Near the top of the function or nearer the use of the function?
Constructor versus inline initialization of member variables. Likewise for static members.