The skirt doesn't look very flattering on you, like at all. Neither does the t-shirt. It just looks like a sloppy outfit you'd throw on when you have to run to the store because you're out of milk. Not something you'd wear seriously to school or when you're out with your friends. The t-shirt is cute to wear to bed. Nothing more. Ditch the denim skirt and find something a bit more flowy and cute, and a bit less 2003.
Change the shirt to a more fitted top and it's a cute outfit! It just looks like you're wearing a casual t-shirt and then a fancy skirt and tights. It doesn't really go well together of compliment your figure.