Не работает команда

Вроде бы всё правильно сделал, но не работает команда "Получение матрицы" Вот код Файл: Spisok.h
#pragma once
class Spisok
{
    int size1;
    int size2;
    int **mas;
 
public:
    Spisok(int,int);
    ~Spisok();
    int get(int, int); // выводим значений отдельных элементов
    void get(); // выводим матрицу
    void sort1(int); //сортировка по номеру строки
    void sort2(int); //сортировка по номеру столбца
};
Файл: Spisok.cpp
#include "stdafx.h"
#include "Spisok.h"
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include<iostream>
#include<conio.h>
 
Spisok::Spisok(int n, int m)
    {
        size1 = n;
        size2 = m;
        mas = new int*[size1];
        for (int i = 0; i < size1; i++)
            mas[i] = new int[size2];
 
        srand(time(NULL));
 
        for (int i = 0; i < size1; i++)
            for (int j = 0; j < size2; j++)
                mas[i][j] = rand() % 100;
} //инициализация поля и заполнение его рандомными числами
Spisok::~Spisok()
{
}
int Spisok::get(int n, int m)
{
    return mas[n][m];
}//вывод определённой ячейки
void Spisok::get()
{
    for (int i = 0; i < size1; i++)
    {
        std::cout <<"\n";
        for (int j = 0; j < size2; j++)
            std::cout <<mas[i][j]<<"\t";
    }
}//вывод матрицы
void Spisok::sort1(int n)
{
    for (int i = 0; i < size2 - 1; i++)
    {
        if (mas[n][i] > mas[n][i + 1])
        {
            int z;
            z = mas[n][i];
            mas[n][i] = mas[n][i + 1];
            mas[n][i + 1] = z;
            i = -1;
        }
    }
}//сортировка по номеру строки
void Spisok::sort2(int n)
{
    for (int i = 0; i < size1 - 1; i++)
    {
        if (mas[i][n] > mas[i + 1][n])
        {
            int z;
            z = mas[i][n];
            mas[i][n] = mas[i + 1][n];
            mas[i + 1][n] = z;
            i = -1;
        }
    }
}//сортировка по номеру столбца
Основной файл:
// Виселица.cpp: определяет точку входа для консольного приложения. 
// 
 
#include "stdafx.h" 
#include "Spisok.h" 
#include <iostream> 
#include <stdio.h> 
#include <locale.h> 
#include <windows.h> 
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "Russian");
    int i = 1, n = 1, m = 1;
    Spisok T(n, m);
    while (i != 0)
    {
        cout << endl << "Напишите что вы хотите сделать";
        cout << endl << "1)Cоздать матрицу";
        cout << endl << "2)Получить число по положению";
        cout << endl << "3)Получить матрицу";
        cout << endl << "4)Отсортировать по строке";
        cout << endl << "5)Отсортировать по столбцу";
        cout << endl << "6)Завершить программу" << endl;
        int b;
        cin >> b;
        switch (b)
        {
        case 1:
        {
            cout << endl << "Введите размеры списка\n";
            int n, m;
            cin >> n >> m;
            Spisok T(n, m);
            break;
        }
        case 2:
        {
            cout << endl << "Введите положение\n";
            int n, m;
            cin >> n >> m;
            cout << "\nЧисло в этом положении =" << T.get(n, m);
            break;
        }
        case 3:
        {
 
            T.get();
            break;
        }
        case 4:
        {
            cout << endl << "Введите номер строки \n";
            int z;
            cin >> z;
 
            T.sort1(z);
            break;
        }
        case 5:
        {
            cout << endl << "Введите номер столбца\n";
            int z;
            cin >> z;
            T.sort2(z);
            break;
        }
        case 6:
        {
            i = 0; break;
        }
        default:
        {
            cout << "Такой ф-ции нет, попробуйте снова";
            break;
        }
        }
    }
    return 0;
}
И дада, мой препод требует, чтобы программа выглядела именно так (3 файла) Помогите, люди добрые)
12 ответов

Ошибка кроется в
int i = 1, n = 1, m = 1;
Spisok T(n, m);
и
case 1:
{
  cout << endl << "Введите размеры списка\n";
  int n, m;
  cin >> n >> m;
  Spisok T(n, m);
  break;
}
И деструктор проверьте )


А можно немножечко поконкретне)


hostel656, в самом начале функции main() вы создаете Spisok T(1, 1). Далее вы switch/case создаете еще один объект Spisok T(n, m). Это два разных объекта. Вам нужно работать с объектом, который был уже создан.


Да, но дело в том, что при его удалении - программа тоже не работает.


Самый простецкий вариант исправления выглядит так:
setlocale(LC_ALL, "Russian");
int i = 1, n = 1, m = 1;
Spisok *T;
case 1:
{
  cout << endl << "Введите размеры списка\n";
  int n, m;
  cin >> n >> m;
  T = new Spisok(n, m);
  break;
}
Не забудьте потом про delete. И вы обрабатываете ситуацию, когда например, пользователь хочет получить матрицу без ее создания?


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


hostel656,
Кстати говоря, у меня теперь пишет, что выражение должно иметь тип класса
По конкретнее можно?И да, в предыдущем посте я немного ошибся, лучше писать так:
Spisok *T = NULL;
Если компилятор поддерживает С++11, то можно так:
Spisok *T = nullptr;


в деструкторе я добавил: "delete[] mas;"


hostel656, вместо T.func() пишите T->func(), T - уже указатель.Почитайте, как правильно освобождать память из под двумерного динамического массива. Псевдокод:
for (...)
{
  delete [] ...;
}
delete [] ...;


и снова ошибка


Spisok.cpp
Кликните здесь для просмотра всего текста
#include "stdafx.h"
#include "Spisok.h"
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include<iostream>
#include<conio.h>
 
Spisok::Spisok(int n, int m)
{
    size1 = n;
    size2 = m;
    mas = new int*[size1];
    for (int i = 0; i < size1; i++)
        mas[i] = new int[size2];
 
    srand(time(NULL));
 
    for (int i = 0; i < size1; i++)
        for (int j = 0; j < size2; j++)
            mas[i][j] = rand() % 100;
} //инициализация поля и заполнение его рандомными числами
Spisok::~Spisok()
{
    for (int i = 0; i < size1; ++i)
    {
        delete[] mas[i];
    }
    delete[] mas;
}
int Spisok::get(int n, int m)
{
    return mas[n - 1][m - 1];
}//вывод определённой ячейки
void Spisok::get()
{
    for (int i = 0; i < size1; i++)
    {
        std::cout << "\n";
        for (int j = 0; j < size2; j++)
            std::cout << mas[i][j] << "\t";
    }
}//вывод матрицы
void Spisok::sort1(int n)
{
    for (int i = 0; i < size2 - 1; i++)
    {
        if (mas[n][i] > mas[n][i + 1])
        {
            int z;
            z = mas[n][i];
            mas[n][i] = mas[n][i + 1];
            mas[n][i + 1] = z;
            i = -1;
        }
    }
}//сортировка по номеру строки
void Spisok::sort2(int n)
{
    for (int i = 0; i < size1 - 1; i++)
    {
        if (mas[i][n] > mas[i + 1][n])
        {
            int z;
            z = mas[i][n];
            mas[i][n] = mas[i + 1][n];
            mas[i + 1][n] = z;
            i = -1;
        }
    }
}//сортировка по номеру столбца
main.cpp
Кликните здесь для просмотра всего текста
// Виселица.cpp: определяет точку входа для консольного приложения. 
// 
 
#include "Spisok.h" 
#include <iostream> 
#include <stdio.h> 
#include <locale.h> 
#include <windows.h> 
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "Russian");
    int i = 1, n = 1, m = 1;
    Spisok *T = nullptr;
    while (i != 0)
    {
        cout << endl << "Напишите что вы хотите сделать";
        cout << endl << "1)Cоздать матрицу";
        cout << endl << "2)Получить число по положению";
        cout << endl << "3)Получить матрицу";
        cout << endl << "4)Отсортировать по строке";
        cout << endl << "5)Отсортировать по столбцу";
        cout << endl << "6)Завершить программу" << endl;
        int b;
        cin >> b;
        switch (b)
        {
        case 1:
        {
            cout << endl << "Введите размеры списка\n";
            int n, m;
            cin >> n >> m;
            T = new Spisok(n, m);
            break;
        }
        case 2:
        {
            cout << endl << "Введите положение\n";
            int n, m;
            cin >> n >> m;
            cout << "\nЧисло в этом положении =" << T->get(n, m);
            break;
        }
        case 3:
        {
            if (T == nullptr)
                std::cout << "error\n";
            else 
                T->get();
            break;
        }
        case 4:
        {
            cout << endl << "Введите номер строки \n";
            int z;
            cin >> z;
 
            T->sort1(z);
            break;
        }
        case 5:
        {
            cout << endl << "Введите номер столбца\n";
            int z;
            cin >> z;
            T->sort2(z);
            break;
        }
        case 6:
        {
            i = 0; break;
        }
        default:
        {
            cout << "Такой ф-ции нет, попробуйте снова";
            break;
        }
        }
    }
    return 0;
}
Проверяйте деструктор, get(n, m). И обратите внимание на case 3, там есть проверка, добавьте ее сами в другие места.


Огромное вам спасибо за помощь. Теперь всё работает!!!