I
прикольная штука, хорошо бы оно еще со спанами работало
Size: a a a
2020 October 21
I
Igor
прикольная штука, хорошо бы оно еще со спанами работало
Ну как форма записи удобно, да. Но реализация сравнения - пиздец
I
switch быстро работает, и мне кажется это будет гораздо быстрее, чем каждое значение целиком проверять
I
я вот кстати давно это знаю (что он быстро работает), только не знаю почему
I
Igor
switch быстро работает, и мне кажется это будет гораздо быстрее, чем каждое значение целиком проверять
Целиком никто не проверяет. Не знаю, сможет ли jit понять что там тупое сравнение памяти или нет, но это будет работать намного медленнее, чем memcmp
I
И его дотнет реализации
I
Не, там простыня
I
В идеале компилятор должен выделить статик массивы и вызывать супер быстрые сравнения через либо Span.SequenceEquals, либо через аналог в Buffer
I
погоди, если у меня там 40 значений, мне придется 40 раз вызвать memcmp?
I
Да, очевидно
I
Сделай 2 версии кода, одну оставь такой, другую сделай через статик массивы в модуле и сделай актив паттерн, который сравнивает массивы, чтобы так же красиво было в записи
I
И используй AsSpan().SequenceEqual
I
Разница будет :)
I
И гоняй бенчмарки
I
да я на строках оставлю, это количество лишнего кода мне нафиг не нужно
I
тем более в этом нет никакого экономического смысла
I
Ну если у тебя приложуха не страдает, то ок)
VS
Но я бы () убрал
а я бы не убирал) потому что где-то есть параметр, где-то нету, единообразие лучше читается
I
И используй AsSpan().SequenceEqual
я почитал, пишут что выгода начинается с длины 8 https://github.com/dotnet/runtime/issues/32363.
Наверное в каких-то сценариях выиграет, но весь этот код придется написать руками. Лучше оставить понятный код
Наверное в каких-то сценариях выиграет, но весь этот код придется написать руками. Лучше оставить понятный код
VS
Igor
а есть дока что делает B? а то лень гуглить)