Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5708)
2. C++ (2739)
3. Куча (2377)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (199)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (33)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

2. PHP / Говнокод #8804

   +153

   1. 1

   'email'   => array('/^[-_a-z0-9\'+*$^&%=~!?{}]++(?:\.[-_a-z0-9\'+*$^&%=~!?{}]+)*+@(?:(?![-.])[-a-z0-9.]+(?<![-.])\.[a-z]{2,6}|\d{1,3}(?:\.\d{1,3}){3})(?::\d++)?$/iD', 'Введите правильный e-mail, пожалуйста.')

   Вот такая проверка валидности email

   Sulik78, 13 Декабря 2011

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

3. Си / Говнокод #8802

   +137

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8
   9. 9

   #define MAKEPTR(p,o) (LPVOID) ( (DWORD)p + (DWORD)o )
   #define WRITE_OPCODE(pCode, x) \
      res = WriteProcessMemory( hProcess, pCode, &x, sizeof(x), &nWritten ); \
      if( !res ) return FALSE; \
      pCode = MAKEPTR(pCode,sizeof(x))
   #define WRITE_DWORD(pCode, x) \
      res = WriteProcessMemory( hProcess, pCode, &x, sizeof(x), &nWritten ); \
      if( !res ) return FALSE; \
      pCode = MAKEPTR(pCode,sizeof(x))

   макрос "сюрприз" хоть бы назвал яснее типа
   WRITE_OPCODE_AND_RETURN_VAL_IF_FAIL
   но чем так писать лучше уж условие не прятать
   
   http://ospy.googlecode.com/svn/!svn/bc/2/trunk/Agent/hooking.cpp

   63F45EF45RB65R6VR, 12 Декабря 2011

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

4. Objective C / Говнокод #8785

   −120

   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

   -(NSString*) getPackName:(int)index
   {	
   	NSString *res=nil;
   	switch (index) 
   	{
   		case 1:
   			res = @"Classic";
   			break;
   		case 2:
   			res = @"Wood";
   			break;
   		case 3:
   			res = @"Cowbell";
   			break;
   		case 4:
   			res = @"Metal";
   			break;
   		case 5:
   			res = @"Electro";
   			break;
   		default:
   			res = @"unknow";
   			break;
   	}	
   	return res;
   }

   Если не ошибаюсь, за тем же авторством, что и предыдущий.

   krypt, 09 Декабря 2011

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

5. PHP / Говнокод #8778

   +162

   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
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43

   /*
   	 * Функция получает выбор пользователя о типе доставки
   	 * проверяет ИД полученного ордера
   	 * и передает ордерИД и delivery_name
   	 */
   
   	public function actionSelectDelivery($OrderId = 1)
   	{
   		$modules = array();
   		foreach($this->module->components as $k => $dir)
   		{
   			$modules[$k] = $dir['show_name'];
   		}
   
   		$model = new Delivery();
   		$model->order_id = $OrderId;
   
   		if(isset($_POST['Delivery']))
   		{
   			$model->attributes = $_POST['Delivery'];
   			$params = $this->module->getParams();
   			$b = false;
   			if(isset($OrderId))
   			{
   				$modelOrder = CActiveRecord::model($params['OrderModel'])->findByPk($OrderId);
   				if(!isset($modelOrder))
   					$b = true;
   			}
   			else
   				$b = true;
   			if(!$b) {
   				$url = $this->createUrl('/delivery/default/createD', array( 'OrderId' => $OrderId, 'DeliveryName' => $model->delivery_name));
   			}
   			else
   				echo 'Error with order #' . $OrderId;
   		}
   
   		$this->render(__FUNCTION__, array(
   				'modules' => $modules,
   				'model' => $model,
   			)
   		);
   	}

   namreg, 09 Декабря 2011

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

6. Objective C / Говнокод #8774

   −115

   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
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63

   @try {
           NSMutableArray *items = nil;
           if(isGrouped){
               items = [NSMutableArray arrayWithArray:[_sortedItemsDict objectForKey:[_alphabet objectAtIndex:section]]];
           }else{
               items = _itemsArray;
           }
           [cell.textLabel setText:[[items objectAtIndex:row] name]?[[items objectAtIndex:row] name]:@"No Title"];
           
           if([[items objectAtIndex:row] isKindOfClass:[Song class]]){
               
               Song *song = (Song*)[items objectAtIndex:row];
               
               [cell.detailTextLabel setText:[NSString stringWithFormat:@"%@ %@", [song albumName]?[song albumName]:[NSString string], [song artistName]]?[song artistName]:[NSString string]];
               
               if([song isAvailableAtItunes])
                   NSLog(@"%@ %d", [song name], [song ID]);
               
               [cell setTag:[song ID]];
               
           }else if([[items objectAtIndex:row] isKindOfClass:[Album class]]){
               
               [cell.detailTextLabel setText:[NSString stringWithFormat:@"%@", [[items objectAtIndex:row] artistName]]?[[items objectAtIndex:row] artistName]:[NSString string]];
               
           }else if([[items objectAtIndex:row] isKindOfClass:[Podcast class]] || [[items objectAtIndex:row] isKindOfClass:[Audiobook class]]){
               [cell setTag:[[items objectAtIndex:row]ID]];
           }
           
           if(!isFromSelfLibrary){
           
               if(items && ([[items objectAtIndex:row] isKindOfClass:[Song class]] || [[items objectAtIndex:row] isKindOfClass:[Podcast class]] || [[items objectAtIndex:row] isKindOfClass:[Audiobook class]])){
               
                   if([[items objectAtIndex:row]isAvailableAtItunes]){
                   
                       for(UIView *view in [cell subviews]){
                           if(view.tag == 111){
                               [view setHidden:NO];
                           }
                       }
                   }else{
                       [[DataController instance] checkItem:[items objectAtIndex:row]];
                   }
                   [cellsDict setValue:cell forKey:[NSString stringWithFormat:@"%d",[cell tag]]];
               }
           }
           
           if(items && ![[items objectAtIndex:row] isKindOfClass:[Song class]] 
              && ![[items objectAtIndex:row] isKindOfClass:[Podcast class]]
              && ![[items objectAtIndex:row] isKindOfClass:[Audiobook class]])
           {
               [cell setAccessoryType:UITableViewCellAccessoryDisclosureIndicator];
           }
           [cell.textLabel setTextColor:[UIColor whiteColor]];
           [cell.textLabel setShadowColor:[UIColor blackColor]];
           [cell.textLabel setShadowOffset:CGSizeMake(0, 1)];
           [cell.detailTextLabel setTextColor:[UIColor lightGrayColor]];
           [cell.detailTextLabel setShadowOffset:CGSizeMake(0, 1)];
           [cell.detailTextLabel setShadowColor:[UIColor blackColor]];
           [tableView deselectRowAtIndexPath: indexPath animated: YES];
       }
       @catch (NSException *exception) {
           NSLog(@"olo1");
       }

   Не то, чтобы отлов исключения меня печалит.... :'(

   lime, 09 Декабря 2011

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

7. PHP / Говнокод #8760

   +156

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8
   9. 9

   case "delete":
      $id=$_GET['id'];
             if(!isset($id)){
             // no id? GTFO   
            header('Location: index.php?go=manage_costumes&act=view'); 
            exit();
            }else{
           // id exists
   ...

   )))

   mimits89, 07 Декабря 2011

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

8. PHP / Говнокод #8711

   +159

   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
   38. 38
   39. 39
   40. 40
   41. 41
   42. 42
   43. 43
   44. 44
   45. 45
   46. 46
   47. 47
   48. 48
   49. 49
   50. 50
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65

   private function isBonusAddOK($code,$ffStr)
   	{
   		switch($code)
   		{
   			case 1:if(
   						($this->isBFFOK($ffStr,40) || $this->isBFFOK($ffStr,38) || $this->isBFFOK($ffStr,37)) && 
   						$this->isBFFOK($ffStr,24) && $this->isBFFOK($ffStr,19) &&
   						($this->isBFFOK($ffStr,42) || $this->isBFFOK($ffStr,43) || $this->isBFFOK($ffStr,44))
   						)return true;return false;
   						
   			case 6:if(
   						($this->isBFFOK($ffStr,40) || $this->isBFFOK($ffStr,38) || $this->isBFFOK($ffStr,37)) && 
   						$this->isBFFOK($ffStr,24) && $this->isBFFOK($ffStr,20) &&
   						($this->isBFFOK($ffStr,42) || $this->isBFFOK($ffStr,43) || $this->isBFFOK($ffStr,44))
   						)return true;return false;
   						
   			case 11:if(
   						$this->isBFFOK($ffStr,24) &&
   						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
   						$this->isBFFOK($ffStr,62) &&
   						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
   						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5))
   						)return true;return false;
   						
   			case 14:if(
   						$this->isBFFOK($ffStr,24) &&
   						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
   						$this->isBFFOK($ffStr,61) &&
   						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
   						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5))
   						)return true;return false;
   			
   			case 17:if(
   						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
   						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
   						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5)) &&
   						$this->isBFFOK($ffStr,17)
   						)return true;return false;
   			
   			case 20:if(
   						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
   						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
   						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5)) &&
   						$this->isBFFOK($ffStr,25) && $this->isBFFOK($ffStr,21)
   						)return true;return false;
   						
   			case 23:if(
   						($this->isBFFOK($ffStr,16) || $this->isBFFOK($ffStr,358) || $this->isBFFOK($ffStr,360) || $this->isBFFOK($ffStr,361)) && 
   						($this->isBFFOK($ffStr,64) || $this->isBFFOK($ffStr,352) || $this->isBFFOK($ffStr,356) || $this->isBFFOK($ffStr,363)) &&
   						($this->isBFFOK($ffStr,3) || $this->isBFFOK($ffStr,4) || $this->isBFFOK($ffStr,5)) &&
   						$this->isBFFOK($ffStr,21)
   						)return true;return false;
   						
   						
   		.... еще куча строк до значения 88
   			
   			default: return true;
   		}
   	}
   	
   	private function isBFFOK($ffStr,$pos)
   	{
   		$val = substr($ffStr,$pos,1);//echo $pos.':'.$val;var_dump(($val != '0'));echo ' ';
   		return ($val != '0');
   		/*if($val == 0)return false;

   ИфЗебок

   kyzi007, 02 Декабря 2011

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

9. Ruby / Говнокод #8691

   −127

   1. 1

   Папка, создаваемая USB Disk Antivirus в корне всех дисков предотвращает заражение компьютера через Autorun. Пожалуйста не изменяйте и не удаляйте ее.

   Напомнило кое-что.

   dos_, 30 Ноября 2011

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

10. ActionScript / Говнокод #8672

    −122

    1. 1

    var mapDress:BitmapData = MathUtils.getTransformationMapTShort(bottomMovie.width, bottomMovie.height, pointFilterTopLeft, pointFilterBezierLeft, point1, pointFilterLimitLeft, pointFilterTopRight, pointFilterBezierRight, point2, pointFilterLimitRight, this.clothes[bottomIndex.layer][bottomIndex.child].colored_set[this.clothes[bottomIndex.layer][bottomIndex.child].currentColorVariant].variants[this.clothes[bottomIndex.layer][bottomIndex.child].currentWearVariant][this.currentView].arm_points_right, this.clothes[bottomIndex.layer][bottomIndex.child].colored_set[this.clothes[bottomIndex.layer][bottomIndex.child].currentColorVariant].variants[this.clothes[bottomIndex.layer][bottomIndex.child].currentWearVariant][this.currentView].arm_points_left);

    "752 см позитива!"

    Z-13, 29 Ноября 2011

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

11. C++ / Говнокод #8666

    +161

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9

    AnsiString DateArray = "";
    int DaysBetween = 0;
    DaysBetween = DateTimePicker2->Date - DateTimePicker1->Date;
    for (int i = 0; i <= DaysBetween; i++ )
    {
     if (i != 0)
     DateArray = DateArray + " OR ";
     DateArray =  DateArray + " date_made = '"+DateToStr(DateTimePicker1->Date + i)+"'";
    }

    Очевидно, подготавливаем запрос для отбора по диапазону дат :)

    labutinpa, 29 Ноября 2011

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

• ← влево
• вправо →

• 1972
• …
• 677
• 676
• 675
• 674
• 673
• 672
• 671
• 670
• 669
• …
• 1