3 ms·
The function is not very long and it's sort of pointless. Besides, what do you name it? collectionInts? That's in the declaration, right there. candidates?
by krinchan 10y ago
The function is not very long and it's sort of pointless. Besides, what do you name it? collectionInts? That's in the declaration, right there. candidates? As obtuse (and useless) as c.
If you're grepping/ag'ing for c you've got another issue, because I can't think of the context that you would grep c in a function that fits on less than 2 24-row terminal screens. At worst, /c and a few presses of 'n' is all you need once you get to the function.
If you're outside the function grepping, it's because you saw printNFirstIntegers(4, someOtherVariableName). You'll hardly have named the variable (or collection literal) "c". So you'll be using ag printNFirstIntegers.
To me, your request is akin to the last time I had a code review for a rake task and someone complained about the 'f' in:
FileList['deploy_assets/ebextensions/*'] do |f|
@helper.process_template(f, ".ebextensions/#{File.basename(f)}")
end
Calling it File is horribly obvious and annoying.
Edit: The function name probably should be run_erb_on or something like that. None of us are free of naming sins. Lol.