Bu 1988 C kodunun nesi var?

Bu kod parçasını "The C Programming Language" (K & R) kitabından derlemeye çalışıyorum. UNIX programının basit bir sürümüdür 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);
}

Ve şu hatayı alıyorum:

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

Bu kitabın 2. baskısı 1988'den ve C'ye oldukça yeniyim. Belki derleyici sürümüyle ilgili olabilir ya da belki sadece saçma konuşuyorum.

Modern C kodunda mainişlevin farklı bir kullanımını gördüm :

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

Bu yeni bir standart mı yoksa hala türsüz bir ana kullanabilir miyim?

Sorununuz, önişlemci tanımlarınızla INve OUT:

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

Bunların her birinde nasıl bir noktalı virgül olduğuna dikkat edin. Önişlemci bunları genişlettiğinde, kodunuz kabaca şöyle görünecektir:

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

Bu ikinci noktalı virgül , ayraç kullanmadığınız için eşleşme olarak öncekinin elseolmamasına neden olur if. Yani, önişlemci tanımlamalardan noktalı virgül kaldırmak INve OUT.

Burada öğrenilen ders, önişlemci ifadelerinin noktalı virgülle bitmesi gerekmediğidir.

Ayrıca her zaman diş teli kullanmalısınız!

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

elseYukarıdaki kodda asılı belirsizlik yoktur .

Bu kodla ilgili temel sorun , K & R'den gelen kod olmamasıdır . Diğerlerinin de belirttiği gibi anlamı değiştiren makro tanımlarından sonra, kitapta yer almayan noktalı virgül içerir.

Kodu anlamak için bir değişiklik yapmanın dışında, anlayana kadar onu yalnız bırakmalısınız. Yalnızca anladığınız kodu güvenli bir şekilde değiştirebilirsiniz.

Bu muhtemelen sizin açınızdan bir yazım hatasıydı, ancak programlama sırasında ayrıntıları anlama ve dikkat etme ihtiyacını gösteriyor.

Makrolardan sonra noktalı virgül olmamalıdır,

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

ve muhtemelen olmalı

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

GİRİŞ ve ÇIKIŞ'ın tanımları şu şekilde görünmelidir:

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

Noktalı virgül soruna neden oluyordu! Açıklama basittir: Hem IN hem de OUT önişlemci direktifleridir, esasen derleyici tüm IN oluşumlarını 1 ile ve OUT'un tüm oluşumlarını kaynak kodunda 0 ile değiştirir.

Orijinal kod, 1 ve 0'dan sonra noktalı virgül içerdiğinden, kodda GİRİŞ ve ÇIKIŞ değiştirildiğinde, numaradan sonraki fazladan noktalı virgül geçersiz kod üretti, örneğin bu satır:

else if (state == OUT)

Böyle görünerek sona erdi:

else if (state == 0;)

Ama istediğin şuydu:

else if (state == 0)

Çözüm: Orijinal tanımdaki sayılardan sonraki noktalı virgülü kaldırın.

Gördüğünüz gibi makrolarda bir sorun vardı.

GCC, ön işlemeden sonra durdurma seçeneğine sahiptir . (-E) Bu seçenek, ön işlemenin sonucunu görmek için kullanışlıdır. Aslında, c / c ++ 'da büyük kod tabanı ile çalışıyorsanız, teknik önemli bir tekniktir. Tipik olarak makefiles, ön işlemeden sonra durdurulacak bir hedefe sahip olacaktır.

Hızlı başvuru için: SO sorusu seçenekleri kapsar - Visual Studio'da önişlemeden sonra bir C / C ++ kaynak dosyasını nasıl görebilirim? . Vc ++ ile başlar, ancak aşağıda belirtilen gcc seçeneklerine de sahiptir .

Tam olarak bir sorun değil ama beyanı main()da tarihli, böyle bir şey olmalı.

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

Derleyici, tek olmayan bir işlev için bir int dönüş değeri varsayacaktır ve derleyici / bağlayıcının argc / argv için bildirim eksikliği ve dönüş değeri eksikliği etrafında çalışacağına eminim, ancak orada olmaları gerekir.

Kod bloklarının etrafına açık parantez eklemeyi deneyin. K & R tarzı belirsiz olabilir.

18. satıra bakın. Derleyici size sorunun nerede olduğunu söylüyor.

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

Basit bir yol, her biri için {} gibi parantez kullanmaktır ifve else:

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

Diğer yanıtların da işaret ettiği gibi, sorun içinde #defineve noktalı virgüldür. Bu sorunları en aza indirmek için sayı sabitlerini her zaman şu şekilde tanımlamayı tercih ederim const int:

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

Bu şekilde birçok sorundan ve olası sorundan kurtulursunuz. Sadece iki şeyle sınırlıdır:

1. Derleyicinizin desteklemesi gerekiyor const- ki bu 1988'de genel olarak doğru değildi, ancak şimdi yaygın olarak kullanılan tüm derleyiciler tarafından destekleniyor. (AFAIK, constC ++ 'dan "ödünç alınmıştır".)

2. Bu sabitleri dizge benzeri bir sabite ihtiyaç duyacağınız bazı özel yerlerde kullanamazsınız. Ama bence programınız öyle değil.