Javascript неправильно проверяет дату

Question

Javascript неправильно проверяет дату

1

Я написал функцию, чтобы проверить, действительна ли дата

Я написал все, кроме одной небольшой части

Функция не работает, и я не могу найти ошибку, можете ли вы любезно помочь?

Я вижу, что, поскольку число 29 не в массиве, оно не работает, но запутано, как заставить его работать

function isValidDate() {
  var dateformat = /^(0?[1-9]|[12][0-9]|3[01])[-](0?[1-9]|1[012])[-]\d{4}$/;

  var readDate = document.getElementById("myDate").value;

  if (readDate.length <= 10) {
    <!--debug-->
    //console.log(readDate);

    /* split date into DD-MM-YYYY format */
    var splitDate = readDate.split('-');

    var day = splitDate[0];

    var month = splitDate[1];

    var year = splitDate[2];

    /* DEBUG - print split date into DD-MM-YYYY format */
    console.log('day ' + day);
    console.log('month ' + month);
    console.log('year ' + year);

    // Create list of days of a month [assume there is no leap year by default]  
    var ListofDays = [31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31];

    //if month is between 1-12
    if (month == 1 || month < 13) {
      //check for invalid month
      if (month == 00) {
        console.log('0 - Invalid MONTH format!');
        return 0;
      }

      //check for invalid day
      if (day == 00) {
        console.log('0 - Invalid DAY format!');
        return 0;
      }

      //check DAY exists in the MONTH
      if (day > ListofDays[month - 1]) {
        console.log('1 - Invalid DATE format!');
        return 0;
      } else {
        console.log('1 - Valid DATE format!');
        return 1
      }
    } else {
      console.log("Invalid MONTH");
      return 0;
    }

    //check for leap year
    if (year % 4 === 0 && year % 100 !== 0) {
      console.log('The year ' + year + ' is a leap year.');
      return 1;
    } else if (year % 4 === 0 && year % 100 === 0 && year % 400 === 0) {
      console.log('The year ' + year + ' is a leap year.');
      return 1;
    } else {
      console.log('The year ' + year + ' is NOT a leap year');
      return 0;
    }
  } else {
    console.log("Invalid DATE length")
  }
}


/*    This is the only bit I did not write:
    
    	if (day > ListofDays[month-1])  
    	{  
        console.log('1 - Invalid DATE format!');  
        return 0;  
    	} 
    	else
        {
        console.log('1 - Valid DATE format!');  
        return 1
    }

*/

<p>Input a date and check it in a)correct format and b)it is a valid date</p>

<input type="text" id="myDate" placeholder="DD-MM-YYYY"> <br><br>

<button type="button" onclick="isValidDate()"> Check Date </button>

simon 31 окт. 2017, в 17:12

Источник

1

stackoverflow.com/a/1353711/7239480
acnn 31 окт. 2017, в 15:18
0

Сделайте себе одолжение и используйте moment.js . Это сделало много тяжелой работы для вас.
Liam 31 окт. 2017, в 15:31
0

@Liam Лиам, чему это научит его. Также очень расточительно для такой простой задачи
mplungjan 31 окт. 2017, в 15:31
0

Гораздо проще разделить строку даты на «-», создать новый объект Date из частей (не забывайте начинать месяцы с 0): var testDate = new Date(year,month-1,day,0,0,0) и проверьте, что части совпадают с деталями даты. Также я не вижу, вы используете регулярное выражение
mplungjan 31 окт. 2017, в 15:33
0

Извините, я новичок в JS, я думал, что он использует регулярные выражения? Как я могу заставить его использовать Regex? И что это делает? (день> ListofDays [месяц - 1]) просто не может решить это
simon 31 окт. 2017, в 15:35
0

У вас есть var dateformat = /^(0?[1-9]|[12][0-9]|3[01])[-](0?[1-9]|1[012])[-]\d{4}$/; но вы не проверяете свою дату против нее
mplungjan 31 окт. 2017, в 15:38
0

извините, я так новичок в JS, как проверить свою дату на это?
simon 31 окт. 2017, в 15:44
0

хорошо, теперь я вижу - var dateformat = new RegExp (/ ^ (0? [1-9] | [12] [0-9] | 3 [01]) [-] (0? [1-9] | 1 [012]) [-] \ d {4} $ /); var readDate = document.getElementById ("myDate"). value; если (dateformat.test (readDate))
simon 31 окт. 2017, в 15:55

Показать ещё 6 комментариев

Теги:

javascript

3 ответа

0

Отбросьте все это.

<input type="date" />

Работа выполнена.

Кроме того, поскольку вы указываете формат, вы можете сделать что-то вроде этого:

var input = document.getElementById("myDate").value;
var parts = input.split("-");
var date = new Date(parts[2],parts[1]-1,parts[0]);
return date.getFullYear() == parts[2]
    && date.getMonth() == parts[1]-1
    && date.getDate() == parts[0];

Это работает, потому что JavaScript "фиксирует" дату, если она выходит за пределы допустимых границ (например, 32 января исправляется до 1 февраля), поэтому просто проверьте, не изменилось ли что-либо, когда она получила дату.

Niet the Dark Absol 31 окт. 2017, в 13:24

1

Спасибо, но мне не нужен JS, чтобы исправить дату. Я действительно хотел, чтобы это работало так, как я делаю это для моего собственного обучения, а затем попытался улучшить
simon 31 окт. 2017, в 15:39

0

function validate(value) {
  var v = false;
  if (value) {
    var r = value.match(/([0-9]{1,2}).?([0-9]{1,2}).?([0-9]{4})/);
    
    if (r) {
      var day1 = parseInt(r[1]);
      var month1 = parseInt(r[2]);
      var year1 = parseInt(r[3]);

      var d = new Date(year1, month1 - 1, day1);

      var day2 = d.getDate();
      var month2 = d.getMonth() + 1;
      var year2 = d.getFullYear();

      v = day1    == day2   &&
          month1  == month2 &&
          year1   == year2;
    }
  }
  
  return v;
}

<input type="text" id="toCheck" />
<button onclick="console.log(validate(document.getElementById('toCheck').value))">Check</button>

Вы можете попробовать с любым разделителем, это только пример (я использовал трюк, чтобы проверить реальную дату). Вы можете изменить эту функцию для личного использования.

Laurianti 31 окт. 2017, в 13:18

0

большое спасибо, но как получается, что мой код не работает? вот что я хочу попробовать и действительно исправить
simon 31 окт. 2017, в 16:01

Ещё вопросы

Сделайте себе одолжение и используйте moment.js . Это сделало много тяжелой работы для вас.
@Liam Лиам, чему это научит его. Также очень расточительно для такой простой задачи
Гораздо проще разделить строку даты на «-», создать новый объект Date из частей (не забывайте начинать месяцы с 0): var testDate = new Date(year,month-1,day,0,0,0) и проверьте, что части совпадают с деталями даты. Также я не вижу, вы используете регулярное выражение
Извините, я новичок в JS, я думал, что он использует регулярные выражения? Как я могу заставить его использовать Regex? И что это делает? (день> ListofDays [месяц - 1]) просто не может решить это
У вас есть var dateformat = /^(0?[1-9]|[12][0-9]|3[01])[-](0?[1-9]|1[012])[-]\d{4}$/; но вы не проверяете свою дату против нее
извините, я так новичок в JS, как проверить свою дату на это?
хорошо, теперь я вижу - var dateformat = new RegExp (/ ^ (0? [1-9] | [12] [0-9] | 3 [01]) [-] (0? [1-9] | 1 [012]) [-] \ d {4} $ /); var readDate = document.getElementById ("myDate"). value; если (dateformat.test (readDate))
Спасибо, но мне не нужен JS, чтобы исправить дату. Я действительно хотел, чтобы это работало так, как я делаю это для моего собственного обучения, а затем попытался улучшить
большое спасибо, но как получается, что мой код не работает? вот что я хочу попробовать и действительно исправить

mplungjan · Accepted Answer · 2017-10-31T13-32-00.000Z

Протестируйте дату против регулярного выражения, которое вы должны уменьшить.
НЕ возвращайся, когда хочешь продолжить

Вот решение, которое поддерживает код как можно ближе к тому, что вы хотите сделать. Более простое решение с использованием объекта Date, которое уже было предложено в комментарии, и было разработано другими

function showError(str) {
  console.log(str)
  return false;
}

function isValidDate() {
  var dateformat = /^(0?[1-9]|[12][0-9]|3[01])[-](0?[1-9]|1[012])[-]\d{4}$/;

  var readDate = document.getElementById("myDate").value;

  if (readDate.length != 10) return showError("Invalid DATE length") // not correct length
  console.log(dateformat.test(readDate));

  if (!dateformat.test(readDate)) {
    return showError("Invalid DATE format") // not matching regex
  }

  /* split date into DD-MM-YYYY format */
  var splitDate = readDate.split('-');

  var day = splitDate[0];
  var month = splitDate[1];
  var year = splitDate[2];

  /* DEBUG - print split date into DD-MM-YYYY format */
  console.log('day ' + day);
  console.log('month ' + month);
  console.log('year ' + year);

  // Create list of days of a month [assume there is no leap year by default]  
  var ListofDays = [31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31];

  //if month is not between 1-12
  if (month <= 0 || month > 12) {
    return showError('0 - Invalid MONTH format!');
  }

  var isLeap = false;
  //check for leap year
  if (year % 4 === 0 && year % 100 !== 0) {
    console.log('The year ' + year + ' is a leap year.');
    isLeap = true;
  } else if (year % 4 === 0 && year % 100 === 0 && year % 400 === 0) {
    console.log('The year ' + year + ' is a leap year.');
    isLeap = true;
  } else {
    console.log('The year ' + year + ' is NOT a leap year');
  }

  //check DAY exists in the MONTH
  var testDay = ListofDays[month - 1]; // array starts at 0
  // testDay += isLeap ? 1 : 0; // add one to testDay using ternary operator
  if (isLeap) testDay++; // less code, does the same as above
  if (day > testDay) {
    return showError('1 - Invalid DATE format!');
  }

  console.log('1 - Valid DATE format!');
  // You can return true here if you want


}

<p>Input a date and check it in a)correct format and b)it is a valid date</p>

<input type="text" id="myDate" placeholder="DD-MM-YYYY"> <br><br>

<button type="button" onclick="isValidDate()"> Check Date </button>

Большое спасибо, я нахожусь в дороге, так что я не могу проверить, позже проверю .... Кстати, как я могу изменить это регулярное выражение, чтобы я не позволял вставлять буквы в текстовое поле? Спасибо
НЕ возвращайся, когда хочешь продолжить - ааа я вижу
testDay + = isLeap? 1: 0 // как это можно упростить, так как я этого не понимаю и не пишу такие IF-выражения, я действительно честно не против того, чтобы это была длинная версия, для моего обучения
testDay += является сокращением для testDay = testDay + ... и isLeap ? 1 : 0 называется троичным и является сокращением для if (isLeap) testDay = testDay+1; else testDay = testDay + 0; это также может быть написано, if (isLeap) testDay++;
возврат покинет функцию. Вы можете использовать возвращаемое значение следующим образом: if (isValidDate()) this.form.submit();
ахх, спасибо, а также как я могу изменить это регулярное выражение, чтобы не допускать вставки букв в текстовое поле только цифрами?
Тебе нельзя. Вы можете добавить обработчик событий для проверки введенного ключа: stackoverflow.com/questions/19966417/…
Я вижу, потому что это текстовое поле ..... однако, если я использую тип ввода = число, то оно автоматически поместит кнопки вверх и вниз в поле ввода, которое мне не нужно ... как-нибудь обойти это?
Да, тип ввода = дата :) или проверить себя
Большое вам спасибо, моя репутация <15, поэтому я не могу проголосовать за вашу большую работу - большое спасибо