3

UNIXで実行されているCで、入力テキストファイルの各文字の数をカウントするプログラムを作成しました。このようなファイルの場合:

「猫は緑のマットの上に座っていた」

出力は次のようになります。

   The letter ’a’ occurs 3 times.
   The letter ’c’ occurs 1 times.
   The letter ’e’ occurs 4 times.
   The letter ’g’ occurs 1 times.
   The letter ’h’ occurs 2 times.
   The letter ’m’ occurs 1 times.
   The letter ’n’ occurs 2 times.
   The letter ’o’ occurs 1 times.
   The letter ’r’ occurs 1 times.
   The letter ’s’ occurs 1 times.
   The letter ’t’ occurs 5 times.

  5                    *
  4     *              *
  4     *              *
  3 *   *              *
  3 *   *              *
  2 *   *  *     *     *
  2 *   *  *     *     *
  1 * * * **    ***  ***
  1 * * * **    ***  ***
  0 **************************
  0 **************************
... abcdefghijklmnopqrstuvwxyz

グラフは、文字が表示される回数を表します。(10を超える場合は、10行目の後に「+」を付けるだけです)。これを実現するために現在作成しているコードは次のとおりです。

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

void drawGraph(int letters[26], char alpha[26]);
void printLetters(int letters[26], char alpha[26]);
void getLetters(FILE *fp, int letters[26], char alpha[26]);

int main(int argc, char *argv[]) {
  FILE *fp;
  int letters[26] = { 0 };
  char alpha[26] = { 'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z' };
  int indexedAlpha[256] = { 0 };
  int j = 1;

  for (i = 97; i <= 127; i++)
    {
      indexedAlpha[i] = j;
      j++;
    }

  //open file
  if ((fp = fopen(argv[1], "r")) == NULL)
    {
      perror("Cannot open file");
      exit(EXIT_FAILURE);
    }

  getLetters(fp, letters, alpha);
  printLetters(letters, alpha);
  printf("\n");
  drawGraph(letters, alpha);
  printf("\n");

  return EXIT_SUCCESS;
}

void getLetters(FILE *fp, int letters[26], char alpha[26]) {
  int c;
  for (int i = 0; (c = fgetc(fp)) != EOF; i++)
    {
      c = fgetc(fp);
      if ( isalpha(c) )
    {
      for ( int j = 0; j < 26; j++ ) //find which letter it is
        {
          if( c == alpha[j] ) 
        {
          letters[j]++;
          break;
        }
        }
    }
    }
}

void printLetters(int letters[26], char alpha[26]) {
  for( int i = 0; i < 26; i++ )
    {
      if(letters[i] != 0){
    printf("The letter '%c' occurs %d times.\n", alpha[i], letters[i]);
      }
    }
}

void drawGraph(int letters[26], char alpha[26]) {
  int x = 11;
  int y;
  while(x >= 0)
    { 
      y = 0;
      while (y < 2)
    {
      if (x == 10)
        {
          printf(" %d ", x);
        }
      else if (x == 11)
        {
          printf("    ");
        }
      else
        {
          printf("  %d ", x);
        }

      for( int i = 0; i < 26; i++ )
        {
          if(letters[i] > 10)
        {
          printf("+");
          letters[i] = 10;
          y++; // Break out of while loop
        }
          else if(letters[i] == x)
        {
          printf("*");
        }
          else
        {
          printf(" ");
        }
          if (letters[i] == x && y == 1)
        {
          letters[i] = letters[i] - 1;
        }
        }
      printf("\n");
      y++;
    }
      x--;
    }
  printf("... ");

  for( int i = 0; i < 26; i++ )
    {
      printf("%c", alpha[i]);
    }
}

しかし、私の現在のコードには2つの問題があります。1.常に10個のY軸ポイントを印刷します。必要な数のY軸ポイントのみを印刷したいのですが、これを実現するための最良の方法は何でしょうか。2.現在、小文字のみがカウントされていますが、どうすればこれに対処できますか?

また、どんな表記法やより良い方法論も大歓迎です、私はまだ学ぼうとしています!

ありがとう!

4

2 に答える 2

2

2番目の質問は私には少し簡単に思えます:

2. Currently only lower case characters are counted, how can i address this?

次のような構造を作成します。

    typedef struct 
    {
     char c; 
     int count;
    }alpha;

    alpha abc[26];

for(i=0 ; i<26 ; i++)
  abc[i].count = 0; // Initialization of count for each alphabet
for(i=0;i<26; i++)
  abc[i].c = 'a' + i;

このようにして、各文字をカウントして追跡できます。

(11 + 1 + 1)ファイル内の各アルファベットの「ヒストグラム」を印刷するには、行が必要です

(少なくとも 13 行を意味します。文字とそのバーを区切るための余分な行にすることができます)

1111アルファベットの出現回数と1前述+1アルファベット自体の行。

*しかし、これらの行を印刷するには、それぞれの(使用したヒストグラム記号)の前のスペースについて余分なことに注意する必要があります。

したがって、配列をループして、次のように出力します。1st question次のようなものを試してください。

int cnt;
for(cnt = 11 ; c >=0 ; c--)
{
 for(i=0; i<26; i++)
 {
  if(abc[i].count >= cnt && cnt == 11)  
  {
   space = abc[i].c - 'a';
   printf("%*c",space,'+'); // setting indentation and printing
  }
  if(abc[i].count == cnt && cnt != 11)  
  {
   space = abc[i].c - 'a';
   printf("%*c",space,'*');  //// setting indentation and printing
  }
  printf("\n");
 } //end of inner for loop
} // end of outer for loop 

printf("abcdefghijklmnopqrstuvwxyz\n");
于 2012-11-28T19:58:50.623 に答える
1

別のデータ表現を使用すると、プログラムが簡素化されます。

1 つのテーブルを使用して、考えられるすべての 1 バイト文字を保持します。char のバイト val がキーとして機能します。出現回数は val です。

の使用に注意してくださいwhile ((c = getchar()) != EOF)。一度に 1 バイトずつ入力を読み取るためのイディオム - あなたの getLetters() ルーチンは 1 文字おきにスキップしました。

以下のプログラムは、大文字と小文字の両方を処理します。ただし、1 バイトのエンコーディングに制限されています - utf8、utf16 などを適切に処理しません。テキスト エンコーディングの詳細については、この記事をお読みください。isalpha() が文字として扱うものは現在のロケールに依存し、setlocaleによって変更できます

#include <stdio.h>
#include <ctype.h>
#include <limits.h>

enum { NROWS = 10 };

void draw_line(int count[], double scale, int level);
void draw(int count[]);

int main() {
    int count[UCHAR_MAX+1] = {0};
    int c;

    while ((c = getchar()) != EOF)
        count[c]++;

    for (c = 0; c <= UCHAR_MAX; c++)
        if (isalpha(c) && count[c] != 0)
            printf("%c %d\n", c, count[c]);

    draw(count);

    return 0;
}

私の描画グラフ バージョンは、大きな値に対して「+」を出力する代わりに、y 値をスケーリングします。カウントを y 軸のレベルと比較して、必要な数の「y 軸スター」のみを出力します。

void draw(int count[]) {
    int c, i;
    int max = 0;
    double scale;

    for (c = 0; c <= UCHAR_MAX; c++)
        if (isalpha(c) && count[c] > max)
            max = count[c];
    scale = (max == 0) : 1.0 : (double)max / NROWS;

    for (i = NROWS; i > 0; i--)
        draw_line(count, scale, i);

    for (c = 0; c <= UCHAR_MAX; c++)
        if (isalpha(c))
            putchar(c);
    putchar('\n');
}

void draw_line(int count[], double scale, int level) {
    int c;

    for (c = 0; c <= UCHAR_MAX; c++) {
        if (isalpha(c) && count[c] / scale >= level)
            putchar('*');
        else if (isalpha(c))
            putchar(' ');
    }
    putchar('\n');
}

グラフの出力例:

$ ./countchars < countchars.c
[..snip..]

                            *                       
                            *     *                 
                            *     *                 
                            *     *    *     *      
                          * *     *    *     *      
                          * * *   *  * *     *      
                          * * *   *  * **  * **     
                          * ***  **  * **  * **     
*                         * **** **  * *** ****     
* *    *    *    *  *  *  * **** **  ***** ******   
ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz
于 2012-11-28T21:57:45.167 に答える