Question

我有这段代码：

private void timer1_Tick(object sender, EventArgs e)
    {
        #region BaseAddress

      Process[] test = Process.GetProcessesByName(process); //Get process handle
      if (test.Any())
      {

          int Base = test[0].MainModule.BaseAddress.ToInt32();
      }
        #endregion
       //lots of other code blocks
     }

我现在想要从timer1_Tick控件中取出区域'BaseAddress'，以使代码更有效，并在程序开始时运行一次。这个控件中的其他代码经常使用变量'Base'，这是什么才能使这个全局可访问，而不必返回使用Base的所有实例，并执行类似MyGlobals.Base的操作？ / p>

Answer 1

将地址延迟加载到静态变量中。在您第一次使用它之前它不会初始化，然后在应用程序的生命周期内保留在内存中。

public static MyGlobals
{
    private static readonly Lazy<int> _processBase = new Lazy<int>(() => GetProcessBase("MyProcessName"));

    // I don't recommend using the word Base, but OK...
    public static int Base { get { return _processBase.Value; } }

    private static int GetProcessBase(string processName)
    {
        int b = 0;
        Process[] p = Process.GetProcessesByName(processName);
        if(p != null && p.Length > 0)
        {
            b = p[0].MainModule.BaseAddress.ToInt32();
        }
        return b;
    }
}

在应用程序的其他部分...

private void timer1_Tick(object sender, EventArgs e) 
{ 
    if(MyGlobals.Base > 0)
    {
         // TODO: change "Base" to "MyGlobals.Base" in code below or it won't compile...
         //lots of other code blocks 
    }
}

Answer 2

我在这种情况下使用的方法是使用 Base 属性创建单例类 ProcessFetcher （例如）。

我的课程将有 fetch（）功能和 isDataPresent 属性。

您可以决定手动调用fetch或将其放在构造函数上。

如何修复这个设计糟糕的代码

2 个答案: