Няшная / Говнокод #6244 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 angry C nerd, (Updated )

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

  • ava TarasB # Ссылка на оригинал 0

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

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

      • ava absolut # Ссылка на оригинал 0 ⇈

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

            • ava gegMOPO4 # Ссылка на оригинал 0 ⇈

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

              • ava absolut # Ссылка на оригинал 0 ⇈

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

                • ava gegMOPO4 # Ссылка на оригинал 0 ⇈

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

    • ava SadKo # Ссылка на оригинал 0 ⇈

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

      • ava turdman # Ссылка на оригинал 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 # Ссылка на оригинал 0 ⇈

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

  • ava gegMOPO4 # Ссылка на оригинал 0

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

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

    • ava SIGSEGV # Ссылка на оригинал 0 ⇈

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

      • ava gegMOPO4 # Ссылка на оригинал 0 ⇈

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

  • ava ctm # Ссылка на оригинал 0

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

    • ava absolut # Ссылка на оригинал 0 ⇈

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

      • ava guest # Ссылка на оригинал 0 ⇈

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

  • ava REDNES # Ссылка на оригинал 0

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

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓