この1988 Cコードの何が問題になっていますか？

私は本「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

この本の第2版は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;;

この2番目のセミコロンは、中括弧を使用していないため、として一致する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 ++で大規模なコードベースを使用している場合に重要です。通常、メイクファイルには、前処理後に停止するターゲットがあります。

クイックリファレンス：SOの質問はオプションをカバーしています-Visual Studioでの前処理後にC / C ++ソースファイルを表示するにはどうすればよいですか？。それはvc ++で始まりますが、以下に述べるgccオプションもあります。

正確には問題ではありませんが、の宣言にmain()も日付が付けられています。これはこのようなものでなければなりません。

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

コンパイラーは、1つはない関数の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セミコロンにあります。これらの問題を最小限に抑えるために、私は常に数値定数をaとして定義することを好みconst intます：

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

このようにして、多くの問題と考えられる問題を取り除くことができます。これは、次の2つの制限があります。

1. お使いのコンパイラはサポートする必要がありますconst-これは1988年には一般的には真実ではありませんでしたが、今では一般的に使用されているすべてのコンパイラでサポートされています。（AFAIKはconstC ++から「借用」されています。）

2. これらの定数は、文字列のような定数が必要になる特別な場所では使用できません。しかし、あなたのプログラムはそうではないと思います。