java - 干净的代码——Java静态函数和变量重用

Question

我正在尝试清理静态函数。让我在这里提供这个函数的模板。它是一个静态函数，仅在代码库中的两个位置使用，其中一个位置 isClient 为 true，另一个位置 isClient 为 false。

public static void validate(BLangFunction resource, DiagnosticLog dlog, boolean resourceReturnsErrorOrNil,
                                boolean isClient) {
        if (!resourceReturnsErrorOrNil) {
            dlog.logDiagnostic(Diagnostic.Kind.ERROR, resource.pos, "Invalid return type: expected error?");
        }
        switch (resource.getName().getValue()) {
            case WebSocketConstants.RESOURCE_NAME_ON_OPEN:
            case WebSocketConstants.RESOURCE_NAME_ON_IDLE_TIMEOUT:
                validateOnOpenResource(resource, dlog, isClient);
                break;
            case WebSocketConstants.RESOURCE_NAME_ON_TEXT:
                validateOnTextResource(resource, dlog, isClient);
                break;
            case WebSocketConstants.RESOURCE_NAME_ON_BINARY:
                validateOnBinaryResource(resource, dlog, isClient);
                break;
            case WebSocketConstants.RESOURCE_NAME_ON_PING:
            case WebSocketConstants.RESOURCE_NAME_ON_PONG:
                validateOnPingPongResource(resource, dlog, isClient);
                break;
            case WebSocketConstants.RESOURCE_NAME_ON_CLOSE:
                validateOnCloseResource(resource, dlog, isClient);
                break;
            case WebSocketConstants.RESOURCE_NAME_ON_ERROR:
                validateOnErrorResource(resource, dlog, isClient);
                break;
            default:
                dlog.logDiagnostic(Diagnostic.Kind.ERROR, resource.pos,
                                   "Invalid resource name " + resource.getName().getValue() + " in service ");
        }

    }

此代码的问题在于，dlog 变量被传递给所有其他被调用的函数以及 isClient 变量。并且 resourceReturnsErrorOrNil 仅在一处使用。

我考虑重构代码以具有非静态函数，以便 dlog 可以是一个类变量，并且可以有两个类，一个用于客户端，另一个用于服务，所有公共(public)代码都在一个 super 一流。然后我想这个函数只在两个地方使用，我们是否应该为这些地方提供实例类？

该函数的源码可见here .

基于干净代码原则的重构的最佳方法是什么？

Answer 1

Then I thought that this function is used only in two places and should we have instance classes for these?

这实际上取决于您的目标。我同意，当前的代码远非理想。你概述的事情都很有道理。因此，回答您的问题:是的，如果您认为这对代码库的质量有帮助，那么拥有一个仅在两个地方使用的“实例”类是完全可以的。

另一件事要仔细看看:也许有一种方法可以摆脱这个开关。例如，您可以使用映射(键:您的常量，值:某些功能接口(interface)...进行相应的验证)。