I just wrote a code review comment that I feel particularly smug about.
The code I was reviewing:
class PointClouds ...
My comment:
It is a bad idea to have a type name in plural. If you need to signify plurality, use a plurality-signifying suffix in singular form, as in:
- PointCloudCollection
- PointCloudSet
- PointCloudGroup
- or go creative with PointCloudFormation
- or take creativity to a whole new level with PointCumulonimbus
Anything but a plural type name.
I have never heard of Cumulonimbus before but that sounds cool, going to use this for all my classes from now on
ReplyDelete(/s) This is a good one! Sound advice in multiple ways 😂
ReplyDelete