Почему программа выдает не тот результат, который нужен?

Question

[Алексей Ярков]

Я уже наверное задолбал своими тупыми вопросами =) Продолжим!
Чтобы не писать длинный вопрос, я дам ссылку:
https://github.com/yarkovaleksei/passgen/issues/1
Код тут:
https://github.com/yarkovaleksei/passgen/blob/mast...
Я сейчас в локальной ветке пытаюсь пофиксить, но тут такой трэш...
Вот как я пытаюсь исправить:

if (length < 1)
	{
		while (isTyped != 1)
		{
			int t;
			char *ct = (char *)malloc(4); // хочу ограничить ввод числом 999
			printf("%s\n", _("Type in a password length"));
			scanf("%s", ct);

			if (isNumber(ct) == 1) // функция isNumber принимает char[], поэтому изврат с scanf("%s", ct)
			{
				t = atoi(ct); // конвертируем в число, если строка из цифр (убейте меня)
				if (t < 1) {
					isTyped = 0;
				} else {
					length = t;
					isTyped = 1;
				}
			} else {
				isTyped = 0;
			}
		}
	}

А теперь внимание на скриншот:

КАКОГО ХРЕНА? Почему в конце дописывается путь к файлу с русской локализацией?
Посмотрите, плиз, весь репозиторий. Чувствую, что я тот еще м@дак...

Comments

[abcd0x00]

Если тебе надо что-то проверить, пиши функцию.
Если тебе надо что-то ввести, пиши функцию.
Если тебе надо что-то создать, пиши функцию.

Пока что у тебя всё навалено в кучу и при любой ошибке ты даже не знаешь, где искать её корни. Просто не читал ты K&R2, и это чувствуется, а потому и не знаешь хорошие практики, используемые при написании кода.

[Алексей Ярков]

abcd0x00: вот именно поэтому сейчас ее и читаю ))

[abcd0x00]

Алексей Ярков: читай, копируй напрямую стиль, чтобы понять, прочувствовать его, зачем он такой. Потом будешь читать по программированию уже что-нибудь (без привязки к языкам, на псевдокоде). У тебя много таких ошибок алгоритмических, всё это из-за того, что ты взялся за код без теоретической подготовки. Такое можно делать только ради мотивирования себя на дальнейшее обучение (нужна мотивация, так как это нелёгкое дело - 25 часов в сутки надо кодить и стирать, кодить и стирать), но не для создания хороших программ. Для создания хороших программ нужно изучать парадигмы программирования, плюсы и минусы этих парадигм. Тоже немало всё это времени займёт. Но зато потом, когда ты будешь знать уже, программы сами будут получаться и, главное, будут делать что-то, что ты руками уже выполнить физически не сможешь, так как не хватит времени. Но чтобы написать такие программы, нужно уметь писать их части, а потом соединять эти части друг с другом. А потом ты будешь соединять такие программы друг с другом, получая программные пакеты или комплексы, которые уже много всяких сложных действий будут прокручивать. Там уже руками нереально выполнить это всё.

[abcd0x00]

Алексей Ярков: но если ты одну функцию написать не можешь нормально, то о части программы и речи быть не может, а уж о всей программе - тем более. Так что всё начинается с малого.

Answer 1

[GavriKos]

Символ окончания строки случайно не забыли?

Comments

[Алексей Ярков]

Японский бог!!! Ну конечно!!!
Строка: https://github.com/yarkovaleksei/passgen/blob/mast...
strcat(result, "\0");
Заменил на

result[counter] = '\0'; // И кавычки, ОЛЕНЬ, ОДИНАРНЫЕ НАДО!!!

От души, человек!

Answer 2

[rPman]

фиг его знает, есть предположение, в main.c

setlocale(LC_ALL, "");

#ifdef PACKAGE_LOCALE_DIR
bindtextdomain(PACKAGE_NAME, PACKAGE_LOCALE_DIR);
#else
#error Please use flag -DPACKAGE_LOCALE_DIR="locale"!
#endif

textdomain(PACKAGE_NAME);

что то из этого выдает в stdout лишний текст путь до локализации, но он закеширован, кеши сбрасываются по завершению программы

Comments

[Алексей Ярков]

Но при использовании ключа -lНЕ_НОЛЬ нормально все. Есть предположение, что вот в этой строке:

// ct is char[]
// t is int
t = atoi(ct);

я творю х@йню =)

Answer 3

[15432]

Переполнение происходит, scanf читает всю строку до символа перевода строки, но буфер всего 4 байта, и происходит перезапись данных, что за буфером. Видимо дальше хранится имя файла, которое и выводится. Необходимо использовать безопасный scanf (в винде это scanf_s, в линуксе не знаю).

Comments

[Алексей Ярков]

Да, я уже нагуглил и читаю. Но проблема то в другом. Если я ввожу число <64, то все норм. А вот если >64, то начинается магия. Вы не пробовали у себя запускать?