if your getting mad at that don't look at mine

Becouse every time when there is string concatenation a new variable is created in separate pice of mamory. Tis how python work, this solution is simple to implement but it use a lot more memory then solutions without string concatenation.

I dont' know where you get that from. Here enumerate doesn't make sense since we only need the indexes, so it would generate an useless additional variable on each iteration. The purpose of enumerate is to provide at the same time an index and the variable at that index. If you don't need both, there is nothing pythonic in using it.

It gets the job done, but using enumarate() is more pythonic.

Why?

Good practice is to add spaces between concatenation.

Dont thik its look like "Best practice", first becouse of string concatenation, second is redundant for loop.