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

    +3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    public function isOverflowed(){
            if(is_null($this->isOverflow)){
                $this->isOverflow = ($this->getMemberCount() > $this->getMaxMembers());
            }
            return $this->isOverflow === true;
    
        }

    Придадим значимости результату.

    AndryG, 26 Ноября 2016

    Комментарии (5)
  2. PHP / Говнокод #21724

    −1

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    switch ($summ) {
                            case '2':
                                $data['summ'] = '2';
                                break;
    			case '7':
                                $data['summ'] = 3';
                                break;
    			case '15':
                                $data['summ'] = '4';
                                break;
    			case '25':
                                $data['summ'] = '5';
                                break;
    			case '97':
                                $data['summ'] = '6';
                                break;
    			default:
                                $data['summ'] = '2';
                                break;
                        }

    Это диагноз!

    drop_database, 25 Ноября 2016

    Комментарии (7)
  3. PHP / Говнокод #21719

    −1

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    public function product(Request $request, $id = null)
        {
            $this->template['product'] = $id ? Product::find($id) : new Product();
            $this->template['product'] || abort(404);
            if(!$this->template['product']->exists && count($input)){
                if(array_key_exists('category', $input) && $input['category']){
                    $category = Category::where('alias','=',$input['category'])->first();
                    if($category){
                        $this->template['product']->category_id = $category->id;
                    }
                }
                if(array_key_exists('brand', $input) && $input['brand']){
                    $car = Car::where('alias','=', $input['brand'])->first();
                    if($car){
                        $this->template['product']->car_id = $car->id;
                    }
                }
                if(array_key_exists('model', $input) && $input['model']){
                    $car_model = CarModel::where('alias','=',$input['model'])->first();
                    if($car_model){
                        $this->template['product']->car_model_id = $car_model->id;
                    }
                }
                if(array_key_exists('year', $input) && $input['year']){
                    $this->template
                }
            }
    
            $this->template['categories'] = Category::all();
            $this->template['brands']     = Brand::all();
            $this->template['years']      = Year::all()->sortBy('year');
            $this->template['cars']       = Car::all();
            $this->template['car_models'] = CarModel::all();
    
    
            return View::make('dashboard.product', $this->template);
        }

    Не знаю наверное у Вас так не принято) но подскажите как избавиться от говноёлки?!)

    Vilintritenmert, 24 Ноября 2016

    Комментарии (3)
  4. PHP / Говнокод #21718

    +1

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    switch ($out['list']) {
      case 'sale': $out['d']=new CShotObjectFlats('ShotObjectFlats',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='flat'; break;
      case 'rent': $out['d']=new CShotObjectFlats('ShotObjectFlats',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='flat'; break;
      case 'exch': $out['d']=new CShotObjectFlats('ShotObjectFlats',$out['date_exch'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='flat_exch'; break;
      case 'purchase': $out['d']=new CShotObjectFlatsZay('ShotObjectFlats',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
      case 'rent_use': $out['d']=new CShotObjectFlatsZay('ShotObjectFlats',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
    
      case 'sale_h': $out['d']=new CShotObjectHouses('ShotObjectHouses',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='house'; break;
      case 'rent_h': $out['d']=new CShotObjectHouses('ShotObjectHouses',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='house'; break;
      case 'exch_h': $out['d']=new CShotObjectHouses('ShotObjectHouses',$out['date_exch'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='house_exch'; break;
      case 'purchase_h': $out['d']=new CShotObjectHousesZay('ShotObjectHouses',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
      case 'rent_use_h': $out['d']=new CShotObjectHousesZay('ShotObjectHouses',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
    
      case 'sale_g': $out['d']=new CShotObjectGarages('ShotObjectGarages',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='garage'; break;
      case 'rent_g': $out['d']=new CShotObjectGarages('ShotObjectGarages',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='garage'; break;
      case 'purchase_g': $out['d']=new CShotObjectGaragesZay('ShotObjectGarages',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
      case 'rent_use_g': $out['d']=new CShotObjectGaragesZay('ShotObjectGarages',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
    
      case 'sale_l': $out['d']=new CShotObjectLands('ShotObjectLands',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='land'; break;
      case 'rent_l': $out['d']=new CShotObjectLands('ShotObjectLands',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); $out['object_list']='land'; break;
      case 'purchase_l': $out['d']=new CShotObjectLandsZay('ShotObjectLands',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
      case 'rent_use_l': $out['d']=new CShotObjectLandsZay('ShotObjectLands',$out['date'],$out['reference'],$out['list_level'],($out['DataIn']->http['TYPE_ID']==2)?0:1); break;
    }

    Отрисовка результата поиска в зависимости от контекста

    SeniorShaurman, 24 Ноября 2016

    Комментарии (11)
  5. PHP / Говнокод #21715

    −1

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    /**
     * Implements hook_form_BASE_FORM_ID_alter().
     */
    function admin_core_form_node_form_alter(&$form, FormStateInterface &$form_state) {
      \Drupal::service('admin_core.post_draft')->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.post_moderation_information')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.feature_post')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.community_verifield')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.editor_plus')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.moderations_elements_access')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.scheduler_form_element_view')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.preview_post')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.min_max')
        ->nodeFormAlter($form, $form_state);
      \Drupal::service('admin_core.seo_fields')
          ->nodeFormAlter($form, $form_state);
    }

    Архитектор Drupal 8

    ldv_a, 23 Ноября 2016

    Комментарии (0)
  6. PHP / Говнокод #21714

    +1

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    switch($_REQUEST["su_type_id"] || $_REQUEST["su_type_id1"]) {
                case "1":
                    $filter[] = " !`contr_user` AND u.id NOT IN (SELECT DISTINCT(`user_id`) FROM `link_sc_chiefs`) AND u.id NOT IN (SELECT DISTINCT(`user_id`) FROM `list_empl`) ";
                    break;
    
                case "2":
                    $filter[] = " `contr_user`>0 ";
                    break;
    
                case "3":
                    # EF: Новая система доступов на СЦ
                    #$filter[] = " (u.id IN (SELECT DISTINCT(`user_id`) FROM `list_empl`)) ";
                    $filter[] = "( exists (select *
      from link_sc_user lsc,
           user_sc_access usa
     where usa.user_id = ".$USER->getId()."
       and lsc.user_id = u.id
       and usa.sc_id = lsc.sc_id) )";
                    break;
    
                case "4":
                    $filter[] = " u.id IN (SELECT DISTINCT(`user_id`) FROM `link_sc_chiefs`) ";
                    break;
    
    
                default:
                    break;
            }

    смотреть на выражение в switch

    pahhan, 23 Ноября 2016

    Комментарии (6)
  7. PHP / Говнокод #21710

    −1

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    list(
    	$sl_year_from,
    	$sl_year_to
    ) = \Company\Module\Database\ModuleDatabase::getYearTimeFrameOfStatisticSales(
    	$this->mysqli_aff, $this->set_user_id, $location
    );
    	list(
    	$cl_year_from,
    	$cl_year_to
    ) = \Company\Module\Database\ModuleDatabase::getYearTimeFrameOfStatisticClicks(
    	$this->mysqli_aff, $this->set_user_id, $location
    );

    без слов, одни эмоции

    kester, 23 Ноября 2016

    Комментарии (0)
  8. PHP / Говнокод #21699

    0

    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
    15. 15
    16. 16
    17. 17
    18. 18
    class VirtualSection {
          const CLASS_NAME = __CLASS__;
          ...
          protected function __construct() {
                ....
          }
          ....
    }
    
    
    $vsect = new VirtualSection ();
    ...
    
    function SetVSectionProps($obj) {
         if(is_object($obj) && get_class($obj) == VirtualSection::CLASS_NAME) {
              ... 
         }
    }

    instanceof от senior PHP developer;

    aat, 20 Ноября 2016

    Комментарии (8)
  9. PHP / Говнокод #21696

    +3

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    /**
    * Конвертация формата времени в удобочитаемый вид
    * @param mixed $data
    * @param mixed $year
    * @param mixed $time
    * @param mixed $second
    */
    public static function data_convert($data, $year, $time, $second){
        $res = "";
        $part = explode(" " , $data);
        $ymd = explode ("-", $part[0]);
        if(count($ymd)==1){
        $ymd = explode (".", $part[0]);
        $ymd[2]="20".$ymd[2];
        $ymd = array_reverse($ymd);
        }
        $hms = explode (":", $part[1]);
        if ($year == 1) {$res .= $ymd[2]; $res .= '.'.$ymd[1]; $res .= '.'.$ymd[0];}
        if ($time == 1) {$res .= ' '.$hms[0]; $res .= ':'.$hms[1]; if ($second == 1) $res .= ':'.$hms[2];}
        return $res;
      }

    На вход из БД могут заходить два вида дат: (mysql)
    - TIMESTAMP "2016-11-13 05:29:38"
    - VARCHAR "13.11.16 05:29"
    На выходе получаем единый формат "16.11.2016 05:29"

    Хранить дату в varchar ? Это модно ))

    Гурманов прошу обратить внимание на строку 19. Как там органично не указаны {} в секундах

    AndryG, 20 Ноября 2016

    Комментарии (73)
  10. PHP / Говнокод #21694

    −1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    $fields = array_keys($data);
    $n = count($data);
    for($i=0; $i < $n; $i++){
        if($data[$fields[$i]]['type']==1){ ?>
            <div class="incConfBox">
    ...

    Для тренировки мозгов читающих коллег.

    AndryG, 20 Ноября 2016

    Комментарии (6)