re: A month of clean code VIEW POST

TOP OF THREAD FULL DISCUSSION
re: IMO, it's better to mirror the use of the collection method inside Set::contains() instead of using in_array, since items seems to be a Collection ...
 

I see your suggestion. However, it's important to remember an array is a collection. No need to introduce an entire framework just to use one method.

 

Ah, I see that the other calls are to the Set class size and contains. I misread - I thought it was calling those on items. My bad. You are correct.

code of conduct - report abuse