C#新手求助：非静态字段报错“需要对象引用”问题

Hey there, fellow C# newbie! Let's break down your problem step by step and fix it while also setting you up for easy global changes later.

First, Why That Error Happens #

The error An object reference is required for the non-static field, method, or property 'RandomColor.GetColors' means you're trying to call the GetColors method directly using the class name (RandomColor.GetColors(...)), but that method is marked as non-static. Non-static methods belong to an instance of the class, not the class itself—so you need to create an object of RandomColor first to call it, or make the method static.

But since you want to avoid duplicating switch logic across two classes and make global changes easy, let's solve both problems at once.

The Best Solution: Extract Switch Logic to a Single Reusable Class #

Instead of having identical switch statements in two separate classes, we'll move all that logic into a single helper class. This way, you only ever need to modify one file when you want to update the logic, and all classes using it will automatically pick up the change.

Step 1: Create a Static Helper Class #

Let's assume your enum looks something like this (adjust to match your actual enum):

public enum ColorCategory
{
    Primary,
    Secondary,
    Accent
}

Now make a static helper class to hold the shared switch logic:

public static class ColorSwitchHelper
{
    // This static method contains all your switch logic in one place
    public static List<Color> GetColors(ColorCategory category)
    {
        switch (category)
        {
            case ColorCategory.Primary:
                return new List<Color> { Color.Red, Color.Blue, Color.Yellow };
            case ColorCategory.Secondary:
                return new List<Color> { Color.Orange, Color.Green, Color.Purple };
            case ColorCategory.Accent:
                return new List<Color> { Color.Magenta, Color.Cyan, Color.Lime };
            default:
                throw new ArgumentOutOfRangeException(nameof(category), category, "Invalid color category");
        }
    }
}

Step 2: Update Your Classes to Use the Helper #

Now in both of your classes, replace the duplicate switch statements with a call to the helper method. No more duplicated code!

Example for one class:

public class FirstColorDependentClass
{
    public void ProcessColors(ColorCategory category)
    {
        // Call the shared logic instead of writing your own switch
        var selectedColors = ColorSwitchHelper.GetColors(category);
        
        // Do your class-specific processing here
        foreach (var color in selectedColors)
        {
            Console.WriteLine($"Processing color: {color.Name}");
        }
    }
}

And the second class will look almost identical, just calling the same helper method:

public class SecondColorDependentClass
{
    public void RenderColors(ColorCategory category)
    {
        var selectedColors = ColorSwitchHelper.GetColors(category);
        
        // Your class-specific rendering logic here
        foreach (var color in selectedColors)
        {
            Console.WriteLine($"Rendering color: {color.Name}");
        }
    }
}

Why This Works Perfectly for Your Needs #

• Fixes the original error: Static methods in ColorSwitchHelper can be called directly using the class name—no need to create an instance first, so you won't get that non-static reference error anymore.
• Global changes made easy: If you need to add a new enum value, modify the colors returned for a category, or adjust the switch logic in any way, you only have to update the ColorSwitchHelper class. All other classes that use it will automatically use the updated logic.
• Eliminates duplicate code: No more maintaining two identical switch statements—less code to debug and update.

If You Must Keep RandomColor.GetColors Non-Static #

If for some reason you can't change RandomColor.GetColors to static, you just need to create an instance of RandomColor before calling the method:

// Create an instance first
var colorGenerator = new RandomColor();
var colors = colorGenerator.GetColors(category);

But this doesn't solve your global change problem—you'd still have duplicate code in both classes. So the helper class approach is strongly recommended.

内容的提问来源于stack exchange，提问作者Nick King