1

为了在每次我想要一个 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);

它是功能性的，并且似乎是一种改进。我想知道的：

  1. 有什么方法可以使它成为一个匿名内部类，或者另一种方法来进一步压缩每个实例的代码？而且不那么紧迫：
  2. 我是不是在滥用“包装”标签，它在模式说话中没有一些特殊的含义吗？
  3. 从 OOP 的角度来看，这种设计“糟糕”吗（我仍在努力在这方面自学）？
4

1 回答 1

1

你总是可以做得更好 SeekBar，并在任何地方使用它：

public class VersatileSeekBar extends SeekBar implements SeekBar.OnSeekBarChangeListener {
    private TextView mTextView;
    private ChangeHandler mChangeHandler;

    public void bindDisplayToChange(TextView textView,ChangeHandler handler) {
        mTextView = textView;
        mChangeHandler = handler;
    }

    public VersatileSeekBar(Context context) {
        super(context);
        init();
    }

    public VersatileSeekBar(Context context, AttributeSet attrs) {
        super(context, attrs);
        init();
    }

    public VersatileSeekBar(Context context, AttributeSet attrs, int defStyle) {
        super(context, attrs, defStyle);
        init();
    }


    @Override
    public void onProgressChanged(SeekBar seekBar, int i, boolean b) {
        if(mChangeHandler != null && mTextView != null){
            mChangeHandler.onChange(i,mTextView);
        }
    }

    @Override
    public void onStartTrackingTouch(SeekBar seekBar) {
    }

    @Override
    public void onStopTrackingTouch(SeekBar seekBar) {
    }

    private void init(){
        this.setOnSeekBarChangeListener(this);
    }

    public static abstract class ChangeHandler{
        public abstract void onChange(int value,TextView textView);
    }
}

调用代码：

myVersatileSeekBar.bindDisplayToChange(myTextView, new VersatileSeekBar.ChangeHandler() {
            @Override
            public void onChange(int value, TextView textView) {
                textView.setText("level :" + value * 100);
            }
        });
于 2012-12-22T07:00:25.357 回答