А зачем ты делаешь конструкцию новой строки просто ради конструкции? Почему сразу не подашь stringOne?
Потому что на результат это не повлияет, но с точки зрения читаемости и дебага , рефакторинга гораздо лучше
вот только суммировать строки (String и) надо
s = s1
s += s2
будет меньше "memory relocation"
String(volt); - не совсем понятно а тип voit какой? если float - то конструктор другой - и произойдет неявное приведение типов
и String отличная штука но *toa - очень простые функции - их собственно он и использует - можно этого посредника тут убрать без усложнения кода
sprintf - тоже очень подойдет для подобных задач - читаться будет даже проще