Question

This might not be the place for this, and if so I do apologize and ask that you point me in the direction where it may be appropriate for me to get an answer. This is a problem I run into when I code alot.

new Vector2(parent.Coordinates.X + (Position + parent.Prompt.Length - parent.visibilityIndex) * parent.font.Width, parent.Coordinates.Y);

That’s quite a line! So I’ll break it up into two:

new Vector2(parent.Coordinates.X + (Position + parent.Prompt.Length - parent.visibilityIndex) * parent.font.Width, 
            parent.Coordinates.Y);

That’s a little better, but still way too long. Anywhere else I try to break up the line seems arbitrary and to serve to obfuscate the code further. Am I wrong? What do you do? Again, I apologize if this is the wrong place for this as I am not sure.

Answer 1

This is subjective, but in this case I would split it out into separate variables.

var fontWidth = parent.font.Width;
var index = parent.visibilityIndex;
var offset = (Position + parent.Prompt.Length - index) * fontWidth;
return new Vector2(parent.Coordinates.X + offset, parent.Coordinates.Y);

Split it up in whichever way makes the most sense to you.