function orderList() の中の下記の部分ですが、

let selectHtml = "<select id=select" + i + ">";

HTML要素のidをダブルクォートで括って、

let selectHtml = '<select id="select' + i + '">';

とする方がよいのではないかと思いました。
元のソースでも動作はしますが。

はい、そのソースでもOKです。
どちらでも大丈夫です。
ありがとうございました。