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

    +157

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    <?php
    
    class ListWidget extends Widget
    {
    	public $itemsPerPage = 10;
    	private static $allowedItemsPerPage = array(10);
    	// ...
    
    	private function getTotalPages()
    	{
    		$this->itemsPerPage = in_array($this->itemsPerPage, self::$allowedItemsPerPage) ? $this->itemsPerPage : 10;
    		// ...
    	}
    }

    Надежная проверка.

    Запостил: ftr, 14 Июля 2014

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

    • Ну ничего уберхуевого (кроме самого жесткого ограничения количества записей на странице в принципе) нет. 10 только хардкодить не надо было, можно же дернуть первый элемент у $allowedItemsPerPage.
      Ответить
      • Смысл создавать массив, чтобы проверить, что число < 10?
        Ответить
        • Не меньше десяти, а в списке разрешенных вариантов. Который потом меняется всего в одном месте, что, признайтесь, удобно, и, очевидно, сделано на последующее расширение. Вот зачем его вообще создавать, а не задать верхнюю границу количества записей - другой вопрос.
          Ответить

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