Sharon Shelton Ответов: 4

Как я могу подсчитать частоту каждого алфавита в текстовом файле?


Я пишу код, чтобы найти частоту каждого алфавита в файле. Но я не получаю правильного ответа. Не могли бы вы сказать мне, что с ним не так?

Что я уже пробовал:

#include <stdio.h>
#include <ctype.h>
#include <stdlib.h>
#include <string.h>
#define tab_size 26
struct alphabet{
	char alpha;
	int count;
};
struct alphabet table[tab_size];
int main(){
	FILE *fptr;
	int buf_size=1000,i,j;
	char name[100],buffer[1000],letter='a';
	printf("\nEnter the file name:");
	scanf("%s",name);
	fptr=fopen(name,"r");
	if(fptr==NULL){
		printf("\nProblem with opening the file");
		exit(1);
	}
	for (i = 0; i < tab_size; i++) {
    	table[i].alpha = letter++;
    	table[i].count = 0;
    //	printf("%c\n",table[i].alpha);
	}
//	fread(buffer,1,buf_size,fptr);
//content is available in the buffer from 0 to 999
while(feof(fptr)==0){
	fread(buffer,1,buf_size,fptr);
	for(i=0;i<buf_size;i++){
		j=0;
		if(isalpha(buffer[i])){
			while((buffer[i]!=table[j].alpha||buffer[i]!=toupper(table[i])&&j<tab_size){
			j++;
		}
		table[j].count++;
	}
	else
		continue;
	}
}
for(i=0;i<tab_size;i++){
	printf("Count of %c is %d\n",table[i].alpha,table[i].count);
}
}

Patrice T

Покажите входной сигнал образца, фактический выход и ожидаемый выход.

4 Ответов

Рейтинг:
33

KarstenK

Вы считаете не те числа. Это работает для персонажей

int table[26] = {0};
// get a valid characters (exclude spaces and numbers)
char c = toupper(buffer);
int index = c - 'A';
table[index]++;
Используйте отладчик, чтобы увидеть детали.


Sharon Shelton

Спасибо. Это было полезно

Рейтинг:
26

CPallini

Я исправил ваш код (в той же степени: он должен работать, но в нынешнем виде он не особенно надежен и эффективен). Попробуй

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

#define tab_size 26

enum
{
  TABLE_SIZE = 26,
  BUFFER_SIZE = 1024,
  STRING_SIZE = 256,
};

struct alphabet
{
  char alpha;
  int count;
};

struct alphabet table[tab_size];

int main()
{
  FILE *fptr;
  int i;

  char filename[STRING_SIZE];
  char buffer[BUFFER_SIZE];
  char letter = 'a';
  printf("\nEnter the file name:");
  scanf("%s", filename);
  fptr = fopen( filename, "r");
  if ( !fptr )
  {
    printf("\nProblem with opening the file");
    exit(1);
  }

  for (i = 0; i < TABLE_SIZE; i++)
  {
      table[i].alpha = letter++;
      table[i].count = 0;
  }

  size_t nread;

  while ( ( nread = fread(buffer, 1, BUFFER_SIZE, fptr) ) > 0)
  {
    for(i=0; i<nread; ++i)
    {
      if( isalpha(buffer[i]) )
      {
        int j;
        for (j = 0; j<TABLE_SIZE; ++j)
        {
          if ( buffer[i] == table[j].alpha || buffer[i] == toupper(table[j].alpha))
          {
              ++table[j].count;
              break; // exit the j-loop
          }
        }
      }
    }
  }

  for(i = 0; i<TABLE_SIZE; ++i)
  {
    printf("Count of %c is %d\n",table[i].alpha,table[i].count);
  }

  fclose(fptr);

  return 0;
}


Patrice T

+5

CPallini

Спасибо.

Sharon Shelton

Спасибо за вашу помощь!

CPallini

Добро пожаловать.

Рейтинг:
2

Patrice T

Цитата:
Я пишу код, чтобы найти частоту каждого алфавита в файле. Но я не получаю правильного ответа. Не могли бы вы сказать мне, что с ним не так?

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.
-----
Совет: Научитесь правильно делать отступы в вашем коде, это покажет его структуру и поможет чтению и пониманию. Это также помогает выявлять структурные ошибки.
#include <stdio.h>
#include <ctype.h>
#include <stdlib.h>
#include <string.h>
#define tab_size 26
struct alphabet{
    char alpha;
    int count;
};
struct alphabet table[tab_size];
int main(){
    FILE *fptr;
    int buf_size=1000,i,j;
    char name[100],buffer[1000],letter='a';
    printf("\nEnter the file name:");
    scanf("%s",name);
    fptr=fopen(name,"r");
    if(fptr==NULL){
        printf("\nProblem with opening the file");
        exit(1);
    }
    for (i = 0; i < tab_size; i++) {
        table[i].alpha = letter++;
        table[i].count = 0;
        //  printf("%c\n",table[i].alpha);
    }
    //  fread(buffer,1,buf_size,fptr);
    //content is available in the buffer from 0 to 999
    while(feof(fptr)==0){
        fread(buffer,1,buf_size,fptr);
        for(i=0;i<buf_size;i++){
            j=0;
            if(isalpha(buffer[i])){
                while((buffer[i]!=table[j].alpha||buffer[i]!=toupper(table[i])&&j<tab_size){
                    j++;
                }
                table[j].count++;
            }
            else
                continue;
        }
    }
    for(i=0;i<tab_size;i++){
        printf("Count of %c is %d\n",table[i].alpha,table[i].count);
    }
}

Стиль отступа - Википедия[^]

Профессиональные редакторы программистов имеют эту функцию и другие, такие как сопоставление скобок и подсветка синтаксиса.
Блокнот++ Главная Страница[^]
личные[^]


Sharon Shelton

Спасибо! Сделаю все возможное, чтобы улучшить

Рейтинг:
1

Garth J Lancaster

Вы не указали, что производит код ... может быть, вам стоит использовать Улучшить вопрос

А пока взгляните на эти два жирных бита

char name[100],buffer[1000],letter='a';

...
while((buffer[i]!=table[j].alpha||buffer[i]!=toupper(table[i])&&j<tab_size){


Так что даже если я соглашусь с остальной логикой, может быть, вы захотите подумать об этом