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

我试图清理一个静态函数。让我在这里提供这个函数的模板。它是一个静态函数，仅在代码库中的两个位置使用，

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

变量一起传递给所有其他调用函数。而

ResourceReturnsErrorNil

仅在一个位置使用

我考虑过重构代码，使之具有非静态函数，以便

dlog

可以是一个类变量，并且可以有两个类，一个用于客户端，另一个用于服务，所有公共代码都在一个超类中。然后我想这个函数只在两个地方使用，我们应该为它们设置实例类吗

可以找到此函数的源代码

基于干净代码原则进行重构的最佳方式是什么

然后我想这个函数只在两个地方使用，我们应该为它们设置实例类吗

这取决于你的目标。我同意，目前的准则远非理想。你概述的东西都有道理。因此，为了回答您的问题：是的，如果您认为这将有助于提高代码库的质量，那么拥有一个只在两个地方使用的“实例”类是非常好的

---

另一件需要仔细观察的事情是：也许有一种方法可以摆脱这个开关。例如，您可以使用一个映射（键：您的常量，值：一些函数接口…进行相应的验证）

该方法应为非静态。这样，您就可以将依赖项注入用于日志功能。

为什么不保持原样呢？如果将其更改为类变量，则可能会有两个线程同时使用它，并且使用了错误的值。如上所述，问题是这不是一个非常干净的代码。我想改进一下。我不认为会有线程问题，因为我在这个应用程序中没有使用多个线程。你能解释解释解释的第二部分吗？我如何用地图消除开关？我想你是说对了？实际上，我也进行了重构，并发送了一个PR:）。谢谢