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

    +157

    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
    $("#SubmitButton").click(function () {
	        for (a_cik = 0; a_cik < 2; a_cik++) {
	            for (c_cik = 0; c_cik < 7; c_cik++) {
	                elem_cik=$('#AvailabilityList_'+a_cik+'__Years_'+c_cik+'_');
	                if (elem_cik != null) {
	                    if (elem_cik.parent().parent().hasClass('qqq')) {  // Проверка элемента на видимость
	                        if (!(((elem_cik.val() >= '1') && (elem_cik.val() <= '9')) || ((elem_cik.val() >= '10') && (elem_cik.val() <= '17')))) {
	                            alert('Ошибка. Возраст ребёнка не указан, или задан в неверном формате. Исправьте ошибку и повторите попытку');
	                            elem_cik.focus();
	                            return false;
	                        }
	                    }
	                }
	            }
	        }
	        $("#BookingForm").submit()
	    });

    Валидация возрастов детей перед их передачей дальше. ATTEMPTION MAGIC NUMBER!

    Запостил: sladkijBubaleh, 16 Октября 2013

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

    • ava Little-Horny 16.10.2013 14:28 # +2

      ATTENTION вот так пишется.

      Ошибка! Ребёнок не наказан!
      Ответить

    • ava wvxvw 16.10.2013 16:08 # 0

      Ну а вдруг ребенок-долгожитель? После ста, там уже наверное все пофигу.
      Ответить

    • ava 1024-- 16.10.2013 18:22 # +2

      Вот что выходит, когда автор не знает про регулярные выражения.
      > if (!(((elem_cik.val() >= '1') && (elem_cik.val() <= '9')) || ((elem_cik.val() >= '10') && (elem_cik.val() <= '17')))) {
      if (!/^[1-8]|^9$/.test(elem_cik.val())) {
      Ответить

      • ava kegdan 16.10.2013 19:17 # 0

        эээ... а почему в инт не конвертнуть, да не проверить на <18?
        Ответить

        • ava 1024-- 16.10.2013 19:39 # +1

          > эээ... а почему в инт не конвертнуть, да не проверить на <18?
          Как говорит наш зелёный друг, - "да Вы, батенька, хам"!
          <18 - это не 20, не 86 и не 235, а код автора допускал эти варианты.
          Ну и конечно же, преобразование в число тормозит
          Ответить

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