Twitter4jを用いてアップロードした画像のmediaIDを取得するmethodを作りたい

###前提・実現したいこと
Twitter4jを用いてアップロードした画像のmediaIDを取得するmethodを作りたい

###発生している問題・エラーメッセージ
どのタイミングでmediaIDをreturnすればいいかわからない

###該当のソースコード

java
1public long twitterUploadImage(final Twitter mTwitter, final String filepath, final Context context){
2        AsyncTask<String, Void, Boolean> task = new AsyncTask<String, Void, Boolean>(){
3
4            @Override
5            protected Boolean doInBackground(String... strings) {
6                try {
7                    //ここで画像をアップロード
8                    media = mTwitter.uploadMedia(new File(filepath));
9                    mediaID = media.getMediaId();
10                    return true;
11                } catch (TwitterException e) {
12                    e.printStackTrace();
13                    return false;
14                }
15            }
16
17            //アップロードの可否をToast
18            @Override
19            protected void onPostExecute(Boolean tweetResult){
20                if(tweetResult){
21                    showToast(context,"画像をアップロードしました");
22                }else{
23                    showToast(context,"画像をアップロードできませんでした");
24                }
25            }
26        };
27        task.execute(filepath);
28        return mediaID;
29    }

と書いたのですが、これだと画像がアップロードされる前にmediaIDがreturnされてしまい、デフォルト値の0しか返ってきません。

###補足情報(言語/FW/ツール等のバージョンなど)
Android Studio3.0、Twitter4j-4.0.6を使用しています

行動規範の内容に同意します

回答1件

ベストアンサー

タイミングは

mediaID = media.getMediaId();

以降ですが、おそらく値の返し方で悩まれているのかと思います。

リスナーなどについて調べる事をお勧めします。

interfaceを使った独自リスナーの作り方
http://android-note.open-memo.net/sub/event__my_listener.html

コード例：書き方は色々あると思います。ビルドを通してないので、typo等あるかもしれません。

[呼び出し側]

final MyListener listener = new MyListener() {
    @Override
    public void onMediaIdChanged(final long mediaID) {
        // mediaIDを使う処理
    }
}

twitterUploadImage(mTwitter, filepath, context, listener);

// まとめて　twitterUploadImage(mTwitter, filepath, context, new MyListener() { ... }); と書くのも可

[リスナ(名前は適当)]

interface MyListener {
    void onMediaIdChanged(final long mediaID)
}

[呼ばれる側(改造版、抜粋)]

// 戻り値の型 long を void に変更、引数 listener を追加
public void twitterUploadImage(final Twitter mTwitter, final String filepath, final Context context, final MyListener listener) {
    AsyncTask<String, Void, Boolean> task = new AsyncTask<String, Void, Boolean>(){

    @Override
    protected Boolean doInBackground(String... strings) {
        try {
            //ここで画像をアップロード
            media = mTwitter.uploadMedia(new File(filepath));
            mediaID = media.getMediaId();
            if (listener != null) {
                listener.onMediaIdChanged(mediaID); // ここで返す
            }
            return true;
        } catch (TwitterException e) {
            e.printStackTrace();
            return false;
        }
    }

    // (省略)

    // return mediaID; // ここではmediaIDを返さない
}

投稿2017/11/12 03:55

編集2017/11/12 11:28