Недавно проходил интервью в одну иностранную компанию - вроде прошел, но чтобы быть более уверенным дополнительно решил скинуть им пример кода. И получил отказ потому-что по их словам у меня в коде очень много анти-паттернов. К сожалению более детальное обьяснение от них получить не удалось.

https://github.com/alexeypetrushin/synchronize/blob/master/sync.js

Подскажите пожалуйста пару проблемных мест, сам я их там не вижу.

Я видимо случайно не на ту категорию щелкнул, ХМЛ конечно-же ни при чем, топик попал сюда по ошибке, нужно удалить.

П.С.

За отзыв спасибо, если число аргументов будет == 0 - будет ошибка. Какой смысл ее проверять отдельно? Если ужпроверять - тогда нужно проверять и то что там действителоьно что-то есть а не null/undefined.

Но частично согласен, я лично не люблю лишний код который делает кучу ненужных проверок, но многие предпочитают именно такой код.