Thread: word count help

hi all,
im very new to programming and require some help with an assignment. im trying to count the occurrences of words in a text file and output the results. this is what ive got so far any help would be greatly appreciated.

Code:

#include "stdafx.h"
#include <stdlib.h>
#include <stdio.h>
#include <conio.h>
#include <dos.h>
#include <windows.h>

int processFile();

int main()
{
	processFile();

	return 0;
}

int processFile()
{
	char c , fname[25] , arr[1000][15]= {0} , oneword[15] ;
	int count[1000] = {0} , x = 0 , y , i = 0 , p ;
	FILE *file1;

	

	printf("\nenter file to process\n");
	scanf("%s" , &fname);
	file1 = fopen(fname , "r");
	if (file1 == NULL)
	{
		printf("error file missing");
		//menu();
		return 1;
	}
 while(!feof(file1))
	{
		c = fscanf(file1 , "%s" , &oneword);
		for( i = 0 ; i < 1000 ; i++)   //arr[i] != '\0'
		{
			if(strcmp(arr[i] , oneword)==0)
			{
				count[i]++;
				break;
			}
			
		}
				x++;
				strcpy(arr[x] , oneword);
    }
	for(p = 0 ; p < 1000 ; p++)
	{
		printf("%8s%11d\n" , arr[p] , count[p]);
		Sleep(100);
	}

	fclose(file1);
	system("pause");
	return 0;
}

I have debugged your code in linux environment . You have made some simple mistakes there ,

1. In your code you used 100 seconds sleep.This is not necessary.
2. You doesn't change the count to 1 when the first occurrence of the word.

I mention those parts in comments,

Code:

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

int processFile();

int main()
{
        processFile();

        return 0;
}

int processFile()
{

                        char c , fname[25] , arr[1000][15]= {0} , oneword[15] ;
                        int count[1000] = {0} , x = 0 , y , i = 0 , p ;
                        FILE *file1;
                        int found=0;


        printf("\nEnter file to process : ");
        scanf("%s" , &fname);
        file1 = fopen(fname , "r");
        if (file1 == NULL)
        {
                printf("error file missing");
                return 1;
        }
        else
        {
                while(!feof(file1))
                {
                        found=0;
                        c = fscanf(file1 , "%s" , &oneword);
                        for( i = 0 ; i < 1000 ; i++)   //arr[i] != '\0'
                        {
                                if(strcmp(arr[i],oneword)==0)
                                {
                                        count[i]++;
                                        found=1; // Have a indicator that the word already occurred
                                        break;
                                }

                        }
                        if(found==0) // Otherwise the word occurring first
                        {
                        strcpy(arr[x] , oneword);
                        count[x]++; // Add the word to the array and have the count to 1
                        x++;
                        }

                }
                for(p = 0 ; p < x ; p++)
                {
                        printf("%8s%11d\n" , arr[p] , count[p]);
                }

                fclose(file1);
                return 0;
        }
        }

cheers.

Originally Posted by karthigayan

1. In your code you used 100 seconds sleep.This is not necessary.

He is coding this for windows.
Sleep(milliseconds) != sleep(seconds)

thanks guys for all your help especially karthigayan.

Code:

for(i= 0 ; i < "infinity" ; i++)
{
     printf("thankyou\n");
}

Thanks for your feed back Salem ,
As per your feed back I modified the code. It will now indicate the empty file.Ans also in the previous code the word which occur last will get one count more because of the while loops last iteration.Now I made the alternative thing by coping '\0' to the oneword array.

Code:

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

int processFile();

int main()
{
        processFile();

        return 0;
}

int processFile()
{

                        char c , fname[25] , arr[1000][15]= {0} , oneword[15] ;
                        int count[1000] = {0} , x = 0 , y , i = 0 , p ;
                        FILE *file1;
                        int found=0;


        printf("\nEnter file to process : ");
        scanf("%s" , &fname);
        file1 = fopen(fname , "r");
        if (file1 == NULL)
        {
                printf("error file missing");
                return 1;
        }
        else
        {
                while(!feof(file1))
                {
                        strcpy(oneword,"\0");
                        found=0;
                        c = fscanf(file1 , "%s" , &oneword);
                        for( i = 0 ; i < 1000 ; i++)   //arr[i] != '\0'
                        {
                                if(strcmp(arr[i],oneword)==0)
                                {
                                        count[i]++;
                                        found=1; // Have a indicator that the word already occurred
                                        break;
                                }

                        }
                        if(found==0) // Otherwise the word occurring first
                        {
                        strcpy(arr[x] , oneword);
                        count[x]++; // Add the word to the array and have the count to 1
                        x++;
                        }

                }
                if(x!=0)
                {
                for(p = 0 ; p < x ; p++)
                {
                        printf("%8s%11d\n" , arr[p] , count[p]);
                }
                }
                else
                        printf("File is empty \n");

                fclose(file1);
                return 0;
        }
        }