С4 Егэ на Си

[interfeys]

Я ученик 11 класса, готовлюсь к Егэ
В этой теме по-возможности собираюсь не реже раза в неделю (в общем то графика не будет, но задач 50 до июня думаю выложу) выкладывать решение задания С4 по информатике на языке С (которое уже корректно работает или не работает по неизвестной мне причине).
От вас мне требуется конструктивная критика по поводу эффективности алгоритма, и иногда уточнить некоторые тонкости.
Думаю, чтобы не загрязнять форум буду выкладывать в одной теме, те, кому интересно следит.
*Именно этот пост не содержит практически никакой смысловой нагрузки, просто интересно мнение пользователей

И почти сразу первое уточнение

Предположим на вход поступают данные:
<Имя> <Фамилия> <Номер>

char str1[20];
char str2[10];
int Num;

Корректно ли ?
scanf("%s %s %d", str1, str2, &Num);

А вот так никогда не делать?
char[10] str1;

[Stilet]

Корректно ли ?

Если не учитывать синтаксических ошибок и необходимости ввода через пробел то да.

А вот так никогда не делать?

Возможно какие-то из Си компилеров и понимают такую инструкцию, я не встречал такого.

[Abstraction]

Если не учитывать синтаксических ошибок и необходимости ввода через пробел то да.

Зависит от понимания "корректного". Это небезопасный код; чтение таким образом пользовательских данных допустимо только при абсолютном доверии к пользователю.

[Stilet]

чтение таким образом пользовательских данных допустимо только при абсолютном доверии к пользователю.

Я же не отрицаю этого )

[interfeys]

Вообще не стоило выкладывать это, но я не могу разобраться в символах
возможно, следовало использовать gets(); но то, что тут не оговоренно кол-во символов, намекает, что это делается по-другому
Литературу, естественно, прочитал, у меня есть представление того, что я хочу сделать (а может и нет)

На вход программы подается текст на английском языке, заканчивающийся точкой (другие символы “.” в тексте отсутствуют). Требуется написать программу, которая будет определять и выводить на экран английскую букву, встречающуюся в этом тексте чаще всего, и количество там таких букв. Строчные и прописные буквы при этом считаются не различимыми. Если искомых букв несколько, то программа должна выводить на экран первую из них по алфавиту. Например, пусть файл содержит следующую запись:
It is not a simple task. Yes!
Чаще всего здесь встречаются буквы I, S и T (слово Yes в подсчете не учитывается, так как расположено после точки). Следовательно, в данном случае программа должна вывести два символа, разделенных пробелом: I 3

#include<stdio.h>
#include<conio.h>
int main(void)
{
   int arr[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 ard[26]={0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0};
   //2 практически необязательных массива
   char c;
   int a,a2,z,z2;
   while((c=getchar())!='.') //Видимо не та функция
   {
      char a = 'a'; 
      char a2 = 'A'; //Это я пытался очень сильно упростить
      char z = 'z'; 
      char z2 = 'Z';        
      if((c=getchar())>a&&(c=getchar())<z)
         ard[c-a]++;
      //По идее, код символа-код буквы "а", должен соответствовать 
     //ячейке в массиве
    //Допустим, если я ввожу a(код 65), то 65-65=0, а это соответствует
   // Тому самому адресу в массиве
  // Который увеличиться на 1
      if((c=getchar())>a2&&(c=getchar())<z2) //Для заглавных букв аналогично
         ard[c-a2]++;
   }
   int max=0;
   int boo=0;
   for(int i=0;i<26;i++)  //Находим самую частую букву
   {
      if(ard[i]>max)
      {
         ard[i]=max;
         boo=i;
      }
   }
   printf("%c %d",arr[boo],max);  //выводим
}

[Stilet]

Я бы делал так:

	char s[225]="It is not a simple task. Yes!";
	int a['z'-'a'];for(int i=0;i<'z'-'a';i++)a[i]=0;
	int max=0;
	for(int i=0;s[i] && s[i]!='.';i++){
		char c=towlower(s[i]);
		if(c>='a' && c<='z') a[c-'a']++;
		max=(a[c-'a']>a[max-'a'])?c:max;
		
	}
	//for(int i=0;i<'z'-'a';i++) 		if(a[i]) printf("%c - %d\n",'a'+i,a[i]);
	printf("%c - %d\n",max,a[max-'a']);
	getchar();
	return 0;

[MooNDeaR]

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

int main(void)
{
	/*Строку взял для примера, по сути надо её считать*/
	char str[30] = "It is not a simple task. Yes";
	
	/*Будем сюда подсчитывать кол-во использованных символов*/
	int ABC[128] = {0};

	/*Сразу найдем длину строки*/
	int len = strlen(str);

	/*Проходим по всей строке и заодно ищем максимальное кол-во вхождений*/
	int max = 0;
	for(int i = 0; i < len; ++i)
	{
		/*Преобразуем символ в верхний регистр*/
		char temp = toupper(str[i]);

		/*Если обнаружили точку*/
		if(temp == '.') break;
		
		/*Дословно: если temp это буква, тогда:
		  прибавляем к ABC[temp] единицу, и проверяем
		  больше ли ABC[temp], чем предыдущее значение
		  max? Если да, тогда max = ABC[temp], если
		  нет, тогда ничего не меняем*/
		if(isalpha(temp))
			max = ++ABC[temp] > max ? ABC[temp] : max;
	}

	/*Находим первую в алфавите букву с максимальным кол-во вхождений*/
	for(int i = 65; i < 90; ++i)
		if(ABC[i] == max)
		{
			printf("%c - %d\n",i,ABC[i]);	
			break;
		}

	return 0;
}

[MooNDeaR]

Я бы делал так:

int a['z'-'a'];for(int i=0;i<'z'-'a';i++)a[i]=0;

А зачем лишний цикл писать (да и еще и в одну строчку с объявлением, жуть одним словом), если можно:

int a['z'-'a'] = {0}

И на будущее ТС:

int ard[26]={0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0};

эквивалентно:

int ard[26]={0};

[Stilet]

если можно:

Можно. Только вот я слышал обратное, что такая инструкция это присвоение только первому элементу нулевое значение, поэтому уже как бы в догадках.
Моя VS тебя поддерживает, но чем си не шутит?

Вообще конечно лучше чтоб спецы по си выступили в пользу этого или отвергли...

[interfeys]

Учел замечания, вот этот код, компилится и правильно работает
//2 массива, это минус, возможно
//Получилось компактно
//Насчет эффективности, не знаю

#include<stdio.h>
#include <ctype.h>
#include <string.h>
int main(void)
{
   char s[255];
   int abc[128]={0}; //Благодарю MooNDeaR
   gets(s);
   int g=0;
   int max=0;
   for(int i=0;s[i]!='.';i++)
   { 
         g=toupper(s[i]);
         if(isalpha(g))
            abc[g]++;
         if(abc[g]>max)
            max=abc[g];
   }
   for(int y=0;y<128;y++)
   {
      if(abc[y]>=max)
      {
         putchar(y);
         printf(" %d\n", max); break;
      }
   }
}