Программа компилируется но не работает

@Alt_Shift · Регистрация: 28.05.2012

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

C++

#include <conio.h>
#include<windows.h>
#include<iostream>
#include<time.h>
using namespace std;
int main()
{
  setlocale(0, "");
    srand(unsigned(time(NULL)));
    int N, SIZE=2,temp;
    cout<<"Введите размер матрицы: ";
    cin >> N;
    int ** matrix = new int * [N];
    for (int i = 0; i < N; i++)
        matrix [i] = new int [SIZE];
 
    for (int i = 0; i < N; i++)
        for (int j = 0; j <SIZE; j++)
            matrix[i][j] = ((rand() % 10));
                        
        
 cout << "Вот Ваша матрица:" << endl << endl;
 for (int i = 0; i < N; i++)
 {
     for(int j = 0; j < SIZE; j++)
         cout << matrix[i][j] << " ";
     cout << endl;
 }
 
 cout << endl;
 
for(int i = 0; i < SIZE; i++)
{
for(int j = 0; j < SIZE - 1; j++)
{
if(matrix[i][j + 1] < matrix[i][j])
{
temp = matrix[i][j + 1];
matrix[i][j + 1] = matrix[i][j];
matrix[i][j] = temp;
}
}
if(matrix[i + 1][0] < matrix[i][SIZE - 1])
{
temp = matrix[i + 1][0];
matrix[i + 1][0] = matrix[i][SIZE - 1];
matrix[i][SIZE - 1] = temp;
}
}
cout << "Вот Ваша матрица:" << endl << endl;
 for (int i = 0; i < SIZE; i++)
 {
     for(int j = 0; j < SIZE; j++)
         cout << matrix[i][j] << " ";
     cout << endl;
 }
  for (int i = 0; i < SIZE; i++)
            delete [] matrix[i];
        delete [] matrix;
}

Создается двумерный динамический массив, показывается и должен сортироваться пузырьком и снова показываться. но вместо этого вылетает ошибка "необработанное исключение"

@Mr. Pyatachok · 27.08.2012, 20:23

добавь задержку system("pause") из библиотеки iostream.h
или
getch() из conio.h

@DaskOFF · 27.08.2012, 20:25

что у вас тут происходит ?
вылетает именно тут, перепроверьте сортировку

C++

for(int i = 0; i < SIZE; i++) {
    for(int j = 0; j < SIZE - 1; j++)   {
        if(matrix[i][j + 1] < matrix[i][j]) {
            temp = matrix[i][j + 1];
            matrix[i][j + 1] = matrix[i][j];
            matrix[i][j] = temp;
        }
    }
    if(matrix[i + 1][0] < matrix[i][SIZE - 1]){
        temp = matrix[i + 1][0];
        matrix[i + 1][0] = matrix[i][SIZE - 1];
        matrix[i][SIZE - 1] = temp;
    }
}

@Alt_Shift · 27.08.2012, 20:29 **[ТС]**

Сообщение от Mr. Pyatachok

добавь задержку system("pause") из библиотеки iostream.h
или
getch() из conio.h

и как это может помочь? через ctrl+F5 тоже нормально задерживается впринципе

@Mr. Pyatachok · 27.08.2012, 20:31

извиняюсь, не до конца прочитал ваше сообщение

@Alt_Shift · 27.08.2012, 20:40 **[ТС]**

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

что у вас тут происходит ?
вылетает именно тут, перепроверьте сортировку

Ну это я и сам вижу,проверял, с статическим массивом работает нормально, с динамикой нет

@DaskOFF · 27.08.2012, 20:43

C++

#include <conio.h>
#include<windows.h>
#include<iostream>
#include<time.h>
using namespace std;
 
int main()
{
  setlocale(0, "");
    srand(unsigned(time(NULL)));
    int N, SIZE=2,temp;
    cout<<"Введите размер матрицы: ";
    cin >> N;
    int ** matrix = new int * [N];
    for (int i = 0; i < N; i++)
        matrix [i] = new int [SIZE];
 
    for (int i = 0; i < N; i++)
        for (int j = 0; j <SIZE; j++)
            matrix[i][j] = ((rand() % 10));
                        
        
 cout << "Вот Ваша матрица:" << endl << endl;
 for (int i = 0; i < N; i++)
 {
     for(int j = 0; j < SIZE; j++)
         cout << matrix[i][j] << " ";
     cout << endl;
 }
 
 cout << endl;
 
for(int i = 0; i < SIZE; i++) {
    for(int j = 0; j < SIZE - 1; j++)   {
        if(matrix[i][j + 1] < matrix[i][j]) {
            temp = matrix[i][j + 1];
            matrix[i][j + 1] = matrix[i][j];
            matrix[i][j] = temp;
        }
    }
    if(matrix[i + 1][0] < matrix[i][SIZE - 1]){
        temp = matrix[i + 1][0];
        matrix[i + 1][0] = matrix[i][SIZE - 1];
        matrix[i][SIZE - 1] = temp;
    }
}
cout << "Вот Ваша матрица:" << endl << endl;
for (int i = 0; i < N; i++){
    for(int j = 0; j < SIZE; j++)
        cout << matrix[i][j] << " ";
    cout << endl;
}
for (int i = 0; i < N; i++)
    delete [] matrix[i];
delete [] matrix;
return 0;
}

@Alt_Shift · 27.08.2012, 20:45 **[ТС]**

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

почему у вас тут SIZE ?

C++

1	for(int i = 0; i < SIZE; i++)

а если массив будет 5 х 2 ?

даже в таком виде

C++

#include <conio.h>
#include<windows.h>
#include<iostream>
#include<time.h>
using namespace std;
int main()
{
  setlocale(0, "");
    srand(unsigned(time(NULL)));
    int SIZE,temp;
    cout<<"Введите размер матрицы: ";
    cin >> SIZE;
    int ** matrix = new int * [SIZE];
    for (int i = 0; i < SIZE; i++)
        matrix [i] = new int [SIZE];
 
    for (int i = 0; i < SIZE; i++)
        for (int j = 0; j <SIZE; j++)
            matrix[i][j] = ((rand() % 10));
                        
        
 cout << "Вот Ваша матрица:" << endl << endl;
 for (int i = 0; i < SIZE; i++)
 {
     for(int j = 0; j < SIZE; j++)
         cout << matrix[i][j] << " ";
     cout << endl;
 }
 
for(int i = 0; i < SIZE; i++)
{
for(int j = 0; j < SIZE - 1; j++)
{
if(matrix[i][j + 1] < matrix[i][j])
{
temp = matrix[i][j + 1];
matrix[i][j + 1] = matrix[i][j];
matrix[i][j] = temp;
}
}
if(matrix[i + 1][0] < matrix[i][SIZE - 1])
{
temp = matrix[i + 1][0];
matrix[i + 1][0] = matrix[i][SIZE - 1];
matrix[i][SIZE - 1] = temp;
}
}
cout << "Вот Ваша матрица:" << endl << endl;
 for (int i = 0; i < SIZE; i++)
 {
     for(int j = 0; j < SIZE; j++)
         cout << matrix[i][j] << " ";
     cout << endl;
 }
  for (int i = 0; i < SIZE; i++)
            delete [] matrix[i];
        delete [] matrix;
}

ошибка та же

@DaskOFF · 27.08.2012, 20:50

пост выше обновил, все работает без ошибок с любым размером
ток он как-то странно сортирует

ValeryS · 27.08.2012, 20:54

во первых поменяй SIZE в некоторых компиляторах она зарезервирована
во вторых

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

SIZE=2,

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

C++

for(int i = 0; i < SIZE; i++)
{
for(int j = 0; j < SIZE - 1; j++)
{
if(matrix[i][j + 1] < matrix[i][j])
{
temp = matrix[i][j + 1];
matrix[i][j + 1] = matrix[i][j];
matrix[i][j] = temp;
}
}
if(matrix[i + 1][0] < matrix[i][SIZE - 1])
{
temp = matrix[i + 1][0];
matrix[i + 1][0] = matrix[i][SIZE - 1];
matrix[i][SIZE - 1] = temp;
}
}

ты массив 2х2 обрабатывешь??? куда N потерял??
в третих вот она ошибка

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

matrix[i + 1][0]

на последней итерации i=SIZE ( а может нет все N зависит при 1 2 точно будет)

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

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

пройди по шагам и посмотри при каких значениях i j вылетает

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

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

ток он как-то странно сортирует

а как ты хотел

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

C++

1 2	for(int i = 0; i < SIZE; i++) { for(int j = 0; j < SIZE - 1; j++) {

при сортировке

DiffEreD · 27.08.2012, 20:54

Могу предположить что должно было быть так

код

C++

#include <conio.h>
#include<windows.h>
#include<iostream>
#include<time.h>
using namespace std;
int main()
{
    setlocale(0, "");
    srand(unsigned(time(NULL)));
    int N, SIZE=5,temp;
    cout<<"Введите размер матрицы: ";
    cin >> N;
    int ** matrix = new int * [N];
    for (int i = 0; i < N; i++)
        matrix [i] = new int [SIZE];
 
    for (int i = 0; i < N; i++)
        for (int j = 0; j <SIZE; j++)
            matrix[i][j] = ((rand() % 10));
 
 
    cout << "Вот Ваша матрица:" << endl << endl;
    for (int i = 0; i < N; i++)
    {
        for(int j = 0; j < SIZE; j++)
            cout << matrix[i][j] << " ";
        cout << endl;
    }
 
    cout << endl;
 
    for (int k = 0; k<N; k++)
    {
        for(int i = 0; i < SIZE - 1; ++i) // i - номер прохода
        {            
            for(int j = 0; j < SIZE - 1; ++j) // внутренний цикл прохода
            {     
                if (matrix[k][j + 1] < matrix[k][j]) 
                {
                    temp = matrix[k][j + 1]; 
                    matrix[k][j + 1] = matrix[k][j]; 
                    matrix[k][j] = temp;
                }
            }
        }
    }
    
    cout << "Вот Ваша матрица:" << endl << endl;
    for (int i = 0; i < N; i++)
    {
        for(int j = 0; j < SIZE; j++)
            cout << matrix[i][j] << " "; 
        cout << endl;
    }
    for (int i = 0; i < N; i++)
        delete [] matrix[i];
    delete [] matrix;
    system("pause");
    return 0;
}

@Alt_Shift · 27.08.2012, 22:05 **[ТС]**

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

Могу предположить что должно было быть так

ну как бы да. но нужно что бы SIZE был равен 2 и сортировка происходила по второму столбцу

Добавлено через 55 минут
такой вариант

C++

#include <conio.h>
#include<windows.h>
#include<iostream>
#include<time.h>
using namespace std;
int main()
{
    setlocale(0, "");
    srand(unsigned(time(NULL)));
    int N, SIZE=2,temp;
    cout<<"Введите размер матрицы: ";
    cin >> N;
    int ** matrix = new int * [N];
    for (int i = 0; i < N; i++)
        matrix [i] = new int [SIZE];
 
    for (int i = 0; i < N; i++)
        for (int j = 0; j <SIZE; j++)
            matrix[i][j] = ((rand() % 10));
 
 
    cout << "Вот Ваша матрица:" << endl << endl;
    for (int i = 0; i < N; i++)
    {
        for(int j = 0; j < SIZE; j++)
            cout << matrix[i][j] << " ";
        cout << endl;
    }
 
    cout << endl;
 
    for (int k = 0; k<N; k++)
    {
        for(int j = 0;j < SIZE - 1; ++j) // i - номер прохода
        {            
            for(int i = 0; i < SIZE - 1; ++i) // внутренний цикл прохода
            {     
                if (matrix[k][2] < matrix[k+1][2]) 
                {
                    temp = matrix[k][2]; 
                    matrix[k][2] = matrix[k+1][2]; 
                    matrix[k+1][2] = temp;
                }
            }
        }
    }
    
    cout << "Вот Ваша матрица:" << endl << endl;
    for (int i = 0; i < N; i++)
    {
        for(int j = 0; j < SIZE; j++)
            cout << matrix[i][j] << " "; 
        cout << endl;
    }
    for (int i = 0; i < N; i++)
        delete [] matrix[i];
    delete [] matrix;
    system("pause");
    return 0;
}

приводит к предыдущей ошибке

ValeryS · 27.08.2012, 22:09

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

приводит к предыдущей ошибке

еще раз

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

пройди по шагам и посмотри при каких значениях i j вылетает

ты за пределы выходишь( по моему)

@Alt_Shift · 27.08.2012, 22:50 **[ТС]**

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

еще раз

ты за пределы выходишь( по моему)

на вот этом этапе

C++

1	for (int k = 0; k<N; k++)

(32 строка)
N=5, а вот k почему то забито мусором вида -854322. Cледующая строчка та же херня с j и потом с i

ValeryS · 27.08.2012, 22:58

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

N=5, а вот k почему то забито мусором вида -854322. Cледующая строчка та же херня с j и потом с i

она еще не успела инициализироваться
на следующем шаге должна быть 0

@Alt_Shift · 27.08.2012, 23:02 **[ТС]**

все верно, но почему так то?

ValeryS · 27.08.2012, 23:22

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

все верно, но почему так то?

попробуй написать так

C++

1	int k = 0;

и тоже самое увидишь
отладчик встал ДО исполнение инициализации а не ПОСЛЕ
грубо говоря ты стоишь в начале строчки и не можешь прочитать что в этой строке

@Alt_Shift · 27.08.2012, 23:34 **[ТС]**

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

попробуй написать так

C++

1	int k = 0;

и тоже самое увидишь
отладчик встал ДО исполнение инициализации а не ПОСЛЕ
грубо говоря ты стоишь в начале строчки и не можешь прочитать что в этой строке

а как правильно?

ValeryS · 28.08.2012, 00:41

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

а как правильно?

в смысле ???
так и есть правильно
это просто работа с отладчиком
все что происходит в строке ты увидишь
когда курсор будет стоять на следующей строке
например

C++

1
2
3

int k=5;// вот здесь ты увидишь в k мусор
k=7+10;// вот здесь ты увидишь в k 5
 ....  // вот здесь ты увидишь в k 17

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

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

C++

for (int k = 0; k<N; k++)
 {
 for(int j = 0;j < SIZE - 1; ++j) // i - номер прохода
 { 
 for(int i = 0; i < SIZE - 1; ++i) // внутренний цикл прохода
 { 
 if (matrix[k][2] < matrix[k+1][2]) 
 {
 temp = matrix[k][2]; 
 matrix[k][2] = matrix[k+1][2]; 
 matrix[k+1][2] = temp;
 }
 }
 }
 }

вот она твоя ошибка
matrix[k][2] ты берешь третий столбец (выход за пределы)
matrix[k+1][2] на последней итерации будет matrix[N][2](опять выход за пределы)
и зачем ты крутишь циклы j и i если не используешь в теле этих значений

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

Добавлено через 21 минуту
если хочешь сортировать 2 столбец то наверное так

C++

for (int k=0; k<N-1; k++) // цикл проходов
 {
 for(int j =0 ;j <N-k ; j++) 
   { 
     if (matrix[j][1] > matrix[j+1][1]) 
     {
       temp = matrix[j+1][1]; 
       matrix[j+1][1] = matrix[j][1]; 
       matrix[j][1] = temp;
     }
   }
 }

"камешком"

C++

for (int k=N-1; k>1; k--) // цикл проходов
 {
 for(int j =k-1 ;j <N-1 ; j++) 
   { 
     if (matrix[j][1] >matrix[j+1][1]) 
     {
       temp = matrix[j+1][1]; 
       matrix[j+1][1] = matrix[j][1]; 
       matrix[j][1] = temp;
     }
   }
 }

"пузырьком"

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

C++

for (int k=1; k<N; k++) // цикл проходов
 {
 for(int j =N-k-1 ;j >=0 ; j--) 
   { 
     if (matrix[j][1] >matrix[j+1][1]) 
     {
       temp = matrix[j+1][1]; 
       matrix[j+1][1] = matrix[j][1]; 
       matrix[j][1] = temp;
     }
   }
 }

еще один "пузырек"

@Alt_Shift · 28.08.2012, 04:17 **[ТС]**

первые два не сортирую вообще

Последняя сортирует но как то странно-только первые несколько строк

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,519
	27.08.2012, 22:09	13
	Сообщение от Alt_Shift приводит к предыдущей ошибке еще раз Сообщение от ValeryS пройди по шагам и посмотри при каких значениях i j вылетает ты за пределы выходишь( по моему) 0

@Alt_Shift 51 / 51 / 20 Регистрация: 28.05.2012 Сообщений: 161
	27.08.2012, 23:02 [ТС]	16
	все верно, но почему так то? 0

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	27.08.2012, 20:23	2
	добавь задержку system("pause") из библиотеки iostream.h или getch() из conio.h 0

@Alt_Shift 51 / 51 / 20 Регистрация: 28.05.2012 Сообщений: 161
	27.08.2012, 20:29 [ТС]	4
	Сообщение от Mr. Pyatachok добавь задержку system("pause") из библиотеки iostream.h или getch() из conio.h и как это может помочь? через ctrl+F5 тоже нормально задерживается впринципе 0

@Mr. Pyatachok 8 / 9 / 1 Регистрация: 30.06.2011 Сообщений: 250
	27.08.2012, 20:31	5
	извиняюсь, не до конца прочитал ваше сообщение 0

@Alt_Shift 51 / 51 / 20 Регистрация: 28.05.2012 Сообщений: 161
	27.08.2012, 20:40 [ТС]	6
	Сообщение от DaskOFF что у вас тут происходит ? вылетает именно тут, перепроверьте сортировку Ну это я и сам вижу,проверял, с статическим массивом работает нормально, с динамикой нет 0

@DaskOFF 113 / 113 / 42 Регистрация: 02.05.2012 Сообщений: 524 Записей в блоге: 1
	27.08.2012, 20:50	9
	пост выше обновил, все работает без ошибок с любым размером ток он как-то странно сортирует 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,519
	27.08.2012, 22:58	15
	Сообщение от Alt_Shift N=5, а вот k почему то забито мусором вида -854322. Cледующая строчка та же херня с j и потом с i она еще не успела инициализироваться на следующем шаге должна быть 0 0

@Alt_Shift 51 / 51 / 20 Регистрация: 28.05.2012 Сообщений: 161
	28.08.2012, 04:17 [ТС]	20
	первые два не сортирую вообще Последняя сортирует но как то странно-только первые несколько строк 0