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

    +144

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    /* возвращает в buf последние 4 символа имени файла filename */
void getfileext(unsigned char *buf, unsigned char *filename)
{
    int i, j;

    for (i = strlen((char *) filename) - 4, j = 0; i < strlen((char *) filename); i++, j++)
        buf[j] = filename[i];

    return;
}

    Запостил: angry C nerd, 06 Апреля 2011

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

    • ava TarasB 06.04.2011 10:04 # +3

      strcpy(buf, filename[strlen(filename) - 4]);

      А если длина имени меньше 4?
      Ответить

      • ava turdman 06.04.2011 10:06 # 0

        undefined behavior, memory corruption, segfault на выбор :3
        Ответить

        • ava absolut 06.04.2011 11:10 # 0

          первым двум вариантам не суждено сбыться:
          будет простое копирование мусора в начале, главное чтобы в buf было достаточно места, но это никак не связано с длиной filename. с третьим - может быть.
          Ответить

          • ava gegMOPO4 06.04.2011 12:09 # +1

            Суждено.
            Ответить

            • ava absolut 06.04.2011 12:45 # 0

              только в силу кривости кода
              Ответить

              • ava gegMOPO4 06.04.2011 13:01 # +3

                Выход за пределы массива — это уже UB. А как оно будет проявляться, ошибкой доступа, зацикливанием, возвратом просто неверных данных, неправильной работой следующих операторов, исключением, остановом процессора — это уже детали, зависящие от окружения и случая.
                Ответить

                • ava absolut 06.04.2011 13:28 # 0

                  Что-то я не припомню, чтобы выход за пределы памяти по чтению приводил, скажем, к зацикливанию.
                  Ответить

                  • ava gegMOPO4 06.04.2011 14:25 # +3

                    Ну, например, сегментная организация памяти, выскочили в соседний сегмент. А strlen предполагает, что вся строка в одном сегменте и инкрементирует только смещение. И в этом сегменте нулевых байтов не оказалось.
                    Ответить

          • ava turdman 06.04.2011 12:11 # 0

            Ступил. Мне запись по [i] приглючилась
            Ответить

      • ava SIGSEGV 06.04.2011 10:34 # +3

        Тогда приду Я!!!
        Ответить

      • ava absolut 06.04.2011 11:06 # +5

        код на С в исполнении Тараса ... дорогого стоит :)))
        Ответить

      • ava Govnocoder#0xFF 06.04.2011 11:26 # −2

        Правильный ответ - Access violation.
        Ответить

      • ava SadKo 06.04.2011 11:49 # −2

        Как бэ лучше вот так ИМХО:
        int len = strlen(filename);
        if (len >= 4)
        strcpy(buf, filename[len - 4]);
        else
        buf[0] = '\0';
        Ответить

        • ava turdman 06.04.2011 11:55 # 0

          filename[len - 4] возвращает char, а не char*.
          длинна известна, так что можно обойтись memcpy
          strlen возвращает size_t, а не int

          size_t len = strlen(filename);
if (len >= 4)
  memcpy(buf, filename + len - 4, 5);
else
  *buf = 0;
          Ответить

        • ava absolut 06.04.2011 11:56 # +1

          TarasB & SadKo :) хватит ошибки копировать. Второй параметр strcpy должен быть указателем, а не символом.
          Ответить

    • ava gegMOPO4 06.04.2011 12:13 # +6

      Как все бросились копировать последние 4 символа. А ничего, что уже 1-я строка (особенно в контексте имени getfileext) — говнокод?

      Сигнатура функции тоже доставляет.
      Ответить

      • ava SIGSEGV 06.04.2011 19:13 # 0

        Особенно интересно, что автор хотел сказать своими unsigned char?
        Ответить

        • ava gegMOPO4 07.04.2011 20:29 # −1

          Неважно, хотел ли автор получить последние 4 символа или «расширение файла», но функция могла бы просто возвращать указатель. Например:
          unsigned char * getfileext(unsigned char *filename)
{
    size_t len = strlen((char *) filename);
    return len > 4 ? filename + (len - 4) : filename;
}
          Ответить

    • ava ctm 06.04.2011 14:13 # +1

      100% ГК - работать не будет нормально даже для 8:3.
      readme
      stdio.h
      я уж не говорю про длинные имена: test.local
      и совсем уж все плохо для файла/папки ".." например
      Ответить

      • ava absolut 06.04.2011 20:21 # +4

        он рассчитан на "правильные" имена формата *.ХХХ :)
        Ответить

        • ava guest 15.04.2011 09:15 # 0

          нуу имена формата *.хххх так же правильные как и *.хх
          Ответить

    • ava Lure Of Chaos 06.04.2011 22:21 # +2

      все-таки говно.
      Ответить

    • ava REDNES 24.08.2021 01:03 # 0

      Потом парень заканчивал, снимал презик, бросал его на пол, а эта самка ожидала следующего героя. Иногда за вечер через Наткину вагину «проходило» до двух десятков парней!
      Ответить

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