ファイル内の大括弧の数を数えよう

Question

テキストファイルをスキャンし、比較のために開いている括弧と閉じた括弧の数を数えるプログラムを作成しようとしています。しかし、これまでのところ、1行目の括弧の最初の出現を数えただけでした。あなたはこのためにstrstr()を使用することはできません

#include <stdio.h> 
#include <string.h> 

int main() 
{ 
    FILE *fileName; 
    fileName = fopen("/home/vagrant/Desktop/tests/t5", "r"); 

    if (fileName != NULL) 
    { 
    int noOfOpenPar=0; 
    int noOfClosePar=0; 
    int noOfOpenSqr=0; 
    int noOfCloseSqr=0; 
    int noOfOpenCurl=0; 
    int noOfCloseCurl=0; 
    char lineOfText[80]; 

    while (fgets(lineOfText, 80, fileName) != NULL) 
    { 
     if (strstr(lineOfText, "(") != NULL) 
     noOfOpenPar++; 
     if (strstr(lineOfText, ")") != NULL) 
     noOfClosePar++; 
     if (strstr(lineOfText, "[") != NULL) 
     noOfOpenSqr++; 
     if (strstr(lineOfText, "]") != NULL) 
     noOfCloseSqr++; 
     if (strstr(lineOfText, "{") != NULL) 
     noOfOpenCurl++; 
     if (strstr(lineOfText, "}") != NULL) 
     noOfCloseCurl++; 
    } 
    printf ("\nThe number of open parenthesies is %d and closed parenthesies  %d\n", noOfOpenPar, noOfClosePar); 
    if (noOfOpenPar!=noOfClosePar) 
    printf("Number of open and closed parenthesies doesn't match\n"); 
    printf ("\nThe number of open square brackets is %d and closed square brackets is %d\n", noOfOpenSqr, noOfCloseSqr); 
    if (noOfOpenSqr!=noOfCloseSqr) 
    printf("Number of open and closed square brackets doesn't match\n"); 
    printf ("\nThe number of open curly brackets is %d and closed curly brackets is %d\n", noOfOpenCurl, noOfCloseCurl); 
    if (noOfOpenCurl!=noOfCloseCurl) 
    printf("Number of open and closed curly brackets doesn't match\n"); 

    } 

} 

Answer 1

、それは単純にすべての文字を反復処理し、それらを検査する方が良いでしょう。あなたは結局1文字のトークンを探しています。代わりに、ループ内でこのような

何か：

for(const char *src = lineOfText; *src != '\0'; ++src) 
{ 
    const char here = *src; 
    if(here == '(') 
    ++noOfOpenPar; 
    else if(here == ')') 
    ++noOfClosePar; 
    else if ... /* more code here */ 
} 

あなたがアイデアを得ます。 もstdioのファイルバッファリングに依存していて、一度に1文字ずつ読むだけでループの数を減らすことができますが、このアプローチはより良いと思います。

もちろん、検索する文字を保持する配列の配列とそれに関連するカウンタを反復することで、データ駆動型にすることもできます。