1. Java / Говнокод #1556

    +71.9

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    List<AbstractPOJO> ordersList = getList();		
int num = CollectionUtils.indexOf(ordersList, orders.getCurrentOrder());
if (num > -1 && num < ordersList.size()) {
	int objectsPerPageCount = orders.getObjectsPerPage();
	for (int i = 0; i < ordersList.size(); i++) {
		if (num >= objectsPerPageCount * i && num < objectsPerPageCount * (i + 1)) {
			orders.setPage(i + 1);
			break;
		}
	}
}

    установка текущей страницы...
    переписалось в:
    List<AbstractPOJO> ordersList = getList();
    int num = CollectionUtils.indexOf(ordersList, orders.getCurrentOrder());
    if (num > -1) {
    orders.setPage(num / orders.getObjectsPerPage());
    }

    Запостил: asdqwezx, 12 Августа 2009

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

    • ava пъянь 14.08.2009 02:14 # 0

      чета сдается мне говнокод и ваш код буит разные страницы показывать
      Ответить

    • ava EfremoffPaul 14.08.2009 13:15 # 0

      +1
      Ответить

    • ava UNV 15.08.2009 12:48 # +1

      int num = getList().indexOf(orders.getCurrentOrder ());
      if (num >= 0) orders.setPage(num / orders.getObjectsPerPage() + 1);
      Ответить

      • ava пъянь 15.08.2009 13:34 # 0

        ой ненавижу иф без скобок
        у говнопостера код лучше этого.

        тока мне кажется это все надо выносить в отдельный метод
        а то кажется мне таких говно кусочков кода в каждом списке

        ну это прочем для любителей сверх задротства... возможно оно и так нормально :)
        Ответить

        • ava пъянь 15.08.2009 13:36 # 0

          чета многа кажется мне сеня а это значит пятница удалась
          Ответить

        • ava UNV 15.08.2009 14:30 # −1

          if (а также while и for) без скобок вполне допустим в случае однострочного оператора небольшой длины. Я раньше тоже всегда требовал полного объёма скобок, но в последнее время сделал себе послабления - получается более компактно без ущерба читаемости и поддерживаемости.

          if (условие) оператор;

          if (условие)
          {
          ____оператор1;
          ____оператор2;
          }

          Однако, нехорошо писать однострочный оператор вот так:

          if (условие)
          ____оператор;

          Чисто из параноидальных соображений, что кто-то впоследствие припишет вот такое:

          if (условие)
          ____оператор1;
          ____оператор2; (!!!)
          Ответить

          • ava UNV 15.08.2009 14:34 # 0

            Потому если однострочный оператор имеет достаточно большую длину или у if'а есть else, то стоит писать полный объём скобок.

            if (условие)
            {
            ____операторБольшойДлины;
            }

            if (условие)
            {
            ____оператор1;
            }
            else
            {
            ____оператор2;
            }

            Это, естественно, лично мои предпочтения к оформлению кода :)
            Ответить

            • ava пъянь 15.08.2009 17:02 # 0

              да я понимаю что одной строкой нормально но все равно не люблю
              очень часто потом приходится чтонить дописывать внутрь иф
              Ответить

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