Как я могу предотвратить дублирование общего цикла в моем решении? - PullRequest
Новая
       26

Как я могу предотвратить дублирование общего цикла в моем решении?

0 голосов
/ 05 мая 2011

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

Как я могу помешать себе переписывать этот цикл снова и снова, чтобы я его дублировал?? 

      foreach (int[] columns in rowsAndColumns)
      {
          foreach (int element in columns)
          {

          }
      }

Ответы [ 5 ]

8 голосов
/ 05 мая 2011

Вы можете написать 

foreach (int element in rowsAndColumns.SelectMany(col => col))
{
    // ...
}

вместо этого. Если вам не нужно постоянно вводить это, вы можете абстрагировать его во вспомогательный метод: 

foreach (int element in rowsAndColumns.Flatten())
{
    // ...
}

// [...]

public IEnumerable<T> Flatten(this IEnumerable<IEnumerable<T>> source)
{
    return source.SelectMany(e => e);
}
1 голос
/ 05 мая 2011

Это зависит от того, что вы хотите сделать, но если вы хотите выполнить действие для каждого типа int, вы можете использовать что-то вроде расширения ниже. Некоторая проверка нуля может быть целесообразной. 

static class RowColExtension
{
    public static void Each(this int[][] rowCols, Action<int> a)
    {
        foreach (var r in rowCols)
        {
            foreach (var c in r)
            {
                a(c);
            }
        }
    }
}
0 голосов
/ 05 мая 2011

Поскольку вы перебираете все элементы независимо от строки или столбца, вы должны превратить ваш зубчатый массив в структуру данных первого класса и реализовать IEnumerable для перебора коллекции с использованием foreach.Одна и та же структура данных первого класса может поддерживать индексаторы с одним и двумя аргументами, проверку диапазона и т. Д.

Редактировать:

Вот один из подходов к использованию абстракциивместо манипулирования низкоуровневыми структурами данных.Это предполагает, что зубчатый массив был размещен в другом месте.В любом случае, суть в том, что теперь мы можем использовать foreach непосредственно в структуре данных: 

public class JaggedArray : IEnumerable<int>
{
    private int[][] array;

    public JaggedArray(int[][] array)
    {
        this.array = array;
    }

    public int this[int row, int column]
    {
        get { return array[row][column]; }
        set { array[row][column] = value; }
    }

    public IEnumerable<int[]> Rows
    {
        get { return array; }
    }

    public IEnumerator<int> GetEnumerator()
    {
        foreach (var row in array)
            foreach (var item in row)
                yield return item;
    }

    System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator()
    {
        return GetEnumerator();
    }
}
0 голосов
/ 05 мая 2011

Метод расширения: 

// It's late and I'm tired, the array declaration might be off.
public static void Visit(this int[][] array, Action<int> visitor)
{
      foreach (int[] columns in array)
      {
          foreach (int element in columns)
          {
              visitor(element);
          }
      }
}

myArray.Visit(elem => Console.WriteLine(elem));

Вы также можете использовать Action<int,int>, чтобы получить строку.

0 голосов
/ 05 мая 2011

Это зависит от того, что вы хотите сделать в цикле.Я бы подошел к этому так (непроверенный код из моей головы!): 

public static class MyHelper {
    public static void ForEach(this IEnumerable<int[]> rowsAndColumns, Action<int> action) {
        foreach (int[] columns in rowsAndColumns) {
             foreach (int element in columns) {
                 action(element);
             }
        }
    }
}

Теперь вы можете назвать это так: 

rowsAndColumns.ForEach(e => Console.WriteLine(e));
...