Enumerable.ToArray #скорость #память
Если мы пишем код по гайдлайнам, то наши методы часто возвращают IEnumerable. Это необходимо для того, чтобы сигнатура метода не изменялись при изменении логики метода. В принципе, очень полезная и весьма здравая мысль.
При получении перечисления из метода, мы часто делаем ToList или ToArray, чтобы получить возможность пробежаться по ней более чем один раз (см. multiple enumerations).
В этой ситуации некоторые программисты допускают логическую ошибку. Они подсматривают код метода, который возвращает перечисление, видят, что за перечислением на самом деле скрывается массив, и делают предположение о том, что если они вызовут метод ToArray, то ничего страшного не произойдёт. Мол, dotnet знает настоящий тип коллекции, которая возвращается из метода, и просто его и вернёт.
Увы, это не так. Если мы посмотрим код, то можно заметить, что при вызове Enumrable.ToArray действительно создаётся новый массив, который и возвращается потребителю.
Можно решить эту проблему собственным методом AsArray, который, я уверен, есть в большом количестве локальных утилит проектов. Он не будет ничего аллоцировать, проверит тип, и, если это действительно массив, просто вернёт его. Скорость сильно выше, никаких дополнительных аллокаций не нужно. На больших массивах экономия будет 'https://t.me/csharp_gepard/118?comment=1191' rel='nofollow'>внушительной. Казалось бы, win-win.
Но нет. И это вторая логическая ошибка.
P.S.: Александр, спасибо!
Если мы пишем код по гайдлайнам, то наши методы часто возвращают IEnumerable. Это необходимо для того, чтобы сигнатура метода не изменялись при изменении логики метода. В принципе, очень полезная и весьма здравая мысль.
При получении перечисления из метода, мы часто делаем ToList или ToArray, чтобы получить возможность пробежаться по ней более чем один раз (см. multiple enumerations).
В этой ситуации некоторые программисты допускают логическую ошибку. Они подсматривают код метода, который возвращает перечисление, видят, что за перечислением на самом деле скрывается массив, и делают предположение о том, что если они вызовут метод ToArray, то ничего страшного не произойдёт. Мол, dotnet знает настоящий тип коллекции, которая возвращается из метода, и просто его и вернёт.
Увы, это не так. Если мы посмотрим код, то можно заметить, что при вызове Enumrable.ToArray действительно создаётся новый массив, который и возвращается потребителю.
Можно решить эту проблему собственным методом AsArray, который, я уверен, есть в большом количестве локальных утилит проектов. Он не будет ничего аллоцировать, проверит тип, и, если это действительно массив, просто вернёт его. Скорость сильно выше, никаких дополнительных аллокаций не нужно. На больших массивах экономия будет 'https://t.me/csharp_gepard/118?comment=1191' rel='nofollow'>внушительной. Казалось бы, win-win.
Но нет. И это вторая логическая ошибка.
P.S.: Александр, спасибо!