1. PHP / Говнокод #15756

    +152

    1. 1
    2. 2
    3. 3
    4. 4
    if (ctype_space($text[$pos] && $pos < $len)) {
    while (ctype_space($text[$pos++]) && $pos < $len);
    $pos--;
}

    Написал я когда-то такое и подумал: а какой смысл в $pos++ в последней итерации, если потом сразу $pos--?
    И тут до меня дошло, что
    while (ctype_space($text[$pos++]) && $pos < $len);
    это не то же самое, что
    while (ctype_space($text[$pos]) && $pos < $len) $pos++;
    ибо в первом случае $pos++ выполнится в последней итерации, даже если ctype_space возвратит false.
    И в итоге заоптимизировал до while (ctype_space($text[$pos]) && $pos < $len) $pos++; (уже без ифа и декремента).

    Мораль: нефиг выпендриваться (ставить инкременты и декременты в условия), если можно написать по-простому.

    Запостил: arzeth, 14 Апреля 2014

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

    • ava bormand 14.04.2014 20:50 # 0 

      while ($pos < $len && ctype_space($text[$pos]))
    $pos++
      Тогда уж.

      P.S. explode() задачу не решает?
      Ответить

    • ava Fike 20.04.2014 04:25 # −1

      мораль он читать нам будет, ты посмотри на него
      Ответить

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