Well, for one thing, your chord progression isn't diationic. This means it does not match the notes of one particular key. This isn't necessarily bad, but for beginners in composition (myself included) I think its easier to stay diatonic. Here is something that helps me a lot (Im assuming you can read roman numerals, if not, google it):
I ii iii IV V vi (vii) I
okay so what does this mean? the uppercase roman numerals represent major chords. The lowercase numerals go with minor chords. I will come back to the one in parentheses later, its a different type of chord.
okay, so say you are in the key of C major, for simplicity's sake. This means only the white keys on the keyboard. The I chord would be a C major chord (can be called the major 1st). The ii is for the minor second, D minor. iii is for E minor, IV is F major (major 4th) , V is G major, and vi is A minor. Make sense so far? Okay, so most of the time, songs (assuming they are in C major) will use these chords, and things work out pretty well.
I hope this is making sense so far. If not, think about it this way, my guitar teacher used this analogy with me, and it helped a ton. Think of a key as a bucket. It has certain notes in it. Most of the time, songs will pick notes only from one bucket at a time. Of course, people change keys sometimes, so they will occasionally switch to another bucket (but dont worry about that too much until you are comfortable working in just one). Most composers can come up with all kinds of wonderful things just working from the one bucket. It has structure in that it has rules. Certain notes arent included in the bucket, so listeners can know what to expect, but there are enough notes that composers can create a ton of variation.
So, what do you do here? If it was me, I would pick a key that has some of the chords that you have there, and adjust the ones that dont fit. If we wanted to keep things simple, with no sharps or flats (white keys only, hint hint) we could easily do it in C major. In C major, the major chords are C, F, and G (I, IV, V). This means you could leave the F and G major chords in your progression alone, they fit this key. The remaining chords that you used were A and E. In C major, the minor chords are D, E, and A (ii, iii, vi). What I would do here then, is change your A major and E major chords into A minor and E minor. This would require a single note change in each chord(In the A major, change the C# to a C, and in the E major, change the G# to a G). You will also have to change the melody to match these chords. Of course, changing these major chords to minor chords will change the "feel" of the groove, so its completely up to you if you like it or not. But its what I would do to start fixing it up pretty. (A major is another key you could fairly easily fit this into, you have the I and V of a major with your A and E chords, but I would advise sticking to C, its just much simpler for starters)
I cant tell for sure, but it seems that you might not have all the info about music theory that you need to get this sounding awesome. If thats not the case, sorry for wall of text and possibly incomprehensible explanations, its late over here. Or rather early xD Otherwise, I hope this is helpful
