Правильно делать так:

Код:
Math.floor (Math.random () * N)
Просто random конечно работает. Но тут встает вопрос в стили кодирования. Если я пишу код с типизацией и на классах, то не использую старые приемы. Иначе это выглядит так как-будто вы купили новинький Mercedes S, а колпаки на него поставили от Запорожца, потому что они дешевле.