c# - 我应该违反 SOLID 中的 S 还是应该违反 DRY 原则？

Question

我有一个像这样发送 GET 请求的方法:

private JArray GetRESTData(string uri)
{
    try
    {
        var webRequest = (HttpWebRequest)WebRequest.Create(uri);
        var webResponse = (HttpWebResponse)webRequest.GetResponse();
        var reader = new StreamReader(webResponse.GetResponseStream());
        string s = reader.ReadToEnd();
        return JsonConvert.DeserializeObject<JArray>(s);
    }
    catch // This method crashes if only one json "record" is found - try this:
    {
        try
        {
            MessageBox.Show(GetScalarVal(uri));
        }
        catch (Exception ex)
        {
            MessageBox.Show(ex.Message);
        }
    }
    return null;
}

...我通过将其嵌入到 webRequest 和 webResponse 的分配中来修改它以处理 POST 请求:

if (uri.ToUpper().Contains("POST"))
{
    webRequest.Method = "POST";
    webRequest.ContentLength = 0;
}

...并将其重命名为 GetOrPostRESTData()

但这违反了单一职责原则。

但是，如果我将其分为两种方法，POST 方法与 GET 方法相同，但在条件(“if Post”)中有另外两行代码除外，我违反了干，因为大部分代码是相同的。

还有第三种方法吗？中间路线？还是我必须在这两种违规行为之间做出选择？我被困在一个干燥的地方和一个坚实的地方之间。

Answer 1

从更高的抽象层次来看怎么样？不用担心它是方法名称中的 GET 还是 POST，只需将其命名为 ProcessRequest。在这种情况下，您可能会争辩说 SRP 仍在遵循 - 您的方法正在做的唯一一件事就是处理给定 URI 中指示的请求 - 而您没有复制任何代码。