colors are bad, boots clash with pants (and are ugly), cardigan is too loose at the waist and too long, and you shouldn't be buttoning the bottom botton, and with a cardigan like that you need a buttondown shirt. peacoat is best piece.
the shirt is too long because it's a dress shirt, and too tight in the waist (because again.. dress shirt). it's not.. bad, but if it was shorter and looser it'd look a lot better. for untucked casual shirts you want them to just skim the top of the pockets, and never cover your entire crotch. i also think they should be loose enough that the placket should meet the entire way down -- a good dress shirt will pull in that situation if its untucked.