控制器中表单收集时，如何高效验证数组元素无重复且含1、2、3？

Hey there! Your current validation works for exactly three elements, but it’s not super flexible if you ever need to adjust the required values later. Plus, it doesn’t account for edge cases like an array with fewer than 3 elements (which would throw an index error). Let’s look at some cleaner, more scalable ways to ensure your vi array contains all three distinct values (1, 2, 3) with no duplicates:

Quick side note first: You can simplify your split line to string[] vi = collection["VoteInfoCatId"].Split(','); instead of using char.Parse(",") — it’s cleaner and does the same thing.

Option 1: Use a HashSet for Full Set Validation #

HashSets are perfect for checking uniqueness and exact set matches. This approach verifies two key things at once: no duplicates, and that the array contains exactly the three required values (no extras, no missing):

string[] vi = collection["VoteInfoCatId"].Split(',');

// Define the exact set of values we expect
var requiredValues = new HashSet<string> { "1", "2", "3" };
var viSet = new HashSet<string>(vi);

bool isValid = viSet.Count == 3 && requiredValues.SetEquals(viSet);

• The HashSet automatically removes duplicates, so if viSet.Count is 3, we know all elements were unique.
• SetEquals confirms that the array’s values match the required set exactly.

Option 2: Concise LINQ Check #

If you prefer a one-liner using LINQ, you can combine checks for uniqueness and presence of all required values:

using System.Linq;

string[] vi = collection["VoteInfoCatId"].Split(',');

bool isValid = vi.Distinct().Count() == 3 && 
               vi.Contains("1") && vi.Contains("2") && vi.Contains("3");

• vi.Distinct().Count() == 3 ensures no duplicates (since distinct count equals the original array length, which should be 3).
• The Contains checks confirm every required value is present.

Option 3: Explicit Edge Case Handling #

If you want to be extra strict (e.g., catch cases where the array has more than 3 elements), add a length check first:

string[] vi = collection["VoteInfoCatId"].Split(',');

bool isValid = vi.Length == 3 && 
               vi.Distinct().Count() == 3 && 
               new HashSet<string>(vi).SetEquals(new[] { "1", "2", "3" });

This ensures the array has exactly 3 elements, no duplicates, and matches the required set perfectly.

Why These Are Better Than Your Original Approach #

• Scalability: If you ever need to add/remove required values, you only update the required set/list instead of rewriting all index comparisons.
• Readability: The intent of the validation is clear at a glance, unlike hardcoded index checks.
• Robustness: These methods handle edge cases (like short/long arrays) that your original code would fail on.

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