Apart from what has been said by Timothy, see Real Life merchant ships of the type you want.
Some points
First the funnels, which although look o.k they are too far apart. Most ships with a dummy funnel had the aft most funnel as a dummy (there were exceptions of course, but as a general rule it is correct)
Deck heights on your ship vary, the deck above the life boats seems too low, especially when compared to the deck below it.
If it is a passenger carrying ship, than most cabins are usually in the hull itself, so you'll have a lot of port holes in the hull
example, or
here.
If it is a passengers ferry/cruise ship than Timothy's example is excellent for a modern ship, and of course there are many others.
Bridge wings are usually open and extend from the bridge. Some have the navigation light, and on Shipbucket it is green as we display the starboard side of the ship (you don't have green and red together)
your fore deck is too empty. Either you have cargo there or more money earning cabins, again look at Real Life ships .
After you have completed the drawing add some details and shading and you'll do fine.
_________________
Thank you Kim for the crest
"Never fear to try on something new. Remember that the Titanic was built by professionals, and the Ark by an amateur"