Почему не правильно выводит значения d?

Question

[fewgesto]

#include <stdio.h>
#pragma warning(disable : 4996)
int main() {
	unsigned char a[81], perm[256], g[81], d = 0, n = 0;
	for (int f = 0; f < 81; f++) {
		scanf("%[^\n]d", &a);
	}
	int f = 0;
	while (a[f] != '\0') {
		n++; f++;
	}
	for (int b = 0; b < 256; b++) {
		perm[b] = b;
		for (int c = 0; c < n; c++) {
			if (a[c] == perm[b]) {
				g[c] = perm[b];
			}
		}
	}
	for (int c = 0; c < n; c++) {
		if (g[c] == g[c + 1]) {
			d++;
		}
		else d = 1;
		//printf(" %d ", g[c]);
		printf(" %c=%d\n ", g[c], d);
	}

}

Comments

[fewgesto]

Код должен посчитать сколько введен каждый символ в строке, строку сам задаешь
Сейчас попытаюсь объяснить...
первый while считает из скольки символов состоит строка
for (int b = 0; b < 256; b++) {
perm[b] = b;
for (int c = 0; c < n; c++) {
if (a[c] == perm[b]) {
g[c] = perm[b];
-> это сортирует строку по аски кодам, те строку "ab " запишет как "32 97 98"
for (int c = 0; c < n; c++) {
if (g[c] == g[c + 1]) {
d++;
}
else d = 1;
//printf(" %d ", g[c]);
printf(" %c=%d\n ", g[c], d);
-> тут должен считать уже сколько раз попадается символ
тк строчка занесена в массив g[c]

[res2001]

fewgesto,

Код должен посчитать сколько введен каждый символ в строке, строку сам задаешь

Во первых, есть же функция strlen, например, и пачка других строковых функций. Почему их не используете?
Во вторых странный подход к решению задачи. Зачем сортировка?
Просто используйте массив счетчиков из 256 элементов. Инициализируйте массив нулями. Для индексации в массиве используйте код символа. Задача решается за один проход по строке, никаких сортировок.
Схематично без ввода строки:

unsigned count[256];
memset(count, 0, sizeof(count));
// Считаем количество символов
for(int i=0; a[i] != '\0'; ++i) {
  ++count[a[i]];
}
// вывод
for(int i=0; i < 256; ++i) {
  if(count[i] > 0)
    printf(" %c=%d\n ", (char)i, count[i]);
}

Answer 1

[GavriKos]

Вы серьезно предлагаете понять что идет не так в алгоритме, у которого нет ни входных данных, ни описания? Да еще и с нечитаемым кодом?

Comments

[fewgesto]

Код должен посчитать сколько введен каждый символ в строке, строку сам задаешь
Сейчас попытаюсь объяснить...
первый while считает из скольки символов состоит строка

for (int b = 0; b < 256; b++) {
    perm[b] = b;
    for (int c = 0; c < n; c++) {
      if (a[c] == perm[b]) {
        g[c] = perm[b];

-> это сортирует строку по аски кодам, те строку "ab " запишет как "32 97 98"

for (int c = 0; c < n; c++) {
    if (g[c] == g[c + 1]) {
      d++;
    }
    else d = 1;
    //printf(" %d ", g[c]);
    printf(" %c=%d\n ", g[c], d);

-> тут должен считать уже сколько раз попадается символ
тк строчка занесена в массив g[c]

[GavriKos]

fewgesto, Берите отладчик и по шагам проходите свой алгоритм. Начните с сортировки.

[fewgesto]

GavriKos, сортировка работает правильно
не правильно записаны условия счита символов

Answer 2

[CityCat4]

Ни единого комментария.

Кому будет интересно разбираться в этой каше? Отладчик в зубы - и по шагам трейсить программу, отслеживая счетчики.

Answer 3

[Александр Ананьев]

Видимо, надо так

for (int c = 0; c < n; c++) {
    if (g[c] == g[c + 1]) {
      d++;
      continue;
    }
    else d = 1;
    //printf(" %d ", g[c]);
    printf(" %c=%d\n ", g[c], d);
  }

Answer 4

[Владимир Дубровин]

%d это int, а передаете unsigned char.