OnActivityResult - おまえら！もうonActivityResultでswitchとif書く時代は終わりだぞ！

OnActivityResult
kyobashi.dex #4
おまえら！もうonActivityResultでswitchとif書く時代は終わりだぞ！

About Me
Shinobu Okano
@operandoOS
Mercari, Inc.
Souzoh, Inc.

Do you know onActivityResult?
https://developer.android.com/
training/basics/intents/result.html
Getting a Result from an Activity

Getting a Result from an Activity
// Start the Activity
static final int PICK_CONTACT_REQUEST = 1; // The request code
private void pickContact() {
Intent pickContactIntent = new Intent(Intent.ACTION_PICK, Uri.parse("content://contacts"));
pickContactIntent.setType(Phone.CONTENT_TYPE); // Show user only contacts w/ phone numbers
startActivityForResult(pickContactIntent, PICK_CONTACT_REQUEST);
}
// Receive the Result
@Override
protected void onActivityResult(int requestCode, int resultCode, Intent data) {
if (requestCode == PICK_CONTACT_REQUEST) {
if (resultCode == RESULT_OK) {
// ...
}
}
}

某フリマアプリから拝借した 
onActivityResult
※定数名とかそこら辺は変えてます

某フリマアプリ onActivityResult
@Override
super.onActivityResult(requestCode, resultCode, data);
switch(requestCode) {
case LOGIN:
if(resultCode == RESULT_OK) {
success();
} else {
failed();
}
break;
}
}

@Override
if(resultCode != RESULT_OK) return;
switch (requestCode) {
case REQUEST_MUGAMUGA:
successItemPost();
break;
}
}

@Override
public void onActivityResult(int requestCode, int resultCode, Intent data) {
if (resultCode != RESULT_OK || data == null) {
mugemuge();
return;
}
if (requestCode == Request.UMUUMU) {
umuumu();
}
}

@Override
if (resultCode == Activity.RESULT_CANCELED) {
cancel();
} else if (requestCode == REQUEST_CODE_SELECT_IMAGE) {
// ....
} else if (requestCode == REQUEST_CODE_TRIMMING) {
// ....
} else {
}
}

@Override
if (requestCode == REQUEST_MUGAMUGA && resultCode == Activity.RESULT_OK) {
if (data != null) {
mugamuga();
}
}
}

@Override
if (resultCode != Activity.RESULT_OK) {
return;
}
switch (requestCode) {
case REQUEST_CODE_CATEGORY:
// ...
break;
case REQUEST_CODE_LOCATION:
// ...
break;
case REQUEST_CODE_CONDITION:
// ...
break;
default:
break;
}
}

onActivityResultの悪さ
• switchとifが多い
• requestCodeとresultCodeをチェックしないといけない
• switchとifの書き方がまちまち
• switch使わない勢 + if else勢😇
• requestCodeとresultCodeどっちに先に判定するか問題
• 人間が書くにはつらい...

OnActivityResult
OnActivityResult annotation compiler
for Android
https://github.com/vanniktech/OnActivityResult

アッテの 
onActivityResultの実装は 
すべてOnActivityResultです！

onActivityResultの実装がこれだけ！！
@Override
ActivityResult.onResult(requestCode, resultCode, data).into(this);
}

受け取ったあとの実装は？？
• それぞのたパターンにあったメソッドを定義して
@OnActivityResultをつける
• @OnActivityResultのrequestCodeとresultCodesで 
どういう結果の場合に呼び出してほしいかを定義する

@OnActivityResult(requestCode = REQUEST_TAKE_PHOTO, resultCodes = RESULT_OK)
void onTakePhoto(@ExtraString(name = CameraActivity.RESULT_TAKE_PHOTO)
final String takePhoto) {
Intent resultIntent = new Intent();
resultIntent.putExtra(RESULT_TAKE_PHOTO, takePhoto);
setResult(RESULT_CODE_TAKE_PHOTO, resultIntent);
finish();
}

// result codeに何が返ってきても処理するメソッド
@OnActivityResult(requestCode = REQUEST_SELECT_IMAGE)
void onHumuHumu() {
Toast.makeText(this, "onHumuHumu", Toast.LENGTH_SHORT).show();
}
@OnActivityResult(requestCode = REQUEST_SELECT_IMAGE, resultCodes = RESULT_CANCELED)
void onCancel() {
Toast.makeText(this, "cancel select image.”, Toast.LENGTH_SHORT).show();
}

onActivityResultの受け取り先の
オブジェクトを指定する
• intoに受け取り先オブジェクトを指定する
• ActivityResult.onResult().into(obj);
• onActivityResultで返ってくるIntentのデータを
ViewModelとかPresenterで受け取る
• 設計によっては嬉しい

onActivityResultの結果を 
ViewModelで受け取る
private MySampleMainViewModel viewModel = new MySampleMainViewModel();
@Override
ActivityResult.onResult(requestCode, resultCode, data).into(viewModel);
}
public class MySampleMainViewModel {
public ObservableField<String> location = new ObservableField<>("0,0");
@OnActivityResult(requestCode = MySampleMainActivity.REQUEST_SELECT_LOCATION,
resultCodes = Activity.RESULT_OK)
void onSelectedLocation(@Extra(name = LocationSelectActivity.RESULT_SELECT_LOCATION) Location location) {
this.location.set(location.toString());
}
}

OnActivityResult sample Code
https://github.com/operando/OnActivityResult/tree/
my_sample_code/onactivityresult-sample
https://github.com/operando/OnActivityResult/tree/my_sample_code/onactivityresult-sample/
build/generated/source/apt/debug/com/vanniktech/onactivityresult/sample
布教活動のために適当にサンプルコード書いてます
annotation processorで作成されるコードも 
入れてあります

Do you know Activity.RESULT_FIRST_USER?
• Start of user-deﬁned activity results.
• Constant Value: 1 (0x00000001)
• https://developer.android.com/reference/android/
app/Activity.html#RESULT_FIRST_USER

どういう場合に使うか
• 画面Aから画面BへRequestをする時 
画面Bからの戻りが2パターン以上ある
場合
• ???

アッテの例) 
募集投稿画面から画像選択をする
• 画像選択画面ではギャラリーの画像を選択するか、カメラで撮影するかの 
2パターンある
• カメラ撮影はまた別画面が起動して、その戻りを画像選択画面が受け取る
• 募集投稿画面への戻りが、ギャラリーの画像を選択、カメラで撮影の2パターンある
• = 募集投稿画面から画像選択画面に対しては1Requestだけど 
2パターンの戻り(Resultパターン)がある
• = 独自のResult Codeを定義したい = Activity.RESULT_FIRST_USERを使って定義

OnActivityResultで実装すると…
// ImageSelectActivity
static final int RESULT_CODE_SELECT_IMAGE = Activity.RESULT_FIRST_USER;
static final int RESULT_CODE_TAKE_PHOTO = Activity.RESULT_FIRST_USER + 1;
@OnActivityResult(requestCode = REQUEST_SELECT_IMAGE,
resultCodes = ImageSelectActivity.RESULT_CODE_SELECT_IMAGE)
void onSelectImage(Intent intent) {
ArrayList<String> selectImages = intent.getStringArrayListExtra(RESULT_SELECT_IMAGES);
Toast.makeText(this, "onSelectImage selectImages : " + selectImages, Toast.LENGTH_SHORT).show();
}
@OnActivityResult(requestCode = REQUEST_SELECT_IMAGE,
resultCodes = ImageSelectActivity.RESULT_CODE_TAKE_PHOTO)
void onTakePhoto(@ExtraString(name = ImageSelectActivity.RESULT_TAKE_PHOTO) final String takePhoto) {
Toast.makeText(this, "onTakePhoto takePhoto : " + takePhoto, Toast.LENGTH_SHORT).show();
}

Support user deﬁned result code
https://github.com/vanniktech/
OnActivityResult/pull/86
俺が！
OnActivityResultでActivity.RESULT_FIRST_USER
使えるようにしてやったぞ！

Contributions
イヤッッホォォォ
オオォオウ！

OnActivityResultの気に入ってるところ
• onActivityResultメソッドの実装はシンプルになる！
• startActivityForResultの呼び出しは既存のままでいい！
• 受け取り後の処理ごとにメソッド定義するので見通しがいい
• 結果受け取り先のオブジェクトを指定できる → into()
• ifとswitchからの開放…

おまえら！
もうonActivityResultで 
switchとif書く時代は 
終わりだぞ！

OnActivityResult - おまえら！もうonActivityResultでswitchとif書く時代は終わりだぞ！

Empfohlen

Empfohlen

Weitere ähnliche Inhalte

Ähnlich wie OnActivityResult - おまえら！もうonActivityResultでswitchとif書く時代は終わりだぞ！

Ähnlich wie OnActivityResult - おまえら！もうonActivityResultでswitchとif書く時代は終わりだぞ！ (20)

Mehr von Shinobu Okano

Mehr von Shinobu Okano (20)

Kürzlich hochgeladen

Kürzlich hochgeladen (12)

OnActivityResult - おまえら！もうonActivityResultでswitchとif書く時代は終わりだぞ！