It is not a good idea to use the String (capital S) class on an Arduino as it can cause memory corruption in the small memory on an Arduino. Just use cstrings - char arrays terminated with 0.
Have a look at the examples in Serial Input Basics - simple reliable ways to receive data.
...R