Firebase API initialization failure.

compileSdkVersion 24, targetSdkVersion 24, サポートライブラリ 24.0.0 で開発してたらFirebaseがErrorを吐いてた。

サポートライブラリを24.0.0にしてるのが原因のようなので、24の機能に依存しているわけではないので、全部23に戻した。

android {
    compileSdkVersion 23
    buildToolsVersion "23.0.3"

    defaultConfig {
        applicationId "io.github.hisaichi5518.akashic"
        minSdkVersion 21
        targetSdkVersion 23
        versionCode 1
        versionName "1.0"
    }
}
    
dependencies {
    compile 'com.android.support:appcompat-v7:23.4.0'
    compile 'com.android.support:recyclerview-v7:23.4.0'
    compile 'com.android.support:design:23.4.0'
}

エラーが出てるけど、通知は受け取れたので、24のままでも良かったかもしれない（気持ち悪いけど）

07-17 19:06:28.532 7303-7303/io.github.hisaichi5518.akashic E/FirebaseApp: Firebase API initialization failure.
                                                                           java.lang.reflect.InvocationTargetException
                                                                               at java.lang.reflect.Method.invoke(Native Method)
                                                                               at com.google.firebase.FirebaseApp.zza(Unknown Source)
                                                                               at com.google.firebase.FirebaseApp.initializeApp(Unknown Source)
                                                                               at com.google.firebase.FirebaseApp.initializeApp(Unknown Source)
                                                                               at com.google.firebase.FirebaseApp.zzbu(Unknown Source)
                                                                               at com.google.firebase.provider.FirebaseInitProvider.onCreate(Unknown Source)
                                                                               at android.content.ContentProvider.attachInfo(ContentProvider.java:1748)
                                                                               at android.content.ContentProvider.attachInfo(ContentProvider.java:1723)
                                                                               at com.google.firebase.provider.FirebaseInitProvider.attachInfo(Unknown Source)
                                                                               at android.app.ActivityThread.installProvider(ActivityThread.java:5158)
                                                                               at android.app.ActivityThread.installContentProviders(ActivityThread.java:4753)
                                                                               at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4693)
                                                                               at android.app.ActivityThread.-wrap1(ActivityThread.java)
                                                                               at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1405)
                                                                               at android.os.Handler.dispatchMessage(Handler.java:102)
                                                                               at android.os.Looper.loop(Looper.java:148)
                                                                               at android.app.ActivityThread.main(ActivityThread.java:5422)
                                                                               at java.lang.reflect.Method.invoke(Native Method)
                                                                               at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
                                                                               at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
                                                                            Caused by: java.lang.IncompatibleClassChangeError: The method 'java.io.File android.support.v4.content.ContextCompat.getNoBackupFilesDir(android.content.Context)' was expected to be of type virtual but instead was found to be of type direct (declaration of 'com.google.firebase.iid.zzg' appears in /data/data/io.github.hisaichi5518.akashic/files/instant-run/dex/slice-com.google.firebase-firebase-iid-9.0.0_71b4f57689b32f0b280a7a5d016ccf0a2694cb94-classes.dex)
                                                                               at com.google.firebase.iid.zzg.zzeC(Unknown Source)
                                                                               at com.google.firebase.iid.zzg.<init>(Unknown Source)
                                                                               at com.google.firebase.iid.zzg.<init>(Unknown Source)
                                                                               at com.google.firebase.iid.zzd.zzb(Unknown Source)
                                                                               at com.google.firebase.iid.FirebaseInstanceId.getInstance(Unknown Source)
                                                                               at java.lang.reflect.Method.invoke(Native Method) 
                                                                               at com.google.firebase.FirebaseApp.zza(Unknown Source) 
                                                                               at com.google.firebase.FirebaseApp.initializeApp(Unknown Source) 
                                                                               at com.google.firebase.FirebaseApp.initializeApp(Unknown Source) 
                                                                               at com.google.firebase.FirebaseApp.zzbu(Unknown Source) 
                                                                               at com.google.firebase.provider.FirebaseInitProvider.onCreate(Unknown Source) 
                                                                               at android.content.ContentProvider.attachInfo(ContentProvider.java:1748) 
                                                                               at android.content.ContentProvider.attachInfo(ContentProvider.java:1723) 
                                                                               at com.google.firebase.provider.FirebaseInitProvider.attachInfo(Unknown Source) 
                                                                               at android.app.ActivityThread.installProvider(ActivityThread.java:5158) 
                                                                               at android.app.ActivityThread.installContentProviders(ActivityThread.java:4753) 
                                                                               at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4693) 
                                                                               at android.app.ActivityThread.-wrap1(ActivityThread.java) 
                                                                               at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1405) 
                                                                               at android.os.Handler.dispatchMessage(Handler.java:102) 
                                                                               at android.os.Looper.loop(Looper.java:148) 
                                                                               at android.app.ActivityThread.main(ActivityThread.java:5422) 
                                                                               at java.lang.reflect.Method.invoke(Native Method) 
                                                                               at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726) 
                                                                               at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616) 

参考

google cloud messaging - IncompatibleClassChangeError after updating to Android Build Tools 25.1.6 GCM / FCM - Stack Overflow

RecyclerView.ItemDecorationの中では分岐しないようにした

RecyclerViewでリストを作る時、RecyclerView.ItemDecorationを使って各Viewの間隔を広げていた。左と右でサイズが違う事と1個だけアイテムを消すとズレる事に気づいた。

最初の表示	消したあと

getSpanIndexが削除前の値を返しているというのもあるけど、outRect.right = 0してるのに効かない時があるようだった。outRect.rightが設定された状態から0にしようとするとダメっぽいので、そもそも分岐するのをやめた。

最初の表示	消したあと

    private class Decoration extends RecyclerView.ItemDecoration {
        private int mLeft, mTop, mRight, mBottom;

        Decoration(int left, int top, int right, int bottom) {
            mLeft = left;
            mTop = top;
            mRight = right;
            mBottom = bottom;
        }

        @Override
        public void getItemOffsets(Rect outRect, View view, RecyclerView parent, RecyclerView.State state) {
            GridLayoutManager.LayoutParams layoutParams = (GridLayoutManager.LayoutParams) view.getLayoutParams();

            int position = layoutParams.getViewAdapterPosition();
            if (position == RecyclerView.NO_POSITION) {
                outRect.set(0, 0, 0, 0);
                return;
            }

            outRect.left = mLeft;
            outRect.top = mTop;
            outRect.right = mRight;
            outRect.bottom = mBottom;
        }
    }

全部のコードはここにおいた。
GitHub - hisaichi5518/RecyclerViewItemDecorationTest

人によって“当たり前”は違う

自分にとって当たり前と思ってることはたくさんある。僕の場合、小学生まで家で食べる天ぷらには醤油とマヨネーズを付けて食べるのが当たり前だった。他の家は天つゆで食べると聞いてマジで驚いた。

コードにおいても、自分の中に当たり前がある。僕にとってはインデントが揃ってるのは当たり前だし、コードに対してテストがあるのも当たり前のことだと思ってる。

コードにおいての当たり前も天ぷらの話と同じように人によって違う。ある人はテストがないのが当たり前なのかもしれないし、インデントバラバラが当たり前なのかもしれない。

自分の当たり前は正しいのか、相手の当たり前はなんなのか、そしてそれは正しいのかを考える。そうやっていくことで、自分の当たり前をより良くし、お金を大量に得たい。

YAP(achimon)C::Asia Hachioji 2016 mid in Shinagawa に出た。

話す予定です pic.twitter.com/qAgGo4JlbN

— ひさいち (@hisaichi5518) 2016年7月2日

既存のAndroidアプリをリファクタリングをしていく話をしました。

目次

• なぜMVPアーキテクチャなのか
• MVPアーキテクチャの課題
• その他課題
• ヤパチー感想

なぜMVPアーキテクチャなのか

他のアーキテクチャも考えたんですが、自分が一番理解しやすくて、他の人を巻き込めるなと思ったのがMVPアーキテクチャだったからです。
ちなみに、アーキテクチャを決める方針として以下を最初に決めていました。

MVPアーキテクチャの課題

今のところヤベ〜〜みたいなのになってない!!!!!!!!!!!!!!!!!
今後出てくる可能性はあるので、その時は話すかもしれません。

その他課題

• テストカバレッジなどの計測まわりがちゃんと出来ていないのでやりたい
• 静的に解析してアレコレみたいなの、小さく始めたい

ヤパチー感想

1日目しか参加してないんですが楽しかった。
Android界隈の知り合いは少ないんですが、ちょっと増えたのも嬉しかった。
またあるなら行きたい。

自分のトークに関しては、もっと元気に自由に話せばよかったという反省をしています。

「Androidチームのこれまでとこれから」という話をした。

Androidチームに新しいメンバーが増えた事とテクニカルリードという役割を僕が担う事になったので、Androidチームのメンバーに僕が現状考えている「テクニカルリードの役割と責任」「Androidチームの方針」「Androidチームの文化」について簡単にまとめました。

方針に関しては、今年のはじめに決めて結構良い感じだったので、そのまま進めるという感じです。

追記: 来てくれ！！！！！１
minne Androidアプリエンジニア（正社員） | エンジニア | 職種詳細 | GMOペパボ株式会社

参考

• 「いるだけで成長できる環境」へ - ペパボテックブログ
• チームが機能するとはどういうことか――「学習力」と「実行力」を高める実践アプローチ
• Tech Lead（TL/テックリード）の役割 - サンフランシスコではたらくソフトウェアエンジニア - Higepon’s blog
• 業務遂行能力 - パルカワ2

東京タラレバ娘 を読んだ。

[まとめ買い] 東京タラレバ娘（Ｋｉｓｓコミックス）

つらい気持ちになるのではないかと思って読んでなかったんだけど読んだ。
本編も面白いけど、おまけ漫画も面白い。

RecyclerView.AdapterでViewTypeによってViewHolderを切り分ける ViewHolderBinder を作った

RecyclerView.Adapterは、Item毎にViewTypeを指定することでItem毎にViewHolderを指定出来る。なので「itemが映画ならFilmViewHolderを利用する」とか「itemがドラマならTvDramaViewHolderを利用する」とか「positionが0ならAdViewHolderを利用する」などが出来る。

ただそれをやっていると以下のようにViewHolderのインスタンスを作成するonCreateViewHolderとViewHolderを利用するonBindViewHolderがItemViewTypeが増えるたび膨らむ。

public FilmographyAdapter(Context context) {
    this.context = context;
}

...

@Override
public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
    switch (viewType) {
        case VIEW_TYPE_1: {
            return new TvDramaViewHolder(LayoutInflater.from(context).inflate(R.layout.row_filmography, parent, false));
        }
        case VIEW_TYPE_2: {
            return new FilmViewHolder(LayoutInflater.from(context).inflate(R.layout.row_filmography, parent, false));
        }
        default:  {
            throw new IllegalArgumentException("みつからんかった");
        }
    }
}

@Override
public void onBindViewHolder(RecyclerView.ViewHolder holder, int position) {
    switch (holder.getItemViewType()) {
        case VIEW_TYPE_1: {
            ((TvDramaViewHolder) holder).render(items.get(position));
            break;
        }
        case VIEW_TYPE_2: {
            ((FilmViewHolder) holder).render(items.get(position));
            break;
        }
        default:  {
            throw new IllegalArgumentException("みつからんかった");
        }
    }
}

RecyclerView-Binderというのもあって、それを利用すれば上記のようにswitchで頑張る必要は なさそう。ただViewHolder以外に自分でBinderクラスを作らなければならない事, Adapterに継承が必要なことが理由で利用しなかった。

なので、aptの勉強がてら自分で作ってみた。
github.com

さっきのswitch文べったりなコードがこうなる。

public FilmographyAdapter(Context context) {
    // Binderクラスが自動で生成されるのでそれのインスタンスを持つ
    this.binder = new FilmographyAdapterViewHolderBinder(context, this);
}

...

// @ViewHolderでviewTypeとlayoutを指定してあげる
void onBindViewHolder(
        @ViewHolder(viewType = VIEW_TYPE_1, layout = R.layout.row_filmography) TvDramaViewHolder holder,
        int position) {
    holder.render(items.get(position));
}

void onBindViewHolder(
        @ViewHolder(viewType = VIEW_TYPE_2, layout = R.layout.row_filmography) FilmViewHolder holder,
        int position) {
    holder.render(items.get(position));
}

@Override
public RecyclerView.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
    // @ViewHolderで指定されたViewType, layoutを元に
    // ViewHolderのインスタンスを作成する。
    return binder.create(parent, viewType);
}

@Override
public void onBindViewHolder(RecyclerView.ViewHolder holder, int position) {
    // @ViewHolderで指定されたViewTypeを元に利用するViewHolderのインスタンスを決めて
    // そのインスタンスにあった onBindViewHolder() を利用する。
    binder.bind(holder, position);
}

ViewHolderのインスタンスを作成する部分は、僕はいつもこう書いてるだけなので、人間が書く必要はないなと感じた。なので、全部自動生成するようにした。

return new ItemViewHolder(layoutInflater.inflate(R.layout.hogehoge, parent, false));

その結果、ViewTypeが増えれば、 対応したViewHolderとonBindViewHolder(...)を追加してやるだけになった。

まとめ

• RecyclerViewでViewType毎にViewHolderを分けれるのは便利
• ただ onCreateViewHolderなどが膨らんでしまい書くコードも多い
• なので、ViewHolderBinderというのを作った

github.com