Ошибка в коде вывода двухмерного массива через функию

@Igoryoka · Регистрация: 17.03.2017

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

Вот код

C++

#include <iostream>
#include <ctime>
 
void Print_Arr(int *ptr_arr, const int size = 5, const int n = 4);
 
void main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    const int size = 5;
    const int n = 4;
    int arr[size][n] =
    {
        { 1, 2, 3, 4 },
        { 6, 7, 8, 9 },
        { 11, 12, 13, 14 }
    };
    int *ptr_arr = arr[size];
}
 
void Print_Arr(int *ptr_arr, const int size = 5, const int n = 4)
{
    for (int i = 0; i < size; i++)
    {
        for (int q = 0; q < n; q++)
        {
            std::cout << *ptr_arr[i][q] << "\t";
        }
        std::cout << std::endl;
    }
}

На этапе

C++

1	std::cout << *ptr_arr[i][q] << "\t";

Выдаёт ошибку Expression must have pointer to object type подсвечивая [q]
Где я ошибся?

@_stanislav · 20.08.2018, 13:04

Igoryoka, вот сравни со своим кодом

C++

#include <iostream>
#include <ctime>
 
void Print_Arr(int **ptr_arr, const int, const int);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    const int size = 5;
    const int n = 4;
    int arr[size][n] =
    {
        { 1, 2, 3, 4 },
        { 6, 7, 8, 9 },
        { 11, 12, 13, 14 }
    };
    int *ptr_arr = arr[size];
 
    return 0;
}
 
void Print_Arr(int **ptr_arr, const int size = 5, const int n = 4)
{
    for (int i = 0; i < size; i++)
    {
        for (int q = 0; q < n; q++)
        {
            std::cout << ptr_arr[i][q] << "\t";
        }
        std::cout << std::endl;
    }
}

@TRam_ · 20.08.2018, 13:05

Сообщение от Igoryoka

int *ptr_arr = arr[size];

указатель ведёт вообще не туда, куда вам бы хотелось. Перечитайте раздел "массивы" и разберитесь почему.

Сообщение от Igoryoka

Print_Arr(int *ptr_arr

ptr_arr в данном случае работает как одномерный массив.

Добавлено через 30 секунд
_stanislav, код всё равно работать не будет.

@_stanislav · 20.08.2018, 13:05

синтаксические ошибки поправил, подразумеваю что есть еще и логические, их искать не так тривиально как синтаксические.

@TRam_ · 20.08.2018, 13:11

Если правильно записать ту логику, которую видимо хотел реализовать автор, то она должна быть примерно такой:

C++

#include <iostream>
#include <ctime>
 
void Print_Arr(void *ptr_arr, const int size = 5, const int n = 4);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    const int size = 5;
    const int n = 4;
    int arr[size][n] =
    {
        { 1, 2, 3, 4 },
        { 6, 7, 8, 9 },
        { 11, 12, 13, 14 }
    };
    void *ptr_arr = arr; 
    Print_Arr(ptr_arr);
    return 0;
}
 
void Print_Arr(void *ptr_arr, const int size, const int n)
{
    int (*converted_arr)[n] = reinterpret_cast< int (*)[n] >(ptr_arr);
    
    for (int i = 0; i < size; i++)
    {
        for (int q = 0; q < n; q++)
        {
            std::cout << converted_arr[i][q] << "\t";
        }
        std::cout << std::endl;
    }
}

@_stanislav · 20.08.2018, 13:12

C++

#include <iostream>
#include <ctime>
 
void Print_Arr(int **ptr_arr, const int size = 5, const int n = 4);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    const int size = 5;
    const int n = 4;
    int** arr = new int*[size]
    {
        new int[n] { 1, 2, 3, 4 },
        new int[n] { 6, 7, 8, 9 },
        new int[n] { 11, 12, 13, 14 },
        new int[n] { 6, 7, 8, 9 },
        new int[n] { 11, 12, 13, 14 }
    };
    Print_Arr(arr);
 
    return 0;
}
 
void Print_Arr(int **ptr_arr, const int size, const int n)
{
    for (int i = 0; i < size; i++)
    {
        for (int q = 0; q < n; q++)
        {
            std::cout << ptr_arr[i][q] << "\t";
        }
        std::cout << std::endl;
    }
}

@TRam_ · 20.08.2018, 13:13

Хотя такая логика сама по себе не особо корректная.

@Igoryoka · 20.08.2018, 13:14 **[ТС]**

Сообщение от TRam_

указатель ведёт вообще не туда, куда вам бы хотелось. Перечитайте раздел "массивы" и разберитесь почему.

TRam_, а почему массивы? Я так понял ошибка в указателях

@TRam_ · 20.08.2018, 13:21

Igoryoka, в двумерном массиве нету указателей (кроме разве что указателя на начало массива), там непрерывно идут числа элементов массива. Указатели будут только в коде как у _stanislav.

Добавлено через 4 минуты
Если очень нужно с указателем, то можно вот так

C++

#include <iostream>
#include <ctime>
 
void Print_Arr(int *ptr_arr, const int size = 5, const int n = 4);
 
int main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    const int size = 5;
    const int n = 4;
    int arr[size][n] =
    {
        { 1, 2, 3, 4 },
        { 6, 7, 8, 9 },
        { 11, 12, 13, 14 }
    };
    int *ptr_arr = reinterpret_cast<int *>(arr);
    Print_Arr(ptr_arr);
}
 
void Print_Arr(int *ptr_arr, const int size, const int n)
{
    for (int i = 0; i < size; i++)
    {
        for (int q = 0; q < n; q++)
        {
            std::cout << ptr_arr[n*i + q] << "\t";    // вот именно так в двумерном массиве находится нужный элемент
        }
        std::cout << std::endl;
    }
}

но это опять же криво, зато объясняет что в двумерном массиве находится

@IP_TCP · 20.08.2018, 14:06

Если ты хотел вывести через функцию двумерный массив, то вот мое решение. В ней же и заполнение массива.
using namespace std;

C++

void Print_Arr(int *ptr_arr[], const int size, const int n)
{
    for (int i = 0; i < size; i++)
    {
        for (int q = 0; q < n; q++)
        {
            ptr_arr[i][q] = 1 + rand() % 20;
            cout << ptr_arr[i][q] << "\t";
        }
            cout << endl;
    }
}
 
int main()
{
    setlocale(LC_ALL, "Russian");
    
    const int size = 3;
    const int n = 4;
    
    int **arr = new int *[size];
    for (int i = 0; i < size; i++)
    {
        arr[i] = new int[n];
    }
 
    Print_Arr(arr, size, n);
 
    delete[] arr;
 
    system("pause");
    return 0;
}

@TRam_ · 20.08.2018, 14:32

IP_TCP, решение у тебя с ошибкой - есть утечка памяти

@rat0r · 20.08.2018, 15:19

Сообщение от TRam_

Если очень нужно с указателем, то можно вот так

Нельзя, это UB.

@IP_TCP · 20.08.2018, 15:27

Сообщение от TRam_

IP_TCP, решение у тебя с ошибкой - есть утечка памяти

Еще какая) Исправил.

C++

    const int size = 3;
    const int n = 3;
 
    int **arr = new int *[size];
    for (int i = 0; i < size; i++)
    {
        arr[i] = new int[n];
    }
 
    Print_Arr(arr, size, n);
 
    for (int i = 0; i < n; i++)
    {
        delete[] arr[i];
    }
 
    delete[] arr;

@TRam_ зомбяк 1584 / 1218 / 345 Регистрация: 14.05.2017 Сообщений: 3,939
	20.08.2018, 13:05	3
	Сообщение от Igoryoka int ptr_arr = arr[size]; указатель ведёт вообще не туда, куда вам бы хотелось. Перечитайте раздел "массивы" и разберитесь почему. Сообщение от Igoryoka* Print_Arr(int ptr_arr ptr_arr в данном случае работает как одномерный массив. Добавлено через 30 секунд* _stanislav, код всё равно работать не будет. 0

@_stanislav "C with Classes" 1646 / 1403 / 523 Регистрация: 16.08.2014 Сообщений: 5,877 Записей в блоге: 1
	20.08.2018, 13:05	4
	синтаксические ошибки поправил, подразумеваю что есть еще и логические, их искать не так тривиально как синтаксические. 0

@TRam_ зомбяк 1584 / 1218 / 345 Регистрация: 14.05.2017 Сообщений: 3,939
	20.08.2018, 13:13	7
	Хотя такая логика сама по себе не особо корректная. 0

@Igoryoka 0 / 0 / 0 Регистрация: 17.03.2017 Сообщений: 20
	20.08.2018, 13:14 [ТС]	8
	Сообщение от TRam_ указатель ведёт вообще не туда, куда вам бы хотелось. Перечитайте раздел "массивы" и разберитесь почему. TRam_, а почему массивы? Я так понял ошибка в указателях 0

@TRam_ зомбяк 1584 / 1218 / 345 Регистрация: 14.05.2017 Сообщений: 3,939
	20.08.2018, 14:32	11
	IP_TCP, решение у тебя с ошибкой - есть утечка памяти 0

@rat0r 285 / 176 / 21 Регистрация: 16.02.2018 Сообщений: 666
	20.08.2018, 15:19	12
	Сообщение от TRam_ Если очень нужно с указателем, то можно вот так Нельзя, это UB. 0