Well there are generally ALOT of ways to achieve the same goal and generally the shortest may or may not be the best.
calling functions more than once in a loop is one thing that will slow down code execution with usually no benefit(In some cases it may be necessary)
memory allocation is another. say for instance you have a char cExample[2046]; if you know that the char will NEVER be closed to filled, its best to allocate the array to a smaller value. Although doing something like this may not really show if there is just one , even though it is bad practice to do so, when you get into bigger , more advanced programs(such as a game) it will make a difference.
if(You.AddCode.toPost)TGC.Users.CanHelp = true;
else return NoCodeMessage;