Androidのお仕事で、あるclassのcodeを読んでいて。

• onTouchListenerの上にonClickListenerを上書き
  あるbuttonをsetOnTouchListener(this);してて。 buttonをonTouchListenerっていうのもなんですが、 onTouch(...)で更にsetOnClickListener(...);してるという...
• timer止めずに新しいinstanceを上書き
  CountDownTimerをinstance fieldとして持ってて、 途中でnewしてるんですが、それが複数箇所あるんですよね... 直前にcancel処理とか特に無いし。大丈夫なのかこれ。
• カタカナや"$","/"にtoLowerCase()/toUpperCase()してindexOf()
  "半角カタカナ".toLowerCase()してからindexOf(...)>-1して、 contains(...)と同じことしてました。indexOfはまだしも、 toLowerCaseしたからって 全角カタカナが半角カタカナになるわけじゃないのに。 え、まさか、とか思ってしまった自分が情けないです。 同様に、記号に対しても"$".toLowerCase()とか謎すぎます。
• 他の(inflateもincludeもしてない)View上のR.idをfindViewById()
  当然nullですfindViewById()しても。実質無害なcodeではありますが。 どうやら他から何も考えずにコピペしたから、らしいです。
• listの2度回し
  なるべく一度で済むように書きますよねぇ、フツーは。 ちょっと違う処理をするから、なのか、 同じlistを直後に2度回して、って。 まー他でも同じtableのDB accessを3回してたりしますからねーこのcode。
• 1830秒?
  随分謎なMagic Numberです。
• loop回すのに中で値を上書き(結局見てるのは最後の値だけ)
  for(i in list){v = i}みたいな。v=list[lastIndex]でいいじゃん。 そういうことされると意図が読めないんですよね。困ります。
• loopの空回し
  waitしたいみたいなんですが、 while(true){if(!flag)break;}ってこれじゃぁCPU無駄遣いでしょ。 改善したっていってdo{i=0;}while(!flag);って、あのねー...
• synchronized wait()で同期 他Activity(dialog)に遷移させ、 その同期にsynchronized(this){wait();}って使ってます。 そういうthread jugglingはやめて欲しい、です。 こういうのってホントはRXですよね。
• method/field名が大文字で始まっててclass名と区別がつかない、 なんていうのは可愛い方で、もう気にもならなくなってますそういえば。 methodも長いし条件分岐も複雑で、 state patternとかなんて知らないんだろうなぁと。
• というか全てがfat ActionでFragmentもなければApplicationもないという (基本的には。後から「訳も分からずダーッとコピペした部分」にはありますが)。

...というように。 こういうcodeと共に仕事するのは、嫌で嫌で仕方ありません。