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

    • ava Fike 14.07.2014 23:43 # 0

      Ну ничего уберхуевого (кроме самого жесткого ограничения количества записей на странице в принципе) нет. 10 только хардкодить не надо было, можно же дернуть первый элемент у $allowedItemsPerPage.
      Ответить

      • ava ftr 14.07.2014 23:52 # 0

        Смысл создавать массив, чтобы проверить, что число < 10?
        Ответить

        • ava Fike 15.07.2014 00:34 # +2

          Не меньше десяти, а в списке разрешенных вариантов. Который потом меняется всего в одном месте, что, признайтесь, удобно, и, очевидно, сделано на последующее расширение. Вот зачем его вообще создавать, а не задать верхнюю границу количества записей - другой вопрос.
          Ответить

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