1

在 Java 中，检查对象是否具有值或正在返回的最佳方法是null什么？我发现的大多数例子都不是很好。基本上我有这个代码：

mDBApi.getSession().setAccessTokenPair(reAuthTokens);
System.out.println(reAuthTokens);
if(reAuthTokens.equals(null)) {
    mDBApi.getSession().startAuthentication(Main.this);
    Log.e(TAG, "Keys not set -- I'm starting authentication");
}

我正在尝试reAuthTokens检查价值，如果没有价值，请继续并进行身份验证。但是，我只是NullPointerException在 if 语句行上得到一个。有什么我可以做得更好的吗？

__ _ __ _ __ _ __ _ OnCreate for rcook _ __ _ __ _ __ _ __ _ __ _ __ _ __ _ __

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    getWindow().requestFeature(Window.FEATURE_NO_TITLE);
    setContentView(R.layout.main);


    //AccessTokenPair tokens=null;      
    //AccessTokenPair tokens = getStoredKeys();   
    //System.out.println(access + "here I am"); 
    //clearKeys();
    //Log.e(TAG, "keys cleared");


    AppKeyPair appKeys = new AppKeyPair(APP_KEY, APP_SECRET);
    AndroidAuthSession session = new AndroidAuthSession(appKeys, ACCESS_TYPE);
    mDBApi = new DropboxAPI<AndroidAuthSession>(session);
    AccessTokenPair reAuthTokens = new AccessTokenPair(APP_KEY, APP_SECRET);
    mDBApi.getSession().setAccessTokenPair(reAuthTokens);

        System.out.println(reAuthTokens);

    if(reAuthTokens == null) {       
    mDBApi.getSession().startAuthentication(Main.this);
    Log.e(TAG, "Keys not set -- I'm starting authentication");



    }


//



    /*read settings*/
    mSettings = getSharedPreferences(PREFS_NAME, 0);
    boolean hide = mSettings.getBoolean(PREFS_HIDDEN, false);
    boolean thumb = mSettings.getBoolean(PREFS_THUMBNAIL, true);
    int space = mSettings.getInt(PREFS_STORAGE, View.VISIBLE);
    int color = mSettings.getInt(PREFS_COLOR, -1);
    int sort = mSettings.getInt(PREFS_SORT, 3);



    mFileMag = new FileManager();
    mFileMag.setShowHiddenFiles(true);
    mFileMag.setSortType(sort);

    if (savedInstanceState != null)
        mHandler = new EventHandler(Main.this, mFileMag, savedInstanceState.getString("location"));
    else
        mHandler = new EventHandler(Main.this, mFileMag);

    mHandler.setTextColor(color);
    mHandler.setShowThumbnails(thumb);
    mTable = mHandler.new TableRow();

    /*sets the ListAdapter for our ListActivity and
     *gives our EventHandler class the same adapter
     */
    mHandler.setListAdapter(mTable);
    setListAdapter(mTable);

    /* register context menu for our list view */
    registerForContextMenu(getListView());

    mStorageLabel = (TextView)findViewById(R.id.storage_label);
    mDetailLabel = (TextView)findViewById(R.id.detail_label);
    mPathLabel = (TextView)findViewById(R.id.path_label);
    mPathLabel.setText("path: /sdcard");

    updateStorageLabel();
    mStorageLabel.setVisibility(space);

    mHandler.setUpdateLabels(mPathLabel, mDetailLabel);

    /* setup buttons */
    int[] img_button_id = {R.id.help_button, R.id.home_button, 
                           R.id.back_button, R.id.info_button, 
                           R.id.manage_button, R.id.multiselect_button,
                           R.id.dropbox_button
                           };

    int[] button_id = {R.id.hidden_copy, R.id.hidden_attach,
                       R.id.hidden_delete, R.id.hidden_move};

    ImageButton[] bimg = new ImageButton[img_button_id.length];

    Button[] bt = new Button[button_id.length];

    for(int i = 0; i < img_button_id.length; i++) {
        bimg[i] = (ImageButton)findViewById(img_button_id[i]);
        bimg[i].setOnClickListener(mHandler);

        if(i < 4) {
            bt[i] = (Button)findViewById(button_id[i]);
            bt[i].setOnClickListener(mHandler);
        }
    }

    Intent intent = getIntent();

    if(intent.getAction().equals(Intent.ACTION_GET_CONTENT)) {
        bimg[5].setVisibility(View.GONE);
        mReturnIntent = true;

    } else if (intent.getAction().equals(ACTION_WIDGET)) {
        Log.e("MAIN", "Widget action, string = " + intent.getExtras().getString("folder"));
        mHandler.updateDirectory(mFileMag.getNextDir(intent.getExtras().getString("folder"), true));

    }
}
4

1 回答 1

5

改为使用if (reAuthTokens == null))。您不是在尝试比较对象的内容；您正在尝试比较参考。“isreAuthTokens指向与 相同的地址null？”

编辑从 OP 更新后：reAuthTokens是类型AccessTokenPair（我敢打赌，许多读者最初认为这是一个List......我知道我做到了）。它在这一行中实例化：

AccessTokenPair reAuthTokens = new AccessTokenPair(APP_KEY, APP_SECRET);

这就是为什么以下条件总是错误的：reAuthTokens == null。这就是为什么在编码时if (reAuthTokens == null)，您会收到“死代码”警告：编译器知道这种情况永远不会为真，因为您reAuthTokens在上面实例化了几行代码。

因此，您所追求的比较类型不是关于参考，而是关于内容。您要检查是否reAuthTokens为“空”。但这从您引用的代码中没有意义。你怎么实例化对象，然后想检查它是否“空”？

我觉得你的逻辑不对。您应该首先从您期望的位置（会话？）获取访问令牌对，并将结果与null​​ . 像这样的东西：

AccessTokenPair reAuthTokens = mDBApi.getSession().getAccessTokenPair();

if (reAuthTokens == null) {
    reAuthTokens = new AccessTokenPair(...);
    mDBApi.getSession().setAccessTokenPair(reAuthTokens);
}
于 2013-02-21T02:01:00.940 回答