1. PHP / Говнокод #6937

    +167

    1. 1
    $r = $this->client->getBerechneteGrundversorgungsTarifebyPLZundVerbrauchKundenart($this->params);

    По сути не говнокод, но нечитабельность налицо...

    Запостил: vov4ik, 13 Июня 2011

    Комментарии (41) RSS

    • ага, это вам не на лицо, а для немцов самое то- описывает суть функции. Единственное что без сокращений ;D. Но немцы они такие- лучше лишний раз написать/сделать/подумать, чем разгребать потом.
      Ответить
      • а для чего комментарии существуют? Тем более там английская частица "by" затесалась с какого-то перепуга... про начальное "get" я уже молчу (традиция)... Во-вторых, традиционно принято давать имена на англ., что явл. правилом хорошего тона.
        Ответить
    • как и get, by традиция... bei- будет на немецком, но эт не так красиво.
      А комментарии можно удалить, а функцию сложнее ;D (перестраховываются)
      Ответить
      • а мне потом как её юзать? Постоянно лезть в исходники и копипастить? Ибо побуквенно перепечатывать с бумажного API в данном случае извращение :)
        Ответить
        • ну если с немцами работаете ))), то нужно язык подучить. Если его знать так название запомнится за секунды
          Ответить
          • немецкий - мой почти родной язык. Работаю с немцами, у немцев... Не в языке дело.
            Ответить
        • А что, в 21-ом веке ещё есть языки программирования и IDE, в которых отсутствует IntelliSense, автодополнение и прочие вкусности? Я ф шоке.
          Ответить
    • а вообще копипаст никто не отменял ), можно шортхэнды создать если не понятно
      Ответить
      • новое слово в пыхопыханье - локализация сырцов!
        Ответить
        • Бля, яб сказал спасибо что это не китайский и не турецкий( yapilacek ) гг
          Ответить
          • хотя по мне так лучше неймить на родном для кодинга языке, тобищь- английском
            Ответить
    • ну если у немцев, то по моему нет к чему придираться, если сократить, можно забыть потом назначение ;D(особенно если крупный проект). Ну а как предложите назвать функцию по другому, чтобы она говорила о своём назначении ?
      Ответить
    • а там нет ли сортировки ??? я всей сути не знаю, но не вижу намёка на сортировку по почтовому коду ??? PLZ ???
      Ответить
      • ню да ладно, дерьмо та не мне выгребать ;D
        Ответить
      • единственный параметр метода - массив, содержащий список всего того (и намного больше), что описано в наименовании метода... Так что какой смысл описывать в наименовании всё, что входит в параметры?
        Исходников нет - есть только API сервиса. Это метод SOAP
        Ответить
    • ну тогда это истинный лазанья код ; D, с чем вас и поздравляю ...
      Ответить
      • Не знаю, из немецкого мне пока нравится порнуха, баварские сосиськии и пивас...
        Ответить
    • нужно попробывать...
      Ответить
      • Хотя это по ходу как балтика 9-ка по вкусу ))) (шнапса туда по традиции вроде как заливают), тогда я лучше фоерцанболе выпью...
        Ответить
    • $this->$hachalnika->ashambeMashambeShaitanaMaByhponyalIline eet($this->stroika);
      Ответить
    • гмм, _КЛАССИЧЕСКАЯ_НЕМЕЦКАЯ_ФАНТАСТИКА_ теперь и на ГК.ру
      Ответить
    • СказкаОТомКакДевочкаВКраснойШапочкеПошла КБабушкеЧерезЛесОтнестиПирожкиИСеромВолк еКоторыйСьелБабушкуАКраснуюШапочкуХотелО бманутьНоОхотникиЕеСпасли
      Ответить
      • while( !детиСпят ) {
        	СказкаОТомКакДевочкаВКраснойШапочкеПошлаКБабушкеЧерезЛесОтнестиПирожкиИСеромВолкеКоторыйСьелБабушкуАКраснуюШапочкуХотелОбманутьНоОхотникиЕеСпасли.Рассказывать();
        }
        Ответить
      • Это вариант для детей. А где взрослый вариант, без цензуры?
        Ответить
        • try {
          	with ( соседка ) {
          		while( детиСпят && женыНетДома ) {
          			СказкаОТомКакДевочкаВКраснойШапочкеПошлаКБабушкеЧерезЛесОтнестиПирожкиИСеромВолкеКоторыйСьелБабушкуАКраснуюШапочкуХотелОбманутьНоОхотникиЕеСпасли.Рассказывать();
          		}
          	}
          } catch (e) {
          	with( жена ) {
          		СказкаОТомКакСоседкаПришлаПопроситьСтаканчикСахараИлиМукиВКраснойШапочке.РассказыватьОченьУбедительно();
          	}
          }
          Ответить
    • АдинЭсники пересаживаются на ПоХаПэ, что может быть хуже?
      Ответить
    • развели тут форум :) Ыыы
      Ответить
    • господа, а никого не смутило то, что аргументом метода класса передается переменная класса?
      $this->что-то-там->где-то-здесь($this->моя-прелесть)
      Ответить
      • всё ок, переменная класса $params содержит array_merge к дефолтному $params, дабы инкапсулировать некий функционал
        Ответить
        • > инкапсулировать функционал
          выскакивает таблица
          Ответить
    • прочитал как $this->client->getBerechneteGrundversordnung...
      Ответить
    • показать все, что скрытоvanished
      Ответить

    Добавить комментарий