javascript - ReferenceError: date is not defined - Stack Overflow

admin管理员组

文章数量:1316020

I have some code here where I get a value from a form represented by "adate". Then I split the string the user enters at the hyphen and separate each value into year, month and day as you can see. I use those values to define a date object. My console correctly displays the date, but I keep getting this error also showing up. Am I defining the date incorrectly? I'm not sure what the issue is.

function getFormData() {
    var task = document.getElementById("task").value;
    if (checkInputText(task, "Please enter a task")) return;

    var who = document.getElementById("who").value;
    if (checkInputText(who, "Please enter a person to do the task")) return;

    var adate = document.getElementById("dueDate").value;
    var reString = new RegExp("[0-9]{4}\\-\[0-9]{2}\\-\[0-9]{2}");

    if ( adate.match(reString)) {
    processDate(adate) } 

    else {
     alert("you did not enter the date in the correct format")
    };

    var id = (new Date()).getTime();
    var todoItem = new Todo(id, task, who, date);
    todos.push(todoItem);
    addTodoToPage(todoItem);
    saveTodoItem(todoItem);

    hideSearchResults();
}    


function processDate(adate) {
     var splitArray = new Array();
     splitArray = adate.split("-");
     var year = splitArray[0];
     var month = splitArray[1] - 1;
     var day = splitArray[2];
     var date = new Date(year, month, day);
     console.log(date);
    }

I have some code here where I get a value from a form represented by "adate". Then I split the string the user enters at the hyphen and separate each value into year, month and day as you can see. I use those values to define a date object. My console correctly displays the date, but I keep getting this error also showing up. Am I defining the date incorrectly? I'm not sure what the issue is.

function getFormData() {
    var task = document.getElementById("task").value;
    if (checkInputText(task, "Please enter a task")) return;

    var who = document.getElementById("who").value;
    if (checkInputText(who, "Please enter a person to do the task")) return;

    var adate = document.getElementById("dueDate").value;
    var reString = new RegExp("[0-9]{4}\\-\[0-9]{2}\\-\[0-9]{2}");

    if ( adate.match(reString)) {
    processDate(adate) } 

    else {
     alert("you did not enter the date in the correct format")
    };

    var id = (new Date()).getTime();
    var todoItem = new Todo(id, task, who, date);
    todos.push(todoItem);
    addTodoToPage(todoItem);
    saveTodoItem(todoItem);

    hideSearchResults();
}    


function processDate(adate) {
     var splitArray = new Array();
     splitArray = adate.split("-");
     var year = splitArray[0];
     var month = splitArray[1] - 1;
     var day = splitArray[2];
     var date = new Date(year, month, day);
     console.log(date);
    }

• javascript
• date

Share Improve this question Follow edited Feb 27, 2013 at 22:55 user2084813 asked Feb 27, 2013 at 22:43 user2084813user2084813 18522 gold badges33 silver badges1212 bronze badges 7

• 1 can you please add what your sample input would look like? ie "12-5-80" or what ever? – Brad Commented Feb 27, 2013 at 22:44
• The date object also takes strings: new Date('01/01/2013') – elclanrs Commented Feb 27, 2013 at 22:46
• 1 The error doesn't e from your posted code. jsbin./otijin/1/edit BTW var splitArray = new Array(); is useless, .split() returns a new Array object so var splitArray = adate.split("-"); cuts out unnecessary overhead. – Fabrício Matté Commented Feb 27, 2013 at 22:46
• 2 @elclanrs: xkcd./1179 :-) – Bergi Commented Feb 27, 2013 at 22:47
• Could you please show us the stack trace of the exception? To me it looks like it es not from the posted code. – Bergi Commented Feb 27, 2013 at 22:49

 |  Show 2 more ments

2 Answers 2

Sorted by: Reset to default 3

Make your function return the date, because the date variable in there is not visible to the outside:

function processDate(adate) {
    var splitArray = new Array();
    splitArray = adate.split("-");
    var year = splitArray[0];
    var month = splitArray[1] - 1;
    var day = splitArray[2];
    return new Date(year, month, day);
}

Then assign to a new variable when you call it:

var date = processDate(adate);

The error actually originated in the following line, because you were referencing a non-existing date variable:

var todoItem = new Todo(id, task, who, date);

Just a ment.

The RegExp constructor is usually only required where the expression is dynamically generated. Where you have a fixed expression, it's simpler to use a literal (as you don't have to quote certain characters). Also, to test the format, a more appropriate method is test rather than match.

If the date format is: yyyy-mm-dd, consider:

var reString = /^\d{4}-\d\d-\d\d$/; // trim leading and trailing white space?

if (reString.test(adate)) {
  processDate(adate);
}

The date string validation should be in the processDate function, which might throw different errors depending on whether the format is incorrect or the date is invalid (e.g. 2013-02-29, which will return a date of 2013-03-01 in your current code).

本文标签： javascriptReferenceError date is not definedStack Overflow