Có gì sai với mã C 1988 này?

Tôi đang cố gắng biên dịch đoạn mã này từ cuốn sách "Ngôn ngữ lập trình C" (K & R). Đây là phiên bản cơ bản của chương trình 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);
}

Và tôi gặp lỗi sau:

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

Ấn bản thứ 2 của cuốn sách này là từ năm 1988 và tôi còn khá mới đối với C. Có lẽ nó liên quan đến phiên bản biên dịch hoặc có lẽ tôi chỉ đang nói những điều vô nghĩa.

Tôi đã thấy trong mã C hiện đại một cách sử dụng khác của mainhàm:

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

Đây có phải là một tiêu chuẩn mới hay tôi vẫn có thể sử dụng một main không loại?

Vấn đề của bạn là với các định nghĩa về bộ tiền xử lý của bạn INvà OUT:

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

Chú ý cách bạn có dấu chấm phẩy ở cuối mỗi cái. Khi bộ tiền xử lý mở rộng chúng, mã của bạn sẽ trông giống như sau:

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

Dấu chấm phẩy thứ hai đó khiến elsecho không có trước đó iflà so khớp, bởi vì bạn không sử dụng dấu ngoặc nhọn. Vì vậy, hãy xóa dấu chấm phẩy khỏi các định nghĩa tiền xử lý của INvà OUT.

Bài học rút ra ở đây là các câu lệnh tiền xử lý không nhất thiết phải kết thúc bằng dấu chấm phẩy.

Ngoài ra, bạn nên sử dụng luôn niềng răng!

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

Không có elsesự mơ hồ nào trong đoạn mã trên.

Vấn đề chính với mã này là nó không phải là mã từ K&R. Nó bao gồm dấu chấm phẩy sau các định nghĩa macro, không có trong sách, như những người khác đã chỉ ra làm thay đổi ý nghĩa.

Ngoại trừ khi thực hiện một thay đổi để cố gắng hiểu mã, bạn nên để nó một mình cho đến khi bạn hiểu nó. Bạn chỉ có thể sửa đổi mã một cách an toàn mà bạn hiểu.

Đây có thể chỉ là lỗi đánh máy của bạn, nhưng nó cho thấy sự cần thiết phải hiểu và chú ý đến các chi tiết khi lập trình.

Không được có bất kỳ dấu chấm phẩy nào sau macro,

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

và nó có lẽ nên

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

Định nghĩa của IN và OUT sẽ giống như sau:

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

Dấu chấm phẩy gây ra sự cố! Giải thích rất đơn giản: cả IN và OUT đều là chỉ thị tiền xử lý, về cơ bản trình biên dịch sẽ thay thế tất cả các lần xuất hiện của IN bằng 1 và tất cả các lần xuất hiện của OUT bằng 0 trong mã nguồn.

Vì mã gốc có dấu chấm phẩy sau số 1 và số 0, khi IN và OUT được thay thế trong mã, dấu chấm phẩy thừa sau số tạo ra mã không hợp lệ, ví dụ như dòng này:

else if (state == OUT)

Kết thúc trông như thế này:

else if (state == 0;)

Nhưng những gì bạn muốn là:

else if (state == 0)

Giải pháp: bỏ dấu chấm phẩy sau các số trong định nghĩa ban đầu.

Như bạn thấy đã có sự cố trong macro.

GCC có tùy chọn dừng sau khi xử lý trước. (-E) Tùy chọn này hữu ích để xem kết quả của quá trình xử lý trước. Trên thực tế, kỹ thuật này là một kỹ thuật quan trọng nếu bạn đang làm việc với cơ sở mã lớn trong c / c ++. Thông thường, các tệp makefiles sẽ có mục tiêu dừng lại sau khi xử lý trước.

Để tham khảo nhanh: Câu hỏi SO bao gồm các tùy chọn - Làm cách nào để xem tệp nguồn C / C ++ sau khi xử lý trước trong Visual Studio? . Nó bắt đầu với vc ++, nhưng cũng có các tùy chọn gcc được đề cập bên dưới .

Không chính xác là một vấn đề, nhưng tuyên bố của main()cũng được ghi ngày tháng, nó phải là một cái gì đó như thế này.

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

Trình biên dịch sẽ giả định một giá trị trả về int cho một hàm w / o, và tôi chắc chắn rằng trình biên dịch / liên kết sẽ giải quyết việc thiếu khai báo cho argc / argv và thiếu giá trị trả về, nhưng chúng sẽ ở đó.

Thử thêm dấu ngoặc nhọn xung quanh các khối mã. Kiểu K&R có thể không rõ ràng.

Nhìn vào dòng 18. Trình biên dịch đang cho bạn biết vấn đề nằm ở đâu.

    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;
    }

Một cách đơn giản là sử dụng các dấu ngoặc như {} cho mỗi ifvà else:

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

Như các câu trả lời khác đã chỉ ra, vấn đề nằm ở #definedấu chấm phẩy và. Để giảm thiểu những vấn đề này, tôi luôn muốn xác định các hằng số dưới dạng const int:

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

Bằng cách này bạn sẽ thoát khỏi nhiều rắc rối và các vấn đề có thể xảy ra. Nó bị giới hạn bởi hai điều:

1. Trình biên dịch của bạn phải hỗ trợ const- điều này nói chung là không đúng vào năm 1988, nhưng bây giờ nó được hỗ trợ bởi tất cả các trình biên dịch thường dùng. (AFAIK constlà "mượn" từ C ++.)

2. Bạn không thể sử dụng các hằng số này ở một số nơi đặc biệt mà bạn cần một hằng số giống như chuỗi. Nhưng tôi nghĩ chương trình của bạn không phải là trường hợp đó.