Ci. Почему все таки происходит выход за грани

[Kozak Illya]

Код: [Выделить]

/* getlines: считует строку в s, возвращает ее длину */
int getlines(char s[])
{
   int c, i;


   for (i = 0; (c = getchar()) != EOF && c != '\n'; ++i)
      {
      if(i < MAX_LINE - 1 ) /* защита от выхода за границы массива. MAX_LINE -размер массива */
         s[i] = c;
      else ;
      }
   if (c == '\n')
      {
      s[i] = c;
      ++i;
      }
      s[i] = '\0';
   return i ;
}А если делаю так

Код: [Выделить]

int getlines(char s[])
{
   int c, i;


   for (i = 0; (c = getchar()) != EOF && c != '\n' && i < MAX_LINE - 1; ++i)
      {
         s[i] = c;
      }
   if (c == '\n')
      {
      s[i] = c;
      ++i;
      }
      s[i] = '\0';
   return i ;
}
то норм

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

Я создал эту тему не потому что не могу реализовать задуманное, а потому что не понимаю почему после того как в первом варианте в цикле for происходит проверка на выход за границы массива, выход все таки происходит?

[SergeyIT]

      else  break;

[Kozak Illya]

else  break; виходит из цыкла for
А я хочу чтоб for добегал до конца с пустым оператором и выдавал значение i.

[SergeyIT]

До какого конца? А если вообще не встретится EOF или '\n'? До бесконечности?

[Kozak Illya]

Дело не в том чтобы сделать код максимально безопасным, а в том что я не могу понять почему не работает мой if else вариант?

[SergeyIT]

А я не могу понять зачем у тебя строчка
 else ;
в коде. Может объяснишь?

[Kozak Illya]

Чтобы ничего не делать, и делать это явно   
без нее все равно выход за пределы 

[wl]

Там у Вас после цикла стоит оператор if, в котором есть запись в s [ i ].
Значение i здесь может быть любым, в том числе больше MAX_LINE.

Я имею в виду, сразу после

else ;

идет закрывающая фигурная скобка.
Так вот, эта скобка завершает цикл for.
Оператор

if (c == '\n')

выполняется один раз после окончания цикла.
В нем есть обращение к s [ i ].


PS. Форматирование кода ужасное.

[SergeyIT]

Код: [Выделить]

/* mygetline: считывает строку в s, возвращает ее длину  - в однобайтовой кодировке*/
int mygetline(char s[])
{
   int c, i, n;
   i = 0;
   while(1)
   {
     c = getchar();
     if( c == EOF) break;
     if(i < MAX_LINE - 1 ) /* записывать только первые MAX_LINE - 1 байт */
        s[i] = c;
     if( c == '\n') break;
     i++;
   }
   n = i++;
   if (i > MAX_LINE - 1) i = MAX_LINE - 1;
   s[i] = '\0';
   return n ;
}

[smart__]

Вот вроде во всех средствах разработки есть отладчик. Воспользоваться не судьба?

[Kozak Illya]

Я использую gedit & gcc. Буду осваивать gdb если он поможет в дальнейшем не делать таких ошибок.
Над примером просидел с час. Смотрел в одну строчку не видя других (Аж стыдно)  . Спасибо за терпение.

[SergeyIT]

Так работает?
(как обрабатывать '\n'- это ваша проблема  )

[Kozak Illya]

GDB открыл мне глаза на поиск ошибок в моих поделках. Я думал его использование будет трудной вещью.
У меня есть архив с статьями с закрывшегося linfoline.homedns.org

(Нажмите, чтобы показать/скрыть)

Манифест GNU
    * Что такое свободное ПО
    * Категории свободного ПО
    * Почему у програм не должно быть хозяев
    * История системы GNU
    * GNU и движение за свободно распространяемое ПО
    * Что такое Copyleft
    * Copyleft: прагматичный идеализм
    * Продажа свободных програм
    * Свободные программы и свободная документация
    * Так ли страшен Microsoft
    * Проблемы лицензии bsd
    * Проблемы лицензии Apple
    * Общественная лицензия Netscape
    * Программы, доступные в системе GNU и рекомендованные для использования с ней
    * Почему вам не стоит использовать Library GPL для вашей новой библиотеки
    * Ловушка X Windows
    * Linux и проект GNU
    * Право читать (The Right to Read), Ричард Столмен (Richard Stallman)
    * Хакеры: Герои компьютерной революции
    * GPL -УНИВЕРСАЛЬНАЯ ОБЩЕСТВЕННАЯ ЛИЦЕНЗИЯ GNU
    * LGPL -СТАНДАРТНАЯ ОБЩЕСТВЕННАЯ ЛИЦЕНЗИЯ ОГРАНИЧЕННОГО ПРИМЕНЕНИЯ
    * GFDL - ЛИЦЕНЗИЯ GNU НА СВОБОДНУЮ ДОКУМЕНТАЦИЮ

   
Руководства
     

         Утилиты системы GNU
         Руководство по GNU Emacs
         Програмирование Emacs Lisp
         Система документации Texinfo
         Стандарт файловой системы
         Стандарт кодирования GNU
         Описание языка ANSI C
         glibc
         gcc-2.7
         Отладка в DDD
         Отладка с помощью GDBN
         GNU Awk (gawk)
         BISON
         GNU Make
         Использование Automake
         autoconf - Создание скриптов конфигурации исходных текстов
         Введение в CVS
         Документация на CVS
         Subversion
         Скрипты BASH(Advanced Bash-Scripting Guide)
         Binutils
         Diff
         Gzip
         Tar
         Cairo руководство
         GTK+ руководство
         GTK+ справочное описание
         GLib cправочное описание
         GObject cправочное описание
         Open MOSIX
         Самосборный Linux
         Руководство iptables
         Руководство XEN
         Sendmail
         TCP/IP
         MySQL
         GNOME
         HOWTO

Кому надо обращайтесь.

[mkarasik]

GDB открыл мне глаза на поиск ошибок в моих поделках.

А это значит, что ты дорос до ddd. А там и до нормальных IDE рукой подать. Например eclipse CDT.

[Kozak Illya]

Говорят для программирования круче чем EMACS только вареные яйца. Думаю со временем перейду на него. Для собственного развития буду учить Lisp (Главное чтоб жена из дома не выгнала  )