Как изменить ошибку: «Индекс находится вне границы массива»?

Question

[embiid]

Пожалуйста, помогите пофиксить проблему с методом EnterMatrix.
Именно в:
MATRIX[j, k] = int.Parse(Console.ReadLine());

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

class Matrix {
    int row, column;
    int[,] MATRIX;

    public int ROW {
        get { return row; }
        set { row = value; }
    }

    public int COLUMN {
        get { return column; }
        set { column = value; }
    }

    public Matrix() {

    }

    public Matrix(int row, int column) {
        this.row = ROW;
        this.column = COLUMN;

        MATRIX = new int[this.COLUMN, this.ROW];
    }

    public void EnterMatrix() {
        Console.Write("Enter the numbers of matrix columns: ");
        COLUMN = int.Parse(Console.ReadLine());
        Console.Write("Enter the numbers of matrix rows: ");
        ROW = int.Parse(Console.ReadLine());

        for (int j = 0; j < COLUMN; j++) {
            for (int k = 0; k < ROW; k++) {
                Console.Write("For matrix cell -> " + (j + 1) + ":" + (k + 1) + " - ");
                MATRIX = new int[j, k];
                MATRIX[j, k] = int.Parse(Console.ReadLine());
            }
        }
    }

    public void DisplayMatrix() {
        Console.Write("The matrix is: ");
        for (int j = 0; j < COLUMN; j++) {
            for (int k = 0; k < ROW; k++) {
                Console.WriteLine("{0}\t", MATRIX[j, k]);
            }
        }
    }

    ~Matrix() {
        Console.WriteLine("Matrix has been denied.");
    }
}

class Vector : Matrix {
    public Vector(int row, int column) {
        
    }

    

    ~Vector() {
        Console.WriteLine("Vector has been denied.");
    }
}

class Program {
    static void Main() {
        Matrix MATRIX = new Matrix();
        MATRIX.EnterMatrix();
        MATRIX.DisplayMatrix();
    }
}

Answer 1

[Мария Мальцева]

MATRIX = new int[j, k];
MATRIX[j, k] = int.Parse(Console.ReadLine());

В цикле Вы каждый раз выделяете в памяти новый массив [j, k] - соответственно, от 0 до (j - 1) и от 0 до (k - 1), а потом обращаетесь к ячейке [j, k].
Нужно вынести создание массива за циклы

Console.Write("Enter the numbers of matrix columns: ");
COLUMN = int.Parse(Console.ReadLine());
Console.Write("Enter the numbers of matrix rows: ");
ROW = int.Parse(Console.ReadLine());
MATRIX = new int[COLUMN, ROW];

Comments

[embiid]

public void EnterMatrix() {
        Console.Write("Enter the numbers of matrix columns: ");
        COLUMN = int.Parse(Console.ReadLine());
        Console.Write("Enter the numbers of matrix rows: ");
        ROW = int.Parse(Console.ReadLine());

        Matrix MATRIX = new Matrix(COLUMN, ROW);

        Console.WriteLine("Enter the dimension of the matrix: ");
        for (int col = 0; col < COLUMN; col++) {
            for (int row = 0; row < ROW; row++) {
                MATRIX[COLUMN, ROW] = int.Parse(Console.ReadLine());
            }
        }
    }

Спасибо!
А как сделать так что бы пользователь инициализировал каждый элемент?
Для этого я и сделал цыкл for, но компилятор ругается на MATRIX[COLUMN, ROW] = int.Parse(Console.ReadLine());

[Мария Мальцева]

MATRIX[col, row] = int.Parse(Console.ReadLine());
Меняются переменные col и row, они и должны быть индексами массива

[embiid]

Мария Мальцева, все я разобрался, большее спасибо за подсказки.

public void EnterMatrix() {
        Console.Write("Enter the numbers of matrix columns: ");
        COLUMN = int.Parse(Console.ReadLine());
        Console.Write("Enter the numbers of matrix rows: ");
        ROW = int.Parse(Console.ReadLine());

        MATRIX = new int[COLUMN, ROW];
        
        for (int col = 0; col < COLUMN; col++) {
            for (int row = 0; row < ROW; row++) {
                Console.Write("Enter the element of matrix cell[" + (col + 1) + ":" + (row + 1) + "]: ");
                int.Parse(Console.ReadLine());
            }
        }
    }

Answer 2

[Александр Юдаков]

https://docs.microsoft.com/en-us/dotnet/csharp/pro...

См. 5-й пункт