1. C# / Говнокод #16336

    +135

    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
    Public CallGridRowModel(CallView callView)
    {
      . . .
      Caller = GetCallerOrCalleeNameColumn(callView, true)
      Callee = GetCallerOrCalleeNameColumn(callView, false)
      . . .
    }
      
      
    private object GetCallerOrCalleeNameColumn(CallView callView, bool isCallerNameColumn)
    {
      if(isCallerNameColumn)
      {
        if(. . .) return . . .;
      }
      if(!isCallerNameColumn)
      {
        if(. . .) return . . .;
      }
      
      if(isCallerNameColumn)
      {
        return . . .;
      }
      else
      {
        return . . .;
      }
    }

    onixar, 15 Июля 2014

    Комментарии (25)
  2. JavaScript / Говнокод #16335

    +155

    1. 1
    Object.prototype.toString.call(o).match(/\[object (\w+)\]/)[1];

    Реализация typeof

    kissarat, 15 Июля 2014

    Комментарии (10)
  3. 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)
  4. Ruby / Говнокод #16324

    −151

    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
    def get_team_choices_with_index
        school_ids = params[:school_id].split(",")
        @index = params[:index]
    
        @teams_array = Array.new
        school_ids.each do |s|
          next if s.eql? "0"
          School.find(s).teams.each do |t|
            @teams_array << t
          end
        end
    
        @teams_array.flatten!
        @teams_array
      end

    А должно быть что-то вроде
    @teams_array = Team.where(school_id: school_ids)

    dj_Andreus, 14 Июля 2014

    Комментарии (1)
  5. 1C / Говнокод #16323

    −160

    1. 1
    2. 2
    3. 3
    Если Год (ТекДата) = 1 Тогда
      ТекДата = ТекущаяДата ();
     КонецЕсли;

    вот так надо проверять дату на заполненность!

    alexinzaz, 14 Июля 2014

    Комментарии (12)
  6. C++ / Говнокод #16322

    +28

    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
    TLSOSI7Command &TLSLongTelegram::GetOSI7Command () const
    {
       if(!m_pOSI7Command) { // private pointer not initialized
                  // WARNING! Quick'n'dirty! is UNINITIALIZED althoug it should be - just for preventing abnormal end!
                  const_cast<TLSOSI7Command*>(m_pOSI7Command)=new TLSOSI7Command;
       } // private pointer not initialized
       else 
       { 
                 // NOT private pointer not initialized
       }; 
       // NOT private pointer not initialized
       return *m_pOSI7Command;
    }

    m)

    blackhearted, 14 Июля 2014

    Комментарии (21)
  7. JavaScript / Говнокод #16321

    +155

    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
    this.resize = function(d) {
        var curSize, newSize;
        if (DOM && settings.resiziable === true) {
            curSize = $(DOM).height();
            if (d < 0) {
                if (settings.min) {
                    if (curSize + d < settings.min) {
                        newSize = settings.min;
                    } else {
                        newSize = curSize + d;
                    }
                } else {
                    newSize = curSize + d;
                }
            } else {
                if (settings.max) {
                    if (curSize + d > settings.max) {
                        newSize = settings.max;
                    } else {
                        newSize = curSize + d;
                    }
                } else {
                    newSize = curSize + d;
                }
            }
    
            if (newSize !== curSize) {
                $(DOM).height(newSize);
            }
        }
    };

    Что-то меня корёжит немного от этого метода.

    dunmaksim, 14 Июля 2014

    Комментарии (18)
  8. JavaScript / Говнокод #16320

    +170

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    > typeof(null)
    "object"
    
    > Object.keys(null)
    TypeError: null is not an object

    bormand, 14 Июля 2014

    Комментарии (10)
  9. JavaScript / Говнокод #16319

    +160

    1. 1
    2. 2
    3. 3
    if (a % 1 == 0) {
        // a - целое число
    }

    Вот такая вот проверка.

    bormand, 14 Июля 2014

    Комментарии (16)
  10. C++ / Говнокод #16318

    +27

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    @@ -50,9 +50,6 @@ void AreaTrigger::RemoveFromWorld()
          ///- Remove the AreaTrigger from the accessor and from all lists of objects in world
          if (IsInWorld())
          {
     -        if (!IsInWorld())
     -            return;
     -
              WorldObject::RemoveFromWorld();
              sObjectAccessor->RemoveObject(this);
          }

    "Core/AreaTrigger: Remove useful check"
    https://github.com/TrinityCore/TrinityCore/commit/d920463fe5fe6eee7c4a8e2d777720fcfbe6f08f

    Actine, 13 Июля 2014

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