По колено в коде — Говнокод.ру

PHP / Говнокод #4483

+168

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45

//Курьерская доставка по Москве оплата при получении
function pay10 ($sum, $num,$weight)
{
$sum=$sum+200;
return $sum; 
}


function pay12 ($sum, $num,$weight)
{
$sum=$sum+200;
return $sum; 
}

//Почта оплата наложенный платеж
function pay20 ($sum, $num,$weight)
{
$sum=$sum+150;
return $sum; 
}

function pay21 ($sum, $num,$weight)
{
$sum=$sum+150;
return $sum; 
}

//Авиа почта оплата наложенный платеж
function pay30 ($sum, $num,$weight)
{
$sum=$sum+300;
return $sum; 
}

function pay31 ($sum, $num,$weight)
{
$sum=$sum+300;
return $sum; 
}

function pay43 ($sum, $num,$weight)
{
$sum=$sum+450;
return $sum; 
}

Библиотека функций в одном известном интернет-магазине...
Каким местом думал автор остается вопросом.

ejiko, 29 Октября 2010

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

PHP / Говнокод #4482

+163

1
2
3
4
5
6
7
8

if ($context['make_event'])
		echo '
			var monthLength = [31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31];

			function generateDays()
			{
...
			}';

К сожалению, так во всем проекте..., весь HTML и JS выводиться через echo, + отсутствует нормальная логика системы..., файлов проекта over 150...

1_and_0, 29 Октября 2010

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

Java / Говнокод #4481

+145

1
2
3
4
5

private static final Color COLOR_GREEN = Color.GREEN;
  private static final Color COLOR_GREEN_DARK = JBoard.COLOR_GREEN;
  private static final Color COLOR_RED = Color.RED;
  private static final Color COLOR_RED_DARK = JBoard.COLOR_RED;
  private static final Color COLR_GRAY = Color.GRAY;

упс, это я случайно )))) если вы это видите, то я это уже рефакторю ))))

Lure Of Chaos, 29 Октября 2010

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

PHP / Говнокод #4479

+162

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

<?php

class Model_Crypt_FactoryCipher {
    
    const BlOWFISH = 'Model_Crypt_BlowFishCipher';
    
    
    /**
     * создает экземпляр cipher для указанного алгоритма
     * 
     * @return Model_Crypt_BlowFishCipher
     */
    public static function createCipher($crypt) {
        
        $cipher = new $crypt;
        
        if ($cipher instanceof Model_Crypt_BlowFishCipher) {
            return new Model_Crypt_BlowFishCipher ();
        } else {
            throw new Exception ( "This algoritm <$crypt> not implement yet" );
        }
    }

}

Гениальная фабрика. Используется один раз путем передачи константы "BlOWFISH" в метод createCipher()

user654321, 29 Октября 2010

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

Perl / Говнокод #4478

−118

1

return (($publish && $filename ne '--') ? 1 : 0);

Особенно умиляют скобочки

krushi, 29 Октября 2010

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

Perl / Говнокод #4477

−122

1

my $exp_data = $map->get_experiments_data($gg, $map, $signals_sign, $distinct_lists);

Пуленепробиваемый код! На случае, если вдруг объект забудет, кто он такой.

krushi, 29 Октября 2010

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

JavaScript / Говнокод #4476

+161

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36

/* Fade-In Page script ©2008 John Davenport Scheuer
   As first seen in http://www.dynamicdrive.com/forums/
   username:jscheuer1 - This credit must remain for legal use.
   */
 
fadeInPage.speed=15; //Set speed of transition for non-IE, lower numbers are faster, 20 is the minimum safe value
fadeInPage.bg='#000'; //Set backgroud style (color or color and image) of transition division for non-IE, should match page background or the predominant color of the page
 
///////////////// Stop Editing /////////////////
 
function fadeInPage(){
var el=document.getElementById("fadeDiv");
el.style[fadeInPage.prprt] = el.style[fadeInPage.prprt] == ''? 1 : el.style[fadeInPage.prprt];
if (el.style[fadeInPage.prprt] > 0){
el.style[fadeInPage.prprt] = el.style[fadeInPage.prprt] - 0.02;
setTimeout("fadeInPage()", fadeInPage.speed);
}
else {
el.style[fadeInPage.prprt] = 0;
if(document.removeChild)
el.parentNode.removeChild(el);
}
}
if(document.documentElement&&document.documentElement.style){
fadeInPage.d=document.documentElement, fadeInPage.t=function(o){return typeof fadeInPage.d.style[o]=='string'};
fadeInPage.prprt=fadeInPage.t('opacity')? 'opacity' : fadeInPage.t('MozOpacity')? 'MozOpacity' : fadeInPage.t('KhtmlOpacity')? 'KhtmlOpacity' : null;
}
fadeInPage.set=function(){
var prop=fadeInPage.prprt=='opacity'? 'opacity' : fadeInPage.prprt=='MozOpacity'? '-moz-opacity' : '-khtml-opacity';
document.write('\n<style type="text/css">\n#fadeDiv {\nheight:'+window.innerHeight+'px;display:block;position:fixed;'+
'z-index:10000;top:0;left:0;background:'+fadeInPage.bg+';width:100%;\n'+ prop +':1;\n}\n<\/style>\n');
}
if(window.addEventListener&&fadeInPage.prprt){
fadeInPage.set();
window.addEventListener('load', fadeInPage, false);
}

Зашел я на сайт родного биллинга. Не работает. Смотрю - остался слой от fade-in (нахрена он в биллинге).
Открыл исходники сайта и увидел там ЭТО.
Отступы авторские. Очень радует "document.write('\n<style" в цикле.

m1el, 29 Октября 2010

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

PHP / Говнокод #4475

+177

1
2
3
4

if (!($user_info['room'] == 900002) || $user_info['room'] != 900002) { 
      header("Location: main.php");  
      die(); 
  }

Разгребаю чужой код и нарыл.
Чтобы условие выглядело посолидней ... или я чего-нить не понимаю.

AndryG, 28 Октября 2010

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

VisualBasic / Говнокод #4474

−84

01
02
03
04
05
06
07
08
09
10
11
12
13

If Not CheckX Then Goto 1
    For i = 0 to USecX
	Goto 2
1:
    For i2 = 0 to USecY
2:
  	...
	If CheckX Then Goto 3
    Next
    If Not CheckX then Goto 4
3:
    Next
4:

Как-то потребовалось выполнить немалый участок кода, но в двух разных случаях нужно было использовать два разных фора. В отдельную процедуру было очень неудобно выносить (пришлось бы передавать слишком много переменных), так что пришлось сделать этот... свичджампинг форов :)

DragDen, 28 Октября 2010