Говнокод #5638 — PHP — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP / Говнокод #5638

+163

function isInt($val) {
    return is_int($val) || (string)(int)$val === (string)$val;
}

Проверка на целое число

Запостил:

govnomes, 11 Февраля 2011

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

nethak 11.02.2011 18:47 # −1

Искусно конечно)
Ответить
Lure Of Chaos 11.02.2011 18:52 # 0

вот ведь, просто is_int почему не хватило? ))
Ответить
- govnomes 11.02.2011 18:55 # 0
  
  а если аргумент приходит в виде строки - "125"
  Ответить
- nethak 11.02.2011 18:55 # −1
  
  Он не всегда работает, точнее не всегда нам возврощает true, когда true очивиден, в итоге возвратит false... С get особенно проблемы бывают, но то что написано сдесь, не поддаётся логике в обще, кошмар и только.
  Ответить
  - code_master 11.02.2011 18:59 # 0
    
    а что бы вы предложили как решение такой проблемы? :)
    Ответить
    - Lure Of Chaos 11.02.2011 19:01 # 0
      
      ну тогда is_numeric
      Ответить
      - govnomes 11.02.2011 19:06 # 0
        
        is_numeric не проверяет на integer, туда можно пихнуть и float
        Ответить
        
        Lure Of Chaos 11.02.2011 19:08 # +1
        
        да кароче $val==(int) $val
        Ответить
        
        nethak 11.02.2011 19:09 # −1
        
        xD
        Ответить
        
        govnomes 11.02.2011 19:16 # 0
        
        $val = "";
        $val == (int)$val вернет true, но в данном случае $val - не число, а пустая строка.
        Ответить
        
        nethak 11.02.2011 19:17 # −1
        
        Он как бы пошутил если что!)))))))
        Ответить
        
        Lure Of Chaos 11.02.2011 19:20 # 0
        
        я сам уже путаюсь, где я серьезен, где шучу, уже такой стиль жизни - шутить )
        Ответить
        
        nethak 11.02.2011 19:22 # −1
        
        Так и надо, жить и не напрегатся! Похер на всё и вся)))
        Ответить
        
        Lure Of Chaos 11.02.2011 19:19 # 0
        
        ну еще и на empty проверять
        Ответить
        
        nethak 11.02.2011 19:22 # −1
        
        Да проще сразу:
        function isInt($val) {
        $val = "1"; # Мне виднее!
        }
        
        xDDDDD
        Ответить
        
        Lure Of Chaos 11.02.2011 19:24 # 0
        
        круче так:
        function isInt(&$val) {
        $val = "1"; # а мне похеру
        }
        Ответить
        
        nethak 11.02.2011 19:30 # −1
        
        Да точно тоже не чего, нам надо развивать гавнокодеров, и открыть помощь говонокодерам, и там им эти варианты предлагать)))
        Ответить
        
        Vasiliy 12.02.2011 09:38 # 0
        
        $val === (int)$val
        Ответить
        
        gegMOPO4 13.02.2011 22:31 # 0
        
        "1"
        Ответить
      - sectus 13.02.2011 16:02 # 0
        
        Может ctype_digit?
        Ответить
    - nethak 11.02.2011 19:08 # −1
      
      Если всё таки это гета, то разумеется сначала его отфильтровать и всёже перевести его в единое число отбросив всё лишнее intval, а уж потом проверил на целое число, наверно так, может был бы искусный говонокод, но речь не о том как написал бы я, а о том ужасе, который выше, так бы я не написал точно. Раз уж целое? Значит должно быть единым, и навернека есть ограничения по длине, смотря от ситуации, но хоть оно и число, проверить всё же надо... А может я с работы туплю )
      Ответить
hidespb 11.02.2011 20:56 # 0

вот как надо по-настоящему:
function isInt($val) {
return preg_match('@^-?\d+$@', $val);
}
:D
Ответить
- Lure Of Chaos 11.02.2011 21:16 # 0
  
  а потом удивляются, откуда в системе тормоза )
  Ответить

Говнокод: по колено в коде.

PHP / Говнокод #5638

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

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