elenalaufeyson / post Goto Github PK

1. В программе есть ограничение на размер ленты, которое устанавливается после считывания ленты. По условию размер ленты должен быть ограничен только доступным объёмом памяти. Нужно реализовать логику изменения размера ленты "на лету", т. е. по ходу выполнения программы. Кроме того, у вас на печать вылезает "мусор" ещё до выхода за границы. Вот результат работы программы, которая двигает ленту влево в цикле: https://gist.github.com/gagarski/59f085e61c4cac7e6d8595e9b8b2af21
   Программа:

Лента:

Обратите внимание на выползающую единицу слева. Скорее всего, вы печатате ячейки за пределами буфера.
2. Приведите в порядок Code Style. Например, тут у вас полнейшее разнообразие из стилей именования переменных: camelCase, через подчёркивание, и то и другое одновременно, слитно без разделения слов. Предпочтительно было бы выбрать первый или второй и использовать везде только один из них.

3.
Если пользователь введёт слово длиннее 5 символов, то буфер переполнится. К сожалению, в Си просто в принципе нельзя безопасно сделать scanf("%s") и к ещё большему сожалению Си позволяет делать такие операции (есть ещё функция gets, которая в принципе ни в каком виде не может нормально использоваться). Вместо этого следует использовать операции чтения с ограничением размера. Например, тот же scanf, то с модификатором длины у %s (%5s), либо fgets. Следует учесть, что после чтения с ограничением размера всё, что не было считано останется во входном потоке. После считывания команды следует дочитать входной поток до конца, игнорируя считанное.
4. Ограничение на число выполняемых команд следует убрать или, как минимум, дать возможность пользователю поменять его.
5. (с минимальным приоритетом). Мне кажется очень странной логика команд 1 и 2.
- Есть ли причины не загружать ленту и программу сразу после старта?Если её не загрузить руками, то всё равно ничего больше сделать нельзя.
- Есть ли смысл ручной перезагрузки ленты и команд? Посреди выполнения всё равно их заменить нельзя, а после выполнения можно было бы просто перезапустить программу, если бы вы их загружали при старте.