Question

所以我写了一个简单的通用矩阵类，遇到了一个我不喜欢我的解决方案的问题，所以我想我会寻求更好的帮助。

考虑这里描述的界面：

public interface IMatrix<T>
{
    void DeleteColumn(int position);
    void DeleteRow(int position);
    // Returns a NEW IMatrix<T>
    IMatrix<T> FromRows(IList<IList<T>> rows);      // would like to remove
    // Returns a NEW IMatrix<T>
    IMatrix<T> FromColumns(IList<IList<T>> columns);// would like to remove
    IList<IList<T>> GetColumns();
    IList<IList<T>> GetRows();
    void InsertColumn(int position);
    void InsertRow(int position);
    void SetValueAt(int row, int column, T value);
}

带扩展程序

public static class MatrixExtensions
{
    /// <summary>
    /// Performs a standard matrix addition
    /// </summary>
    public static IMatrix<T> Add<T>(this IMatrix<T> matrix, IMatrix<T> other, IScalarOperators<T> operators)
    {
        JoinCells<T> joiner = new JoinCells<T>();
        return joiner.Join(matrix, other, null, operators.OperatorAdd);
    }

    /// <summary>
    /// Adds a row to the end of the matrix
    /// </summary>
    public static void AddRow<T>(this IMatrix<T> matrix);

    /// <summary>
    /// Adds a number of rows to the end of the matrix
    /// </summary>
    public static void AddRows<T>(this IMatrix<T> matrix, int rows);

    /// <summary>
    /// Adds a column to the end of the matrix
    /// </summary>
    public static void AddColumn<T>(this IMatrix<T> matrix);

    /// <summary>
    /// Adds a number of columns to the end of the matrix
    /// </summary>
    public static void AddColumns<T>(this IMatrix<T> matrix, int columns);

    /// <summary>
    /// Gets the column at the specified position
    /// </summary>
    public static IList<T> ColumnAt<T>(this IMatrix<T> matrix, int position);

    /// <summary>
    /// Gets the number of columns in the matrix
    /// </summary>
    public static int ColumnCount<T>(this IMatrix<T> matrix);

    /// <summary>
    /// Sets the number of columns in the matrix
    /// </summary>
    public static void ColumnCount<T>(this IMatrix<T> matrix, int columns);

    /// <summary>
    /// Deletes the last column from the matrix
    /// </summary>
    public static void DeleteLastColumn<T>(this IMatrix<T> matrix);

    /// <summary>
    /// Deletes the last row from the matrix
    /// </summary>
    public static void DeleteLastRow<T>(this IMatrix<T> matrix);

    /// <summary>
    /// Gets the value at the specified position in the matrix
    /// </summary>
    public static T GetValueAt<T>(this IMatrix<T> matrix, int row, int column);

    /// <summary>
    /// Multiplies this matrix with the other matrix and returns the result
    /// </summary>
    public static IMatrix<T> Multiply<T>(this IMatrix<T> matrix, IMatrix<T> other, IVectorOperators<T> vectorOperators, IScalarOperators<T> scalarOperators)
    {
        JoinRowColumn<T> joiner = new JoinRowColumn<T>();
        return joiner.Join(matrix, other, vectorOperators.OperatorAdd, scalarOperators.OperatorMultiply);
    }

    /// <summary>
    /// Gets the row at the specified position
    /// </summary>
    public static IList<T> RowAt<T>(this IMatrix<T> matrix, int position);

    /// <summary>
    /// Gets the number of rows in the matrix
    /// </summary>
    public static int RowCount<T>(this IMatrix<T> matrix);

    /// <summary>
    /// Sets the number of rows in the matrix
    /// </summary>
    public static void RowCount<T>(this IMatrix<T> matrix, int rows);
}

考虑乘法方法。乘以IMatrix对象的结果是众所周知的。为简单起见，仅考虑Matrix的整数实现。为了计算结果，除了Multiply（int，int）和Add（int，int）如何工作之外，我们不需要了解矩阵的任何信息。由于它们都是已知的，因此我不需要任何其他东西来返回具有该结果的新矩阵。但是，我不确定这样做的最佳方式。

我的方法是将两个方法FromRows和FromColumns添加到接口。这似乎是错误的，因为我不应该以这种特定的方式强制构造矩阵（或者我觉得）。但是，这是我能弄清楚如何返回此接口实例的唯一方法。我将使用IList在joiner类中构建矩阵，并确保该集合是行或列定义，然后使用FromRows方法。也许这将通过一个例子更有意义：

/// <summary>
/// Class used for joining by combining rows and columns
/// </summary>
/// <typeparam name="T">
/// Type of the values contained in the matrix
/// </typeparam>
class JoinRowColumn<T> : IJoinMatrix<T>
{
    public IMatrix<T> Join(IMatrix<T> a, IMatrix<T> b, IOperateVector<T> vectorOperation, IOperateScalar<T> cellOperation)
    {
        // ensure that the matricies can be joined
        if (a.ColumnCount() != b.RowCount())
        {
            throw new ArgumentException("Cannot join matricies.  Invalid dimensions");
        }

        IList<IList<T>> rowDefinition = IMatrixHelpers.GetRowDefinition<T>(a.RowCount(), b.ColumnCount());
        for (int row = 0; row < a.RowCount(); row++)
        {
            IList<T> aRow = a.RowAt(row);
            for (int col = 0; col < b.ColumnCount(); col++)
            {
                IList<T> bCol = b.ColumnAt(col);
                rowDefinition[row][col] = vectorOperation.Operate(aRow, bCol, cellOperation);
            }
        }
        // I do not like this because it is unclear that the
        // method is returning a NEW instance of IMatrix<T>
        // based on the row definition.  It does not update
        // a to contain the matrix defined by rowDefinition
        return a.FromRows(rowDefinition); // UGLY!
    }
}

所以在方法的最后，我使用给我的一个矩阵来生成一个（可能）相同类型的新矩阵（尽管对于具体实现，矩阵返回的内容没有限制）。有一部分问题; FromRows返回一个新实例。然而，这并不明显，人们可能会认为它正在更新调用该方法的矩阵。

是否有更好的模式可以用来构建接口的具体实现？或者这种方法看起来好吗？

我只是熟悉仿制药，所以如果我没有看到明显的东西，请耐心等待。

Answer 1

在您的界面上包含一个名为Construct(int xDimension, int yDimension)的方法，并返回一个新的实例
设计在这种情况下使用的默认实现。在编写接口编码时，没有人应该采用特定的实现方式。

就个人而言，我会选择第二种选择。无论如何，您正在编写接口编码，实现并不重要。您可以轻松返回矩阵的默认实现，并且调用者将能够使用它。另外，您应该考虑将其用于其他方法 - 而不是操纵传入的矩阵，创建一个新的并操纵它。

这与LINQ的工作方式类似，可以防止bug在途中潜入。如果要操作当前对象，则不需要扩展方法。

返回接口类型的扩展方法

1 个答案: