- 1
- 2
- 3
- 4
- 5
- 6
- 7
if (isset($_POST['email']))
{
$email = $_POST['email'];
if ($email == '') {
unset($email);
}
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+170
if (isset($_POST['email']))
{
$email = $_POST['email'];
if ($email == '') {
unset($email);
}
}
все предусмотрел...
+164
function maybe_unserialize( $original ) {
if ( is_serialized( $original ) ) // don't attempt to unserialize data that wasn't serialized going in
return @unserialize( $original );
return $original;
}
function maybe_serialize( $data ) {
if ( is_array( $data ) || is_object( $data ) )
return serialize( $data );
if ( is_serialized( $data ) )
return serialize( $data );
return $data;
}
Мне нравится подход, есть в этом что-то занятное... : )
WP
+164
$this->db->Query("DELETE FROM `u_proposal`
WHERE `id` = '%d' AND `user` = '%d'", $item, $this->user->id);
$this->db->Query("DELETE FROM `p_options`
LEFT JOIN `u_proposal` ON `p_options`.`portfolio` = `u_proposal`.`id`
WHERE `p_options`.`portfolio` = '%d' AND `p_options`.`type` = 'project' AND `u_proposal`.`user` = '%d'", $item, $this->user->id);
3 часа еб**ся не мог понять почему записи из p_options не удаляются ))
+81
public void actionPerformed(java.awt.event.ActionEvent e)
{
/* code here */
thr = new Potok(Spisok,f,papk);
thr.setWork(true);
try { thr.join(); }
catch (InterruptedException e1)
{
// TODO Auto-generated catch block
e1.printStackTrace();
}
/* another code here */
}
/* in the galaxy far-far away */
class Potok extends Thread
{
private boolean isWork = true;
public Potok() {}
public void run()
{
while ( isWork )
{
/* code here */
}
}
public void setWork(boolean work)
{
isWork = work;
}
}
Внутри обработчика нажатия на кнопку раскрывается вся суть многопоточных приложений.
+177
if(msg == "" || msg == "{}"){
// странный баг, форма очищается, но не прячется
return;
$("#variants_div").html('');
$("variants_div").hide();
}
Даже Джумла первая была радостным кодом по сравнению с этим.
−129
static public function getCamera():Camera
{
// Return the same camera if it has been successfully requested before
if (cam != null)
{
if (cam.muted) Security.showSettings(SecurityPanel.PRIVACY);
return cam;
}
// Get the camera
cam = Camera.getCamera();
if (cam != null)
{
// Set properties if a camera was found
if((cam.name.toLowerCase().indexOf("usb") != -1) &&
(flash.system.Capabilities.os.toLowerCase().indexOf("mac") != -1))
{
// this is a mac default cam, dont change mode
Logger.getInstance().debug("has usb + mac: no set mode");
}
else
{
cam.setMode(CAMERA_WIDTH, CAMERA_HEIGHT, 30, true);
Logger.getInstance().debug("setMode to cam");
}
cam.addEventListener(StatusEvent.STATUS, NeaveCamera.statusListener);
return cam;
}
else
{
// No camera found
Security.showSettings(SecurityPanel.CAMERA);
return new Camera();
}
}
В отчаянии.
+139
Response.Write("<" + '/'.ToString() + "script>");
Даже не знаю, кому и зачем это понадобилось...
−110
var p2pStream:P2PStream = this
var client:Object = new Object();
client.onPeerConnect = function(subscriber:NetStream):Boolean{
return p2pStream.onPeerConnect()}
nsPublish = new NetStream(netConnection, NetStream.DIRECT_CONNECTIONS);
nsPublish.client = client;
Это AS3 (хотя это было бы говном в AS2 тоже, но там хоть причину можно было понять).
−102
($sec,$min,$hour,$mday,$mon,$year) = localtime(time);
if ($sec < 10) {$sec = "0$sec";}
if ($min < 10) {$min = "0$min";}
if ($hour < 10) {$hour = "0$hour";}
if ($mday < 10) {$mday = "0$mday";}
if ($mon < 10) {$mon = "0$mon";}
$mon++;
$year = 1900 + $year;
print "[$mday-$mon-$year $hour:$min:$sec]\n";
Добиваем нулями
+106
public static bool KeyToString(Keys key, bool shitKeyPressed,
out char character)
{
bool result = false;
character = ' ';
CharPair charPair;
if ((Keys.A <= key && key <= Keys.Z) || key == Keys.Space)
{
// Use as is if it is A~Z, or Space key.
character = (shitKeyPressed) ? (char)key : Char.ToLower((char)key);
result = true;
}
else if (keyMap.TryGetValue(key, out charPair))
{
// Otherwise, convert by key map.
if (!shitKeyPressed)
{
character = charPair.NormalChar;
result = true;
}
else if (charPair.ShiftChar.HasValue)
{
character = charPair.ShiftChar.Value;
result = true;
}
}
return result;
}
Разбираю код, скачанный с майкросовтовского App Hub. В коде и комментариях полно маленьких грамматический ошибок, но это...
Код выложен на полном серьёзе и качается тут http://create.msdn.com/en-US/education/catalog/utility/performance_utility
P.S. для тупых спойлер: вместо shift написано shit.