Question

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

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方法出错。
总之，把这句话展开成一个函数，多加一些检查，才能保持函数的可靠。

Answer 2

substring是分割字符串用的java基本类库方法。看你的用例，只是简单的封装了下对于字符串的操作方法，本身执行没有什么区别。但是你在封装的方法中加了isEmpty()验证，因此不会在调用substring时抛出空指针异常。因为对象json.optString("name")如果是null，调用substring方法会报错。而你封装后的方法，传递的参数做了处理因此不会抛出错误。