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

    +137.6

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    for( unsigned int i = 0; i < strlen(str); i++)
{
    if ( str[i] == '-')
    {
        str[i] = '\0';
        break;
    }
}

    'strlen' в условии цикла - это вычисление инварианта в теле цикла.
    Вообще этот цикл реально можно заменить одним вызовом 'strchr'

    Запостил: benderlog, 18 Марта 2010

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

    • ava guest 18.03.2010 10:27 # −3.2

      Компилятор всеравно оптимизирует, и не факт что вариант со strchr будет быстрее.
      Ответить

      • ava benderlog 18.03.2010 15:55 # +4.8

        Конечно, компилятор всё всё вусмерть оптимизирует, а бесконечное кол-во обезьян напишут всего Шекспира.
        Но это не повод говнякать направо и налево.
        Ответить

      • ava crox 02.04.2010 16:47 # +1.8

        ничего он не оптимизирует. Строка меняется в цикле...
        Ответить

    • ava guest 18.03.2010 10:37 # −1.2

      Все зависит от компилятора, то что strchr быстрее работает я более чем уверен.
      Ответить

      • ava guest 18.03.2010 14:49 # −0.6

        ага, кто знает, может strlen постоянно вызываться будет... нельзя так наверняка, если пишется кроссплатформ. софт.
        Ответить

        • ava guest 22.03.2010 11:39 # 0

          А это уже проблема платформы, если все свое писать, так надо писать на асме, эффективно. Но кого волнует, что разработка затянется на десятилетия.
          Ответить

    • ava del 18.03.2010 12:11 # +0.2

      > for( unsigned int i = 0; ...

      Оно еще и скомпилится только c99-компилятором.
      Ответить

      • ava guest 18.03.2010 16:32 # +1.6

        Шо? Имеете ввиду, что сегодня нужно обратно совместимо с с90 писать?
        Ответить

    • ava guest 18.03.2010 22:24 # −1.2

      Блин. Хоть не в Си++ положили!
      Ответить

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