Author fails to follow own advice:<p><pre><code>  int printFirstNPositive(int maxToPrint, Collection&lt;Integer&gt; c)
</code></pre>
The method name uses &quot;N&quot;, but the param for N is named &quot;maxToPrint&quot;. Confusing and also violates his&#x2F;her wasted space rule.<p>Instead:<p><pre><code>  int printFirstNPositive(int n, Collection&lt;Integer&gt; c)
</code></pre>
I think the short variable names &quot;n&quot; and &quot;c&quot; are fine since they are part of the method signature and have unambiguous meaning, and also because the implementation is so short (so no problem remembering that n and c are the inputs.