Jan. 23rd, 2014

metaclass: (Default)
Заглянул в дебри одного давно существующего проекта, обнаружил кусок кода парсера CSV строк. По номеру поля получает значение поля из строки, разделенной запятыми.
Код работает.
Но он на каждый 1 символ в строке выполняет выделение памяти и копирование строки целиком, за исключением первого символа. Еще он делает подстроку из первого символа и сравнивает ее с разделителем. И это повторяется количество раз, равное номеру поля. А потом он выполняет поиск разделителя в оставшей от предыдущей содомии строки. В итоге, я даже думать не хочу, сколько раз на каждую запись оно долбится в менеджер памяти и трешит кэш проца.
Это все вместо простейшего конечного автомата и получения списка полей за один раз по списку индексов.

Внимание, вопрос: какие меры воздействия на автора этого и подобного ему кода являются этически допустимыми?

Profile

metaclass: (Default)
metaclass

April 2017

S M T W T F S
      1
2345678
9101112 131415
16171819202122
23242526272829
30      

Most Popular Tags

Style Credit

Expand Cut Tags

No cut tags
Page generated Aug. 9th, 2025 03:28 am
Powered by Dreamwidth Studios