El uso de un MatrizParam, pero que requiere al menos un parámetro

Question

Lo que solía tener:

Public Sub Subscribe(channel As ChannelType) 
Public Sub Subscribe(channels As IEnumerable(Of ChannelType)) 

El primero sólo llama a la segunda con {channel} para convertir su parámetro en una matriz.

Decidí que tener que crear una lista de canales para pasar al método era incómodo y elegí combinar las dos sobrecargas en un método que toma un ParamArray.

Public Sub Subscribe(ParamArray channels() As ChannelType) 

'Usage 
Subscribe(ChannelType.News) 
Subscribe(ChannelType.News, ChannelType.Sports) 
Subscribe() 'Oops... this is valid 

¿Qué es la "mejor práctica" aquí? Me gusta la flexibilidad que ParamArray me da para dejar pasar cosas, pero no ayuda al desarrollador a "fallar-más rápido" a través de los comentarios de errores del compilador ... eso significa que algo como un ArgumentException está fuera de cuestión aquí ya que las personas consumir este método puede no estar escribiendo pruebas unitarias. Una opción es la siguiente ...

Public Sub Subscribe(channel As ChannelType) 
Public Sub Subscribe(channel As ChannelType, ParamArray channels() As ChannelType) 

pero me siento como que me pone cerca de volver al punto de partida, es confuso, y requiere mi aplicación de ese método a ser menos sencilla.

Answer 1

otra opción a considerar sería

Module ParamArrayTest 
    Sub ShowThings(ParamArray MyThings() As Integer) 
     For Each thing As Integer In MyThings 
      Debug.Print("{0}", thing) 
     Next 
    End Sub 

    ' Don't try to call without parameters: 
    <Obsolete("Must have at least one parameter", True)> Sub ShowThings() 
     Throw New ArgumentException("Must specify at least one parameter") 
    End Sub 

    Sub Test() 
     ShowThings(3, 4, 5) 
     ShowThings() 
    End Sub 
End Module 

la etiqueta <Obsolete()> con un segundo parámetro de True informa al compilador que intentar utilizar el método de marcado debe r esult en un error de compilación. Dado que el método en cuestión se usaría cuando, y solo cuando, se intenta invocar el método sin ningún parámetro, causaría un error solo en esos momentos. Tenga en cuenta que no se usará el método si se intenta pasar el método a una matriz de cero elementos de Integer; en ese caso, se usaría la forma normal ParamArray.

Answer 2

Creo que la opción que mencionaste es la mejor opción. El uso de nombres más claras para sus parámetros hará que sea menos confuso:

Public Sub Subscribe(mainChannel As ChannelType, ParamArray otherChannels() As ChannelType) 

La otra opción es hacerla cumplir en tiempo de ejecución, pero como usted ha dicho que no dejaría tan rápido:

Public Sub Subscribe(ParamArray channels() As ChannelType) 
    If channels.Count = 0 then 
     Throw new InvalidOperationException("At least one channel is needed") 
    End If 
End Sub