SeekBar とそれに対応する TextView が必要になるたびに少し少ないコードを記述しようとして、次の抽象クラスを作成しました。
abstract public class SeekBarWrapper {
SeekBar bar;
TextView valueText;
int value = 0;
int minValue;
int divisor;
public SeekBarWrapper(SeekBar sb, TextView tv, int value, int minValue,
int divisor){
this.bar = sb;
this.valueText = tv;
this.value = value;
this.minValue = minValue;
this.divisor = divisor;
setListener();
}
private void setListener(){
bar.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
@Override
public void onStopTrackingTouch(SeekBar seekBar) { }
@Override
public void onStartTrackingTouch(SeekBar seekBar) { }
@Override
public void onProgressChanged(SeekBar seekBar, int progress,
boolean fromUser) {
if(!fromUser) return;
value = progress + minValue;
valueText.setText(Integer.toString(value/divisor));
sendValue();
}
});
}
abstract protected void sendValue();
public void updateValue(int newValue){
if(newValue == value) return;
value = newValue;
valueText.setText(Integer.toString(value));
bar.setProgress(value*divisor-minValue);
}
}
具体的な SeekBar ごとに、ネストされたクラスを記述します。次に例を示します。
class VolumeBarWrapper extends SeekBarWrapper{
public VolumeBarWrapper(SeekBar s, TextView t, int v, int min, int div){
super(s, t, v, min, div);
}
public void sendValue(){
someCallback.volume(this.value);
}
}
そして次のようにインスタンス化します:
VolumeBarWrapper volume;
// later:
volume = new VolumeBarWrapper((SeekBar) view.findViewById(R.id.volume_bar),
(TextView) view.findViewById(R.id.volume_value), 300, 0, 70);
それは機能的で、一見改善されています。私が知りたいこと:
- これを匿名の内部クラスにする方法、またはインスタンスごとのコードをさらに圧縮する別の方法はありますか? そして、それほど差し迫ったことではありません:
- 私は「ラッパー」というラベルを悪用していますか?それはパターンで言えば特別な意味を持っていませんか?
- この設計は OOP の観点から「悪い」ものですか (私はまだこの点について自分自身を教育しようとしています)?