重用部分代码

Question

我是一个真正的 Java 新手，尤其是这个面向对象的编程，所以我将不胜感激任何输入。

我已经制作了功能，可以更改每个评分栏选择的颜色，并且由于我想在其他评分栏上重复使用此功能，因此我尝试将整个代码插入方法中每个对象的名称和资源 id 作为参数，但我显然不知道我在做什么，因为我收到错误，因为名称变量已在范围中定义并且 findViewById 是非静态方法并从静态上下文中调用.

//rating bar
static void starLight(String name, int resId) {
    RatingBar name = (RatingBar) findViewById(resId);
    name.setOnRatingBarChangeListener(new RatingBar.OnRatingBarChangeListener() {
        @Override
        public void onRatingChanged(RatingBar ratingBar, float 
                rating, boolean fromUser) {
            int whole = Math.round(rating);

            Drawable progress = ratingBar.getProgressDrawable();
            if (whole == 1) {
                DrawableCompat.setTint(progress, 
                    ResourcesCompat.getColor(getResources(), R.color.colorGreen, null));
            }
            if (whole == 2) {
                DrawableCompat.setTint(progress, 
                    ResourcesCompat.getColor(getResources(), R.color.colorOrange, null));
            }
            if (whole == 3) {
                DrawableCompat.setTint(progress, 
                    ResourcesCompat.getColor(getResources(), R.color.colorRed, null));
            }
        }
    });
}

如果您能给我一些启发或指出正确的方向，我将不胜感激。

Answer 1

为什么不传入特定的 RatingBar 而不是 Resource id？然后您可以将方法签名保留为静态。正如其他人指出的那样，不需要String name参数。像这样：

static int green = ResourcesCompat.getColor(getResources(), R.color.colorGreen, null));


static void starLight(RatingBar ratingBar) {

 Drawable progress = ratingBar.getProgressDrawable();
        if (whole == 1) {
            DrawableCompat.setTint(progress, green);
...}

Answer 2

看来只需要改函数签名，去掉String参数即可：

static void starLight(/*String name,*/ int resId) {

我不知道你打算用它来做什么，但看起来你实际上并没有用它来做任何事情。

您收到编译器错误的原因是因为您有两个名为“name”的变量：一个作为参数，一个作为方法主体内的局部变量。

---

附带说明，您当前的版本重复了很多代码。你可以重构这个：

Drawable progress = ratingBar.getProgressDrawable();
if (whole == 1) {
    DrawableCompat.setTint(progress, 
        ResourcesCompat.getColor(getResources(), R.color.colorGreen, null));
}
if (whole == 2) {
    DrawableCompat.setTint(progress, 
        ResourcesCompat.getColor(getResources(), R.color.colorOrange, null));
}
// etc.

类似

Drawable progress = ratingBar.getProgressDrawable();
int colour;
switch (whole)
{
    case 1:
        colour = R.color.colorGreen;
        break;
    case 2:
        colour = R.color.colorOrange;
        break;
    case 3:
        colour = R.color.colorRed;
        break;
    default:
        colour = //something. Or throw an exception maybe?
}
DrawableCompat.setTint(progress, ResourcesCompat.getColor(getResources(), colour, null));

有趣的是，这确实比你现在得到的要长，但我认为更容易看到正在发生的事情，因为信息不那么密集。此外，如果您更改设置色调的方式，现在您只需在一处进行。