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

    +142

    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
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    $("#idSave").click(function () {
            var img_src = $("#idImg").attr('src');
            var web_src = $("#idFromWebSite").attr('href');
            var pamp_id = $.cookie('pamp');
            var title_src = $("#idTitle").val();
            var brand_src = $("#idBrand").val();
            var tags_src = $("#idTags").val();
            var price_src = $("#idPrice").val();
            var privacy_src = $("#idPrivacy").val();
    
            var data = new Object();
            data.Url = img_src;
            data.UrlFrom = web_src;
            data.PampID = pamp_id;
            data.Title = title_src;
            data.Brand = brand_src;
            data.Tags = tags_src;
            data.Price = price_src;
            data.Privacy = privacy_src;
    
            $.ajax({
                url: url,
                type: "POST",
                dataType: "json",
                data: data,
                success: successProductSave,
                error: errorProductSave
            });
        });

    - $().serialize() and $().serializeArray() ?? WTF ??
    - RTFM ....

    Запостил: DarkThinker, 25 Мая 2012

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

    • вы несёте ухйню товарищ.
      Ответить
      • показать все, что скрытообоснуйте...
        я таким методом решал задачи первые 2 мес знакомства с javascript и jQuery.
        если вы считаете, что это не говнокод, то вы нуб.
        кроме того, для чего переприсваивать несколько раз одни и теже значения, когда можно сделать код более компактным. В моем варианте этот код занял бы на 20 строчек меньше.
        лечите чувство юмора, товарищ
        Ответить
        • речь не идёт о том, кто во сколько строчек уложится.
          Ответить
          • правильно, дело не встрочках, дело в подходе и логики в целом.
            Ответить
        • Как бы вы получили значения:
          $("#idImg").attr('src');
          $("#idFromWebSite").attr('href');
          $.cookie('pamp');

          с помощью $(form).serialize() ?
          Ответить

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