Ксакеп, ну, это лучше, чем было.

Финальный вариант, со всеми оптимизациями и комментариями

Было

Было

[копировать] [скачать]

1. if ((($rights == 3 || $rights >= 6) && $rights >= $res['rights']) || (($res['user_id'] == $user_id && !$set_forum['upfp'] && ($start + $i) == $colmes && $res['time'] > $realtime - 900)AND !$type1['edit']) || (($res['user_id'] == $user_id && $set_forum['upfp'] && $start == 0 && $i == 1)AND !$type1['edit']) || (($type1['user_id']==$user_id && $i==1 && $start == 0) AND !$type1['edit'])) {
2.     if((($rights == 3 || $rights >= 6) && $rights > $res['rights']) and $user_id != $res['user_id']) {

Стало

Стало

[копировать] [скачать]

1. function rightsToControlPost($post_info, $topic_info, $start, $i, $post_count, $is_descending_sort, $is_current_user_curator)
2. {
3.     global $rights, $user_id, $realtime;
4.  
5.     $can_edit_post = false;
6.     $can_delete_post = false;
7.     $can_ban_user = false;
8.  
9.     $is_post_by_current_user = ($post_info['user_id'] == $user_id);
10.  
11.     $is_current_user_moderator = ($rights == 3 || $rights >= 6);
12.     if ($is_current_user_moderator)
13.     {
14.         // Модератор может изменять все посты
15.         $can_edit_post = true;
16.         // Модератор может удалять все посты
17.         $can_delete_post = true;
18.         // Модератор может банить младших по званию, но не себя
19.         $can_ban_user = $rights > $post_info['rights'] && !$is_post_by_current_user;
20.  
21.         return array($can_edit_post, $can_delete_post, $can_ban_user);
22.     }
23.  
24.     if ($is_current_user_curator)
25.     {
26.         // Куратор может изменять все посты
27.         $can_edit_post = true;
28.         // Куратор может удалять все посты
29.         $can_delete_post = true;
30.  
31.         return array($can_edit_post, $can_delete_post, $can_ban_user);
32.     }
33.  
34.     $is_ascending_sort = !$is_descending_sort;
35.     $is_first_post = ($start == 0 && $i == 1);
36.     $is_last_post = (($start + $i) == $post_count);
37.     $is_first_post_in_all_sorting_modes = ($is_ascending_sort && $is_first_post) || ($is_descending_sort && $is_last_post);
38.  
39.     $is_current_user_topicstarter = ($topic_info['user_id'] == $user_id);
40.     if ($is_current_user_topicstarter)
41.     {
42.         // Автор темы может изменять первый пост
43.         $can_edit_post = $is_first_post_in_all_sorting_modes;
44.     }
45.  
46.     // Пользователь может изменять и удалять свой последний пост в течение 15 минут
47.     $is_time_to_edit_post = $post_info['time'] > $realtime - 15 * 60;
48.     $can_edit_post = $can_edit_post || $is_time_to_edit_post;
49.     $can_delete_post = $can_delete_post || $is_time_to_edit_post;
50.  
51.  
52.     return array($can_edit_post, $can_delete_post, $can_ban_user);
53. }
54.  
55. list ($can_edit_post,
56.       $can_delete_post,
57.       $can_ban_user) = rightsToControlPost($res, $type1, $start, $i, $colmes, $set_forum['upfp'], $is_curator_user);
58. if ($can_edit_post)
59. {
60.     if ($can_ban_user)
61.     {

Так как эти проверки были в двух местах (веб-версия и мобильная версия), то пришлось засунуть в функцию.

aNNiMON, не понял это за тихий ужас? ООП не улучшило бы код или это противоречит совершенному коду?

Dinisimys, ООП бы улучшило, но там где были эти изменения, ООП встроить тяжело.

ООП, JohnCMS.
ООП, JohnCMS.

Dinisimys, Какое ООП? Это джон 3 версии, тогда и букв таких не знали. Тут абы человеку понять, а то только интерпритатор и разбирал

aNNiMON, не знал что джон на столько бездушно относился к прогерам. Но смысл тогда писать этот код сюда, если он все же не совершенный?

Dinisimys, не нашёл темы по рефакторингу, поэтому написал сюда. Тот пост - продолжение этого поста. Просто на память оставил и как пример возможного рефакторинга.
На совершенный код не претендую.

Для джона это уже совершенный код Один и тот же код в функцию запилить

Naik, Пост #489764, ну вот InsertSectorsType превратить в интерфейс с методом Date getDate(Context context, int index, BaseEvent model); и строчки с 22 по 57 превратятся в date = getDate(context, i, model);, а если избавиться от костыля if (i == 0 || ..) на 36 строке, то int index уберётся и будет красивее.

Объект Date today у тебя используется только в ветке SEE_EVENT_STATUS, так что можно его перенести с начала метода туда.

На 19 строке у тебя удар по перформансу, если объектов в events много и кому-то приспичит LinkedList передать. events.size() в отдельную переменную до цикла и будешь спокоен. Тем более, что размер списка у тебя и так на 12 строке впервые используется.

На 18 строке у тебя лишнее создание объекта. Год можешь взять с todayLocalDate.

Ну а то, что дальше внутри ветки if (date != null), какой-то легаси-велосипед, там без пяти капель не разобраться.

там сама суть
Проверка нужно ли вставлять категорию (если уже вставлено к примеру Today то уже не нужно его вставлять. Ну и месяц вставляется если нужно. Также в сектор даты записывается сколько дней до той даты, т.к. при выводе если дата в пределах недели то она отображается по другому и может развернуться в почасовый таймлайн