CodeSOD: Странная сортировка Заметка
The Daily WTF на русском

CodeSOD: Странная сортировка

Скрипт предназначен для вывода всех активных пользователей Active Directory и времени их последнего входа, но использует неэффективный и запутанный подход. Он перебирает каждую букву алфавита для фильтрации учетных записей пользователей, что не гарантирует полной алфавитной сортировки. Для каждой буквы создается новый объект DirectorySearcher, явно запрашивается только свойство 'name', а затем находятся все соответствующие учетные записи.Впоследствии, для каждого полученного имени скрипт создает еще один DirectorySearcher. Этот новый поисковик используется для запроса конкретной учетной записи пользователя с целью получения всех ее свойств. Хотя для поиска по имени пользователя ожидается только один результат, используется метод FindAll(), возвращающий массив.Это требует ненужного цикла для перебора массива из одного элемента. Наконец, выводится имя пользователя и время последнего входа, объединенные запятой. Дизайн скрипта крайне неэффективен из-за повторного создания поисковиков, избыточных запросов и нелогичного метода фильтрации. Несмотря на недостатки, его способность генерировать CSV для менеджеров делает его "критически важным" в организации.