Стоит ли использовать блоки кода для ограничения области видимости?

Question

[MaxShell]

Решил почитать детали языка C и наткнулся на следующее:

#include <stdio.h>

int main() {
    {
        char* msg = "Hello world!";
        puts(msg); // Работать будет
    }
    puts(msg); // Работать не будет
}

Вопросы такие: Стоит ли использовать такие блоки кода для ограничения области видимости? Если стоит, то где их можно применить.

Comments

[res2001]

Стоит/не стоит - каждый решает сам для себя.
Если функция из 3 строчек, как в примере, то в этом смысла нет.
Альтернатива блокам - блок кода выделить в функцию. Компилятор ее заинлайнит и эффект с областью видимости будет такой же. Чаще предпочитаю этот вариант, но иногда бывает и блоки использую, если блок прям ну очень короткий 1-2 строки и больше не повторяется.
Но самый частый вариант - забить на дополнительные скобки и писать в одном блоке кода все что нужно - при выходе из функции все равно все стековые переменные уничтожатся. Уничтожаться они на пару строк раньше или позже, обычно роли не играет. Но всегда могут быть исключения.
Кстати, далеко не факт, что компилятор вообще вставит хоть какие-то команды на закрывающей скобке, по идее он должен поменять регистр указателя стека, но так же он может счесть это не нужным. Но проверить, что переменная вышла из области видимости он не забудет в любом случае.

[Adamos]

Ограничения области видимости внутри одной функции нужны только для того, чтобы случайно не использовать в двух местах одну и ту же переменную. Адекватное наименование переменных решает эту проблему эффективнее.
Например, если вместо msg назвать ее resultMessage - вряд ли где-то еще в коде еще появится переменная с таким именем.

[Дмитрий]

Adamos, если не страдать копипастой)

Answer 1

[CityCat4]

Вкусовщина, имхо. Может понадобиться только тогда, когда тебе вот позарез захотелось назвать переменную так, но у тебя уже есть другая переменная с таким именем.

Comments

[Дмитрий]

Больше путаницы будет, как по мне

[CityCat4]

Дмитрий, Да по мне тоже, я так никогда не делаю :)

Answer 2

[Дмитрий]

Код нужно писать так, чтобы его было удобно читать! Основная задача кода — быть прочитанным человеком, в первую очередь, а потом уже компилятором.
Эти скобочки вносят колоссальный мусор и шум в беглое чтение, при том, что реально без них обойтись можно в 99.999% случаев.

Единственный, мне известный случай, когда это реально надо — в низкоуровневом программировании драйвера или ядра своей ОС. Когда ты знаешь захардкоженное значение размера стека и тебе нужно в одной функции выделить что-то размером в почти весь твой стек, что-то с этим значением сделать и резко сразу же его очистить, чтобы не стек уплыл в оверфлоу при использовании ниже. Тогда можно воспользоваться этим хаком. В других случаях адекватный человек этим пользоваться не будет.

Answer 3

[Mercury13]

Иногда, чтобы прикрыть теоретическую возможность важной ошибки.
Чаще нужно в Си++ с его автодеструкторами.
На чтение особо не влияет.

Вот пример на Си…

{ char* buf = calloc(n, sizeof(char));  // Ради чего я создал блок — ставлю в той же строке
    snprintf(buf, n, some_shit);
    strcpy(dest, buf);
    free(buf);
}

Ну а это чистые «кресты».

{ std::ofstream os(fname);
   os << someData;
}  // Тут файл закрыт