item2を選択する際、

const item2=document.querySelectorAll('li')[2];

で選択していますが、item2は

const item2=document.querySelectorAll('li')[3];

と選択するのではないのでしょうか？
なぜ[2]なのでしょうか？