Should use a StringBuilder