- 1
- 2
- 3
if (strcasecmp($fieldValue, "Array") == 0) {
...
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+153
if (strcasecmp($fieldValue, "Array") == 0) {
...
}
+71
public class Pages extends TabActivity {
@Override
protected void onResume() {
super.onResume();
if (!loading) {
loading = true;
new EmptyThread().execute();
}
}
@Override
protected void onPause() {
Data.killNow = true;
super.onPause();
}
private class EmptyThread extends AsyncTask<Void, String, Void> {
@Override
protected void onPreExecute() {
Data.killNow = false;
}
@Override
protected Void doInBackground(Void... unused) {
while (!Data.killNow) {
try {
Thread.sleep(1000);
} catch (InterruptedException e) {
}
}
return null;
}
@Override
protected void onPostExecute(Void unused) {
loading = false;
}
}
}
В продолжение предыдущей темы (http://govnokod.ru/9256): из-за столь вольного обращения с модификатором static и адского обращения с AsyncTask*ами программист нарвался на утечку памяти и AsyncTask*и у него прекратили запускаться (не факт, что из-за утечек, я их исходный код не смотрел). Выше вам был представлен костыль, благодаря которому асинхронки всё-таки запускались. Как - я так и не постиг.
Для справки - когда экран выходит на передний план, срабатывает onResume(), когда уходит с него - onPause().
+68
public class Pages extends TabActivity {
public static TabHost host;
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
host = getTabHost();
// много говнокода
}
}
public class Table extends Activity {
//много говнокода
@Override
protected void onResume() {
super.onResume();
if (!loading) {
loading = true;
new MainTask().execute();
}
}
private class MainTask extends AsyncTask<Void, String, Void> {
//безумнейший парсинг в donInBackground()
@Override
protected void onPostExecute(Void unused) {
//разбираемся с тем, что пришло - для начала отлавливаем ошибки в парсере.
if (!timeOut) {
AlertDialog dlg = new AlertDialog.Builder(TableOfGroups.this).setTitle(Data.lowConnectionTitle)
.setMessage(Data.lowConnectionText).setOnCancelListener(new DialogInterface.OnCancelListener() {
@Override
public void onCancel(DialogInterface dialog) {
loading = false;
if (!update) {
Pages.host.setCurrentTab(Data.savePageIndex - 1);
}
}
}).setPositiveButton("OK", new DialogInterface.OnClickListener() {
@Override
public void onClick(DialogInterface dialog, int whichButton) {
loading = false;
if (!update) {
Pages.host.setCurrentTab(Data.savePageIndex - 1);
}
}
}).create();
dlg.show();
if (!update)
return;
}
//ещё около 150 строк подобной ловли
}
}
}
Задача (Android): имеется активити со вкладками (Pages), внутри него - дочерние активити. Если в активити Table на 4-й вкладке происходит косяк, то нужно показать диалог и перепрыгнуть на 1-ю вкладку.
+162
if(object[k] !== undefined ? replace !== false : true){
object[k] = extendData[k];
}
Поддерживаемый и читаемый код? Не, не слышал
+158
<td class="index"><?=isset($idx)?++$idx:$idx=1;?></td>
Просто и понятно, особенно ночью
+151
<li><a href="#">О нас</a></li>
<li><a href="#">Каталог мест</a></li>
<li><a href="#">Правила</a></li>
<li class="footer_write_link"><a href="#">Написать нам</a></li>
<li class="footer_for_company">
<?php if($account): ?>
<a href="/acount/place/index/<?php echo $account->account->id; ?>/"><?php echo $account->account->name; ?> </a>
<?php else: ?>
<a href="/account/new/"> <span>компаниям</span></a>
<?php endif; ?>
</li>
Вот такая отладка из Kohana уже второй день появляется на spooon у всех, кто получил инвайт
+188
<?php
...
$Sex = $_POST['sex']; //Пол юзера из поискового запроса
$searchSex = $_POST['searchsex']; //Пол искомого партнёра
...
if ($Sex == $searchSex) //Осторожно, педерасты!
{
...
//Вывод капчи
...
}
?>
Из написанного знакомым движка для сайта знакомств. Комментарии оригинала.
+171
if(in_array(strlen($_POST['password']), range(6,30))) {
////.....
}
но кода на несколько символов все равно меньше, согласитесь
+154
http://habrahabr.ru/blogs/web_security/137196/
Нанотехнологии в действии
+166
if ($id == 1) {
$source = 'GallerySkins/s/skin_01.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 2) {
$source = 'GallerySkins/s/skin_02.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 3) {
$source = 'GallerySkins/s/skin_03.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 4) {
$source = 'GallerySkins/s/skin_04.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 5) {
$source = 'GallerySkins/s/skin_05.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 6) {
$source = 'GallerySkins/s/skin_06.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 7) {
$source = 'GallerySkins/s/skin_07.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 8) {
$source = 'GallerySkins/s/skin_08.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 9) {
$source = 'GallerySkins/s/skin_09.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 10) {
$source = 'GallerySkins/s/skin_10.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 11) {
$source = 'GallerySkins/s/skin_11.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
elseif ($id == 12) {
$source = 'GallerySkins/s/skin_12.png';
$dest = "MinecraftSkins/$user.png";
if(!copy($source, $dest)) {
echo "Ошибка<br />\n";
}
}
Разгребал систему скинов и наткнулся на это... даже не знаю, как это чудо назвать.
Заменяется элементарным
$source = 'GallerySkins/s/skin_'.$id.'.png'; // Исходная директория файла
$dest = "MinecraftSkins/$user.png"; // Конечная директория файла
if(!copy($source, $dest)) // Если файл не скопирован:
{
echo "Ошибка<br />\n";
}