..and reinventing themselves. It's as if I can take any random file in the solution today and find parts of the CLR reimplemented. Some more totally pointless gems: public static List<T> GetList<T>(params T[ items) { List<T> list = new List<T>(items.Length); list.AddRange...

This is really elementary stuff, but I acually said "WTF" (in longhand... voice?) when I saw it during a routine peer code review, so naturally, I had to immediately post it here. if ($this->hasSomeCondition){ $query = "SELECT * from myTable WHERE itemId = '$itemId'"; ...

Just stumbled over something ugly in our code base: void SomeClass::reset() { memset((char*)this, 0x00, sizeof(SomeClass)); timestamp = 0.0; datestamp = 0; // some more initialization } This method is called by the constructor and in at least 2 other places in the code. If it wasn't followed by initialization...