Работа с динамическими массивами

@Виктор_Сен · Регистрация: 01.08.2011

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

Добрый день! Написал программу на си, которая рисует окружность в псевдографике:

C++

// Clocks.cpp : Defines the entry point for the console application.
 
#include "stdafx.h"
#include <iostream>
#include <math.h>
using namespace std;
 
//Рисует круг в псевдографике
void circleDrawing(short int dimX, short int dimY, char **matrix, char color, float minRadius, float maxRadius, short int x, short int y)
{
    for (register short int j=0; j<dimY; j++)
    {
        for (register short int i=0; i<dimX; i++)
        {
            float l=sqrtf(powf((float)x-i,2.0f)+powf((float)y-j,2.0f));
            if ((l>minRadius)&&(l<maxRadius))
            {
                matrix[j][i]=color;             
            }
        }
    }
}
 
//Выводит на экран матрицу символов
void printMatrix(short int dimX, short int dimY, char **matrix)
{
    for (register short int j=0; j<dimY; j++)
    {
        for (register short int i=0; i<dimX; i++)
        {
            cout<<matrix[j][i];
        }
        cout<<"\n";
    }
}
 
int _tmain(int argc, _TCHAR* argv[])
{
    //Размеры экрана
    const short int xMax=79;
    const short int yMax=25;
    
    //Матрица символов
    char **matrix = (char**) new char [yMax][xMax];
 
    //Инициализация матрицы символов
    for (register short int j=0;j<yMax;j++)
    {
        for (register short int i=0;i<xMax;i++)
        {
            matrix[j][i]=' ';//Ошибка времени выполнения
        }
    }
 
    //Рисование окружности
    circleDrawing(xMax,yMax,matrix,'+',10,12,xMax/2+1,yMax/2+1);
 
    //Вывод результата
    printMatrix(xMax,yMax,matrix);
 
    system("pause");
    return 0;
}

Программа компилируется нормально, но при выполнении возникает ошибка: "Unhandled exception at 0x00411894 in Clocks.exe: 0xC0000005: Access violation writing location 0xcdcdcdcd" в строке "matrix[j][i]=' ';". Не подскажите, почему так происходит?

@OstapBender · 08.08.2011, 15:02

в этой теме почитайте про то как создавать 2-d динамический массив
Как объявить двумерный динамический массив
да и забудьте про эту ерунду register short int , const short int - пиши просто int. такая экономия памяти уже лет 15 не в моде, а переменные i и j компилер сам заоптимизирует .

@~~LosAngeles~~ · 08.08.2011, 15:04

Сообщение от Виктор_Сен

char **matrix = (char**) new char [yMax][xMax];

ошибка?

@Виктор_Сен · 08.08.2011, 15:40 **[ТС]**

Вот так работает:

C++

// Clocks.cpp : Defines the entry point for the console application.
 
#include "stdafx.h"
#include <iostream>
#include <math.h>
using namespace std;
 
//Рисует круг в псевдографике
void circleDrawing(int dimX, int dimY, char **matrix, char color, float minRadius, float maxRadius, int x, int y)
{
    for (int j=0; j<dimY; j++)
    {
        for (int i=0; i<dimX; i++)
        {
            float l=sqrtf(powf((float)(x-i),2.0f)+powf((float)(y-j),2.0f));
            if ((l>minRadius)&&(l<maxRadius))
            {
                matrix[j][i]=color;             
            }
        }
    }
}
 
//Выводит на экран матрицу символов
void printMatrix(int dimX, int dimY, char **matrix)
{
    for (int j=0; j<dimY; j++)
    {
        for (int i=0; i<dimX; i++)
        {
            cout<<matrix[j][i];
        }
        cout<<"\n";
    }
}
 
int _tmain(int argc, _TCHAR* argv[])
{
    //Размеры экрана
    const int xMax=79;
    const int yMax=25;
    
    //Матрица символов
    char **matrix = new char *[yMax];
    for (int j=0;j<yMax;j++)
    {
        matrix[j]=new char [xMax];
    }
 
    //Инициализация матрицы символов
    for (int j=0;j<yMax;j++)
    {
        for (int i=0;i<xMax;i++)
        {
            matrix[j][i]=' ';
        }
    }
 
    //Рисование окружности
    circleDrawing(xMax,yMax,matrix,'+',10,12,xMax/2+1,yMax/2+1);
 
    //Вывод результата
    printMatrix(xMax,yMax,matrix);
 
    system("pause");
    return 0;
}

Конечно хотелось записать создание массива более компактно, как в 1 случае, но видимо придётся через цикл.

Добавлено через 8 минут

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

ошибка?

В том то и дело, что компилятор не выдаёт здесь ошибки.

@~~Сыроежка~~ · 08.08.2011, 17:25

Сообщение от Виктор_Сен

Добрый день! Написал программу на си, которая рисует окружность в псевдографике:

C++

// Clocks.cpp : Defines the entry point for the console application.
 
#include "stdafx.h"
#include <iostream>
#include <math.h>
using namespace std;
 
//Рисует круг в псевдографике
void circleDrawing(short int dimX, short int dimY, char **matrix, char color, float minRadius, float maxRadius, short int x, short int y)
{
    for (register short int j=0; j<dimY; j++)
    {
        for (register short int i=0; i<dimX; i++)
        {
            float l=sqrtf(powf((float)x-i,2.0f)+powf((float)y-j,2.0f));
            if ((l>minRadius)&&(l<maxRadius))
            {
                matrix[j][i]=color;             
            }
        }
    }
}
 
//Выводит на экран матрицу символов
void printMatrix(short int dimX, short int dimY, char **matrix)
{
    for (register short int j=0; j<dimY; j++)
    {
        for (register short int i=0; i<dimX; i++)
        {
            cout<<matrix[j][i];
        }
        cout<<"\n";
    }
}
 
int _tmain(int argc, _TCHAR* argv[])
{
    //Размеры экрана
    const short int xMax=79;
    const short int yMax=25;
    
    //Матрица символов
    char **matrix = (char**) new char [yMax][xMax];
 
    //Инициализация матрицы символов
    for (register short int j=0;j<yMax;j++)
    {
        for (register short int i=0;i<xMax;i++)
        {
            matrix[j][i]=' ';//Ошибка времени выполнения
        }
    }
 
    //Рисование окружности
    circleDrawing(xMax,yMax,matrix,'+',10,12,xMax/2+1,yMax/2+1);
 
    //Вывод результата
    printMatrix(xMax,yMax,matrix);
 
    system("pause");
    return 0;
}

Программа компилируется нормально, но при выполнении возникает ошибка: "Unhandled exception at 0x00411894 in Clocks.exe: 0xC0000005: Access violation writing location 0xcdcdcdcd" в строке "matrix[j][i]=' ';". Не подскажите, почему так происходит?

Тема указателей и их связь с массивами достаточно сложная и не всегда интуитивно понятна. В вашем случае, когда вы пишите

C++

1	char matrix = (char) new char [yMax][xMax];

то вы на самом деле создаете следующий указатель

C++

1	char ( *matrix )[xMax];

Поэтому когда выполняется арифметика указателей, как, например, matrix + 1, то это выражение указывает не на следующий элемент вашего массива, как это может показаться, а на следующую строку массива. То есть на самом деле выражение matrix + 1 означает, что к адресу, на который указывает matrix, прибавляете sizeof( char ) * xMax. Отсюда у вас и возникает ошибка выполнения.

Добавлено через 21 минуту
Я в предыдущем своем сообщении написал не то, что следовало бы написать. На самом деле вы имеете указатель на указатель, а потому при арифметике с указателями выражение matrix + 1 указывает на адрес, которы

Добавлено через 4 минуты
Я в предыдущем своем сообщении написал не то, что следовало бы написать. На самом деле вы имеете указатель на указатель, а потому при арифметике с указателями выражение matrix + 1 указывает на адрес, который отстоит от исходного на размер указателя, который обычно на 32-битовых платформах равен 4 байтам.

Добавлено через 2 минуты
Я в предыдущем своем сообщении написал не то, что следовало бы написать. На самом деле вы имеете указатель на указатель, а потому при арифметике с указателями выражение matrix + 1 указывает на адрес, который отстоит от исходного на размер указателя, который обычно на 32-битовых платформах равен 4 байтам.

Добавлено через 11 минут
Вам надо было объявить указатель как

C++

1	char ( *matrix )[ xMax ];

И тогда бы ваш цикл в main по инициализации элементов массива работал бы нормально.

@OstapBender 594 / 532 / 76 Регистрация: 22.03.2011 Сообщений: 1,585
	08.08.2011, 15:02	2
	в этой теме почитайте про то как создавать 2-d динамический массив Как объявить двумерный динамический массив да и забудьте про эту ерунду register short int , const short int - пиши просто int. такая экономия памяти уже лет 15 не в моде, а переменные i и j компилер сам заоптимизирует . 0

@~~LosAngeles~~ Заблокирован
	08.08.2011, 15:04	3
	Сообщение от Виктор_Сен char matrix = (char) new char [yMax][xMax]; ошибка? 0

	08.08.2011, 17:25

Опции темы