Должен ли я создавать сложные вложенные структуры с Linq или традиционными циклами?

Question

так из 3 следующих примеров, которые делают то же самое, я действительно склоняюсь к первому, но это действительно избыточна и злоупотребление LINQ, чтобы делать то, что путь, где вы можете создать все это как выражение

var Rand = new Random(); 
     Hosts = "abcdef".Select(x => 
        { 
         return new HostMachineToUpdate(x + "_Host", 
          Enumerable.Range(1, Rand.Next(3, 8)) 
          .Select(y => new VirtualMachineToUpdate(x + y.ToString() + "_VM")). 
          ToList() 
          ); 
        } 
       ) 
       .ToList(); 

     //traditional 
     Hosts = new List<HostMachineToUpdate>(); 
     for (int x = (int)'a'; x < (int)'e'; x++) 
     { 
      var Guests = new List<VirtualMachineToUpdate>(); 
      for (int y = 1; y < (new Random().Next(3, 8));y++) 
      { 
       Guests.Add(new VirtualMachineToUpdate((char)x + y.ToString() + "_VM")); 
      } 
      Hosts.Add(new HostMachineToUpdate((char) x + "Host",Guests)); 
     } 

     //very traditional. 
     Hosts = new List<HostMachineToUpdate>(); 
     int lower = (int)'a'; 
     int upper = (int)'e'; 
     for (int x = lower; x < upper; x++) 
     { 
      List<VirtualMachineToUpdate> Guests = new List<VirtualMachineToUpdate>(); 
      int randomItemNum = new Random().Next(3, 8); 
      for (int y = 1; y < randomItemNum; y++) 
      { 
       string vmname = (char)x + y.ToString() + "_VM"; 
       VirtualMachineToUpdate vm = new VirtualMachineToUpdate(vmname); 
       Guests.Add(vm); 
      } 
      string hostname = (char)x + "Host"; 
      HostMachineToUpdate host = new HostMachineToUpdate(hostname, Guests);     
      Hosts.Add(host); 
     } 

Answer 1

Мне лично не нравится сумма литье использовано в вашем традиционное решение.

Нужно ли все кастинг?

Не будет ли этот (непроверенный) код делать то, что требуется?

// traditional. 
    Hosts = new List<HostMachineToUpdate>(); 
    foreach (char x in "abcd") 
    { 
     List<VirtualMachineToUpdate> Guests = new List<VirtualMachineToUpdate>(); 
     int randomItemNum = new Random().Next(3, 8); 
     for (int y = 1; y < randomItemNum; y++) 
     { 
      Guests.Add(new VirtualMachineToUpdate(x + y.ToString() + "_VM")); 
     } 
     Hosts.Add(new HostMachineToUpdate(x + "Host", Guests)); 
    } 

Answer 2

Я предпочитаю декларативный подход, поэтому что-то вроде первого варианта. Но я предпочел бы использовать синтаксис C#. Что-то неопределенно:

(from x in Enumerable.Range('a', 'e'-'a') 
select new HostMachineToUpdate(
    (char)x + "_Host", 
    (from y in Enumerable.Range(1, new Random.Next(3,8)) 
    select new VirtualMachineToUpdate((char)x + y.ToString() + "_VM")).ToList()) 
.ToList(); 

Он чувствует себя близко. Может быть недостающим (или).

Answer 3

Если каждое из ваших решений выполняется достаточно быстро для ваших нужд, то можно с уверенностью сказать, что машина не может отличить эти параметры.

Важное соображение сводится к тому, насколько выразительным или ясным является код для читателя, поскольку в будущем кому-то придется понимать, отлаживать или расширять код. В приведенном выше случае попробуйте это. Оставьте код в покое на две недели. Когда вы вернетесь, решите, какой из параметров легче всего понять.

Для меня это проверка реальности. Раньше я гордился написанием некоторого умного кода, но на самом деле более простым решением было правильное решение.