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

    +159.8

    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
    function check_date() {
	var	message = "Дата невалидна.";
		days = [31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31];

	var str = $('#registration-form [name="birthday"]').val();

	var m = str.match(/(\d*)\.(\d*)\.(\d*)(.*)/);

	if (!m ||
		!m[1] ||
		!m[2] ||
		!m[3] ||
		m[4] ||
		parseInt(m[2], 10) < 1 ||
		parseInt(m[2], 10) > 12 ||
		parseInt(m[1], 10) < 1 ||
		parseInt(m[1], 10) > days[parseInt(m[2], 10)-1] ||
		parseInt(m[3], 10) < 1920 ||
		parseInt(m[3], 10) > 2000 ) {
		tooltip($('#registration-form [name="birthday"]').get(0), message);
	} else {
		removett($('#registration-form [name="birthday"]').get(0));
	}
}

    Не очень говно, но вполне себе код.

    Запостил: ixth, 03 Марта 2010

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

    • ava ixth 03.03.2010 16:15 # 0

      И да, валидация даты, конечно же.
      Ответить

    • ava ixth 03.03.2010 16:31 # 0

      Переписал как-то так и жажду одобрения:
      function check_date() {
	var	message = "Дата невалидна.",
		m = $('#registration-form [name="birthday"]').val().split('.'),
		date = new Date(m[2], m[1] - 1, m[0]);

	if (date.getDate() != m[0] ||
		date.getMonth() != m[1] - 1 ||
		date.getFullYear() != m[2]) {
		tooltip($('#registration-form [name="birthday"]').get(0), message);
	} else {
		removett($('#registration-form [name="birthday"]').get(0));
	}
}
      Ответить

      • ava murz 03.03.2010 16:42 # 0

        Не учитывается диапазон годов
        Ответить

        • ava ixth 03.03.2010 16:48 # 0 

          || !(1920 < m[3] && m[3] < 2000)
          ммм?
          Ответить

          • ava guest 06.03.2010 12:21 # 0

            А что же будет с нашими уважаемыми ветеранами? Что ж теперь, 90-летним в интернет не ходить?!
            Ответить

      • ava guest 03.03.2010 16:45 # 0

        Не понял, опять реализуется сценарий "я признаю, что мой код - говно, а вы даёте мне годных советов"? =)
        Ответить

      • ava murz 03.03.2010 16:45 # 0

        нет проверки на правильность ввода, например, "5.12" или "", или "произвольный текст"
        Ответить

        • ava murz 03.03.2010 16:50 # 0

          или "dd.mm.yyyy"
          Ответить

          • ava ixth 03.03.2010 17:16 # 0

            Оно просто не отпарсится в дату и зафэйлит валидацию.
            Ответить

            • ava murz 03.03.2010 17:28 # 0

              хорошо, тогда "12.06.1990.456.789"
              Ответить

              • ava ixth 03.03.2010 18:09 # 0

                Ок, тогда так:
                var	message = "Дата невалидна.",
	m = document.registration.birthday.value.match(/^(\d{1,2})\.(\d{1,2})\.(\d{4})$/),
	date = m ? new Date(m[3], m[2] - 1, m[1]) : null;

if (!date ||
	date.getDate() != m[1] ||
	date.getMonth() != m[2] - 1 ||
	date.getFullYear() != m[3] ||
	!(1920 < m[3] && m[3] < 2000)) {
	tooltip($('#registration-form [name="birthday"]').get(0), message);
} else {
	removett($('#registration-form [name="birthday"]').get(0));
}
                Ответить

                • ava guest 03.03.2010 22:39 # 0

                  29.02.2010?
                  Ответить

                  • ava ixth 04.03.2010 00:49 # 0

                    Превратится в первое марта и отфильтруется этим:
                    date.getDate() != m[1] ||
	date.getMonth() != m[2] - 1 ||
	date.getFullYear() != m[3] ||
                    Ответить

    • ava mrbig66 04.03.2010 16:23 # 0

      Лучше намутить виджет "Календарь". Он не даст ошибиться.
      Ответить

    • ava mrbig66 04.03.2010 16:23 # 0 

      var	message = "Дата невалидна.",


      Прочитал: Два инвалида. )))
      Ответить

    • ava guest 05.03.2010 21:49 # −1

      сам исправляй свои баги, говнопостер!
      Ответить

    • ava bugmenot 02.06.2010 00:20 # 0

      теплое, мягкое jquery
      Ответить

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