This will just be an embellishment of @user15077’s answer.
This is the beginning of your piece as you’ve notated it:
Here is what it would look like with a more standard approach:
As you can see, many of the notes are expressed as tied notes now. For example, the quarter-note D-sharp in the first measure is written as a sixteenth tied to a dotted eighth, even though there’s “room in the measure” for you just to write a quarter note. The reason for writing it with a tie is so that the performer can more easily see where the beat boundary is—in this case, partway through the D-sharp.
In 4/4 time, in general, each of the four beats of the measure should be marked by a new note head, unless a note longer than a quarter note has already started.
(If by any chance you’re familiar with the concept of data alignment from computer science, this is very much analogous.)
Edit: By popular demand, here is a version with the dotted eighths written as dotted eighths:
I’m not used to seeing this rhythm—a dotted eighth followed by a sixteenth is so much more familiar than the reverse—but after seeing it written this way I agree it’s a lot more clear what’s going on.