来自 EditText 的 getText()：是否有可能获得“null”？

Question

我正在编写一个我正在编写的 Android 应用程序，并试图使我的代码完全符合 Android Studio 的 lint 建议。

我有以下代码发出警告（省略了一些代码）：

final EditText input = (EditText)view.findViewById(R.id.edit_text);
Button button        = (Button)view.findViewById(R.id.button);
button.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        String value = input.getText().toString();
        if (value == null || value.length() == 0) {

Android Studio 给我一个警告：

条件 'value == null' 始终为 false。

当我允许 Android Studio 为我“修复”问题时，它建议：

将 'value == null' 简化为 false

然后代码变成：

if (value.length() == 0) {

我查看了 Android 源代码 (http://www.grepcode.com)，我很困惑。 EditText 的文档说“EditText 是对 TextView 的薄薄的一层，它将自身配置为可编辑。”那么getText()方法定义如下：

@Override
public Editable getText() {
    return (Editable) super.getText();
}

当我去getText() 寻找TextView（“超级”）时，我看到了这个：

public CharSequence getText() {
    return mText;
}

TextView 的 setText() 方法似乎不允许 null 值，因为这是该方法的开始：

private void setText(CharSequence text, BufferType type, boolean notifyBefore, int oldlen) {
    if (text == null) {
        text = "";
    }

默认构造函数也是这样开始的：

public TextView(Context context, AttributeSet attrs, int defStyle) {
    super(context, attrs, defStyle);
    mText = "";

因此，getText() 似乎无法返回 null 值，但 this answer 上的 cmets 表明它是。 this question 的答案似乎也表明这是可能的。

我想练习防御性编码，这就是我从一开始就按照我的方式构建代码的原因，但我不想做null 检查不可能是null 的东西.那么，在这种情况下，最佳做法是什么？

Answer 1

我认为 lint 检查告诉您的情况略有不同：

String value = input.getText().toString();
if (value == null || value.length() == 0) {

条件 'value == null' 始终为 false。

在这个 if 语句中，EditText.getText() 是否可以返回 null 实际上并不重要。因为如果它返回 null，那么在到达 if 语句之前，.toString() 将导致 NullPointerException。因此，如果 if 语句完全可达，则 value 不能为 null。

你可以翻遍Android源码，也许你会发现EditText不可能返回null。如果是这种情况，那么如果您知道标准的 EditText 在您的应用程序中，则可以忽略 lint 警告。

但是，在更一般的情况下，没有什么能阻止我将其粘贴在视图层次结构中：

public class EvilEditText extends EditText {
    // Constructors

    @Override
    public Editable getText() {
        return null;
    }
}

所以最安全的选择是将代码重写为完全为空安全：

Editable e = input.getText();
String value = (e == null ? null : e.toString());
if (TextUtils.isEmpty(value)) {  //hat tip to Deepak Goyal's answer. Could also use isEmpty(e)

Answer 2

你可以使用 TextUtils.isEmpty(value);

final EditText input = (EditText)view.findViewById(R.id.edit_text);
Button button        = (Button)view.findViewById(R.id.button);
button.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View v) {
        String value = input.getText().toString();
        if (TextUtils.isEmpty(value)) {
          // value is empty
        }else{
          // have value
        }

Answer 3

您可能会收到此警告，因为您的 value 不包含从 EditText#getText()（返回 CharSequence）返回的内容，而是从 CharSequence#toString()（返回 String）返回的内容.

toString() 对非空 CharSequence 的调用应该不可能返回空值，这就是为什么 lint 警告您空值检查是不必要的。你应该对EditText#getText() 的返回做空检查。

换句话说，如果您的input.getText() 为空，那么它已经失败而无需调用toString()，因此您的空检查将永远无法到达。

Answer 4

表达式中的值

String value = input.getText().toString();

不会是null。 因为如果输入是null，它将为方法调用input.getText() 生成一个NPE。因此下一条语句将不会执行。如果你处理这个NPE 类似

            String value = null;
            if (input!=null) {
                value = input.getText().toString();
            }
            if (value == null || value.length() == 0){

            }

那么你就不会得到那个指示了。