2016-07-11 8 views
0

Ich versuche, das aktuelle Datum auf der Seite als Zeitstempel zu drucken, wenn eine neue Aufgabe (Obj) angezeigt wird, aber neue Zeitstempel alle dasselbe Datum sind, unabhängig davon, wann der Zeitstempel erstellt wird. Jede Hilfe würde sehr geschätzt werden.Neue Zeitstempel sind alle das gleiche Datum

function getDate() { 
    var today = new Date(); 
    var dd = today.getDate(); 
    var mm = today.getMonth() + 1; 
    var yyyy = today.getFullYear(); 
    var time = today.getTime(); 
    if (dd < 10) { 
     dd = '0' + dd 
    } 

    if (mm < 10) { 
     mm = '0' + mm 
    } 

    today = mm + '/' + dd + '/' + yyyy + ' ' + time; 

    localStorage.setItem('newDate', today); 
    var newDate = localStorage.getItem('newDate'); 
    return (newDate); 
} 

function show() { 
    var todos = get_todos(); 
    var date = localStorage.getItem('newDate'); 
    var html = '<table>'; 
    for (var i = 0; i < todos.length; i++) { 
     html += '<br><tr><strong>' + 
      '<input type="image" src="Pictures/remove.png" class="remove" id="' + i + 
      '"></input>' + todos[i] + ' - ' + date + '</strong><input type="checkbox" name="cBox" id="isDone"><label for="cBox"</label></tr><br>'; 
    }; 
    html += '</table>'; 


    document.getElementById('todos').innerHTML = html; 

    var buttons = document.getElementsByClassName('remove'); 
    for (var i = 0; i < buttons.length; i++) { 
     buttons[i].addEventListener('click', remove); 
    }; 
} 

document.getElementById('add').addEventListener('click', add); 
show(); 
+2

Wo Sie getDate nennen()? So scheint Ihre localStorage eingestellt zu sein. – JonSG

+1

Warum verwenden Sie localStorage überhaupt? Bitte lerne auch [deinen Code einrücken] (http://jorendorff.blogspot.com.au/2012/01/why-do-we-indent-code.html). – nnnnnn

+0

Sie benötigen keine Klammern um die Variable, die Sie zurückgeben. Es ist auch nicht besonders wichtig, den Wert von localStorage in 'newDate' zu ​​ziehen, um ihn zurückzugeben. Sie könnten einfach 'today' zurückgeben, es wird genau der gleiche Wert sein. Wenn Sie einen Lesevorgang von localStorage hinzufügen, verlangsamt sich dies nur. –

Antwort

1
var date = localStorage.getItem('newDate'); 

Ich denke, sollte

var date = getDate(); 
Verwandte Themen