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

    +165

    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
    once = true;
$(document).ready(function () {
    $('#Town').change(function () {
        if (once) {
            if ($(this).val() != $("#TownName").val()) {
                alert("При смене города произойдет потеря всех товаров в корзине");
                once = false;
            }

        }
    }
    })
);

    вместо
    $('#Town').one('change', function() {})

    Запостил: nur, 29 Июня 2010

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

    • ava Lure Of Chaos 29.06.2010 16:37 # +6

      "При смене города произойдет потеря всех товаров в корзине", а при смене телефона - всех товаров в магазине
      Ответить

      • ava 3.14159265 29.06.2010 16:51 # +4

        lol
        боюсь представить что произойдет при смене браузера
        Ответить

      • ava Анонимус 29.06.2010 18:11 # +5

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

    • ava turdman 29.06.2010 17:06 # +1

      once небось глобальная?
      Ответить

      • ava Lure Of Chaos 29.06.2010 17:12 # 0

        а как же
        Ответить

      • ava babaika 29.06.2010 21:14 # −3

        Ну а что такого? Все равно надо где-то флажок сохранить, показали алерт или нет. Либо в недрах джиквери, либо так, стикером на лоб. Вот кстати этот самый one:
        function( type, data, fn ) {
		var one = jQuery.event.proxy( fn || data, function(event) {
			jQuery(this).unbind(event, one);
			return (fn || data).apply( this, arguments );
		});
		return this.each(function(){
			jQuery.event.add( this, type, one, fn && data);
		});
	}
        Ответить

        • ava Анонимус 29.06.2010 22:35 # +2

          А Вы инкапсуляцию данных не проходили еще?
          Ответить

          • ava babaika 30.06.2010 00:36 # −3

            О да, давайте защитим флажок от повреждения. Не дай бог кто сломает нечаянно и снова этот алерт увидит, вот тут точно всем кранты.
            Говнокодом тут является проверка на "поменялся ли город" после change, да и сам факт потери товаров из корзины не радует.
            Ответить

            • ava Анонимус 30.06.2010 00:38 # +2

              угу, а глобальные перменные -- это нормально. Вы всегда так пишите -- и ни одной проблемы, правда? А если две переменные случай одинаково назовутся -- так это легко отловить же, да
              Ответить

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