我应该在方法参数中传递对象还是在Java的父构造函数中传递对象

Question

我有一个我无法解决的困境。 在这种情况下，我应该在方法参数或父构造函数中传递对象吗？

第一个想法：

public class TempChannelsPlugin extends JolssyPlugin {

    private JDA client;
    private TempChannelsListener listener;
    private JdaCommandManager jdaCommandManager;

    @Inject
    public TempChannelsPlugin(JDA client, JdaCommandManager jdaCommandManager, TempChannelsListener listener) {
        super(Category.UTILITIES, "Temporary Channels");

        this.client = client;
        this.jdaCommandManager = jdaCommandManager;
        this.listener = listener;
    }


    @Override
    protected void setup() {
        this.registerListener(this.client, this.listener);
    }
}

在 JolssyPlugin 中：

protected void registerListener(JDA client, ListenerAdapter listenerAdapter) {
   this.registerListeners(client, listenerAdapter);
}

第二个想法：

public class TempChannelsPlugin extends JolssyPlugin {

    private JDA client;
    private TempChannelsListener listener;
    private JdaCommandManager jdaCommandManager;

    @Inject
    public TempChannelsPlugin(JDA client, JdaCommandManager jdaCommandManager, TempChannelsListener listener) {
        super(client, Category.UTILITIES, "Temporary Channels");

        this.jdaCommandManager = jdaCommandManager;
        this.listener = listener;
    }


    @Override
    protected void setup() {
        this.registerListener(this.listener);
    }
}

在 JolssyPlugin 中：

protected void registerListener(ListenerAdapter listenerAdapter) {
   this.registerListeners(this.client, listenerAdapter);
}

这两种情况有什么区别？为什么我应该支持一个案例而不是另一个案例？

请注意，我正在使用 Guice 进行 DI

Answer 1

在我看来，如果方法 registerListener(ListenerAdapter listenerAdapter) 位于 TempChannelsPlugin 类中：

protected void registerListener(ListenerAdapter listenerAdapter) {
   this.registerListeners(this.client, listenerAdapter);
}

那么第二个选项更好，因为 vatiable private JDA client; 具有整个班级的范围。所以不需要声明不必要的参数，因为我们可以在类TempChannelsPlugin 的任何地方使用private JDA client; 变量。

更新：

在我看来，如果方法registerListener(ListenerAdapter listenerAdapter) 位于TempChannelsPlugin 类，那么第二个选项更好，因为可变private JDA client; 具有整个类的范围。 所以不需要声明不必要的参数。

我会将JDA client 的依赖模式设置为JolssyPlugin，就像您在JolssyPlugin 中拥有registerListener 一样。 通过这样做，我们可以从registerListener 和registerListeners 等方法中删除一个参数。 所以代码看起来像这样：

public abstract class JolssyPlugin {

    protected Set<ListenerAdapter> listeners = new HashSet<>();

    protected JdaCommandManager jdaCommandManager;
    protected Category category;
    protected String name;
    private JDA jdaClient;

    public JolssyPlugin(Category category, String name, JDA jdaClient) {
        this.category = category;
        this.name = name;
        this.jdaClient = jdaClient;
    }
    
    protected abstract void setup();

    protected void registerListener( ListenerAdapter listenerAdapter) {
       this.registerListeners(client, listenerAdapter);
    }

    protected void registerListeners(ListenerAdapter... listenerAdapter) {
        List<ListenerAdapter> listeners = Arrays.asList(listenerAdapter);

        client.addEventListener(listeners);
        this.listeners.addAll(listeners);
    }
}