0

我有以下代码和体系结构（实际上这是一个非常简化的版本），其中包含一个抽象类和一个具体类。

public abstract class AbstractProcessor {
    public void updateDataFromUrl(String url) {
        //Download url
        //Decrypt, and do a lot of stuff, 
        String data = "abc"; //Result from downloading

        String processed = processData(data);

        //Do a lot of other things with the transformed data
    }

    public abstract String processData(String data);
}


final class ConcreteProcessor extends AbstractProcessor {
    public void updateData(int year, int month, int day) {
        String url = String.format("http://%d.%d.%d", year, month, day);
        updateDataFromUrl(url);
    }

    @Override
    public String processData(String data) {
        //Process the data

        //---------------
        //PROBLEM:
        //----------------
        //Need access to year, month, day ....

        return null;
    }
}

该updateDataFromUrl方法包含很多processData我想在多个处理器中重用的代码（展位，调用之前和之后），因此我将代码放入一个抽象类中。

问题是：我想访问提供给新添加updateData方法的数据（此处year为 , month, day）。当调用流经不知道这些参数的抽象类时，这些信息就会丢失。如何修改此架构以保留此信息？

我想到了以下解决方案，它们都有明显的缺点：

  1. 提供带有 url 的抽象方法并再次从那里提取参数。（问题：有哪些参数我只需要在procecssData方法中而不需要在 url 中？）
  2. 将该updateDataFromUrl方法拆分为两个两个方法（processData调用之前的部分和之后的部分）。现在直接在 ConcreteProcessor 中使用这些方法。（问题：该updateDataFromUrl方法在调用之前和之后都有很多我需要的上下文processData。如何在新创建的方法之间传输这些数据？）
4

2 回答 2

2

两个想法：

  1. 使用实例变量ConcreteProcessor来存储年、月、日。两种方法都可以访问对象的实例变量。但是，抽象类无法访问它们。注意：该对象不再是线程安全的。您必须有多个独立的处理器才能同时处理数据。

  2. 反转依赖和使用组合。泛型类Processor包含通用逻辑。当您调用时，updateDataFromUrl您还传递了一个ProcessingStrategy实现的实例processData（实际上是一种回调）。

可能有几种可能的想法变体。

于 2012-11-22T14:24:40.463 回答
1

只有微小的修改会导致这个解决方案（如果我没有错过某个点（未经测试））。

您将获得以下好处：

  • 信息传递非常简单（使用实例变量）
  • 你保留线程安全
  • 从外部（公共）的角度来看，ConcreteProcessor可以保持不变

最好不要返回ConcreteProcessorfromupdateData而是一些表示转换后的数据的不可变类型。

public abstract class AbstractProcessor {
    public void updateDataFromUrl(String url) {
        //Download url
        //Decrypt, and do a lot of stuff, 
        String data = "abc"; //Result from downloading

        String processed = processData(data);

        //Do a lot of other things with the transformed data
    }

    public abstract String processData(String data);
}


final class ConcreteProcessor extends AbstractProcessor {
    public static ConcreteProcessor updateData(int year, int month, int day) {
            ConcreteProcessor p = new ConcreteProcessor(year, month, day);
            p.updateDataFromUrl(url);
            return p;
    }

    private /* instance vars for year, month, day, url, ... */

    private ConcreteProcessor(int year, int month, int day) {
            this.year = year;
            this.month = month;
            this.day = day;
            this.url = String.format("http://%d.%d.%d", year, month, day);
    }


    @Override
    public String processData(String data) {
        //Process the data

        //---------------
        // NO PROBLEM:
        //----------------
        //Easy to access to year, month, day using the instance vars

        return null;
    }
}

但整体概念看起来像一个线性管道，因此最好使管道更明确。您将需要某种状态承载，但我认为使用泛型应该不是什么大问题。

于 2012-11-22T14:52:40.967 回答