java中直接用substring截取字符串和将substring封装在方法里的区别？

Question

strName = json.optString("name").substring(0, json.optString("name").indexOf(" "));
和
strName = subName(json.optString("name"));
private String subName( String gameName )
{
if ( ! gameName.isEmpty() )
{
String str[] = gameName.split(" ");
String subNewName = str[0];
return subNewName;
}
else
{
return null;
}
}
在代码执行上有什么区别吗？为什么在代码中用第一种方法，app运行有时候会出错，有时候又不会？

Answer 1

strName = json.optString("name").substring(0, json.optString("name").indexOf(" "));这样写是不安全的。原因是json.optString("name")有可能因为种种原因返回null，或者其中不含空格导致json.optString("name").indexOf(" ")返回-1进而导致substring方法出错。总之，把这句话展开成一个函数，多加一些检查，才能保持函数的可靠。