Ошибка на этапе выполнения при том, что программа выполняется до конца

@Jeironica · Регистрация: 27.02.2013

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

Есть матрица с заданным кол-вом столбцов и строк, которую надо сдвинуть, на выбор, во все 4 направления и наискосок. Программа, в принципе, работает, но часть, сдвигающая строку вправо, выдаёт ошибку, хотя даже последняя строка программы выполняется. И это удивительно. Вот урезанная программа с работающим без сбоев кодом и со сбоями (при выборе варианта 1 идёт сбой, а при 2 - всё впорядке). Help...

А, j сделать равным не size-1-num, a size-2-num, то ошибки не выдаётся, но последний столбец оказывается незаполненным. Я в полнейшем замешательстве.

C++

#include <iostream>
using namespace std;
 
void main ()
{
int size;
int size2;
 
cout<<"Vvedite kolvo strok: ";
cin>>size;
cout<<"\nVvedite kolvo stolbcov: ";
cin>>size2;
cout<<endl;
 
int **pa=new int* [size];
for (int i=0; i<size; i++)
pa[i]=new int [size2];
 
for (int i=0; i<size; i++)
{
for (int j=0; j<size2; j++)
{
pa[i][j]=i+j;
cout<<pa[i][j]<<"\t";
}
cout<<endl;
}
cout<<endl;
 
cout<<"Sdvinut vpravo - 1, po vniz - 2, po diagonali sleva sverhu vpravo vniz- 3, vlevo - 4, vverh - 5, po diagonali sprava sverhu nalevo vniz - 6, po diagonali sleva snizu napravo vverh - 7, po diagonali sprava snizu nlevo vverh - 8"<<endl;
int sdvig;
cin>>sdvig;
int num;
cout<<"\nVvedite kolichestvo yacheek, na kotoriye nado sdvinut matricu. Esli obratno, to chislo dologno byt otricatelnim"<<endl;
cin>>num;
cout<<endl;
 
if (sdvig==1)
{
    size2+=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=size2-1+num; j>=0; j--)
    {
    pa[i][j+num]=pa[i][j];
    pa[i][j]=0;
 
    //cout<<j<<"\t"<<pa[i][j+num]<<"\t"<<j+num<<"\t"<<i<<endl;
    }
    }
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
    //cout<<"La-la"<<endl;
}
else if (sdvig==2)
{
 
    size+=num;
    for (int i=size-1; i>size-1-num; i--)
    pa[i]=new int[size2];               //Добавление новых строк.
 
    for (int i=size-1; i>=num; i--)
    {                               //Сдвиг значений вниз.
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i-num][j];
    pa[i-num][j]=0;
    }
    }
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
cout<<"Bla-Bla"<<endl;
}
 
[size="1"][color="grey"][I]Добавлено через 49 секунд[/I][/color][/size]
Вот на всякий случай полный код. Везде, где присутствует это фрагмент, идёт ошибка.
 
#include <iostream>
using namespace std;
 
void main ()
{
int size;
int size2;
 
cout<<"Vvedite kolvo strok: ";
cin>>size;
cout<<"\nVvedite kolvo stolbcov: ";
cin>>size2;
cout<<endl;
 
int **pa=new int* [size];
for (int i=0; i<size; i++)
pa[i]=new int [size2];
 
for (int i=0; i<size; i++)
{
for (int j=0; j<size2; j++)
{
pa[i][j]=i+j;
cout<<pa[i][j]<<"\t";
}
cout<<endl;
}
cout<<endl;
 
cout<<"Sdvinut vpravo - 1, po vniz - 2, po diagonali sleva sverhu vpravo vniz- 3, vlevo - 4, vverh - 5, po diagonali sprava sverhu nalevo vniz - 6, po diagonali sleva snizu napravo vverh - 7, po diagonali sprava snizu nlevo vverh - 8"<<endl;
int sdvig;
cin>>sdvig;
int num;
cout<<"\nVvedite kolichestvo yacheek, na kotoriye nado sdvinut matricu. Esli obratno, to chislo dologno byt otricatelnim"<<endl;
cin>>num;
cout<<endl;
 
if (sdvig==1)
{
    size2+=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=size2-1+num; j>=0; j--)
    {
    pa[i][j+num]=pa[i][j];
    pa[i][j]=0;
 
    //cout<<j<<"\t"<<pa[i][j+num]<<"\t"<<j+num<<"\t"<<i<<endl;
    }
    }
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
    //cout<<"La-la"<<endl;
}
else if (sdvig==2)
{
 
    size+=num;
    for (int i=size-1; i>size-1-num; i--)
    pa[i]=new int[size2];               //Добавление новых строк.
 
    for (int i=size-1; i>=num; i--)
    {                               //Сдвиг значений вниз.
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i-num][j];
    pa[i-num][j]=0;
    }
    }
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
 
else if (sdvig==3)
{
    size+=num;
    for (int i=size-1; i>size-1-num; i--)
    pa[i]=new int[size2];               //Добавление новых строк.
 
    for (int i=size-1; i>=num; i--)
    {                               //Сдвиг значений вниз.
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i-num][j];
    pa[i-num][j]=0;
    }
    }
 
 
        size2+=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=size2-1-num; j>=0; j--)
    {
    pa[i][j+num]=pa[i][j];
    pa[i][j]=0;
    }
    }
 
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
else if (sdvig==4)
{
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2-num; j++)
    {
    pa[i][j]=pa[i][j+num];
 
    }
    }
    size2-=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
else if (sdvig==5)
{
    for (int i=0; i<size-num; i++)
    {
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i+num][j];
    }
    }
    size-=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
else if (sdvig==6)
{
    size+=num;
    for (int i=size-1; i>size-1-num; i--)
    pa[i]=new int[size2];               //Добавление новых строк.
 
    for (int i=size-1; i>=num; i--)
    {                               //Сдвиг значений вниз.
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i-num][j];
    pa[i-num][j]=0;
    }
    }
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2-num; j++)
    {
    pa[i][j]=pa[i][j+num];
 
    }
    }
    size2-=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
else if (sdvig==7)
{
    size2+=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=size2-1+num; j>=0; j--)
    {
    pa[i][j+num]=pa[i][j];
    pa[i][j]=0;
    }
    }
 
    for (int i=0; i<size-num; i++)
    {
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i+num][j];
    }
    }
    size-=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
else if (sdvig==8)
{
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2-num; j++)
    {
    pa[i][j]=pa[i][j+num];
 
    }
    }
    size2-=num;
 
    for (int i=0; i<size-num; i++)
    {
    for (int j=0; j<size2; j++)
    {
    pa[i][j]=pa[i+num][j];
    }
    }
    size-=num;
 
    for (int i=0; i<size; i++)
    {
    for (int j=0; j<size2; j++)
    {
    cout<<pa[i][j]<<"\t";
    }
    cout<<endl;
    }
    cout<<endl;
}
//cout<<"Bla-Bla"<<endl;
}

@Kuzia domovenok · 18.03.2013, 13:55

кто это читать будет, ты подумал?

Croessmah · 18.03.2013, 15:56

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

И это удивительно.

Ничего удивительного. Попортили память - программа упала.

@Jeironica · 20.03.2013, 21:22 **[ТС]**

Croessmah, а как это исправить?

@Kuzia domovenok · 20.03.2013, 22:07

лучше пиши заново, даже если б программа работала, это всё равно есть говнокод.

@Jeironica · 22.03.2013, 21:20 **[ТС]**

Программа работает, но моё сообщение правили и теперь в том коде 2 программы и текст между ними. Разуйте глаза, уважаемый.

@Kuzia domovenok · 22.03.2013, 21:38

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

Разуйте глаза, уважаемый.

как была стена в 350 строк индусского кода так и осталась. Кто это вообще читать должен?
Работает? Очень приятно. Но тебе стоит подумать, как убрать из программы весь копипаст.
Подсказка. Есть такие штуки: называются циклы. Ещё есть такие штуки: функции. Они сильно упрощают программирование, бывает.

@Jeironica · 22.03.2013, 22:35 **[ТС]**

Так вот, если бы вы разули глаза, уважаемый, вы бы заметили, что в начале там есть программа с выделенным неработающим кодом. Советую сначала попробовать, а потом писать более адекватные сообщения. А не фигачить в компилятор 2 порграммы, текст между ними и жаловаться, что что-то там не работает. Естественно.

alsav22 · 23.03.2013, 00:27

Как-то так:

C++

if (sdvig == 1)
{
    size2 += num;
    
    int **temp = new int*[size];
    for (int i = 0; i < size; i++)
        temp[i] = new int [size2];
    
    for (int i = 0; i < size; ++i)
        for (int j = 0; j < size2 - num; ++j)
            temp[i][j] = pa[i][j];
    
    for (int i = 0; i < size; ++i)
        delete [] pa[i];
    delete [] pa;
    
    pa = temp;
                
    for (int i = 0; i < size; i++)
    {
        for (int j = size2 - 1 - num; j >= 0; j--)
        {
            pa[i][j + num] = pa[i][j];
            pa[i][j] = 0;
        }
    }
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j < size2; j++)
        {
            cout << pa[i][j] << "\t";
        }
        cout << endl;
    }
    cout << endl;
    //cout<<"La-la"<<endl;
}

alsav22 · 24.03.2013, 01:37

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

@Jeironica · 28.03.2013, 00:07 **[ТС]**

Хм, тогда у меня всё-таки есть несколько вопросов )

1. Почему при pa=temp pa не изменило свои значения?
2. Я так понимаю, для изменения размерности массива недостаточно просто изменить size и size2? Нужен ещё и второй массив? Но тогда почему это прокатило в условии, равном 2м??

Добавлено через 3 минуты
Хм, тогда у меня всё-таки есть несколько вопросов )

1. Почему при pa=temp pa не изменило свои значения?
2. Я так понимаю, для изменения размерности массива недостаточно просто изменить size и size2? Нужен ещё и второй массив? Но тогда почему это прокатило в условии, равном 2м??

alsav22 · 28.03.2013, 00:31

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

1. Почему при pa=temp pa не изменило свои значения?

Откуда видите, что не изменило? Под значением pa, что имеете ввиду? Адрес?

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

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

Но тогда почему это прокатило в условии, равном 2м??

Даже если и прокатило, то это не значит, что всё было правильно сделано.

@Jeironica · 29.03.2013, 04:00 **[ТС]**

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

Откуда видите, что не изменило? Под значением pa, что имеете ввиду? Адрес?

cout видит )

Добавлено через 2 минуты
Количество столбцов изменилось, но не значения первых 5ти. Вот мне и интересно, как это вышло...

@Kuzia domovenok · 29.03.2013, 04:01

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

cout видит )

а больше, чем два слова ответить сложно? alsav задал нужный вопрос, потому что есть подозрение, что ты вообще не понимаешь, что за значение там у тебя изменилось при выполнении pa=temp

alsav22 · 29.03.2013, 04:14

C++

#include <iostream>
using namespace std;
 
int main()
{
 
int size;
int size2;
 
cout<<"Vvedite kolvo strok: ";
cin>>size;
cout<<"\nVvedite kolvo stolbcov: ";
cin>>size2;
cout<<endl;
 
int **pa=new int* [size];
for (int i=0; i<size; i++)
pa[i]=new int [size2];
 
for (int i=0; i<size; i++)
{
for (int j=0; j<size2; j++)
{
pa[i][j]=i+j;
cout<<pa[i][j]<<"\t";
}
cout<<endl;
}
cout<<endl;
 
cout<<"Sdvinut vpravo - 1, po vniz - 2, po diagonali sleva sverhu vpravo vniz- 3, vlevo - 4, vverh - 5, po diagonali sprava sverhu nalevo vniz - 6, po diagonali sleva snizu napravo vverh - 7, po diagonali sprava snizu nlevo vverh - 8"<<endl;
int sdvig;
cin>>sdvig;
int num;
cout<<"\nVvedite kolichestvo yacheek, na kotoriye nado sdvinut matricu. Esli obratno, to chislo dologno byt otricatelnim"<<endl;
cin>>num;
cout<<endl;
 
if (sdvig == 1)
{
    size2 += num;
    
    int **temp = new int*[size];
    for (int i = 0; i < size; i++)
        temp[i] = new int [size2];
    
    for (int i = 0; i < size; ++i)
        for (int j = 0; j < size2 - num; ++j)
            temp[i][j] = pa[i][j];
    
    for (int i = 0; i < size; ++i)
        delete [] pa[i];
    delete [] pa;
    
    pa = temp;
                
    for (int i = 0; i < size; i++)
    {
        for (int j = size2 - 1 - num; j >= 0; j--)
        {
            pa[i][j + num] = pa[i][j];
            pa[i][j] = 0;
        }
    }
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j < size2; j++)
        {
            cout << pa[i][j] << "\t";
        }
        cout << endl;
    }
    cout << endl;
    //cout<<"La-la"<<endl;
}
 
    system("pause");
    return 0;
}

alsav22 · 29.03.2013, 04:20

Это к тому, что видит cout:

alsav22 · 29.03.2013, 04:51

Проще это так сделать:

C++

if (sdvig == 1)
{
    size2 += num;
    
    int **temp = new int*[size];
    for (int i = 0; i < size; i++)
    {
        temp[i] = new int [size2];
        for (int j = 0; j < size2; ++j)
            temp[i][j] = 0;
    }
 
    for (int i = 0; i < size; ++i)
        for (int j = num, k = 0; j < size2; ++j, ++k)
            temp[i][j] = pa[i][k];
 
    for (int i = 0; i < size; ++i)
        delete [] pa[i];
    delete [] pa;
    
    pa = temp;
                
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j < size2; j++)
        {
            cout << pa[i][j] << "\t";
        }
        cout << endl;
    }
    cout << endl;
 }

@Igor3D · 29.03.2013, 09:41

Jeironica, Вы усложнили себе задачу - ну раз так в 10

Если готовы забыть написанный код как страшный сон - поговорим как сделать по уму. А нет, хотите все-таки домучить кошку - не будем тратить время.

@Jeironica · 30.03.2013, 04:30 **[ТС]**

Сообщение от Kuzia domovenok

а больше, чем два слова ответить сложно? alsav задал нужный вопрос, потому что есть подозрение, что ты вообще не понимаешь, что за значение там у тебя изменилось при выполнении pa=temp

Если вы не понимаете смысла моих слов, это характеризует вас далеко не с лучшей стороны.)
Кстати, это уже второй раз, когда вы не въезжаете. И просто ляпаете в теме, невпопад. Это что, желание общаться? оО

alsav22, до меня дошло, наконец-то... Вы вначале перенесли значения из pa в temp, потом стерли ра, а потом заново записали туда. Именно поэтому ра до перемещения столбцов, сразу после приравнивания его с temp, имеет те же значения, что и раньше. Да-да, иногда я торможу.)

C++

cout - вот этот вот, если вставить его вразу после pa=temp;
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j < size2; j++)
        {
            cout << pa[i][j] << "\t";
        }
        cout << endl;
    }
    cout << endl;

Результат вывода на рисунке во вложении. 1я таблица - это ра, вторая - ра после приравнения с temp, а третья - конечный результат.
Объяснять уже ничего не надо - я всё поняла... спасибо за помощь, alsav22).

alsav22 · 30.03.2013, 05:58

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

Именно поэтому ра до перемещения столбцов, сразу после приравнивания его с temp, имеет те же значения, что и раньше.

Я почему и уточнил про адрес:

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

Под значением pa, что имеете ввиду? Адрес?

Адрес, естественно, сразу менялся, значения по адресу - после записи со смещением. В последнем коде всё упрощено: до присваивания pa = temp всё уже записано со смещением.

@Jeironica 0 / 0 / 0 Регистрация: 27.02.2013 Сообщений: 16
	28.03.2013, 00:07 [ТС]	11
	Хм, тогда у меня всё-таки есть несколько вопросов ) 1. Почему при pa=temp pa не изменило свои значения? 2. Я так понимаю, для изменения размерности массива недостаточно просто изменить size и size2? Нужен ещё и второй массив? Но тогда почему это прокатило в условии, равном 2м?? Добавлено через 3 минуты Хм, тогда у меня всё-таки есть несколько вопросов ) 1. Почему при pa=temp pa не изменило свои значения? 2. Я так понимаю, для изменения размерности массива недостаточно просто изменить size и size2? Нужен ещё и второй массив? Но тогда почему это прокатило в условии, равном 2м?? 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	28.03.2013, 00:31	12
	Сообщение от Jeironica 1. Почему при pa=temp pa не изменило свои значения? Откуда видите, что не изменило? Под значением pa, что имеете ввиду? Адрес? Добавлено через 2 минуты Сообщение от Jeironica Но тогда почему это прокатило в условии, равном 2м?? Даже если и прокатило, то это не значит, что всё было правильно сделано. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	29.03.2013, 04:20	16
	Это к тому, что видит cout: Миниатюры 0

@Kuzia domovenok 4064 / 3318 / 924 Регистрация: 25.03.2012 Сообщений: 12,495 Записей в блоге: 1
	18.03.2013, 13:55	2
	кто это читать будет, ты подумал? 0

Croessmah Неэпический 17870 / 10635 / 2054 Регистрация: 27.09.2012 Сообщений: 26,737 Записей в блоге: 1
	18.03.2013, 15:56	3
	Сообщение от Jeironica И это удивительно. Ничего удивительного. Попортили память - программа упала. 1

@Jeironica 0 / 0 / 0 Регистрация: 27.02.2013 Сообщений: 16
	20.03.2013, 21:22 [ТС]	4
	Croessmah, а как это исправить? 0

@Kuzia domovenok 4064 / 3318 / 924 Регистрация: 25.03.2012 Сообщений: 12,495 Записей в блоге: 1
	20.03.2013, 22:07	5
	лучше пиши заново, даже если б программа работала, это всё равно есть говнокод. 0

@Jeironica 0 / 0 / 0 Регистрация: 27.02.2013 Сообщений: 16
	22.03.2013, 21:20 [ТС]	6
	Программа работает, но моё сообщение правили и теперь в том коде 2 программы и текст между ними. Разуйте глаза, уважаемый. 0

@Kuzia domovenok 4064 / 3318 / 924 Регистрация: 25.03.2012 Сообщений: 12,495 Записей в блоге: 1
	22.03.2013, 21:38	7
	Сообщение от Jeironica Разуйте глаза, уважаемый. как была стена в 350 строк индусского кода так и осталась. Кто это вообще читать должен? Работает? Очень приятно. Но тебе стоит подумать, как убрать из программы весь копипаст. Подсказка. Есть такие штуки: называются циклы. Ещё есть такие штуки: функции. Они сильно упрощают программирование, бывает. 0

@Jeironica 0 / 0 / 0 Регистрация: 27.02.2013 Сообщений: 16
	22.03.2013, 22:35 [ТС]	8
	Так вот, если бы вы разули глаза, уважаемый, вы бы заметили, что в начале там есть программа с выделенным неработающим кодом. Советую сначала попробовать, а потом писать более адекватные сообщения. А не фигачить в компилятор 2 порграммы, текст между ними и жаловаться, что что-то там не работает. Естественно. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	24.03.2013, 01:37	10
	Наверное, можно сразу заполнить новый массив нулями и записать старый массив в конец нового. 0

@Jeironica 0 / 0 / 0 Регистрация: 27.02.2013 Сообщений: 16
	29.03.2013, 04:00 [ТС]	13
	Сообщение от alsav22 Откуда видите, что не изменило? Под значением pa, что имеете ввиду? Адрес? cout видит ) Добавлено через 2 минуты Количество столбцов изменилось, но не значения первых 5ти. Вот мне и интересно, как это вышло... 0

@Kuzia domovenok 4064 / 3318 / 924 Регистрация: 25.03.2012 Сообщений: 12,495 Записей в блоге: 1
	29.03.2013, 04:01	14
	Сообщение от Jeironica cout видит ) а больше, чем два слова ответить сложно? alsav задал нужный вопрос, потому что есть подозрение, что ты вообще не понимаешь, что за значение там у тебя изменилось при выполнении pa=temp 0

@Igor3D 1824 / 732 / 99 Регистрация: 01.10.2012 Сообщений: 3,746
	29.03.2013, 09:41	18
	Jeironica, Вы усложнили себе задачу - ну раз так в 10 Если готовы забыть написанный код как страшный сон - поговорим как сделать по уму. А нет, хотите все-таки домучить кошку - не будем тратить время. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	30.03.2013, 05:58	20
	Сообщение от Jeironica Именно поэтому ра до перемещения столбцов, сразу после приравнивания его с temp, имеет те же значения, что и раньше. Я почему и уточнил про адрес: Сообщение от alsav22 Под значением pa, что имеете ввиду? Адрес? Адрес, естественно, сразу менялся, значения по адресу - после записи со смещением. В последнем коде всё упрощено: до присваивания pa = temp всё уже записано со смещением. 1