Comprobar si la colección está vacía o no

Question

public ActionResult Create(FormCollection collection, FormCollection formValue) 
{ 
    try 
    { 
     Project project = new Project(); 

     TryUpdateModel(project, _updateableFields); 

     var devices = collection["devices"]; 
     string[] arr1 = ((string)devices).Split(','); 
     int[] arr2 = Array.ConvertAll(arr1, s => int.Parse(s)); 

     project.User = SessionVariables.AuthenticatedUser; 
     var time = formValue["Date"]; 
     project.Date = time; 
     project.SaveAndFlush(); 

     foreach (int i in arr2) 
     { 
      Device d = Device.Find(i); 
      d.Projects.Add(project); 
      d.SaveAndFlush(); 
     } 

     return RedirectToAction("Index"); 
    } 
    catch (Exception e) 
    { 
     return View(e); 
    } 
} 

Quiero envolver el foreach en una sentencia if que comprueba si

var devices = collection["devices"]; 

está vacía o no. Si está vacío, no se debe ejecutar para cada uno. Para el registro, la colección ["dispositivos"] es una colección de valores de casilla de verificación de un formulario.

Answer 1

No necesita comprobar si la colección está vacía; si está vacía, el código dentro de ForEach no se ejecutará, vea mi ejemplo a continuación.

using System; 
using System.Collections.Generic; 

namespace Test 
{ 
    class Program 
    { 
     static void Main(string[] args) 
     { 
      List<string> emptyList = new List<string>(); 

      foreach (string item in emptyList) 
      { 
       Console.WriteLine("This will not be printed"); 
      } 

      List<string> list = new List<string>(); 

      list.Add("item 1"); 
      list.Add("item 2"); 

      foreach (string item in list) 
      { 
       Console.WriteLine(item); 
      } 

      Console.ReadLine(); 
     } 
    } 
} 

Answer 2

Usted puede utilizar el campo Count para comprobar si la colección está vacía o no

por lo que va a terminar con algo como esto:

if(devices.Count > 0) 
{ 
    //foreach loop 
} 

Answer 3

¿Qué hay de comprobar la longitud de la matriz

if (arr2.length > 0) 
{ 
    foreach (int i in arr2) 
    { 
     Device d = Device.Find(i); 
     d.Projects.Add(project); 
     d.SaveAndFlush(); 
    } 
} 

Answer 4

Su código, tal como está, no funcionará, ya que dice que collection["devices"] es una colección de valores de casilla de verificación, y aún así lo está enviando a un string. ¿Quieres decir collection son los valores de la casilla de verificación? ¿Cuál es el tipo exacto de collection?

Cualquier objeto que implemente ICollection o ICollection<T> se puede comprobar si está vacío o no verificando si la propiedad Count es mayor que cero.

Answer 5

Usted puede utilizar el método Any para saber si una colección como cualquier elemento.

if (devices.Any()) 
{ 
    //devices is not empty 
}