1. Си / Говнокод #6799

    +146

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    int main(argc,argv)
int argc;
char *argv[];
{
int raw=0,httpcont=0;
int i,ilatex=0,ititle=1;
char *spoint=0;
char ttver[]=TTH_VERSION;
char ttname[20];
...
    strcpy(ttname,"Tt");
    strcat(ttname,TTH_NAME);
    strncpy(spoint-10-strlen(ttname),ttname,strlen(ttname));

    Понадобился мне конвертёр TeX в HTML.
    Скачал (http://hutchinson.belmont.ma.us/tth/), всё работает, но оказалось, что он не умеет работать с командой \begin{cases}
    Ну я решил его подправить, благо исходник открыт.
    Открыл я исходник и... решил написать с нуля свой.

    ПС Посоны, что делает 13я строчка?
    ППС Си - говно.

    Запостил: TarasB, 31 Мая 2011

    Комментарии (21) RSS

    • ava absolut 31.05.2011 16:09 # 0

      >что делает 13я строчка?
      копирует не более strlen(ttname) символов из ttname в (spoint-10-strlen(ttname)).
      Т.е в данном случае не копируется завершающий ноль.
      Для того, чтобы понять первый аргумент - не помешает развернуть 10ю строку многоточий.
      Ответить

    • ava sayidandrtfm 31.05.2011 16:31 # +3

      > Посоны, что делает 13я строчка?
      #ifdef TTH_GOLD
#define TTH_NAME "Hgold"
#else
#define TTH_NAME "H"
#endif
...
char tth_DOC[]="\n\
                Version XXXX (c)1997-2010 Ian Hutchinson\n\
...
if((spoint=strstr(tth_DOC,"XXXX"))){ /* Make version strings */
...

      Способ создания строки с инфо о версии. 

      printf(spoint);
      уже наконец.
      Ответить

      • ava sayidandrtfm 31.05.2011 16:42 # 0

        Конечно же, printf("%s",spoint); , если printf() не своя.
        Ответить

      • ava TarasB 31.05.2011 17:02 # 0

        Блин, не приметил.
        Получилась загадка в стиле Урсуса.
        Ответить

    • ava SmackMyBitchUp 31.05.2011 16:56 # 0

      Чет не понял
      spoint-10-strlen(ttname)
      Адрес первого символа spoint - 10 - длина строки ttname O_o
      Ответить

    • ava odmin 31.05.2011 19:09 # +3

      Аффтар - гавно.
      Си - рулит.
      Ответить

    • ava Govnocoder#0xFF 31.05.2011 20:32 # 0

      Минус один Тарасу за последний постскриптум.
      Ответить

    • ava TheCalligrapher 31.05.2011 23:18 # 0

      За одно только использование 'strncpy' для "ограниченного копирования" - расстрел на месте.
      Ответить

      • ava absolut 01.06.2011 06:41 # 0

        Чем плох strncpy? Кроме некопирования нуля в отдельных случаях конечно.
        Ответить

        • ava TheCalligrapher 01.06.2011 07:59 # +3

          'strncpy' плох тем, что для таких целей совершенно не предназначен.

          Назначение функции 'strncpy' - конвертирование нуль-терминированных строк (т.е. обычных С-шных строк) в строки "фиксированной ширины" (которые не обязательно нуль-терминированы). Строки фиксированной ширины использовались в файловой системе одной из старинных версий Юникса (System V). Там имя файла представлялось строкой фиксированной ширины в 14 символов. Вот специально для этого и была заведена функция 'strncpy'. Никакого отношения к ограниченному копированию нуль-терминированных строк эта функция не имеет (немотря на сбивающее с толку имя). К сожалению, функции ограниченного копирования нуль-терминированных строк в стандартной библиотеке С вообще нет.

          Многие реализации однако предоставляют функцию ограниченного копирования нуль-терминированных строк по имени 'strlcpy'. Это фактически стало де-факто стандартом для имени этой функции. Т.е. пользоваться в таких случаях надо именно 'strlcpy'. Если ваша реализация такой функции не предоставляет - напишите ее сами. А пользоваться для этих целей 'strncpy' - это забивание шурупов молотком. К сожалению, многие источники тупо втирают новичкам байки о том, что 'strncpy' - это якобы "функция безопасного копирования нуль-терминированных строк в С", что является банальным враньем. А потом начинается это кривое использование 'strncpy' для "безопасного копирования"...
          Ответить

    • ava guest8 09.04.2019 10:51 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий