Share via


CA1025: Replace repetitive arguments with params array

TypeName

ReplaceRepetitiveArgumentsWithParamsArray

CheckId

CA1025

Category

Microsoft.Design

Breaking Change

Non-breaking

Cause

A public or protected method in a public type has more than three parameters, and its last three parameters are the same type.

Rule Description

Use a parameter array instead of repeated arguments when the exact number of arguments is unknown and the variable arguments are the same type, or can be passed as the same type. For example, the WriteLine method provides a general-purpose overload that uses a parameter array to accept any number of Object arguments.

How to Fix Violations

To fix a violation of this rule, replace the repeated arguments with a parameter array.

When to Suppress Warnings

It is always safe to suppress a warning from this rule; however, this design might cause usability issues.

Example

The following example shows a type that violates this rule.

using System;

namespace DesignLibrary
{
   public class BadRepeatArguments
   {
      // Violates rule: ReplaceRepetitiveArgumentsWithParamsArray. 
      public void VariableArguments(object obj1, object obj2, object obj3, object obj4) {}
      public void VariableArguments(object obj1, object obj2, object obj3, object obj4, object obj5) {}
   }

   public class GoodRepeatArguments
   {
       public void VariableArguments(object obj1) {}
       public void VariableArguments(object obj1, object obj2) {}
       public void VariableArguments(object obj1, object obj2, object obj3) {}
       public void VariableArguments(params Object[] arg) {}
   }
}