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

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

+75

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
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61

public static void getShops(Favorite favoriteBrands,
			Favorite favoriteShops, StationItem station,
			WorkingTimeFilter workTimeFilter, boolean[] filter, String keyword,
			GeoLocationPointItem userLocation,
			GeoLocationPointItem searchLocation, String catalogId,
			String netId, String page, String pageSize, String api,
			AsyncTask callback, ConnectionWatcher watcher) {
//...
//упаковка кучи параметров


		boolean emptyFavBrands = false;
		if (favoriteBrands != null) {
			int i = 0;
			for (FavoriteItem item : favoriteBrands) {
				String id = item.getId();
				Log.i(tag, "addFaforiteBrand " + id);
				params.add(new BasicNameValuePair(PARAMS_FAVORITE_BRAND_ID
						+ "[" + i + "]", id));
				i++;
			}
			if (i == 0)
				emptyFavBrands = true;
		}

		boolean emptyFavShops = false;
		if (favoriteShops != null) {
			int i = 0;
			for (FavoriteItem item : favoriteShops) {
				String id = item.getId();
				Log.i(tag, "addFaforiteShop " + id);
				params.add(new BasicNameValuePair(PARAMS_FAVORITE_SHOP_ID + "[" + i
						+ "]", id));
				i++;
			}
			if (i == 0)
				emptyFavShops = true;
		}

		boolean error = false;

		if (emptyFavBrands && emptyFavShops) {
			if (favoriteBrands != null || favoriteShops != null)
				error = true;
		}

		if (emptyFavShops) {
			if (favoriteBrands == null && favoriteShops != null)
				error = true;
		}

		if (emptyFavBrands) {
			if (favoriteBrands != null && favoriteShops == null)
				error = true;
		}

		if (error) {
			callback.onError(JsonEntity.FAVORITE_ERROR, new IOException());
			return;
		}
}

Обращение к веб-сервису. Вызов этого метода при переносе IDE "растекается" на 5-6 строк.
Самое интересное - проверка корректности параметров находится в самом конце метода (длиной 150 строк)

rphx, 23 Февраля 2011

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

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

+166

1
2
3
4

function assign($params)
{              
        return $params[0];
}

Функция с одной коммерческой cms. Можно же обойтись и без функции. Да и желательно сделать проверку на существование переменной.

codosub, 23 Февраля 2011

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

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

+171

01
02
03
04
05
06
07
08
09
10

if($nav_url=="blog.php")
{
$ahaha='2';
eval('$code["$elementtype"] .= "' . fetch_template('navbar_link') . '";');
$ahaha='';
}
else
{
eval('$code["$elementtype"] .= "' . fetch_template('navbar_link') . '";');
}

Ахаха!

symbix, 23 Февраля 2011

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

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

+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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54

<?php
$t1 = microtime();
define('CMS_LOAD', TRUE);

require_once('./inc/config.inc.php');
require_once('./inc/main_fnc.inc.php');


cms_MysqlConnect();
unset($cms); // попутно используемые данные скрипта
unset($vars); // $_POST, $_COOKIE, $_GET данные
cms_RegVars();

$cms['page'] = !empty($vars['page']) ? $vars['page'] : 'main';
$cms['md_page'] = !empty($vars['md_page']) ? $vars['md_page'] : '';

$cms['ESTRING'] = !get_magic_quotes_gpc() ? TRUE : FALSE;
$cms['strpos'] = strpos($cms['page'], '_');

$cms['q_page'] = mysql_query("SELECT name, ptext FROM ".CMS_PREFIX."_pages WHERE page='".cms_MysqlEStr($cms['page'])."' LIMIT 1");
$cms['is_page'] = mysql_num_rows($cms['q_page']);


// если в _pages нет такой страницы, перемещаем на главную
if($cms['is_page'] > 0)
{
	list($cms['page_name'], $cms['page_text']) = mysql_fetch_row($cms['q_page']);
}
else
{
	header("Location: /");
}
mysql_free_result($cms['q_page']);

if($cms['strpos'] > 0 && !empty($cms['md_page']))
{
	$cms['q_md_product1'] = mysql_query("SELECT name, descript2, cost FROM ".CMS_PREFIX."_productions WHERE id=".intval($cms['md_page'])." LIMIT 1");
	
	if(mysql_num_rows($cms['q_md_product1']) > 0)
	{
		list($cms['page_name'], $cms['md_prod_descr2'], $cms['md_prod_cost']) = mysql_fetch_row($cms['q_md_product1']);
	}
	else
	{
		header("Location: /");
	}
	
	mysql_free_result($cms['q_md_product1']);
}


//edited by DEMIS [[email protected]]

$cms['page_text_header'] = $cms['page_name'];

Index.php в самописной CMS улыбает. И это только начало +)

be3, 23 Февраля 2011

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

ActionScript / Говнокод #5783

−91

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19

/**
 *  @private
 *  The exponent used in the easer function for the main part of the throw animation.
 *  NOTE: if you change this, you need to re-differentiate the easer
 *  function and use the resulting derivative calculation in createThrowMotionPath. 
 */
private static const THROW_CURVE_EXPONENT:Number = 3.0;

...

// Set the velocity for the next loop iteration.  Make sure it matches the actual velocity in effect when the
// throw reaches the end of the list.
//
// The easer function we use for the throw is 1-((1-x)^3), the derivative of which is 3*x^2-6*x+3.
// (I used http://www.numberempire.com/derivatives.php to differentiate the easer function).
// Since the slope of a curve function at any point x (i.e. f(x)) is the value of the derivative at x (i.e. f'(x)),
// we can use this to determine the velocity of the throw at the point it reached the beginning of the bounce.
var x:Number = partialTime / effectTime;
var y:Number = 3 * Math.pow(x, 2) - 6 * x + 3;

Flex 4.5 Hero.
Благо на работу кода это не сказывается, но всё-таки...

Разработчикам: Вообще класс и непосредственно алгоритм touch/kinetic скролла там сделан отвратительно, но с некоторыми весьма хорошими фишками (отпределение скорости, например). Советую вам покопаться.

fljot, 23 Февраля 2011

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

SQL / Говнокод #5782

−143

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
46
47
48
49
50

IF @Detail is not null
      AND (SELECT
            COUNT(AppraisalResponseID)
            FROM AppraisalResponseHistory
            WHERE AppraisalResponseID = @AppraisalResponseID) >0
      BEGIN
            INSERT INTO AppraisalResponseHistory 
            ( AppraisalResponseID, 
                  PermissionID, 
                  AppraisalTypeQuestionPromptID, 
                  Detail, 
                  ModifiedDate, 
                  PersonID )
      
            SELECT AppraisalResponse.ID, 
                  AppraisalResponse.PermissionID,
                   AppraisalResponse.AppraisalTypeQuestionPromptID,
                   AppraisalResponse.Detail, 
                  AppraisalResponse.ModifiedDate,
                  PersonID
            FROM AppraisalResponse
            WHERE AppraisalResponse.ID=@AppraisalResponseID AND AppraisalResponse.Detail<>@Detail
      
      END
ELSE
      IF COALESCE(@Detail, '') = ''
--if the reviewer is inserting a blank record after having filled in a response, insert into history
--if the blank record is the place holder, do not insert into history
      AND (SELECT
            COUNT(AppraisalResponseID)
            FROM AppraisalResponseHistory
            WHERE AppraisalResponseID = @AppraisalResponseID) >0
      BEGIN
            INSERT INTO AppraisalResponseHistory 
            ( AppraisalResponseID, 
                  PermissionID, 
                  AppraisalTypeQuestionPromptID, 
                  Detail, 
                  ModifiedDate, 
                  PersonID )
      
            SELECT AppraisalResponse.ID, 
                  AppraisalResponse.PermissionID,
                   AppraisalResponse.AppraisalTypeQuestionPromptID,
                   AppraisalResponse.Detail, 
                  AppraisalResponse.ModifiedDate,
                  PersonID
            FROM AppraisalResponse
            WHERE AppraisalResponse.ID=@AppraisalResponseID AND AppraisalResponse.Detail<>@Detail
END

If @Detail is not null or blank, I need to check if AppraisalResponse.Detail is blank or null (- I think this is where I am running into so much trouble). If it is blank or null then
I need to check if there is already a record in AppraisalHistory with this ID. If there is, I need to insert the blank record to history, otherwise not.

If @Detail is null.
I need to check if there is already a record in Appraisal History with this ID. If there is, I need to insert the blank record to history, otherwise not.

Даже описание скопипащено

3.14159265, 23 Февраля 2011

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

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

+179

01
02
03
04
05
06
07
08
09
10

$kPOST = @array_keys($_POST);

for (@$i=0;$i<@count(@$kPOST);@$i++) {
	$_POST[@$kPOST[@$i]] = @htmlspecialchars(@$_POST[@$kPOST[@$i]]);
}

$kGET = @array_keys(@$_GET);
for (@$i=0;@$i<@count(@$kGET);@$i++) {
	$_GET[@$kGET[@$i]] = @htmlspecialchars(@$_GET[@$kGET[@$i]]);
}

Ну теперь точно без ошибок!

Sych, 23 Февраля 2011

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

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