มีอะไรผิดปกติกับรหัส 1988 C นี้?

ฉันกำลังพยายามรวบรวมโค้ดชิ้นนี้จากหนังสือ "The C Programming Language" (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;
}

นี่เป็นมาตรฐานใหม่หรือฉันยังสามารถใช้ main-type ได้หรือไม่

ปัญหาของคุณคือคำจำกัดความของตัวประมวลผลล่วงหน้า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 ++ โดยปกติ makefiles จะมีเป้าหมายให้หยุดหลังจากประมวลผลล่วงหน้า

สำหรับการอ้างอิงอย่างรวดเร็ว: คำถาม 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. คุณไม่สามารถใช้ค่าคงที่เหล่านี้ในสถานที่พิเศษบางแห่งที่คุณต้องการค่าคงที่เหมือนสตริง แต่ฉันคิดว่าโปรแกรมของคุณไม่เป็นอย่างนั้น