JS todo list item选中，并添加item

Question

我的目标是创建一个类似于https://www.w3schools.com/howto/howto_js_todolist.asp的JS todo列表

我已经创建了我的待办事项列表应用程序的基本结构和关闭按钮的功能。 这些工作正常，但我在添加新的待办事项列表项以及选中和取消选中待办事项时遇到问题。 我不确定我是否很好地使用了类列表切换属性，也无法弄清楚为什么添加按钮根本不起作用。

var todoitemlist = document.getElementsByClassName('todo-item');

var i;
for (i = 0; i < todoitemlist.length; i++) {
  var span = document.createElement("SPAN");
  span.innerHTML = "Close";
  span.className = "closebutton";
  todoitemlist[i].appendChild(span);
}

var close = document.getElementsByClassName("closebutton");

var i;
for (i = 0; i < close.length; i++) {
  close[i].onclick = function() {
    var listitem = this.parentElement;
    listitem.style.display = "none";
  }
}
var todoitemlistx = document.getElementsByClassName('todo-item');
//checked element
var i;
for (i = 0; i < todoitemlistx.length; i++) {
  todoitemlistx[i].onclick = function(ev) {
    ev.style.backgroundColor = "red";
    ev.classList.toggle("todo-item-checked");
  }
}

//add another list item
function add() {
  var listitem = document.createElement("LI");
  listitem.className = "todo-item";
  var text = document.getElementById('todoinput').value;
  var myul = getElementById('todo-list');
  var t = document.createTextNode(text);
  listitem.appendChild(t);
  myul.appendChild(listitem);

  var span = document.createElement("SPAN");
  span.innerHTML = "Close";
  span.className = "closebutton";
  listitem[i].appendChild(span);

  for (i = 0; i < close.length; i++) {
    close[i].onclick = function() {
      var div = this.parentElement;
      div.style.display = "none";
    }
  }
}

* {
  box-sizing: border-box;
}

body {
  text-align: center;
  margin: 0;
  padding: 0;
  background-color: #dbf9fc;
  font-family: Arial, Helvetica, sans-serif;
  color: rgba(0, 27, 39);
}

#todoinput {
  margin: 5px;
  padding: 5px;
  width: 65%;
}

#add-button {
  padding: 5px;
  margin: 5px;
  width: 5%;
  background-color: rgba(0, 27, 39);
  color: #dbf9fc;
  border: none;
  border-radius: 5px;
  height: 1fr;
  cursor: pointer;
}

#add-button:hover {
  background-color: black;
}

#todo-list {
  display: inline-block;
  margin: 0;
  padding: 0;
  list-style: none;
  width: 70%;
}

.todo-item {
  position: relative;
  display: flex;
  justify-content: flex-start;
  background-color: white;
  border: 2px solid black;
  padding: 5px;
  margin: 5px;
}

.closebutton {
  cursor: pointer;
  justify-self: flex-end;
  background-color: #e6772d;
  position: absolute;
  right: 0;
  top: 0;
  color: white;
  float: right;
  padding: 5px;
  width: 30%;
  margin: 0;
}

.closebutton:hover {
  background-color: #c46526;
}

.todo-item-checked {
  position: relative;
  display: flex;
  justify-content: flex-start;
  background-color: rgb(187, 187, 187);
  border: 2px solid black;
  padding: 5px;
  margin: 5px;
  text-decoration: line-through;
}

.black {
  background-color: black;
}

<main class="centered">
  <h1>ToDo List JS</h1>
  <h3>js project</h3>
  <form action="">

    <input type="text" name="" id="todoinput" placeholder="Enter the activity you've wented to do">
    <input type="button" value="Add" id="add-button" onclick="add()">
  </form>
  <ul id="todo-list">
    <li class="todo-item">
      Hit the lights
    </li>
    <li class="todo-item">
      Hit the lights
    </li>
    <li class="todo-item">
      Hit the lights
    </li>
    <li class="todo-item-checked todo-item">
      Hit the lights
    </li>
  </ul>
</main>

Answer 1

我记得开始开发的感觉和难度，所以我坚持了下来。 我原以为会是一些小的改动，但结果变成了大规模的重写。

Una grande padulo，就像意大利人喜欢说的那样。

我希望你能或试着理解我在这里做了什么。

最大的教训是，不要重复编写相同的代码，将其放在单独的函数中。

所以这似乎有效。 如果没有，请告诉我。

 var todoitemlist=document.getElementsByClassName('todo-item');

 for(var i=0;i<todoitemlist.length;i++)
 {
  myawesomeclosebutton(todoitemlist[i]);
  todoitemlist[i].addEventListener('click',myawesomebackground);
 }

 function myawesomeclosebutton(myawesomeitem)
 {
  var span=document.createElement('span');
  span.innerHTML='Close';
  span.className='closebutton';
  span.addEventListener('click',myawesomecloseevent);
  myawesomeitem.appendChild(span);
 }

 function myawesomebackground(ev)
 {
  if(ev.target.style.backgroundColor!='red')
  {
   ev.target.style.backgroundColor='red';
  }
  else
  {
   ev.target.style.backgroundColor='transparent';
  }
 }

 function myawesomecloseevent(event)
 {
  var div=event.target.parentElement;
  div.style.display='none';
 }

 function add()
 {
  var listitem=document.createElement('li');
  listitem.className='todo-item';
  var myawesomeinput=document.getElementById('todoinput');
  var text=myawesomeinput.value;
  var myul=document.getElementById('todo-list');
  var t=document.createTextNode(text);
  listitem.appendChild(t);
  myul.appendChild(listitem);
  listitem.addEventListener('click',myawesomebackground);
  myawesomeclosebutton(listitem);
  myawesomeinput.value='';
 }