The Daily WTF на русском
Подписаться
Код, который портит день: Считаем все
Сегодня в США день выборов, и подсчет голосов находится в центре внимания многих людей. Код, который обсуждается, не связан с подсчетом голосов, но он все равно проблематичен. Код происходит из старого приложения, выпущенного в 2007 году, и ссылается на ограничение строк Excel в 65 536, которое было увеличено до 1 000 000 в Excel 2007. Однако код на самом деле читает файл CSV, а не файл Excel. Код дважды конструирует имя файла, что является избыточным. Основная проблема заключается в том, как код считает строки, что является неэффективным и работает плохо с большими файлами. Код читает весь файл в память, разделяет его на новые строки и затем считает длину. Этот подход ужасен для больших файлов, которых было много. К сожалению, никто не хочет исправлять код из-за правила "не трогай старый код" для старого, устаревшего кода. Вместо этого руководство обещает запустить новый проект замены, но на данный момент код остается неисправленным.