PaulS:
Why do you suggest to stop using the string class completely?
A little research, on your part, would answer that. The destructor of the String class calls free(), but free() has some fundamental problems that cause the Arduino to crash. Nothing that the String class does can not be done using NULL terminated char arrays.
PaulS,
Thanks for the Info. I'm a newbie so I'm not too familiar with the best method but did look up char arrays. So the suggestion is to take out the ,10%, 20%, 30%....(from the code) and put them into an array table, then call from it?
Paul