Character that appears max times in string

Hello All,

I have developed code for finding character that appears max times in a string. Wanted to know, any better version of this. If two characters appear same number of times then whichever appaers first is given priority.

RT

Code:

#include<stdio.h>

#define MAX_CHAR	26

unsigned char cFreqTable[MAX_CHAR];

main()
{
	unsigned char string[200],cMaxChar=0;
	unsigned iCurrFreq,iMaxFreq;
	unsigned char *buf = string;

	iCurrFreq = iMaxFreq = 0;
	printf("Enter the string\n");

	scanf("%s",buf);

	while (*buf)
	{
		// convert to lower case
		if ( ( (*buf) >= 'A' ) && ( (*buf) <= 'Z' ) )
			(*buf) = (*buf) + ('a' - 'A') ;

		// Increment frequency table
		cFreqTable['a'-(*buf)] = cFreqTable['a'-(*buf)] + 1;

		iCurrFreq = cFreqTable['a'-(*buf)];

		if ( iCurrFreq > iMaxFreq )
		{
			iMaxFreq = iCurrFreq;
			cMaxChar = *buf;
		}
		buf++;
	}	
	printf("Character %c appears for max %d times \n",cMaxChar,iMaxFreq);
}

> If two characters appear same number of times then whichever appaers first is given priority.
So have another array which records the index each time you see a char for the first time

A couple points:

• cFreqTable doesn't need to be global
• main() should be: int main(void)
• there needs to be a return 0; at the end of the main function
• that code will only work in an enviroment that has contigious letters
• use tolower() from <ctype.h> to convert to lower instead