1. JavaScript / Говнокод #21901

    −49

    1. 1
    2. 2
    3. 3
    4. 4
    function PhotoDetail(uid, id, key) {
    var uid = uid;
    var id = id;
    var key = key;

    No comments

    Запостил: gorsash, 26 Декабря 2016

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

    • ava Steve_Brown 26.12.2016 13:21 # +1

      Может, это перемещает переменные в какой-нибудь контекст замыкания или что-то в этом роде.
      Забавен набор аргументов: идентификатор, идентификатор, идентификатор.
      Ответить

      • ava gorsash 26.12.2016 13:24 # 0

        А разве в замыканиях что-то поменяется, кроме this? Вот вся функция:

        function PhotoDetail(uid,id,key)
        {
        var uid = uid;
        var id = id;
        var key = key;
        var lastid = $('#border_val_photo').val();

        $.ajax({
        type: "GET",
        url: "photo_detail.php?uid="+uid+"&id="+i d,
        success: function(msg){
        if(lastid)
        {
        document.getElementById("td_"+lastid).st yle.borderColor="#cbdbea";
        }
        document.getElementById("td_"+key).style .borderColor="#FF0000";
        $('#border_val_photo').val(key);
        $('#photo_detail').html(msg);
        PhotoAlbumDetail(uid,id);
        }
        });
        }
        Ответить

        • ava Vince 27.12.2016 08:49 # +1

          а вот это уже говнокод. И jQuery и pure js в кучу.
          Ответить

      • ava gorsash 26.12.2016 13:37 # 0

        Вроди и без локальных переменных всё работает

        function test(a) {

        setTimeout(function() {
        console.log(a);
        }, 1000);

        }

        test(1);
        test(2);
        test(3);
        Ответить

      • ava gorsash 26.12.2016 13:40 # 0

        Насколько я понимаю, имеет смысл делать копии переменных только, если они потом меняются в самой функции
        Ответить

      • ava 1024-- 26.12.2016 13:58 # +2

        Аргументы же и так как надо будут. Разве что var that = this нужно в некоторых случаях делать.
        Ответить

      • ava eternalko 12.01.2017 05:09 # 0

        Не перемещает :)
        Ответить

    • ava 1024-- 26.12.2016 13:55 # +3

      Лол. Говнокод обучающий. Не знал, что так можно.

      Меня деды обучили примером
      var x = 1;
function(){
   alert(x); // 1
}
function(){
   alert(x); // undefined
   if(false) { var x = 0; }
}


      Да и в том же интерпретаторе, где ОП-код работает,
      var x = 1;
function(){
  var x = x;
  alert(x); //undefied
}
      Ответить

      • ava gorsash 26.12.2016 13:57 # +1

        Ну это немного из другой оперы. Тут глобальные переменные
        Ответить

        • ava 1024-- 26.12.2016 14:02 # 0

          Переменные из внешнего контекста!
          Ответить

        • ava 1024-- 26.12.2016 14:07 # +1

          Впрочем, из той же оперы.

          Определения локальных переменных как бы вставляются в начало функции, демонстрируя единственно логичные следствия:
          function(x) { var y; } -> function var x, y (x) {}
function(x) { y; if(0) { var y; } } -> function var x, y (x) { y; }
function(x) { var x = x } -> function var x (x) { x = x; }

          Я раньше думал, что они только в начало тела вылезают.
          Ответить

          • ava bakagaijin 26.12.2016 20:29 # +1

            Объявления функций и переменных всплывают к началу области видимости (которая и есть функция). Присваивания остаются на месте. Стандартный вопрос - что упадёт?

            foo();
            function foo() {}

            или

            foo();
            var foo = function() {};
            Ответить

      • ava barop 26.12.2016 13:58 # +1

        Джаваскрипт такой непредсказуемый, круче плюсов
        Ответить

      • ava Fike 27.12.2016 23:59 # 0

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

        • ava Fike 28.12.2016 00:06 # 0

          и в очередной раз я отвечаю, неправильно поняв суть исходного комментария
          Ответить

          • ava bayan 28.12.2016 02:06 # 0

            яиратнеммок огондохси ьтус вяноп оньливарпен 'юачевто я зар йондеречо в и
            Ответить

      • ava 1024-- 07.01.2017 21:53 # 0

        Ну ты тупой! Уже джва года как знал же: http://govnokod.ru/16497#comment242907
        Ответить

        • ava nihau 12.01.2017 11:59 # 0

          не стыдно не знать джаваскрипта, стыдно его знать
          Ответить

          • ava bormand 12.01.2017 17:15 # +2

            Не стыдно знать экмаскрипт, стыдно этим гордиться.
            Ответить

            • ava barop 12.01.2017 18:02 # 0

              Не стыдно знать экмаскрипт, стыдно знать что-то еще и писать на экмаскрипте
              Ответить

              • ava guestinho 12.01.2017 19:27 # 0

                А если что-то еще - это пыха?
                Ответить

                • ava bormand 12.01.2017 19:28 # +3

                  От пыхи и жс не зарекаются.
                  Ответить

                  • ava nihau 12.01.2017 19:30 # +1

                    не заикаются
                    Ответить

                  • ava barop 13.01.2017 14:23 # +2

                    Конечно. Кризис же в стране. Вдруг придется разнорабочим поработать?
                    Ответить

                    • ava defecate-plusplus 13.01.2017 20:10 # +6

                      > разнорабочим
                      ээ, теперь это называется фулл-стек
                      Ответить

                      • ava inkanus-gray 13.01.2017 20:15 # +5

                        Фулл-стек по водоснабжению дома: рытьё траншеи, прокладка полиэтиленовых труб, установка фитингов, монтаж сантехники, разработка водоотведения.
                        Ответить

                        • ava barop 13.01.2017 20:36 # +5

                          А я, кстати, знаю таких вот фулл-стек мастеров, и обычно не очень им доверяю.

                          Чуваки которые одновременно делают электрику, сантехнику, лоджию, потолки, прокладывают телефон и клеют обои обычно делают это всё одинаково хуёво.
                          Ответить

                          • ava bakagaijin 13.01.2017 22:06 # +2

                            Два стакана стекломоя этому господину прорабу.

                            Вот кстати у меня такое чувство, что все "евроремонты" двухтысячных (или десятых, зависит от географии) - это в немалой степени заслуга тех самых, которые как-то с гипсокартоном умеют работать, как-то с плиткой, как-то с полами... за исключением знания тех самых маленьких частностей, которые не дают отделке превратиться в говно. Зато дёшево.
                            Ответить

    • ava barbaricus 25.01.2017 13:39 # 0

      Просто так спокойнее.
      Ответить

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