1. ActionScript / Говнокод #16727

    −84

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    public function isContainsHiddenId(hiddenId: String): Boolean
 		{
-			for (var index: int = 0; index < hiddenIds.length; index++)
+			/*for (var index: int = 0; index < hiddenIds.length; index++)
 			{
 				if (hiddenIds[index] == hiddenId)
 				{
 					return true;
 				}
 			}
-			return false;
+			return false;*/
+			return hiddenIds.indexOf(hiddenId) == -1 ? false : true;

    От автора MathAndGeometryUtils, после того, как ему посоветовали использовать indexOf.

    Запостил: wvxvw, 19 Сентября 2014

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

    • ava kyzi007 19.09.2014 12:14 # 0

      Это не лечится.
      Ответить

    • ava bormand 19.09.2014 12:17 # +2

      > как ему посоветовали использовать indexOf
      Везёт вам в экшонскрипте... А вот в блядском IE8 нету indexOf.
      Ответить

      • ava Elvenfighter 19.09.2014 12:50 # +4

        А вот в блядском IE8 нету <подставь что угодно>.
        Ответить

        • ava inkanus-gray 18.11.2014 10:03 # 0

          Я недавно подставил data:URI, а оказалось, что в IE8 это есть. Хотя... выяснилось, что оно там так реализовано, что лучше бы его не было.
          Ответить

      • ava wvxvw 19.09.2014 12:56 # +1

        У меня такое подозрение, что создание более-менее нетривиальных приложений в ж.скрипте в любом случае обязывает использовать какой-нибудь модрнзр, или что-то такое, где эту неприятность исправят. Принимая во внимание то, что это проект с использованием Флекса, тут никто не поскупился бы загрузить библиотеку, если бы она была нужна, какие-то пару десятков килобайтов погоды не сделают.
        Ответить

    • ava makc3d 18.11.2014 03:57 # +1

      по рукам надо давать за "isContains". это блядские code conventions for people with no brain
      Ответить

      • ava inkanus-gray 18.11.2014 07:39 # 0

        А isContaining будет ближе к естественному языку или всё равно лажа?
        Ответить

        • ava bormand 18.11.2014 07:47 # +1

          present continous или как там его?

          Object is containing hidden id at this moment.
          Ответить

        • ava roman-kashitsyn 18.11.2014 07:50 # +2

          Не, факты принято выражать в Present Simple.
          Ответить

          • ava inkanus-gray 18.11.2014 08:06 # 0

            У меня есть ещё варианты:
            1. isHiddenIdContained.
            2. doesContainHiddenId.
            Ответить

            • ava bormand 18.11.2014 09:13 # 0

              > isHiddenIdContained...
              ...inTheObject
              Ответить

            • ava wvxvw 18.11.2014 09:45 # +3

              Contained и contains - разные вещи. Contained - чаще используется как прилагательное "сдержаный", contains как глагол "содержит".
              А вообще, желание писать isX идет вразрез с тем, как такие слова строятся в естесственном языке. Т.е. схема is[прилагательное|отглагольное существительное] воспринимается как новояз, потому что в естесственном языке у свойств есть "полярность", т.е. когда мы говорим, например, "высокий", нам не нужно уточнять, что объект не невысокий, мы бы просто не использовали слово "высокий" для описания чего-то низкого.
              А если мы копнем глубже, например, почитаем Хайдеггера (Бытие и время) или Хана (Глагол "быть" в греческой философии), то поймем, что использование этого глагола в таком качестве ведет к отсутствию когерентности, упадку и забвению человеческой мысли и, вобщем, все умерли, очень печально.
              Ответить

              • ava roman-kashitsyn 18.11.2014 10:50 # +3

                Трилобиты умерли, динозавры умерли, а человек так вообще обречён.
                Ответить

                • ava makc3d 18.11.2014 11:09 # +2

                  if (trilobitesAreDead) // не, нихуя, у нас так не пишут
                  if (areTrilobitesDead) // лучше, но не с из
                  if (isTrilobitesDead) // тимлид улыбается - я иду в правильном направлении
                  if (isDeadTrilobites) // "схема is[прилагательное|отглагольное существительное]" ??
                  if (isDeadEveryTrilobite) // а теперь даже число правильное - коммитим!
                  Ответить

                  • ava roman-kashitsyn 18.11.2014 11:14 # 0 

                    if (all(trilobites, _.dead()))
                    Ответить

                    • ava guest 18.11.2014 11:27 # 0

                      переголова, у нас уже в переменной все посчитано.
                      Ответить

                    • ava 3.14159265 18.11.2014 16:10 # 0

                      >_
                      Кто-нибудь может мне объяснить, что за любовь к именованию переменных таким образом?
                      Ответить

                      • ava roman-kashitsyn 18.11.2014 16:28 # 0

                        магическое подчёркивание же

                        В скалке это сахар для простых лямбд
                        _.prop === x => x.prop

                        В хаски для переменных, начинающихся с подчёркивания, не выводятся ворнинги при их неиспользовании (есть идиома использовать подчёркивание как шаблон "catch all" при паттерн-матчинге)
                        Ответить

                        • ava 1024-- 18.11.2014 16:38 # 0

                          > В скалке это сахар для простых лямбд
                          Какая прелесть, вот бы в JS Это было.

                          Хотя, в ES6 можно и самостоятельно сделать
                          var _ = new Proxy({}, {
  get: function(_, name){
    return function(x){
      return x[name];
    }
  }
});
                          Ответить

                        • ava 3.14159265 18.11.2014 17:39 # 0

                          Не, я вот видел такие именования без привязки к специфике языка.
                          Ответить

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