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

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

+158

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

<script type="text/javascript" src="swfobject.js"></script>
<script type="text/javascript">
	function  myTags(mytags){
		mytags=mytags.replace(/<A/g, '<a')
			.replace(/\/A>/g, "/a>")
			.replace(/(\starget=_)(\w*)/g, ' target="_$2"')
			.replace(/(\sclass=)(?!")(\w*)/g, ' class="$2"')
			.replace(/(\sname=)(?!")(\w*)/g, ' name="$2"')
			.replace(/(\sid=)(?!")(\w*)/g, ' id="$2"')
			.replace(/(\srel=)(?!")(\w*)/g, ' rel="$2"');
		mytags=encodeURIComponent(mytags).replace(/!/g, '%21')
			.replace(/'/g, '%27').replace(/\(/g, '%28')
			.replace(/\)/g, '%29').replace(/\*/g, '%2A');
		var rnumber = Math.floor(Math.random()*9999999);
		var flashvars = {
			tcolor:"0x2A62C8",
			tcolor2:"0x000000",
			hicolor:"0xB12AC8",
			tspeed:"110",
			distr:"true",
			mode:"tags",
			tagcloud:mytags
		};
		var params = {
			allowScriptAccess:"always",
			bgcolor:'#ffffff'
		};
		var attributes = {
			id:"flash_cloud"
		};
		swfobject.embedSWF("tagcloud.swf?r="+rnumber,
						   "tags", "600", "420", "9.0.0",
						   "expressInstall.swf", flashvars,
						   params, attributes);
	}
	window.onload=function(){
		var mytags="<tags>"
		+document.getElementById('tags').innerHTML
		+"</tags>";
		myTags(mytags);
	};
</script>

qbasic, 22 Февраля 2011

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

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

+158

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

<?
//include('../root/start.php');
include('start.php');




$hotels_menu_id = GetSettingsParam('hotels_menu_id');

if ($image_id = $_GET['image_id']) {
    $row = mysql_fetch_assoc(mysql_query(
                    'select co.id co_id, co.title co_title, ' .
                    'ci.id ci_id, ci.title ci_title, ' .
                    'h.id h_id, h.title h_title ' .
                    'from ' . _mysql_tbl_prefix . 'countries co ' .
                    'left join ' . _mysql_tbl_prefix . 'cities ci on co.id = ci.country_id ' .
                    'left join ' . _mysql_tbl_prefix . 'hotels h on ci.id = h.city_id ' .
                    'left join ' . _mysql_tbl_prefix . 'hotel_images hi on hi.hotel_id = h.id ' .
                    'where hi.image_id = ' . $image_id));
    echo mysql_error();
# $src = '../../hotel_images/'.(int)($image_id/2000).'/'.$image_id.'.jpg';
    $src = './hotel_images/' . (int) ($image_id / 2000) . '/' . $image_id . '.jpg';
}
?>
<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=windows-1251" />
        <title>Фото отеля "<?= $row['h_title'] ?>" (<?= $row['ci_title'] . ', ' . $row['co_title'] ?>) - Туроператор</title>
    </head>
    <link rel="stylesheet" href="../hotel_image.css">
    <body>
        <table height="100%" width="100%">
            <tr>
                <td valign="middle" align="center">
                    <h1 class="hotel-image-title"><?=
'Фото отеля "<a href="../index.php?menu_id=' . $hotels_menu_id . '&hotel_id=' . $row['h_id'] . '">' . $row['h_title'] . '</a>" ' .
        '(<a href="../index.php?menu_id=' . $hotels_menu_id . '&city_id=' . $row['ci_id'] . '">' . $row['ci_title'] . '</a>, ' .
        '<a href="../index.php?menu_id=' . $hotels_menu_id . '&country_id=' . $row['co_id'] . '">' . $row['co_title'] . '</a>)'
?></h1>
                    <a href="#" onClick="window.close()"><img src="<?= $src ?>" class="hotel-image" alt="Фото отеля "<?= $row['h_title'] ?>" (<?= $row['ci_title'] . ', ' . $row['co_title'] ?>)"></a>
                </td>
            </tr>
        </table>
    </body>
</html>

Попросили так сазать исправить))
я был ошеломлен "магическим числом" 2000

rainerg, 22 Февраля 2011

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

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

+146

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

package gargoyle.xenox.game;

import gargoyle.util.log.Log;

import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.ObjectInputStream;
import java.io.ObjectOutputStream;
import java.io.Serializable;

@SuppressWarnings("rawtypes")
public abstract class Persistent<T extends Persistent> implements Serializable {
  private static final long serialVersionUID = 1L;

  private File file(final Class<? extends Persistent> clazz) {
    return new File(System.getProperty("user.home"), clazz.getName() + ".dat");
  }

  @SuppressWarnings("unchecked")
  final protected T load(final Class<T> clazz) {
    ObjectInputStream in;
    try {
      in = new ObjectInputStream(new FileInputStream(this.file(clazz)));
      return (T) in.readObject();
    } catch (final IOException e) {
      Log.error(e);
    } catch (final ClassNotFoundException e) {
      Log.error(e);
    }
    return null;
  }

  final protected void save(final T o) {
    ObjectOutputStream os = null;
    try {
      os = new ObjectOutputStream(new FileOutputStream(this.file(o.getClass())));
      os.writeObject(this);
      os.flush();
    } catch (final IOException e) {
      Log.error(e);
    } finally {
      try {
        if (os != null) {
          os.close();
        }
      } catch (final IOException e) {
        Log.error(e);
      }
    }
  }
}

такой вот забавный сериализатор получился
причина - запутался в генериках

Lure Of Chaos, 21 Февраля 2011

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

C++ / Говнокод #5757

+165

1
2
3
4
5

template<typename Class>
void Raise(int Code)
{
  throw Class(Code);
};

Продолжение эпоса из уже всем знакомого проекта, доставшегося по наследству, в котором активно используются исключния.

Говногость, 21 Февраля 2011

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

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

−102

1

import com.adobe.protocols.dict.events.ErrorEvent;

Ну хоть бы циферку добавили...

wvxvw, 21 Февраля 2011

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

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

+84

01
02
03
04
05
06
07
08
09
10
11

transDate = ( ExtBaApiStaging.class.equals( extExcep.getClass() ) )?((ExtBaApiStaging)extExcep).getBeginDate():
		( ExtRsApiCancel.class.equals( extExcep.getClass() ) )?((ExtRsApiCancel)extExcep).getTransDate():
		( ExtRsApiWriteCustContact.class.equals( extExcep.getClass() ) )?((ExtRsApiWriteCustContact)extExcep).getTransDate():
		( ExtRsApiPayAdjust.class.equals( extExcep.getClass() ) )?((ExtRsApiPayAdjust)extExcep).getTransDate():
		( ExtRsApiChgCustomer.class.equals( extExcep.getClass() ) )?((ExtRsApiChgCustomer)extExcep).getTransDate():
		( ExtRsApiChgService.class.equals( extExcep.getClass() ) )?((ExtRsApiChgService)extExcep).getTransDate():
		( ExtRsApiChgBoxData.class.equals( extExcep.getClass() ) )?((ExtRsApiChgBoxData)extExcep).getTransDate():
		( ExtCLApiIsfMnp.class.equals( extExcep.getClass() ) )?((ExtCLApiIsfMnp)extExcep).getLoadDate():
		( ExtCLApiNoTruckSro.class.equals( extExcep.getClass() ) )?((ExtCLApiNoTruckSro)extExcep).getLoadDate():
		( ExtCLApiNsfHoldEcOsa.class.equals( extExcep.getClass() ) )?((ExtCLApiNsfHoldEcOsa)extExcep).getLoadDate():
		( ExtRsApiSendAHit.class.equals( extExcep.getClass() ) )?((ExtRsApiSendAHit)extExcep).getTransDate() : null;

Это писала одна тимлид

tr00_gr1m_doomster, 21 Февраля 2011

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

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

−123

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

sub check_interface
{
	my $int_input = shift;
	my $all_intr_local = [];
	$_ = qx[ip a];
	s[\d{1,}:[ ]{1,}([^ ]{1,}):.*]<unshift(@$all_intr_local, $1)>ge;
	if ( ! grep( { /^$config_params{$int_input}$/ } @$all_intr_local ))
	{
		$warning->debug("Error: interface $int_input can't found local!!!");
		exit 1;         
	}
	else
	{
		$info->debug("Load param $int_input = $config_params{$int_input}.");   
	}
}

А вот так мы сканируем сетевые интерфейсы...

SadKo, 21 Февраля 2011

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

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

−124

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

sub check_dir
{
	my ($param, $mode) = @_;
	$mode ||= 'sr';

	unless ( $mode eq 'sc')
	{
		my $full_path = $config_params{$param};
		$full_path =~ /^(.+)\/(.+)$/;
		my $put = $1;

		while(1)
		{
			if (-l $full_path)
			{
				$info->debug("Load param $param = $full_path it is symlink. Readlink...");
				$full_path = readlink $full_path;
			}
			elsif (-d $full_path or -d "$put/$full_path")
			{
				$info->debug("Load param $param = $full_path. It is directory and it is exists");
				last;
			}
			else
			{
				$warning->debug("Not valid config. Error parsing param: $param");
				exit 1;
			}
		}
	}
	else
	{
		my $full_path = $config_params_sc{$param};
		$full_path =~ /^(.+)\/(.+)$/;
		my $put = $1;

		while(1)
		{
			if (-l $full_path)
			{
				$info_sc->debug("Load param $param = $full_path it is symlink. Readlink...");
				$full_path = readlink $full_path;
			}
			elsif (-d $full_path or -d "$put/$full_path")
			{
				$info_sc->debug("Load param $param = $full_path. It is directory and it is exists");
				last;
			}
			else
			{
				$warning_sc->debug("Not valid config. Error parsing param: $param");
				exit 1;
			}
		}
	}
}

По крайней мере, этот код можно сократить вдвое... И подобное по всей утилите...

SadKo, 21 Февраля 2011

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

C++ / Говнокод #5752

+167

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

const StringId sidPathName = [=]() -> const char* const
{
	switch (path)
	{
		case PATH_TOWN_HERO:
			return "game/careers/town_hero";
		case PATH_MAD_SCIENTIST:
			return "game/careers/mad_scientist";
		case PATH_SPECIAL_AGENT:
			return "game/careers/special_agent";
		case PATH_PSYCHIC_PAINTER:
			return "game/careers/psychic_painter";
		default:
			RZT_ASSERT(false);
			return "";
	}
}();

Не удержался: давно мечтал о возможности ставить const на такие переменные, не отказываясь при этом от свитчей!

Kirinyale, 21 Февраля 2011

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

C# / Говнокод #5751

+117

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

public static class EventExtensions
    {
        public static void Fire(this EventHandler eventHandler, object sender)
        {
            eventHandler.Fire(sender, EventArgs.Empty);
        }

        public static void Fire(this EventHandler eventHandler, object sender, EventArgs args)
        {
            eventHandler.FireDelegate(sender, args);
        }

        public static void Fire<TEA>(this EventHandler<TEA> eventHandler, object sender, TEA args)
            where TEA : EventArgs
        {
            eventHandler .FireDelegate(sender, args);
        }

        public static void FireDelegate(this Delegate eventHandler, params object[] parameters)
        {
            if (eventHandler != null)
            {
                Delegate[] delegates = eventHandler.GetInvocationList();
                foreach (var del in delegates)
                {
                    try
                    {
                        del.Method.Invoke(del.Target, parameters);
                    }
                    catch (Exception e)
                    {
                        // use logger here
                        throw;
                    }
                }
            }
        }
    }

Надоело писать инвокаторы для ивентов? Как тебе это?

walash, 21 Февраля 2011