Czy (PRAWDA) dobry pomysł w C?

Question

W języku programowania C rozumiem, że zmienne mogą być zdefiniowane tylko na początku bloku kodu, a zmienna będzie miała zakres bloku, w którym została zadeklarowana. Z tą myślą zastanawiałem się, czy uważa się złą praktyką sztucznie utworzyć nowy zakres, jak w poniższym przykładzie:

void foo() 
{ 
    ... Do some stuff ... 

    if(TRUE) 
    { 
     char a; 
     int b; 

     ... Do some more stuff ... 
    } 

    ... Do even more stuff ... 
} 

Zakładając PRAWDA jest ustawiony na 1 w definicji makra, to kod ten należy uznać za „dobry kod”, czy też zrobić programiści doświadczeni warować na samą myśl o tym?

Dzięki za twój wkład!

EDYCJA: W odpowiedzi na niektóre z odpowiedzi, kod, nad którym pracuję, musi działać z dość starymi, starszymi systemami. Chociaż dobrze byłoby działać na założeniu C99, nie możemy zagwarantować, że będą go mieli.

Answer 1

Nie potrzebujesz nawet instrukcji if. Możesz tworzyć bloki za pomocą {}

Prawdopodobnie powinna to być jednak osobna funkcja.

Przykład tutaj:

#include <stdio.h> 

int 
main(int argc, char **argv) { 
    int i = 0; 
    { 
     int i = 10; 
     printf("%d\n", i); 
    } 
    printf("%d\n", i); 
} 

Answer 2

Prawdopodobnie chcesz utworzyć nową funkcję dla tego zakresu.
Jeśli naprawdę musi mieć swój własny zakres, to prawdopodobnie jest to osobna funkcja logiczna.

Answer 3

Nie nazwałbym się doświadczonym, ale jestem trochę skulony.

Mój problem polega na tym, że instrukcja if sprawi, że ktoś uwierzy, że coś jest aktualnie analizowane ... ale w czasie wykonywania makro jest prawdziwe lub fałszywe, nie ma zmiany tego, że jest czymś innym. Powinieneś albo podać kod, albo nie.

Jeśli to, co masz na myśli, aby zrobić coś jak #ifdef DEBUG to należy zrobić, aby wskazać, że jest to czytnik kodu debug ...

Answer 4

Nie sądzę, trzeba if (true) część.

Tylko {} są wymagane do zmiany zakresu zmiennych.

Answer 5

Moja odpowiedź jest następująca:

To sprawia, że ​​programiści doświadczeni warować na samą myśl o tym.

Answer 6

Ponieważ możesz po prostu utworzyć blok zasięgu bez znaku, jeśli, to jest lepszy pomysł.

void foo() { 
    ... Do some stuff ... 
    { 
     char a; 
     int b; 
     ... Do some more stuff ... 
    } 
    ... Do even more stuff ... 
} 

Answer 7

O ile wiem, można utworzyć zakres bez.

używać tylko szelki takie jak ten:

{ 
    int x; 

} 

A ja odradzam

if (TRUE) 

ponieważ utrudnia czytelność.

Answer 8

można usunąć

if(TRUE) 

i po prostu zostawić szelki, które same określają nową składniowej zablokuj - do compound statement.

Jest to zdecydowanie czystsze niż fałsz, jeśli wcześniej, ale nadal możesz zadać sobie pytanie, dlaczego chcesz utworzyć nowy blok - czy byłoby lepiej zdefiniować podprogram?

Answer 9

Po pierwsze, nowy blok nie musi być blokiem if. Może to być po prostu fragment kodu otoczony szelek, tak:

void foo() { 
... Do some stuff ... 

{ 
    char a; 
    int b; 

    ... Do some more stuff ... 
} 

... Do even more stuff ... 
} 

Po drugie, w każdym nowoczesnym kompilator C, który jest zgodny ze standardem C (myślę, C99), można deklarować zmienne w dowolnym miejscu na bloku, więc nie musisz w ogóle tworzyć nowego bloku.

Answer 10

zostawiając drzwi otwarte dla niektórych ludzi twórczych:

#define TRUE 0 
#define FALSE 1 

Wystarczy użyć szelki zadeklarować zakres.

Answer 11

C99 pozwala zadeklarować zmienne prawie w dowolnym miejscu. Powstrzymaj się jednak od tego bez ważnego powodu. Spróbuj najpierw podzielić swoją funkcję na mniejsze (prawdopodobnie inline) funkcje.

Jedynym miejscem, w którym takie coś może mieć sens, jest sytuacja, gdy masz zmienną zainicjowaną w środku funkcji, np. podobne do tworzenia obiektu w C++.

Answer 12

Myślę, że pracujesz z pewnymi nieaktualnymi założeniami. Od kilku miesięcy koduję proste C używając GCC i nie musisz deklarować zmiennych na początku bloku, mimo że druga edycja K & R mówi, że musisz. Można zadeklarować zmienną w dowolnym miejscu, podobnie jak to niezbyt użyteczny przykład:

char* palstring; 
palstring = malloc(LARGEST_STRING); 
memset(palstring, 0, sizeof palstring); 
fgets(palstring, LARGEST_STRING, fin); 

char* cur = palstring; 
char letter; 
letter = *cur; 

Więc nie ma potrzeby, aby robić to, co sugerujesz. Język się przeniósł.

Innym dobrym dodatkiem do języka C są tablice o zmiennej długości, które pozwalają przekazać tablicę do funkcji wraz z jej rozmiarem. W dawnych czasach wszystko, co mogłeś zrobić, to przekazać wskaźnik.

Answer 13

Jak już powiedzieliśmy w wielu odpowiedziach, nie potrzebujesz rzeczy "jeśli". Po prostu utwórz goły blok. Ale chcę dostać w innym punkcie. W języku C można tworzyć deklaracje zmiennych w dowolnym miejscu bloku, nie tylko na początku. W C89 masz to ograniczenie. Począwszy od C99 (to już 10 lat), nie masz już tego ograniczenia, chociaż niektóre kompilatory i tak będą jęczeć. GCC nie będzie jednak, jeśli powiesz mu, aby używał najbardziej "ostatniego" C Standardu z opcją -std = c99.

Ponieważ w dalszym ciągu istnieją kompilatory, które domyślnie narzekają, nie chciałbym jednak mieszać deklaracji i kodu. Nadal składałem deklaracje na początku bloków ze względu na kompatybilność.

Answer 14

Należy zauważyć, że w C99 dopuszczalne jest deklarowanie lokalnych zmiennych w środku bloków.

C99 to wersja standardu C z 1999 roku; obsługuje go większość nowoczesnych kompilatorów języka C.

Answer 15

Zakładając, że używasz starego kompilatora (tak jak ja, to dla starego sprzętu), pomijasz jeśli (PRAWDA), jak sugerowali inni, i że masz naprawdę OGROMNĄ funkcję (nie powinieneś mieć w pierwsze miejsce), wtedy myślę, że jest w porządku. Zrobiłem to, ale nie czułem się dobrze ...

Answer 16

Podobno jestem w mniejszości, ale uważam, że "tylko aparaty ortodontyczne" są znacznie trudniejsze do odczytania, ponieważ odbiegają od typowego schematu. Na (co prawda rzadko) okazjach gdzie chcę scoped blok bez definiowania inną funkcję, wolę zawierać „jeśli”, ale bez makr i wraz z komentarzem, aby wyjaśnić dlaczego:

if(1) // just to establish scope 
{ 
    // do stuff here 
} 

Answer 17

Wystarczy zdefiniować zmienne na początku bloku lub użyj innej funkcji. Dodanie sztucznego zakresu za pomocą pustych {} s lub dowolnej alternatywy nie jest dobrą praktyką.