Неопределенный идентификатор

@rOJIoC 6orA · Регистрация: 09.03.2020

Author24 — интернет-сервис помощи студентам

Написал код, который позволяет вводить размерность матрицы через консоль и заполняет ее рандомными числами.
Единственная ошибка по которой не запускается: "идентификатор "j" не определен. Больше никаких ошибок не выдает. Убедительная просьба, не менять мой код, а лишь идентифицировать "j". Если что меняете, то пожалуйста, распишите подробный комментарий для новичка. Спасибо!

C++

#include <iostream>
#include <ctime>
 
    using namespace std;
 
    int main()
{
    srand(time(0));
    int m, n, **matrix;
    cin >> m >> n;
    cout << endl;
    matrix[0] = new int[m * n];
 
    for (int i = 0; i < m; i++)
    
        {
            for (int j = 0; j < n; j++);
            matrix[i][j] = rand() % 50;
            cout << matrix[i][j] << "\t";
        }
    system("pause");
    return 0;
}

@Annemesski · 11.03.2020, 16:55

C++

#include <iostream>
#include <ctime>
 
using namespace std;
 
int main()
{
srand(time(0));
int m, n, **matrix;
cin >> m >> n;
cout << endl;
matrix[0] = new int[m * n];
 
for (int i = 0; i < m; i++)
 
{
for (int j = 0; j < n; j++){ // вот тут ";" поставил, а надо "{" открыть тело цикла for
matrix[i][j] = rand() % 50;
cout << matrix[i][j] << "\t";
} // а вот тут его же закрыть надобно.
}
system("pause");
return 0;
}

@rOJIoC 6orA · 11.03.2020, 17:11 **[ТС]**

Спасибо за ответ! Я поставил скобки, однако теперь пишет что: "использована неинициализированная локальная переменная "matrix" ". Не подскажете где ошибка прячется?

C++

#include <iostream>
#include <ctime>
 
    using namespace std;
 
    int main()
{
    cout << "Введите размерность матрицы: ";
    srand(time(0));
    int m, n, **matrix;
    cin >> m >> n;
    cout << endl;
    matrix[0] = new int[m * n];
 
    for (int i = 0; i < m; i++)
 
 
 
 
    {
        for (int j = 0; j < n; j++)
        {
            matrix[i][j] = rand() % 50;
            cout << matrix[i][j] << "\t";
        }
    }
    system("pause");
    return 0;
}

@Annemesski · 11.03.2020, 17:22

массив не инициализирован

C++

int n, m;
cin >> n >> m;
// выделяем память под массив
int **matrix = new int*[n];
for (int i = 0; i < n; i++)
    matrix[i] = new int[m];
 
// далее заполняйте массив значениями

@liv · 11.03.2020, 18:14

rOJIoC 6orA, Ваше

C++

1	matrix[0] = new int[m * n];

мне напомнило об вот таком интересном способе построения матрицы

C++

#include <iostream>
#include <ctime>
 
using namespace std;
 
int main()
{
    cout << "Введите размерность матрицы: ";
    srand(time(0));
    int n, m, **matrix;
    cin >> n >> m;
    cout << endl;
 
    matrix = new int*[n];
    matrix[0] = new int[m * n];
 
    for (int i = 1; i < n; i++)
        matrix[i] = matrix[0] + (i * m);
 
    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++)
        {
            matrix[i][j] = rand() % 50;
            cout << matrix[i][j] << "\t";
        }
        cout << endl;
    }
    
    delete[]matrix[0];
    delete[]matrix;
 
    system("pause");
    return 0;
}

	11.03.2020, 18:14