классы

[Dimarik]

добрый вечер! пишу программу по классам, написал 2 реализации причём одна работает некорректно, вторая вообще не пашет.

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

namespace laba1
{
    class matrix
    {
        public int dl1, dl2;
        public int[,] mas;
        public  matrix()//конструктор по умолчанию
        {

        }

        public void vvod()//ввод
        {
            Console.WriteLine("Введите размер массива");
            dl1 = Convert.ToInt16(Console.ReadLine());
            dl2 = Convert.ToInt16(Console.ReadLine());
            Console.WriteLine("Введите массив");
            int[,] mas = new int[dl1, dl2 ];
            for (int i = 0; i < dl1; i++)
            {
                for (int j = 0; j < dl2; j++)
                {

                    mas[i,j] = Convert.ToInt16(Console.ReadLine());
                }

            }
            //вывод
//эта часть кода написана в методе vivod, но здесь она работает нормально, а в отдельном методе - нет
            Console.WriteLine("Вывод");
            for (int i = 0; i < dl1; i++)
            {
                for (int j = 0; j < dl2; j++)
                {

                    Console.WriteLine(mas[i, j]);
                }

            }
        }
      public  void vivod()//вывод
    {
        for (int i = 0; i < dl1; i++)
        {
            for (int j = 0; j < dl2; j++)
            {

                Console.WriteLine(mas[i, j]);
            }

        }
    }
    }
    class Program
    {
        static void Main(string[] args)
        {
            Console.BackgroundColor = ConsoleColor.White;
            Console.Clear();
            Console.ForegroundColor = ConsoleColor.Black;
            Console.WriteLine("Hi!");
            matrix matr1 = new matrix();
            matr1.vvod();
            matr1.vivod();
        }
    }
}

возникает такая ошибка в процессе выполнения: Object reference not set to an instance of an object. не понимаю я что делать. пробовал написать и так
public void vivod()//вывод
{
for (int i = 0; i < dl1; i++)
{
for (int j = 0; j < dl2; j++)
{

Console.WriteLine(this.mas[i, j]);
}

}
}
ну просто судя по ошибке подумал что this нужен, но странно, зачем он нужен. не захотела компилиться. потом я подумал, что всё дело в том, что функция ввода имеет тип vioid. Ну и я подумал сделать малость иначе

public int vvod()//ввод
{
Console.WriteLine("Введите размер массива");
dl1 = Convert.ToInt16(Console.ReadLine()) ;
dl2 = Convert.ToInt16(Console.ReadLine()) ;
Console.WriteLine("Введите массив");
int[,] mas = new int[dl1, dl2 ];
for (int i = 0; i < dl1; i++)
{
for (int j = 0; j < dl2; j++)
{

mas[i,j] = Convert.ToInt16(Console.ReadLine()) ;
//добавил
return mas[i, j];
}

}
//вывод
//эта часть кода написана в методе vivod, но здесь она работает нормально, а в отдельном методе - нет
Console.WriteLine("Вывод");
for (int i = 0; i < dl1; i++)
{
for (int j = 0; j < dl2; j++)
{

Console.WriteLine(mas[i, j]);
return mas[i, j];
}

}
} и тут какая-то ошибка. короче бился я бился, так и не сделал по-нормальному(((

[BOZKURT]

Потому, что глобальная переменная "mas" не инициализирована.
Ты инициализируешь в методе "vvod()" не глобальную (внутри класса "matrix") переменную "mas", а переменную внутри метода "vvod()".
Решение - убери объявление переменной как массив из целых чисел из тела метода, т.е. замени строку :

int[,] mas = new int[dl1, dl2];

на

mas = new int[dl1, dl2];

[Dimarik]

эээээээ... чёт я не догнал разницу для компилятора((((

[Cpluser]

что не работает?Попробуй так

public static int[] mas=new int[n];

[MaTBeu]

Раз уж пишете классы, то пишите их нормально.
Вот рабочий пример программы с матрицей
Класс CMatrix

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

namespace MatrixApplication
{
    class CMatrix
    {
        private int line_number;
        private int column_number;
        public int[,] mas;
        public CMatrix(int lines, int columns) //конструктор принимает размерности матрицы
        {
            if(lines == 0 && columns == 0)
            {
                lines = 10;
                columns = 10;
            }
            line_number = lines;
            column_number = columns;
            mas = new int[line_number, column_number]; //создаем матрицу
        }
        public void Enter() //вводим матрицу
        {
            for (int i = 0; i < line_number; i++)
            {
                for (int j = 0; j < column_number; j++)
                {
                    mas[i, j] = Convert.ToInt32(Console.ReadLine());
                }
            }
        }
        public void Print()  //выводим матрицу
        {
            for (int i = 0; i < line_number; i++)
            {
                for (int j = 0; j < column_number; j++)
                {
                    Console.Write(mas[i, j]);
                    Console.Write(" ");
                }
                Console.Write("\n");
            }
        }
    }
}

Главный класс

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

namespace MatrixApplication
{
    class Program
    {
        static void Main(string[] args)
        {
            int l = 0, c = 0;
            Console.WriteLine("Enter number of lines in the matrix: ");
            l = Convert.ToInt32(Console.ReadLine());
            Console.WriteLine("Enter number of columns in the matrix: ");
            c = Convert.ToInt32(Console.ReadLine());
            CMatrix matrix = new CMatrix(l, c);
            Console.WriteLine("Enter elements of the matrix: ");
            matrix.Enter();
            Console.WriteLine("Elements of the matrix: ");
            matrix.Print();
        }
    }
}

Надеюсь, вы сможете понять в чем разница. Удачи.

[Dimarik]

а! кажется я понял почему всё работало через одно место. всё дело в конструкторе! я его коряво прописал. точнее я его пустым сделал. правильно?

[TaTT DoGG]

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

[BOZKURT]

Dimarik - у меня просто нет слов.
Я тебе явно указал на ошибку.
Если ты не понял разницы между локальным и глобальным переменными, беги из программирования.......

[Dimarik]

почему не понял? я недавно перешёл с с++ на шарп. вот и думаю: так как есть public int[,] mas; то по идее она должна быть видна внутри класса и даже вне его. Хотя авторы многих книг не рекомендуют пользоваться Public так как это небезопасное программирование

[Dimarik]

всё) понял в чём тут фигня! дело в том что я как в с++ просто объявлял переменные и не инициализировал их, а в C# такого делать низя)))