This week's book giveaways are in the Refactoring and Agile forums.
We're giving away four copies each of Re-engineering Legacy Software and Docker in Action and have the authors on-line!
See this thread and this one for details.
Win a copy of Re-engineering Legacy Software this week in the Refactoring forum
or Docker in Action in the Cloud/Virtualization forum!
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic

Spacing issue

 
Matthew Phillips
Ranch Hand
Posts: 2676
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
I am very confused on spacing issues. My code is below.

The nitpick was to inlude blank lines between declaring value and the following if statement. I was also told to include blank lines between return value and the following brace. In a previous program I was told not to include blank lines between initializing variable and an if statement and I cannot see a reason for including them after the return value. Would someone please explain when to use blank lines. I have read the style guide and it didn't address this. Thanks.
Matthew Phillips
 
bill bozeman
Ranch Hand
Posts: 1070
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
I can't speak for the nitpicker here, but I think that is a suggestion as opposed to a matter of something required. I don't beleive blank lines is mentioned in the style guide except to use them between methods and to space out large chuncks in methods.
However, it does make it more readable to have identifiers and if statements separated by a blank line. But I think that is a matter of preference.
Correct me if I am wrong.
Bill
 
Marilyn de Queiroz
Sheriff
Posts: 9059
12
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
spacing or blank lines?

"Spacing" refers to spaces on both sides of an identifier, (horizontally) on the same line, not between lines (vertically).

Bill's statements regarding blank lines are correct.
 
Matthew Phillips
Ranch Hand
Posts: 2676
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
I must have misunderstood your comments. When I saw that you added blank lines and wrote spacing I thought you meant for me to add blank lines. Disregard my submission. I will resubmit Leap today.
Matthew Phillips
 
Marilyn de Queiroz
Sheriff
Posts: 9059
12
  • Mark post as helpful
  • send pies
  • Quote
  • Report post to moderator
We nitpickers try to surround our comments with blank lines to make them more noticeable.
 
  • Post Reply
  • Bookmark Topic Watch Topic
  • New Topic