Работа со строками Опции

[Jenya7]

Тема навеяна гневными диффирамбами уважаемого jcxz. Вернемся к тем же баранам

Код

char *s_MainMenu[] =
{
    "MAINT ",
    " AUTO ",
    "MANUAL",
    "FAILS ",
    "MUXBUS"
};

void set_display_state(uint8_t state_val, uint8_t state_maxind, char *words[])
{
    g_DisplayStates.MenuState = state_val;
    g_DisplayStates.MaxStringIndex = state_maxind;
    g_DisplayStates.WordstoDisplay = words;
}
set_display_state(MAIN_STATE,MAX_MAIN_INDEX, s_MainMenu);

я так понял это плохо. а так хорошо?

Код

const char *s_MainMenu[] =
{
    "MAINT ",
    " AUTO ",
    "MANUAL",
    "FAILS ",
    "MUXBUS"
};

void set_display_state(uint8_t state_val, uint8_t state_maxind, const char *words[])

Сообщение отредактировал Jenya7 - Jul 24 2018, 12:25

[Сергей Борщ]

"гневные дифирамбы" - это что-то вроде "сухая вода". В который раз поражаюсь узости вашего кругозора.

а так хорошо?

Да. Если ваша функция принимает значение по указателю и не изменяет значение по этому указателю - всегда нужно объявлять параметр как указатель на константу. Это и вас предохранит от случайного изменения значения, и компилятору даст больше свободы в оптимизации.
А еще лучше будет сделать const char * const s_MainMenu[] = ... Подумайте, почему.

[Jenya7]

"гневные дифирамбы" - это что-то вроде "сухая вода". В который раз поражаюсь узости вашего кругозора.
Да. Если ваша функция принимает значение по указателю и не изменяет значение по этому указателю - всегда нужно объявлять параметр как указатель на константу. Это и вас предохранит от случайного изменения значения, и компилятору даст больше свободы в оптимизации.
А еще лучше будет сделать const char * const s_MainMenu[] = ... Подумайте, почему.

я понял. спасибо.

[jcxz]

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

Точно!
Я и член-функции классов, не изменяющие членов класса, тоже в обязательном порядке объявляю с модификатором const. Именно по этом-же причинам.