Welcome to the forums.
I have edited your thread title. "Please help" is not a useful title. Use a phrase from your question as your thread title so you can find it later. (In another forum, a user has posted probably 50 threads all with the title "Proofreading?" I don't know how he finds them again.)
Your sentence is okay as written.
Student or Learner