What is bad in this code (except concatenation thing)?