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

    +6

    1. 1
    2. 2
    3. 3
    if(is_int((int)$companyID)) {
       ...
    }

    Конкретная проверка

    aat, 29 Ноября 2016

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

    +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
    function to_array($a)
    {
        if (is_array($a))
            return $a;
        elseif ((false === $a) || (NULL === $a))
            return array();
        else
            return array($a);
    }
    
    function _or($a, $b)
    {
        if ($a) return $a;
        return $b;
    }

    Немножко хелперов

    SeniorShaurman, 29 Ноября 2016

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

    +1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    function mtime()
    {
      list($usec, $sec) = explode(' ', microtime());
      return $usec + $sec;
    }

    Что-то со временем

    SeniorShaurman, 29 Ноября 2016

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

    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
    foreach($mainCategories as $category){
                $subcategory = AdCategory::where('parent_id', $category->id)->get();
                if(!$subcategory->isEmpty()){
                    foreach($subcategory as $item){
                        $subcategoryArray[] = $item->id;
                    }
    	
                    $category->adsCount = Ad::whereIn('category_id', $subcategoryArray)->onlyRegion($currentRegion->id)->active()->count();
                    $subcategoryArray = [];
                }else{
                    $category->adsCount = Ad::where('category_id', $category->id)->onlyRegion($currentRegion->id)->active()->count();
                }
            }

    Получаем количество элементов в категориях

    infolevelru, 28 Ноября 2016

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

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    if ($project->Budget
                            and ((((new DateTime($project->Budget->update_time)) < (new DateTime((new DateTime('-1 day'))->format('Y-m-d'))))
                            and (((new DateTime('now')) > (new DateTime($project->Budget->from))->add(new \DateInterval('P1D')))
                            and ((new DateTime('now')) < (new DateTime($project->Budget->to))->add(new \DateInterval('P2D')))))
                            or ((new DateTime($project->Budget->update_time)) < (new DateTime($project->Budget->to))))
                            and (count(unserialize($project->campaignIDS)) > 0)
                        ) {
                              // Делвем обновление данных в кэше по бюджету 
                        }

    Пришло время посмеяться над собой и переписать свой старый говнокод =) условие количеством скобок лисп напоминает.

    fens, 26 Ноября 2016

    Комментарии (13)
  6. 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)
  7. 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)
  8. 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)
  9. 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)
  10. 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)