ISO C90 zabrania mieszanych deklaracji i kodu w C

Question 1

Zadeklarowałem zmienną w ten sposób:

int i = 0;

Otrzymuję ostrzeżenie:

ISO C90 zabrania mieszanych deklaracji i kodu

Jak mogę to naprawić?

Question 2

Myślę, że powinieneś przenieść deklarację zmiennej na początek bloku. To znaczy

{
    foo();
    int i = 0;
    bar();
}

do

{
    int i = 0;
    foo();
    bar();
}

Question 3

Aż do standardu C99 wszystkie deklaracje musiały występować przed jakimikolwiek instrukcjami w bloku:

void foo()
{
  int i, j;
  double k;
  char *c;

  // code

  if (c)
  {
    int m, n;

    // more code
  }
  // etc.
}

C99 pozwala na mieszanie deklaracji i instrukcji (jak C ++). Wiele kompilatorów nadal domyślnie C89, a niektóre kompilatory (takie jak Microsoft) nie obsługują C99 w ogóle .

Musisz więc wykonać następujące czynności:

Sprawdź, czy Twój kompilator obsługuje C99 lub nowszy; jeśli tak, skonfiguruj go tak, aby kompilował C99 zamiast C89;
Jeśli Twój kompilator nie obsługuje C99 lub nowszego, musisz albo znaleźć inny kompilator, który go obsługuje, albo przepisać kod tak, aby wszystkie deklaracje znajdowały się przed wszelkimi instrukcjami w bloku.

Question 4

Po prostu użyj kompilatora (lub podaj mu potrzebne argumenty), tak aby kompilował się dla nowszej wersji standardu C, C99 lub C11. Np. Dla rodziny kompilatorów GCC tak będzie -std=c99.

Question 5

Upewnij się, że zmienna znajduje się w górnej części bloku, a na wypadek, gdybyś ją kompilował -ansi-pedantic, upewnij się, że wygląda następująco:

function() {
    int i;
    i = 0;

    someCode();
}

Question 6

Aby zdiagnozować, co naprawdę powoduje błąd, najpierw spróbuję usunąć = 0

Jeśli błąd zostanie wyzwolony, najprawdopodobniej deklaracja nastąpi po kodzie.
Jeśli nie ma błędu, może to być związane z flagami wymuszania / kompilacji standardu C LUB ... czymś innym.

W każdym razie zadeklaruj zmienną na początku bieżącego zakresu. Możesz następnie zainicjować go osobno. Rzeczywiście, jeśli ta zmienna zasługuje na swój własny zakres - ogranicz jej definicję w {}.

Gdyby PO mógł wyjaśnić kontekst, nastąpiłaby bardziej ukierunkowana reakcja.

Question 7

-Wdeclaration-after-statement minimalny powtarzalny przykład

main.c

#!/usr/bin/env bash

set -eux

cat << EOF > main.c
#include <stdio.h>

int main(void) {
    puts("hello");
    int a = 1;
    printf("%d\n", a);
    return 0;
}
EOF

Daj ostrzeżenie:

gcc -std=c89 -Wdeclaration-after-statement -Werror main.c
gcc -std=c99 -Wdeclaration-after-statement -Werror main.c
gcc -std=c89 -pedantic -Werror main.c

Nie ostrzegaj:

gcc -std=c89 -pedantic -Wno-declaration-after-statement -Werror main.c
gcc -std=c89 -Wno-declaration-after-statement -Werror main.c
gcc -std=c99 -pedantic -Werror main.c
gcc -std=c89 -Wall -Wextra -Werror main.c
# https://stackoverflow.com/questions/14737104/what-is-the-default-c-mode-for-the-current-gcc-especially-on-ubuntu/53063656#53063656
gcc -pedantic -Werror main.c

Ostrzeżenie:

main.c: In function ‘main’:
main.c:5:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
     int a = 1;
     ^~~

Testowano na Ubuntu 16.04, GCC 6.4.0.

Answer 1

83

Zadeklarowałem zmienną w ten sposób:

int i = 0;

Otrzymuję ostrzeżenie:

ISO C90 zabrania mieszanych deklaracji i kodu

Jak mogę to naprawić?

c variables Peter Mortensen
źródło

2

Brzmi jak GCC -pedanticlub -std=c89, w takim przypadku, możesz skompilować, -std=gnu99jeśli chcesz.

Dietrich Epp

3

Jeśli używasz gcc, możesz określić użycie C99, który zezwala na mieszane deklaracje i kod (flaga -std=c99).

hmjd

11

Jedną z opcji byłoby zaprzestanie kompilowania do standardu, który ma 22 lata, a nawet zastąpiono go zamiennikiem.

Stephen Canon

5

@StephenCanon, powiedz o tym firmie Microsoft. :)

hmjd

2

@hmjd: Microsoft nie jest zainteresowany dostarczaniem kompilatora C; to prawdopodobnie się nie zmieni. Na szczęście istnieje wiele doskonale dobrych kompilatorów, których celem jest platforma dostarczana przez innych dostawców.

Stephen Canon

Answer 2

2

Brzmi jak GCC -pedanticlub -std=c89, w takim przypadku, możesz skompilować, -std=gnu99jeśli chcesz.

Dietrich Epp

Answer 3

3

Jeśli używasz gcc, możesz określić użycie C99, który zezwala na mieszane deklaracje i kod (flaga -std=c99).

hmjd

Answer 4

11

Jedną z opcji byłoby zaprzestanie kompilowania do standardu, który ma 22 lata, a nawet zastąpiono go zamiennikiem.

Stephen Canon

Answer 5

5

@StephenCanon, powiedz o tym firmie Microsoft. :)

hmjd

Answer 6

2

@hmjd: Microsoft nie jest zainteresowany dostarczaniem kompilatora C; to prawdopodobnie się nie zmieni. Na szczęście istnieje wiele doskonale dobrych kompilatorów, których celem jest platforma dostarczana przez innych dostawców.

Stephen Canon

Answer 7

127

Myślę, że powinieneś przenieść deklarację zmiennej na początek bloku. To znaczy

{
    foo();
    int i = 0;
    bar();
}

do

{
    int i = 0;
    foo();
    bar();
}

Johan Kotlinski
źródło

5

... lub otwórz nowy kontekst:{foo(); {int i=0; bar();}}

alk

3

@alk s / context / block Tak to się nazywa w standardzie C.

Jens,

@Johan Kotlinski Ale dlaczego to ma znaczenie?

ocean800

1

@ ocean800 Specyfikacja C90 mówi, że nie możesz mieć deklaracji po nie-deklaracjach w tym samym bloku.

MM

Answer 8

5

... lub otwórz nowy kontekst:{foo(); {int i=0; bar();}}

alk

Answer 9

3

@alk s / context / block Tak to się nazywa w standardzie C.

Jens,

Answer 10

@Johan Kotlinski Ale dlaczego to ma znaczenie?

ocean800

Answer 11

1

@ ocean800 Specyfikacja C90 mówi, że nie możesz mieć deklaracji po nie-deklaracjach w tym samym bloku.

MM

Answer 12

Aż do standardu C99 wszystkie deklaracje musiały występować przed jakimikolwiek instrukcjami w bloku:

void foo()
{
  int i, j;
  double k;
  char *c;

  // code

  if (c)
  {
    int m, n;

    // more code
  }
  // etc.
}

C99 pozwala na mieszanie deklaracji i instrukcji (jak C ++). Wiele kompilatorów nadal domyślnie C89, a niektóre kompilatory (takie jak Microsoft) nie obsługują C99 w ogóle .

Musisz więc wykonać następujące czynności:

Sprawdź, czy Twój kompilator obsługuje C99 lub nowszy; jeśli tak, skonfiguruj go tak, aby kompilował C99 zamiast C89;
Jeśli Twój kompilator nie obsługuje C99 lub nowszego, musisz albo znaleźć inny kompilator, który go obsługuje, albo przepisać kod tak, aby wszystkie deklaracje znajdowały się przed wszelkimi instrukcjami w bloku.

Answer 13

11

Po prostu użyj kompilatora (lub podaj mu potrzebne argumenty), tak aby kompilował się dla nowszej wersji standardu C, C99 lub C11. Np. Dla rodziny kompilatorów GCC tak będzie -std=c99.

Jens Gustedt
źródło

6

Ta odpowiedź jest w najlepszym razie niepełna. Naprawia problem, ale nie wyjaśnia, co go spowodowało. Ta odpowiedź wyklucza sytuacje, w których nowszy kompilator nie jest możliwy (kilka możliwych powodów ...) lub środowiska kompilacji, w których przełączanie opcji na kompilator w ten sposób nie jest możliwe. Głos negatywny, ponieważ nie mogę się zgodzić, to jest „prawdziwa” odpowiedź.

Andrew Falanga

1

W niektórych przypadkach -std=gnu89jest używany podczas kompilacji modułów jądra Linuksa.

cbix

Answer 14

6

Ta odpowiedź jest w najlepszym razie niepełna. Naprawia problem, ale nie wyjaśnia, co go spowodowało. Ta odpowiedź wyklucza sytuacje, w których nowszy kompilator nie jest możliwy (kilka możliwych powodów ...) lub środowiska kompilacji, w których przełączanie opcji na kompilator w ten sposób nie jest możliwe. Głos negatywny, ponieważ nie mogę się zgodzić, to jest „prawdziwa” odpowiedź.

Andrew Falanga

Answer 15

1

W niektórych przypadkach -std=gnu89jest używany podczas kompilacji modułów jądra Linuksa.

cbix

Answer 16

4

Upewnij się, że zmienna znajduje się w górnej części bloku, a na wypadek, gdybyś ją kompilował -ansi-pedantic, upewnij się, że wygląda następująco:

function() {
    int i;
    i = 0;

    someCode();
}

Ron Nuni
źródło

Nadal możesz zainicjować zmienną, po prostu nie możesz umieścić kodu przed jego zadeklarowaniem.

SS Anne

Answer 17

Nadal możesz zainicjować zmienną, po prostu nie możesz umieścić kodu przed jego zadeklarowaniem.

SS Anne

Answer 18

Aby zdiagnozować, co naprawdę powoduje błąd, najpierw spróbuję usunąć = 0

Jeśli błąd zostanie wyzwolony, najprawdopodobniej deklaracja nastąpi po kodzie.
Jeśli nie ma błędu, może to być związane z flagami wymuszania / kompilacji standardu C LUB ... czymś innym.

W każdym razie zadeklaruj zmienną na początku bieżącego zakresu. Możesz następnie zainicjować go osobno. Rzeczywiście, jeśli ta zmienna zasługuje na swój własny zakres - ogranicz jej definicję w {}.

Gdyby PO mógł wyjaśnić kontekst, nastąpiłaby bardziej ukierunkowana reakcja.

Answer 19

-Wdeclaration-after-statement minimalny powtarzalny przykład

main.c

#!/usr/bin/env bash

set -eux

cat << EOF > main.c
#include <stdio.h>

int main(void) {
    puts("hello");
    int a = 1;
    printf("%d\n", a);
    return 0;
}
EOF

Daj ostrzeżenie:

gcc -std=c89 -Wdeclaration-after-statement -Werror main.c
gcc -std=c99 -Wdeclaration-after-statement -Werror main.c
gcc -std=c89 -pedantic -Werror main.c

Nie ostrzegaj:

gcc -std=c89 -pedantic -Wno-declaration-after-statement -Werror main.c
gcc -std=c89 -Wno-declaration-after-statement -Werror main.c
gcc -std=c99 -pedantic -Werror main.c
gcc -std=c89 -Wall -Wextra -Werror main.c
# https://stackoverflow.com/questions/14737104/what-is-the-default-c-mode-for-the-current-gcc-especially-on-ubuntu/53063656#53063656
gcc -pedantic -Werror main.c

Ostrzeżenie:

main.c: In function ‘main’:
main.c:5:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
     int a = 1;
     ^~~

Testowano na Ubuntu 16.04, GCC 6.4.0.

ISO C90 zabrania mieszanych deklaracji i kodu w C

Odpowiedzi: