По колено в коде — Говнокод.ру

Python / Говнокод #16203

−100

01
02
03
04
05
06
07
08
09
10

#Было
def extract_brand(x):
                x = x[0].lower()
                for brand in brands:
                    if x.count(brand):
                        return brand
il.brand_out = extract_brand

#Стало после кодревью
il.brand_in = lambda x: [brand for brand in brands if brand.lower() in x[0].lower()]

Кто то смог прочитать второй вариант?

kyzi007, 22 Июня 2014

Комментарии (90)

C# / Говнокод #16202

+145

1

Ошибка сервера в приложении '/'. Выполнение этой команды требует открытого доступного соединения. Текущее состояние соединения: разорвано. Описание: Необработанное исключение при выполнении текущего веб-запроса. Изучите трассировку стека для получения дополнительных сведений о данной ошибке и о вызвавшем ее фрагменте кода. Сведения об исключении: System.InvalidOperationException: Выполнение этой команды требует открытого доступного соединения. Текущее состояние соединения: разорвано. Ошибка источника: Необработанное исключение при выполнении текущего веб-запроса. Информацию о происхождении и месте возникновения исключения можно получить, используя следующую трассировку стека исключений. Трассировка стека: [InvalidOperationException: Выполнение этой команды требует открытого доступного соединения. Текущее состояние соединения: разорвано.]    System.Data.Objects.ObjectContext. EnsureConnection() +11438147    System.Data.Objects.ObjectContext. CreateFunctionObjectResult (EntityCommand entityCommand, ReadOnlyMetadataCollection`1 entitySets, EdmType[] edmTypes, MergeOption mergeOption) +133    System.Data.Objects.ObjectContext. ExecuteFunction(String functionName, MergeOption mergeOption, ObjectParameter[] parameters) +411    System.Data.Objects.ObjectContext. ExecuteFunction(String functionName, ObjectParameter[] parameters) +78    Dmir.ContentCommon.Entities. ContentCommonDb.ContentGet (Nullable`1 contentID, Nullable`1 isModerator) in D:\Projects \DmirCollection\CommonObject \Dmir.ContentCommon \Dmir.ContentCommon\Entities \Model1.cs:745    Dmir.ContentCommon.Entities. ContentCommonRepository. ContentGet(Guid contentID, Boolean isModerator) in D:\Projects \DmirCollection\CommonObject \Dmir.ContentCommon \Dmir.ContentCommon\Entities \ContentCommonRepository.cs:30    Dmir.ContentCommon.Services. ContentCommonService.ContentGet (Guid contentID, Boolean isModerator)  in D:\Projects\DmirCollection \CommonObject \Dmir.ContentCommon \Dmir.ContentCommon\Services \ContentCommonService.cs:142    Dmir.Realty.Content.MvcApp. Controllers.ContentHelperController. View(Guid contentViewID, Int32 page, Boolean isModerator) in D: \Projects\DmirCollection\WebSite \Dmir.Realty.Content \Dmir.Realty.Content.MvcApp \Controllers \ContentHelperController.cs:211    lambda_method(Closure , ControllerBase , Object[] ) +252    System.Web.Mvc. ReflectedActionDescriptor.Execute (ControllerContext controllerContext, IDictionary`2 parameters) +261    System.Web.Mvc. ControllerActionInvoker. InvokeActionMethod (ControllerContext controllerContext, ActionDescriptor actionDescriptor, IDictionary`2 parameters) +39    System.Web.Mvc.Async.<>c__ DisplayClass42.< BeginInvokeSynchronousActionMethod >b__41() +34    System.Web.Mvc.Async.<>c__ DisplayClass39.< BeginInvokeActionMethodWithFilters> b__33() +124    System.Web.Mvc.Async.<>c__ DisplayClass4f.<InvokeActionMethodF ilterAsynchronously>b__49() +839035    System.Web.Mvc.Async.<>c__ DisplayClass37.< BeginInvokeActionMethodWithFilters> b__36(IAsyncResult asyncResult) +15    System.Web.Mvc.Async.<>c__ DisplayClass2a.<BeginInvokeAction>b __20() +33    System.Web.Mvc.Async.<>c__ DisplayClass25.<BeginInvokeAction>b __22(IAsyncResult asyncResult) +839620    System.Web.Mvc.<>c__ DisplayClass1d.<BeginExecuteCore>b __18(IAsyncResult asyncResult) +28    System.Web.Mvc.Async.<>c__ DisplayClass4.<MakeVoidDelegate>b_ _3(IAsyncResult ar) +15    System.Web.Mvc.Controller. EndExecuteCore(IAsyncResult asyncResult) +65    System.Web.Mvc.Async.<>c__ DisplayClass4.<MakeVoidDelegate>b_ _3(IAsyncResult ar) +15    System.Web.Mvc.Controller. EndExecute(IAsyncResult asyncResult) +51    System.Web.Mvc.<>c__ DisplayClass8.<BeginProcessRequest> b__3(IAsyncResult asyncResult) +42    System.Web.Mvc.Async.<>c__ DisplayClass4.<MakeVoidDelegate>b_ _3(IAsyncResult ar) +15    System.Web.Mvc.MvcHandler. EndProcessRequest(IAsyncResult asyncResult) +51 System.Web.CallHandlerExecutionStep .System.Web.HttpApplication. IExecutionStep.Execute() +606    System.Web.HttpApplication. ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) +288 Информация о версии: Платформа Microsoft .NET Framework, версия:4.0.30319; ASP.NET, версия:4.0.30319.18446

http://dm-realty.ru/RepairConstruction/View?contentViewID=317f4680-bdb0-4d4e-a423-041a0d833e71

И нахуй вот это простому пользователю?

Pythoner, 21 Июня 2014

Комментарии (98)

Java / Говнокод #16201

+80

1

private HttpClient httpServer = new DefaultHttpClient();

У нас в приложении есть свой сервер!!1

javahutt, 21 Июня 2014

Комментарии (3)

Java / Говнокод #16200

+75

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

public void onBackPressed() {
	if (getSupportFragmentManager().getBackStackEntryCount()==0 || getSupportFragmentManager().getBackStackEntryCount()==1) { 
							finish();
							android.os.Process.killProcess(android.os.Process.myPid());
							((MainActivity)instance).onDestroy();
	}
}

/* чуть дальше */

            JSONObject jsonObjectA = null, jsonObjectB = null;
            try {
                jsonObjectA = new JSONObject(a.toString());
                jsonObjectB = new JSONObject(b.toString());
            } catch (JSONException e1) {
                e1.printStackTrace();
            }

            if ((jsonObjectA != null || jsonObjectB != null) || (jsonObjectA != null && jsonObjectB != null)) {
                try {
                    valA = jsonObjectA.getString("name");
                    valB = jsonObjectB.getString("name");
                } catch (JSONException e) {
                    e.printStackTrace();
                }
            }

Первый фрагмент кода - это больше специфика Android - при нажатии кнопки назад: закрыть приложение, убить процесс и дернуть метод onDestroy() который в норме должен автоматом вызваться при закрытии приложения но не вызывается из-за того что мы убиваем процесс. Про то что условие if можно оптимизировать я умолчу.

На if() во втором фрагменте заглючил даже встроенный в intelliJ Idea инструмент рефакторинга...

TAX, 20 Июня 2014

Комментарии (7)

PHP / Говнокод #16199

+159

1
2
3
4

if(!preg_match('/@rambler.ru/', $item['email']) ) { 
   cmsCore::addSessionMessage('Можно использовать почту только <strong>rambler.ru</strong>', 'error');
    $errors = true;
}

// код надо расширить добавив другие домены, тот кто это писал явно не думал что придется это делать..

ma-ix, 20 Июня 2014

Комментарии (13)

Objective C / Говнокод #16198

−195

1
2

// optimization. Should only be used to read it. Never to write it.
extern CGFloat	__ccContentScaleFactor;

Может быть, я бунтарь!
__ccContentScaleFactor = -666;

tirinox, 20 Июня 2014

Комментарии (3)

C# / Говнокод #16197

+142

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19

public Config(string[] str)
        {
            InitializeComponent();
            try
            {
                textBox5.Text = str[0];
                textBox6.Text = str[1];
                textBox1.Text = str[2];
                textBox2.Text = str[3];
                textBox3.Text = str[4];
                textBox4.Text = str[5];
                textBox7.Text = str[6];
                raz = str[7];
            }
            catch (Exception)
            {
                return;
            }
        }

Классика. Говнокод эпохи барокко. Строк мало, но пованивает.

GreatMASTERcpp, 19 Июня 2014

Комментарии (22)

PHP / Говнокод #16196

+154

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25

$this->db->from("admin_sidebar");
        $this->db->where("folder", 1);
        $this->db->order_by("sort", "asc");
        $query = $this->db->get();
        $data[0] = $query->result();
        $this->db->from("admin_sidebar");
        $this->db->where("folder", 2);
        $this->db->order_by("sort", "asc");
        $query = $this->db->get();
        $data[1] = $query->result();
        $this->db->from("admin_sidebar");
        $this->db->where("folder", 3);
        $this->db->order_by("sort", "asc");
        $query = $this->db->get();
        $data[2] = $query->result();
        $this->db->from("admin_sidebar");
        $this->db->where("folder", 4);
        $this->db->order_by("sort", "asc");
        $query = $this->db->get();
        $data[3] = $query->result();
        $this->db->from("admin_sidebar");
        $this->db->where("folder", 5);
        $this->db->order_by("sort", "asc");
        $query = $this->db->get();
        $data[4] = $query->result();

asdozzz, 19 Июня 2014

Комментарии (2)

SQL / Говнокод #16195

−165

1

passport integer

Из дипломной работы. Человек хоть раз открывал свой паспорт?

bormand, 19 Июня 2014

Комментарии (49)

PHP / Говнокод #16194