первый аргумент метода сплайсинга - индекс массива, а второй аргумент - сколько элементов вы хотите удалить из массива. Если вы используете
todolist.splice(i, 0);
, это означает, что вы удаляете 0 элемент, поэтому, если вы хотите удалить 1 элемент, вы должны использовать приведенный ниже код
todolist.splice(i, 1);