about the lenght, I assure you I'm not, more like 5'11
This looks not dissimilar from some of my early fits here so let me give you some thoughts.
The cardi is simultaneously too small and too big for you. It's too small in the shoulders/chest, and is not hugging the collar of your shirt which looks bad as there's a huge gap there. But then it's too big around the waist and looks slouchy, which can be a cool look but it would need to be slouchy all over and this isn't.
A tie with jeans is almost never going to look good. It looks like a knit tie so I'm sure you're going to say it's a casual tie but that's just what GQ wants you to believe and it's not really true. Women have a rule about taking one thing off before you walk out the door and if I were apply this to your fit, the tie would definitely be the thing to lose. The shirt is actually ok and the cardi might be ok if unbuttoned (along with unbuttoning the shirt at the collar).
Those jeans aren't the best but not the worst I've ever seen. They are verging on dad-jean status but not quite there. Typically people here like to see jeans either cuffed or stacked and it looks like you might have just enough length for a single cuff which is what I would recommend here.
Shoes are also ok but definitely not good in context with the tie (but again, lose the tie first). I honestly don't really know what to recommend on the shoes so maybe someone else can chime in here, but they are definitely more casual so either the rest of the outfit needs to be, or you need to change the shoes to something else.