Czy właściwym jest ustawienie profilu użytkownika w programie jako static?

Witam, mam taki dylemat:

Piszę program w którym tworzymy sobie profil, następnie możemy się zalogować i działać na profilu do którego się zalogowaliśmy więc jest tylko 1 instancja profilu użytkownika i jako, że muszę odwoływać się w wielu miejscach do danych przechowywanych wewnątrz tej klasy to tworzę tą klasę jako static przez co łatwo z dowolnego miejsce jest mi się odwołać do potrzebnych danych.

Zastanawia mnie tylko, czy jest to właściwe rozwiązanie?

Nie. Powinieneś zastosować wstrzykiwanie zależności.
Wersja dla amatorów to zastosowanie singletona.

Static oznacza, że dany obiekt jest dostępny w ramach całej maszyny. Dla ciebie rozwiązaniem jest użycie DI i zakresu sesji.