Помогить отладить маленькую программку (segmentation fault)

Вместо:
scanf("%s", chars);
надо:
scanf("\n%c", &chars);
\n - это маленький хак из-за особенностей scanf.

Вместо:
printf("%s", fillCharacter);
надо:
printf("%c", fillCharacter);

Очевидно!

char chars;
.
.
.
scanf("%s", chars);

во-первых scanf("%s", &chars);во-вторых если хочешь считывать строку то объяви массив char chars[xxx](напрмер 256), или же с помощью оператора malloc:
char* chars;

chars = (char*)malloc(num);, где num число байтов которое ты хочешь выделить.
scanf("%s", chars); в этом случае в сканфе пишится chars без &.

Если хочешь считать один символ, то у тебя всё правильно за исключением:
scanf("%c", &chars);

void draw(int side, char* fillCharacter) - так правильно, если хочешь передавать строку в функцию.

соответственно вызов этой функции должен выглядеть так: draw(side, &chars);(естественно если ты используешь malloc для выделения памяти массиву chars, то у тебя написан вызов функции правильно).

ЗЫ Моё ИМХО: даже начиная изучать C, уже можно протрассировать прогу и
найти ошибки в ней......

На самом деле, scanf - одна из самых дурацких функций C. Применяется только в школе, да на лабах в институте :) Если надо читать символ, то, как правило - getchar, если строку - fgets (но не gets!). А если уж совсем для пользователя удобно надо сделать - readline.

> во-вторых если хочешь считывать строку то объяви массив char chars[xxx](напрмер 256), или же с помощью оператора malloc

А что сделает scanf, если я вбабахаю ей на вход пару килобайт текста?

> А что сделает scanf, если я вбабахаю ей на вход пару килобайт текста?

Ответ : буферизует ее :-)

А кто сказал что scanf читает только одну строку или всю
строку до конца?
Рекомендую попробовать на кошках:
main(){
int i,j;
scanf("%d", &i);
scanf("%d", &j);
printf("%d - %d\n", i, j);
}
i и j могут быть в одной строке с мусором далее, могут быть в разных.

>На самом деле, scanf - одна из самых дурацких функций C.
>Применяется только в школе, да на лабах в институте :

Спасибо за науку, буду иметь в виду. Просто когда писалась та книжка, по которой я C изучаю, линукса еще в проекте не было :) Да и меня тоже :) И не потому, что я такой молодой ;)

>ЗЫ Моё ИМХО: даже начиная изучать C, уже можно
>протрассировать прогу и найти ошибки в ней......

А как, можно узнать? Я ведь раньше "программировал" только в маздае, а на каком языке даже не скажу - стыдно ;) Так там удобно было - можно было ставить точки останова, узнавать значения переменных во время работы программы, прерывать ее выполнение в любой момент и т.п. А такое можно сделать в линуксе?

PS а программа заработала :)

> А кто сказал что scanf читает только одну строку или всю строку до конца?

Я сделаю так, что прочитает. И запишет. Если в той паре килобайт текста не будет пробелов, переводов строки, табуляций?, ...?, то scanf как миленькая запишет весь этот хлам любой длины в буфер, переданный спецификатору %s.

Твой пример не очень удачен. %d, %f, %c, ... использовать достаточно безопасно - их результат имеет фиксированную, известную на этапе компиляции длину. А вот с %s сложнее. Длина результата заранее неизвестна, а scanf не позволяет указать ограничение на неё и не занимается сама выделением памяти нужного размера под результат. Посмотри для сравнения на gets и fgets и подумай, почему gets считается небезопасной, а fgets - безопасной, если писать правильно.

>А как, можно узнать? Я ведь раньше "программировал" только в маздае, а на каком языке даже не скажу - стыдно ;) Так там удобно было - можно было ставить точки останова, узнавать значения переменных во время работы программы, прерывать ее выполнение в любой момент и т.п. А такое можно сделать в линуксе?


Программил ты наверное на Visual Basic'е. Смеятся не буду, сам(когда-то давно-давно, хотя я ни такой и старый:)) начинал с него. Потом был VC++.И только потом перешел на Линукс и программлю теперь уже на С, использую компилятор gcc. Вообще ничего стыдного в этом нету, все мы когда-то с чего-то начинали.
Ладно это было лирическое отступление.

А отлаживать проги в Линуксе можно с помощью gdb. Трудно сказать чего НЕ
умеет эта прога. Почитай доки, маны по этой проге и заценишь её...

OK, спасибо, поискал, нашел доку.

Кстати, когда у меня была Mandrake 9.0, там в домашнем каталоге (или где-то еще, не помню точно), после падения прог создавались файлы *.core. А в Slackware 9.1 я таких файлов чего-то не вижу. Может, их создание отключено?

Да отключено по-умолчанию сделай ulimit -sC 10000000.