Использование идентификатора юзера в моделях. Корректно ли?
Как по мне так вполне корректно, другое дело что лучше на хардкодить, а передать параметром в конструктор.
Плачевными последствиями может обернуться наличие (и их дальнейшее увеличение) 5-10 методов в модели. :) Используйте паттерн "репозиторий" или CQRS.
Если же всё нормально, то наличие определение userId в модели - норм.