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

    • ava Freeman 25.05.2012 18:12 # +4

      вы несёте ухйню товарищ.
      Ответить

      • ava DarkThinker 25.05.2012 21:40 # −6

        показать все, что скрытообоснуйте...
        я таким методом решал задачи первые 2 мес знакомства с javascript и jQuery.
        если вы считаете, что это не говнокод, то вы нуб.
        кроме того, для чего переприсваивать несколько раз одни и теже значения, когда можно сделать код более компактным. В моем варианте этот код занял бы на 20 строчек меньше.
        лечите чувство юмора, товарищ
        Ответить

        • ava Freeman 26.05.2012 01:47 # 0

          речь не идёт о том, кто во сколько строчек уложится.
          Ответить

          • ava DarkThinker 26.05.2012 14:03 # +1

            правильно, дело не встрочках, дело в подходе и логики в целом.
            Ответить

        • ava alex322 26.05.2012 09:11 # +3

          Как бы вы получили значения:
          $("#idImg").attr('src');
$("#idFromWebSite").attr('href');
$.cookie('pamp');

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

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