vak: (Default)
Serge Vakulenko ([personal profile] vak) wrote2020-04-11 02:39 pm

Си++ вам не Си

В частности, передавать массивы в качестве параметров лучше всего по ссылке:
void foo(int (&array)[10])
{
    //...
}

Такую процедуру не удастся вызвать с параметром - массивом другого, неподходящего размера.
Если же написать, как в Си:
void bar(int array[10])
{
    //...
}

то такой параметр фактически обрабатывается как "int *array", и никакой проверки размера массива не делается.
ircicq: (Default)

[personal profile] ircicq 2020-04-11 10:11 pm (UTC)(link)
Рекомендуется всё же форма: std::array<int, 10>
norian: (Default)

[personal profile] norian 2020-04-11 10:30 pm (UTC)(link)
этот темплейтный класс с кучей потрохов вообще низачем не нужен

вот что не рекомендуецца так это числовые константы в коде, их лучше определять в топовом хедере как const size_t myArrayLimit = 10; и не искать потом по всему коду
spamsink: (Default)

[personal profile] spamsink 2020-04-12 02:02 am (UTC)(link)
Если процедура умеет только один конкретный тип данных, то его надо один раз назвать именованным классом и не мучиться с магическими константами. А в начале процедуры, если очень лень писать имя поля, можно написать и что-то типа int (&array)[] = arg.array; - тип уже проверен.
juan_gandhi: (Default)

[personal profile] juan_gandhi 2020-04-12 02:06 am (UTC)(link)
Ой как интересно нынче. А 10 - это только литерал может быть? А константа? А что-нибудь вычисляемое?
ircicq: (Default)

[personal profile] ircicq 2020-04-12 02:28 am (UTC)(link)
Да, в C++ сейчас есть constexpr функции
Их можно вычислять в Compile time

[personal profile] sergegers 2020-04-12 04:36 am (UTC)(link)
Лучше void foo(std::span<int, 10>), но всё это синтаксический сахар.
Edited 2020-04-12 04:37 (UTC)

[personal profile] sergegers 2020-04-12 08:13 pm (UTC)(link)
Он обслуживает и C array и std::array и динамические массивы. Причём статические массивы с проверкой в compile time.
dluciv: (Default)

[personal profile] dluciv 2020-04-12 07:19 am (UTC)(link)
Дополню: в Си не делается для старшего индекса, который для одномерного массива единственный. У многомерного массива все размерности, кроме старшей, являются частью типа, иначе его просто индексировать не получится.
pappadeux: (Default)

[personal profile] pappadeux 2020-04-13 03:04 am (UTC)(link)
всегда считал, что нет многомерных массивов
fenikso: (Default)

[personal profile] fenikso 2020-04-14 06:15 am (UTC)(link)
Для передачи массива как параметра в варианте с неизвестным заранее размером массива еще бывает полезен absl::Span (https://abseil.io/tips/93), но это когда можно дополнительные библиотеки подключать. Еще удобен тем что в функцию которая принимает absl:Span, можно передать вектор и т.п.
vlkamov: Рембрандт. Автопортрет с широко открытыми глазами. (Default)

[personal profile] vlkamov 2020-04-15 03:38 pm (UTC)(link)
Мало что понял, особенно комментарии, но вот здесь
> void bar(int array[10])
получается, что 10 - буквально ни о чем. Что с таким же успехом можно ставить 20, 2
Так ?

pappadeux: (Default)

[personal profile] pappadeux 2020-04-16 04:53 am (UTC)(link)
Да

массив становится указателем на 1й елемент

https://opensourceforu.com/2016/09/decayintopointers/
vlkamov: Рембрандт. Автопортрет с широко открытыми глазами. (Default)

[personal profile] vlkamov 2020-04-16 06:59 am (UTC)(link)
Howdo - это само собой.

Просто я люблю, чтобы код был читабелен даже козе, например
https://vlkamov.livejournal.com/685130.html

Если массив всегда из 10 элементов, то 10-ка должна быть прописана в коде функции.
Тогда ею вообще не следует захламлять вызов и объявление, например

void bar(int array[])
или для ясности
void bar(int array_10[])

Но если массив то там то сям разной длины, функции следует сообщить, сколько чего, например
void bar(int array[], 10)
...
void bar(int array[], 22)

Хотя не уверен, что компилятор напрямую скушает такие конструкции.
Edited 2020-04-16 07:05 (UTC)
pappadeux: (Default)

[personal profile] pappadeux 2020-04-17 03:55 am (UTC)(link)
> void bar(int array[10])
> или для ясности
> void bar(int array_10[])

так проблема в том, что человеку, возможно, это и помогает, а железяке - нет. Нет возможности громко закричать если юсер запулил туда массив из 9ти элементов.

> void bar(int array[], 22)


это Фортран с точки зрения С