Add AC setting to optionally add For loop variable in Next statement when block-completing For...Next

[SmileyFtW]

When auto-completing For Each/Next have the option to have the each object be included in the closing Next:

For Each thing in things
Next thing.

Similarly for With/End With except it would need to be added as a comment.

With thing
End With 'thing

[retailcoder]

IMO if you need to be that explicit, then you either have too much nesting, or a procedure doing too many things. I can see Next thing as a ...thing (it's part of the language syntax and could conceivably be a personal preference, just like an explicit Step 1 might be), but I wouldn't have Rubberduck add "what" comments, when good comments should say "why".

[SmileyFtW]

Thanks! I do the “Next thing” out of a personal preference. I’ve seen enough “simple” code with enough nesting that the syntax would be helpful. The comment thing for With/End With was an add on; I don’t really ever do that… <grin>. Thanks to what I am learning from Rubberduck I am sure much of what I do now will be shed in favor of better code. Many thanks! David From: Mathieu Guindon [mailto:notifications@github.com] Sent: Friday, August 31, 2018 3:02 PM To: rubberduck-vba/Rubberduck Cc: SmileyFtW; Author Subject: Re: [rubberduck-vba/Rubberduck] Autocomplete add options (#4347) IMO if you need to be that explicit, then you either have too much nesting, or a procedure doing too many things. I can see Next thing as a ...thing (it's part of the language syntax and could conceivably be a personal preference, just like an explicit Step 1 might be), but I wouldn't have Rubberduck add "what" comments, when good comments should say <https://codereview.stackexchange.com/a/90113/23788> "why". — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#4347 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AoQ_F_KZA69cqaZHSNI20dQnE-e8Uo68ks5uWZZFgaJpZM4WVx5I> . <https://github.com/notifications/beacon/AoQ_F_IA-PeazJmUfJCsseTPpi4eGKkfks5uWZZFgaJpZM4WVx5I.gif>