Что не так с кодом C 1988 года?

Я пытаюсь скомпилировать этот фрагмент кода из книги «Язык программирования C» (K&R). Это простая версия программы UNIX wc:

#include <stdio.h>

#define IN   1;     /* inside a word */
#define OUT  0;     /* outside a word */

/* count lines, words and characters in input */
main()
{
    int c, nl, nw, nc, state;

    state = OUT;
    nl = nw = nc = 0;
    while ((c = getchar()) != EOF) {
        ++nc;
        if (c == '\n')
            ++nl;
        if (c == ' ' || c == '\n' || c == '\t')
            state = OUT;
        else if (state == OUT) {
            state = IN;
            ++nw;
        }
    }
    printf("%d %d %d\n", nl, nw, nc);
}

И я получаю следующую ошибку:

$ gcc wc.c 
wc.c: In function ‘main’:
wc.c:18: error: ‘else’ without a previous ‘if’
wc.c:18: error: expected ‘)’ before ‘;’ token

Второе издание этой книги датировано 1988 годом, и я новичок в C. Может быть, это связано с версией компилятора, а может, я просто несу чушь.

Я видел в современном коде C другое использование mainфункции:

int main()
{
    /* code */
    return 0;
}

Это новый стандарт или я все еще могу использовать основной без типов?

Ваша проблема связана с определениями препроцессора INи OUT:

#define IN   1;     /* inside a word */
#define OUT  0;     /* outside a word */

Обратите внимание, что у вас есть точка с запятой в конце каждого из них. Когда препроцессор расширяет их, ваш код будет выглядеть примерно так:

    if (c == ' ' || c == '\n' || c == '\t')
        state = 0;; /* <--PROBLEM #1 */
    else if (state == 0;) { /* <--PROBLEM #2 */
        state = 1;;

Эта вторая точка с запятой приводит к elseтому, что предыдущее ifсовпадение отсутствует, потому что вы не используете фигурные скобки. Итак, удалите точки с запятой из определений препроцессора INи OUT.

Урок, усвоенный здесь, состоит в том, что операторы препроцессора не должны заканчиваться точкой с запятой.

Кроме того, всегда следует использовать скобки!

    if (c == ' ' || c == '\n' || c == '\t') {
        state = OUT;
    } else if (state == OUT) {
        state = IN;
        ++nw;
    }

В приведенном elseвыше коде нет неоднозначности.

Основная проблема с этим кодом в том, что это не код от K&R. Он включает точки с запятой после определений макросов, которых не было в книге, что, как указывали другие, изменяет значение.

За исключением случаев внесения изменений в попытке понять код, вы должны оставить его в покое, пока вы его не поймете. Вы можете безопасно изменять только тот код, который вам понятен.

Вероятно, это была просто опечатка с вашей стороны, но она иллюстрирует необходимость понимания и внимания к деталям при программировании.

После макроса не должно быть точек с запятой,

#define IN   1     /* inside a word */
#define OUT  0     /* outside a word */

и это, вероятно, должно быть

if (c == ' ' || c == '\n' || c == '\t')

Определения IN и OUT должны выглядеть так:

#define IN   1     /* inside a word  */
#define OUT  0     /* outside a word */

Проблема была в точках с запятой! Объяснение простое: и IN, и OUT являются директивами препроцессора, по сути, компилятор заменит все вхождения IN на 1 и все вхождения OUT на 0 в исходном коде.

Поскольку в исходном коде после 1 и 0 была точка с запятой, при замене IN и OUT в коде дополнительная точка с запятой после числа приводила к недопустимому коду, например, к этой строке:

else if (state == OUT)

В итоге получилось так:

else if (state == 0;)

Но вы хотели вот этого:

else if (state == 0)

Решение: удалите точку с запятой после чисел в исходном определении.

Как видите, возникла проблема с макросами.

У GCC есть опция остановки после предварительной обработки. (-E) Эта опция полезна, чтобы увидеть результат предварительной обработки. На самом деле этот метод важен, если вы работаете с большой базой кода на c / c ++. Обычно у make-файлов есть цель, которую нужно остановить после предварительной обработки.

Для быстрой справки: вопрос SO охватывает варианты - как мне увидеть исходный файл C / C ++ после предварительной обработки в Visual Studio? . Он начинается с vc ++, но также имеет параметры gcc, упомянутые ниже .

Не совсем проблема, но декларация main()тоже датирована, должно быть примерно так.

int main(int argc, char** argv) {
    ...
    return 0;
}

Компилятор примет возвращаемое значение int для функции без одного, и я уверен, что компилятор / компоновщик будет работать с отсутствием объявления для argc / argv и отсутствием возвращаемого значения, но они должны быть там.

Попробуйте добавить явные скобки вокруг блоков кода. Стиль K&R может быть неоднозначным.

Посмотрите на строку 18. Компилятор сообщает вам, в чем проблема.

    if (c == '\n') {
        ++nl;
    }
    if (c == ' ' || c == '\n' || c == '\t') { // You're missing an "=" here; should be "=="
        state = OUT;
    }
    else if (state == OUT) {
        state = IN;
        ++nw;
    }

Самый простой способ - использовать скобки вроде {} для каждого ifи else:

if (c == '\n'){
    ++nl;
}
if (c == ' ' || c == '\n' || c == '\t')
{
    state = OUT;
}
else if (state == OUT) {
    state = IN;
    ++nw;
}

Как указывали другие ответы, проблема заключается в #defineточках с запятой. Чтобы минимизировать эти проблемы, я всегда предпочитаю определять числовые константы как const int:

const int IN = 1;
const int OUT = 0;

Таким образом вы избавитесь от множества проблем и возможных проблем. Он ограничен всего двумя вещами:

1. Ваш компилятор должен поддерживать const- что в 1988 году в целом было неправдой, но теперь оно поддерживается всеми обычно используемыми компиляторами. (AFAIK const"заимствовано" из C ++.)

2. Вы не можете использовать эти константы в некоторых особых местах, где вам понадобится строковая константа. Но я думаю, ваша программа не в этом.