如何实现这个 C# 类层次结构

Question

我想做的事情其实很简单。我有一个名为 clsSQLInterface 的类来处理数据库执行。该类包含一个名为 bool isSQLSafe 的静态函数，如果发送执行的 SQL 被认为是危险的，它将返回 false。这样我就可以过滤掉任何恶意行为。

现在，我的程序的另一部分实际上需要能够执行诸如 UPDATE、DELETE 等操作。所以我想我应该继承 clsSQLInterface类并使用始终返回 true 的内容覆盖 isSQLSafe 函数。

顺便说一句，这不是关于数据库安全性的问题！

好吧，我这样做了......

public class clsSQLInterface //Code not shown, just definitions
{
  private static string connectionString(string sKey){...}

  public static bool isSQLSafe(string sSQL){...}

  public static DataTable executeSQLText(string sSQL, string sConnectionKey){...}

  public static DataTable executeGenericQuery(clsGenericSQL query,string sDBKey){...}
}

还有最重要的类......

public class clsSQLInterface_unsafe : clsSQLInterface
{
    public clsSQLInterface_unsafe()
    {   
    }

    public new static bool isSQLSafe(string sSQL) //overriding the base method
    { return true; }
}

好吧。这种方法的问题在于，从方法 executeSQLText 和 executeGenericQuery 中调用 isSQLSafe。我希望这些方法做的是调用重写的 isSQLSafe ，它总是返回 true。然而，他们没有。他们称之为基础实现。

我是否还必须重写每个调用 isSQLSafe 的方法？ 这看起来像是浪费代码。

当然，当我继承该类时，我实际上是在“复制”所有基本方法，并且它们的行为应该就像它们现在是 clsSQLInterface_unsafe 的一部分一样？

Answer 1

您不能覆盖静态方法。它们不是继承的，它们是类的方法，而不是类的实例的方法。基类中的静态方法将始终调用同一类中的静态方法。
只需使方法不是静态和虚拟的，然后在派生类中重写它们就可以解决您的问题。

编辑：new static修饰符只是告诉编译器您打算隐藏基类的方法（尝试删除它并查看警告），但它不会覆盖任何内容。

覆盖意味着该函数的派生类版本正在取代虚拟表中的基类版本。
虚拟表是与实例关联的方法的索引。没有实例，没有虚拟表，因此您无法覆盖静态方法。

PS：在这里查看什么是虚拟表的更好解释：http://en.wikipedia。 org/wiki/Virtual_method_table

Answer 2

问题来自于static修饰符。

您可以重新考虑使用以下内容重构您的代码，为什么不呢：

    using System;
    using System.Collections.Generic;
    using System.Linq;
    using System.Text;

    namespace ConsoleApplication1
    {
        public abstract class BaseSqlInterface
        {
            protected abstract bool IsSafe(string instruction);

            public void Execute(string sqlStatement)
            {
                if (IsSafe(sqlStatement))
                {
                    // run the sql command
                }
                else
                {
                    throw new Exception("You're evil");
                }
            }
        }

        public class SqlInterfaceSafe : BaseSqlInterface
        {
            public override bool IsSafe(string instruction)
            {
                return instruction.Contains("I'm not evil, I promise");
            }
        }
        public class SqlInterfaceUnsafe : BaseSqlInterface
        {
            public override  bool IsSafe(string instruction)
            {
                return true;
            }
        }


        public static class SqlInterfaceFactory
        {
            public static BaseSqlInterface GetInstance()
            {
                // return the actual object using IOC, switch, ... whichever method you want
                return DateTime.Now.Day % 2 == 0 ? (BaseSqlInterface)new SqlInterfaceSafe() : new SqlInterfaceUnsafe();
            }
        }
    }