- 1
$("input[id='yearB']")
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+175
$("input[id='yearB']")
+158
public function search() {
$ret = array();
foreach($this->dir() as $file)
if($file->folder())
$ret = array_merge($ret,$file->search()->asArray());
else
$ret[] = $file;
return new mod_file_list($ret);
}
Красота, в результате работы функции мы получаем объект, состоящий из массива объектов класса file (этот метод в этом классе)... "чудо-поиска" (хотя это вовсе и не поиск, а банальный список файлов всех дочерних папок), особенно будет забавно, если в списке будет несколько тысяч файлов...
В этом классе в принципе много мест, которые наровят насоздавать кучу экземпляров этого же класса, вместо замены одной приватной переменной...
+162
public static function perform($table, $data, $action = 'insert', $parameters = '') {
reset($data);
if ($action == 'insert') {
$query = 'INSERT INTO ' . $table . ' (';
while (list($columns, ) = each($data)) {
$query .= $columns . ', ';
}
$query = substr($query, 0, -2) . ') values (';
reset($data);
while (list(, $value) = each($data)) {
switch ((string)$value) {
case 'now()':
$query .= 'NOW(), ';
break;
case 'null':
$query .= 'NULL, ';
break;
default:
$query .= '\'' . mysql_escape_string($value) . '\', ';
break;
}
}
$query = substr($query, 0, -2) . ')';
} elseif ($action == 'update') {
$query = 'UPDATE ' . $table . ' SET ';
while (list($columns, $value) = each($data)) {
switch ((string)$value) {
case 'now()':
$query .= $columns . ' = NOW(), ';
break;
case 'null':
$query .= $columns .= ' = NULL, ';
break;
default:
$query .= $columns . ' = \'' . mysql_escape_string($value) . '\', ';
break;
}
}
$query = substr($query, 0, -2) . ' WHERE ' . $parameters;
}
self::request($query);
}
Вот так у нас вставляют в базу :)
+157
struct Event
{
int priority;
bool operator<(Event e) const
{
return e.priority>priority;
}
virtual void operator() () = 0;
};
priority_queue<Event*, deque<Event*> > eventQueue;
template <typename T, void (T::*g)()>
struct TEvent : Event
{
T* parent;
virtual void operator() ()
{
(parent->*g)();
}
TEvent (T* _parent) : parent(_parent){}
};
struct Entity
{
int foo;
void AFunc()
{
cout<<foo;
}
void BFunc()
{
cout<<"Ololo";
}
typedef TEvent<Entity,&Entity::AFunc> someEvent;
Entity(int _foo): foo(_foo){}
void addEvent()
{
eventQueue.push(new someEvent(this));
}
};
int main()
{
Entity foo(10);
foo.addEvent();
(*eventQueue.top())();
return 0;
}
Один из самых сложных и неочевидных способов вывести в консоль число 10.
(Этот код - проверка, будет ли работать прототип велосипедного делегата)
+159
$this->setLayout('empty');
$this->setTemplate('step3');
if ($this->getBrokerId() == 19 || $this->getBrokerId() == 29)
{
$this->setLayout('empty');
$this->setTemplate('step3');
$this->brokerId = $this->getBrokerId();
}
Нашел в чужом проекте. При этом $this->getBrokerId() не просто возвращает переменную, а производит сравнение строк (то есть вызывать этот метод 3 раза - явный перебор).
+123
string callPage = CallPage;
if (callPage != "Checkout.aspx")
{
callPage = "Checkout.aspx";
}
ServerTransfer(callPage + "?makerefresh=true", Server, Request);
Найди лишнее
+159
$ar=$arFields['GROUP_ID'];
$arFields['GROUP_ID']=array();
foreach($ar as $a) $arFields['GROUP_ID'][]=$a;
Смысл данной конструкции не ясен, если так нужно обнулить ключи для массива то array_values() в помощь.
+113
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace ConsoleApplication1
{
class Program
{
static void Main(string[] args)
{
string[] exit_program = { "1", "2", "3", "4", "5", "6" };
bool flag = true;
while(flag)
{
Console.WriteLine("Введите команду");
string ss = Console.ReadLine();
for (int i = 0; i < exit_program.Length&&flag;i++)
{
if (ss == exit_program[i])
{
Console.WriteLine("Вы ввели {0} и теперь можете выйти из цикла",ss);
flag = false;
break;
}
}
}
Console.ReadLine();
}
}
}
Говнокод, позволяющий выйти из консольного приложение по нажатию цифр 1,2....6
+164
<?php
class Gcont extends Controller {
// ...
function gcont(){
$this->load->model('am_loader');
$title = $this->am_loader->gettitle();
echo '<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"';
echo '<html xmlns="http://www.w3.org/1999/xhtml">';
echo '<head><title>'.$title.'</title>';
// ...
$this->load->view('gcont');
}
}
Суровая генерация контента нубом из контроллера MVC.
+116
public static class ControlManager
{
public static UIElement GetControlByName(UIElement control, string name)
{
if (EntityType.GetValue(control, "Name").Equals(name))
return control;
if (EntityType.IsProperty(control, "Children"))
{
foreach (var element in (UIElementCollection)EntityType.GetValue(control, "Children"))
{
if (EntityType.GetValue(element, "Name").Equals(name))
return element;
var temp = GetControlByName(element, name);
if (temp != null)
return temp;
}
}
if (EntityType.IsProperty(control, "Child"))
{
var element = (UIElement)EntityType.GetValue(control, "Child");
if (EntityType.GetValue(element, "Name").Equals(name))
return element;
var temp = GetControlByName(element, name);
if (temp != null)
return temp;
}
if (EntityType.IsProperty(control, "SelectionElement"))
{
var element = (UIElement)EntityType.GetValue(control, "SelectionElement");
if (EntityType.GetValue(element, "Name").Equals(name))
return element;
var temp = GetControlByName(element, name);
if (temp != null)
return temp;
}
return null;
}
public static UIElement GetControlByType(UIElement control, Type type)
{
if (control.GetType() == type)
return control;
if (EntityType.IsProperty(control, "Children"))
{
foreach (var element in (UIElementCollection)EntityType.GetValue(control, "Children"))
{
if (element.GetType() == type)
return element;
var temp = GetControlByType(element, type);
if (temp != null)
return temp;
}
}
if (EntityType.IsProperty(control, "Child"))
{
var element = (UIElement)EntityType.GetValue(control, "Child");
if (element.GetType() == type)
return element;
var temp = GetControlByType(element, type);
if (temp != null)
return temp;
}
if (EntityType.IsProperty(control, "SelectionElement"))
{
var element = (UIElement)EntityType.GetValue(control, "SelectionElement");
if (element.GetType() == type)
return element;
var temp = GetControlByType(element, type);
if (temp != null)
return temp;
}
return null;
}
}
Автор тот же.
Этот код пошатнул мою психику. А может так и надо. а?
всё-таки stringly-static programming....