metaclass: (Default)
metaclass ([personal profile] metaclass) wrote2007-11-16 05:41 pm

Жыдоиндусский Мелкософт

кусочек кода примера с их сайта:

if(message.GetType() == typeof(SoapServerMessage))


Никто не знает, почему они не написали так:

if(message is SoapServerMessage)

[identity profile] metaclass.livejournal.com 2007-11-16 08:45 pm (UTC)(link)
Проблема потенциальная - такой код может сломаться, если кто-нибудь унаследуется от SoapServerMessage и впихнет экземпляр наследника при посылке. Мне такая практика кажется вполне легитимной. Второй вариант так же читабельнее.
И вообще, если автор кода хочет знать, что это именно тот класс что он хочет, а не его наследник - у автора очевидные змеи в голове, ибо это нарушение ООП, согласно канонам которого, наследники должно быть можно подсовывать всюду, где нужны базовые классы.