Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+94
if MessageDlg('Удалить проводку №'+srcTransact.DataSet.FieldByName('Trans_ID').AsString+' и ее спецификацию?',
mtConfirmation, [mbYes, mbNo], 0) = mrYes then
begin
CMD.SQL.Add('EXEC dbo.asp_DelTransact '+srcTransact.DataSet.FieldByName('Trans_ID').AsString );
CMD.Execute;
_refresh(srcTransact.DataSet);
end;
end;
Из реального проекта (учётная система).
При определённых условиях можно удалить не только то что пользователь собирался удалить :).
Да и вообще, сам код - говно.
Fes,
30 Июля 2010
+102
private static string GetRootedCurrentConfigurationFile(string configurationFile)
{
if (string.IsNullOrEmpty(configurationFile))
{
throw new ArgumentException(Resources.ExceptionStringNullOrEmpty, "configurationFile");
}
if (!File.Exists(configurationFile))
{
throw new FileNotFoundException(string.Format(CultureInfo.CurrentCulture, Resources.ExceptionConfigurationLoadFileNotFound, new object[] { configurationFile }));
}
if (!Path.IsPathRooted(configurationFile))
{
return Path.Combine(AppDomain.CurrentDomain.BaseDirectory, configurationFile);
}
return configurationFile;
}Собснно Говнокод by Microsoft (Ent Lib 5.0.414.0)
TrueLauncher,
30 Июля 2010
+115
public static string GetTable(int type, string title, string[][] rows)
{
bool flag2;
int num;
bool isEmpty = false;
string str = "";
switch (type)
{
case 1:
str = str + "\r\n <div class='clear'> </div>\r\n\t\t\r\n\t\t <div class='down'>\r\n \r\n\t\t\t\t\t <div class='ProperyDetailHeading'>\r\n\t\t\t\t\t\t <span class='color2'>" + title + "</span>\r\n\t\t\t\t\t </div>\r\n\t\t\t\t\t <div class='PropertyDetailContent'>\r\n\t\t\t\t\t\t <div class='PropertyDetailInformation content_inner'>\r\n\t\t\t\t\t\t\t <table summary='Test Table'>\r\n\t\t\t\t\t\t\t\t <tbody>";
if ((rows != null) && (rows.Length > 0))
{
flag2 = true;
for (num = 0; num < rows.Length; num++)
{
if ((rows[num] != null) && (rows[num].Length > 0))
{
str = str + GetTableRow(ref flag2, rows[num][0], rows[num][1], ref isEmpty);
}
}
}
break;
case 2:
......ASP.NET, пакистанские кодеры. Вообще весь код с сайта можна поместить на сайте отдельным разделом. В проекте вообще не используются контролы типа DataGrid/FormView.. даже DataSource. У них и тэмплэйты свои :(.. Грустно так все...(
Coffeeholic,
30 Июля 2010
+164
function isGood($race,$class) {
switch ($race) {
case 1:
if ($class == 1 || $class == 2 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
break;
case 2:
if ($class == 1 || $class == 3 || $class == 4 || $class == 6 || $class == 7 || $class == 8 || $class == 9) { return true; }
break;
case 3:
if ($class == 1 || $class == 2 || $class == 3 || $class == 4 || $class == 5 || $class == 6) { return true; }
break;
case 4:
if ($class == 1 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 11) { return true; }
break;
case 5:
if ($class == 1 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
break;
case 6:
if ($class == 1 || $class == 3 || $class == 6 || $class == 7 || $class == 11) { return true; }
break;
case 7:
if ($class == 1 || $class == 4 || $class == 6 || $class == 8 || $class == 9) { return true; }
break;
case 8:
if ($class == 1 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 7 || $class == 8) { return true; }
break;
case 10:
if ($class == 2 || $class == 3 || $class == 4 || $class == 5 || $class == 6 || $class == 8 || $class == 9) { return true; }
break;
case 11:
if ($class == 1 || $class == 2 || $class == 3 || $class == 5 || $class == 6 || $class == 7 || $class == 8) { return true; }
break;
}
return false;
}
Oops,
30 Июля 2010
+156
// в данном случае count($xls_data) не подойдет, потому что
// он меньше изза того что индексы не всегда по +1 идут
end($xls_data);
list($xls_dataCount,$unused) = each($xls_data);
// тут некоторый код, тоже пахнет
for($i = 3; $i <= $xls_dataCount; $i++){
$row = $xls_data[$i];
if(count($row) == 1 && $estnames[$row[1]])$ename = $row[1];
else{
foreach($years as $yindex => $year)
$data[$ename][$row[1]][$year] = $row[$yindex];
}
}Сижу и ковыряюсь в говнице, по уши увяз((((
asics,
30 Июля 2010
+164
try {
...
} catch ( Exception $O_o ) {
error_log( $O_o->getMessage() );
...
}Совсем неожиданный эксепшин O_o
eee_qqq,
30 Июля 2010
+74
for (Integer i = 0; i<_pwiList.size(); i++)Вот к чему приводит бездумное использование классов-обёрток "для унификации". Конечно, оптимизатор может здесь сообразить, но не факт. А если данный фрагмент трактовать буквально, то i++ приведёт к unboxing, увеличению и последующему boxing.
konsoletyper,
30 Июля 2010
+75
PriceWithInterval pwi = new PriceWithInterval();
pwi.setPrice(p);
pwi.setStart(dateFormat.parse(dateFormat.format(p.getStartDate())).getTime());
Long endTm = dateFormat.parse(dateFormat.format(p.getEndDate())).getTime();
//Больше 50 лет - техническая бесконечность
if (((endTm-pwi.getStart())/1000/3600/24/365)>50)
{
endTm = dateFormat.parse("31.12.9999").getTime();
}
// Где dateFormat объявлен как
new SimpleDateFormat("dd.MM.yyyy");Вот такое вот масло масляное. Думается, что этот трюк применён здесь для того, чтобы получить начало дня. Чем не устраивает Calendar, неясно. Ну в крайнем случае, можно было целочисленно поделить и умножить на 1000 * 3600 * 24. Ну и вычисление в общем-то константной "технической бесконечности" радует неимоверно.
konsoletyper,
30 Июля 2010
+147
<?php
require_once 'MDB2.php';
$dsn = "mysql://user:pass@localhost/db";
$mdb2 = & MDB2::singleton($dsn);
if (PEAR::isError($mdb2)) {
die($mdb2->getMessage());
}
class DB {
static private $instance = NULL;
static private $mdb2 = NULL;
private function __construct() {
self::$mdb2 = & MDB2::singleton();
self::$mdb2->exec("SET NAMES utf8");
self::$mdb2->setFetchMode(MDB2_FETCHMODE_ASSOC);
}
static function getInstance() {
if(self::$instance == NULL) {
self::$instance = & new DB();
}
return self::$instance;
}
public function query($sql = false) {
$res = self::$mdb2->query($sql);
if (PEAR::isError($res)) {
die($res->getMessage());
}
if(!$res->numRows()) {
return FALSE;
}
return $res;
}
private function __clone() {
}
}
class Page{
public $limit = 10;
private $conn = FALSE;
function __construct() {
$this->conn = & DB::getInstance();
}
public function getPageList() {
$result = FALSE;
$sql = "SELECT * FROM table LIMIT ".$this->limit;
$res = $this->conn->query($sql);
if($res) {
$result = $res->fetchAll();
}
return $result;
}
}
$p = & new Page();
$nodes = $p->getPageList(25);
print '<pre>'.print_r($nodes, 1).'</pre>';
?>Дайте, пожалуйста, оценку такой конструкции. Не говнокод ли?
cartman,
29 Июля 2010
+63
<p>
<strong>
<strong><strong>
<strong><strong><strong>
<strong><strong><strong><strong>
<strong>.... ещё пара сотен открывающихся <strong> ....<strong>
<strong>24.07.2010 - <a href="http://www.korea-dpr.com/ocn"><span class="style34">OCN Articles</span></a> added !<br> </strong></strong>
</strong></strong></strong></strong></strong></strong>
... ещё пара сотен закрывающихся </strong>...
</strong></strong></strong></strong>
</strong></strong></strong>
</strong></strong>
</strong>
</p>Исходный код официальной страницы КНДР - http://www.korea-dpr.com. Именования стилей "style-какие-то две цифры" прилагаются. Безумный хтмл и прочая тёмная сторона силы.
Rsk,
29 Июля 2010
Follow us!