Функция для сортировки массива пузырьком

@Олег Фирамов · Регистрация: 14.11.2017

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

Помогите исправить код. Скорей всего проблема в сортировке.
Условие:
Дана матрица (N+15)x(N+15), отсортировать парные рядки в прямом порядке, а непарные в обратном, числа генерировать случайно в интервале [- N-15; N+15].

Код:

C++

#include <iostream>
#include <ctime>
#include <iomanip>
#include <stdio.h>
using namespace std;
int N;
void input(int **mass, int size)
{
    for (int i = 0; i < size; i++) 
    {
        for (int j = 0; j < size; j++)
        {
            mass[i][j]=rand() % (size * 2+1)-size;
            cout<<setw(4) << mass[i][j];
        }
        cout<<endl;
    }
    cout<<endl;
}
void sort(int **mass, int size)
{
    int tmp;
 
    for (int m = 0; m < size; m++) 
    {
 
        for (int i = 1; i > size; ++i) 
        {
 
            for (int j = i; j < size; j--) 
            {
 
                if (m % 2 == 0) 
                {
                    if (mass[m][j] > mass[m][j - 1]) 
                    {
 
                        tmp = mass[m][j];
                        mass[m][j] = mass[m][j - 1];
                        mass[m][j - 1] = tmp;
                    }
                }
                else{
                    if (mass[m][j] < mass[m][j - 1]) 
                    {
 
                       tmp = mass[m][j];
                        mass[m][j] = mass[m][j - 1];
                        mass[m][j - 1] = tmp;
                    }
                }
            }
        }
    }
}
 
 
void main()
{
    srand(time(NULL));
    cin>>N;
    int size = N+15;
    int **mass = new int* [size];
    for (int i = 0; i <= size; i++)
    {
        mass[i]=new int[size];
    }
    input(mass,size);
    void sort(int** mass,int size);
    for (int i = 0; i <= size; i++)
    {
        for (int j = 0; j < size; j++)
        {
            cout<<setw(4)<<mass[i][j];
        }
        cout<<endl;
    }
    for (int i = 0; i <= size; i++)
    {
        delete[] mass[i];
    }
 
system ("PAUSE");
}

@nmcf · 14.11.2017, 21:42

Рядки в огороде.
Каждую группу раздельно сортируй.

@Олег Фирамов · 14.11.2017, 21:53 **[ТС]**

nmcf, можешь конкретней подсказать что убрать и что изменить?

@nmcf · 14.11.2017, 21:55

Делаешь цикл по нечётным - с шагом 2. В нём сортируешь текущую строку обычным способом.
То же с чётными.

@Олег Фирамов · 14.11.2017, 22:17 **[ТС]**

nmcf, Я что-то неправильно делаю. Результат все тот же. И почему вылазит последняя строка цифр, как ее убрать?

@nmcf · 14.11.2017, 22:22

Показывай.

@Олег Фирамов · 14.11.2017, 22:26 **[ТС]**

nmcf,

C++

#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <iomanip>
#include <stdio.h>
using namespace std;
int N;
void input(int **mass, int size)
{
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j < size; j++)
        {
            mass[i][j] = rand() % (size * 2 + 1) - size;
            cout << setw(4) << mass[i][j];
        }
        cout << endl;
    }
    cout << endl;
}
void sort(int **mass, int size)
{
    int tmp;
 
    for (int m = 0; m < size; m+=2)
    {
        for (int i = 1; i > size; ++i)
        {
            for (int j = i; j < size; j--)
            {
                if (m % 2 == 0)
                {
                    if (mass[m][j] > mass[m][j - 1])
                    {
                        tmp = mass[m][j];
                        mass[m][j] = mass[m][j - 1];
                        mass[m][j - 1] = tmp;
                    }
                }
            }
        }
    }
    for (int m = 0; m < size; m+=2)
    {
 
        for (int i = 1; i > size; i += 2)
        {
 
            for (int j = i; j < size; j--)
            {
 
                if (m % 2 == 1)
                {
                    if (mass[m][j] < mass[m][j - 1])
                    {
 
                        tmp = mass[m][j];
                        mass[m][j] = mass[m][j - 1];
                        mass[m][j - 1] = tmp;
                    }
                }
            }
        }
    }
}
 
    
 
void main()
{
    srand(time(NULL));
    cin >> N;
    int size = N + 15;
    int **mass = new int*[size];
    for (int i = 0; i <= size; i++)
    {
        mass[i] = new int[size];
    }
    input(mass, size);
    void sort(int** mass, int size);
    for (int i = 0; i <= size; i++)
    {
        for (int j = 0; j < size; j++)
        {
            cout << setw(4) << mass[i][j];
        }
        cout << endl;
    }
    for (int i = 0; i <= size; i++)
    {
        delete[] mass[i];
    }
 
    system("PAUSE");
}

@nmcf · 14.11.2017, 22:39

Чётные пузырьком:

C++

    for (int m = 1; m < size; m+=2)
    {
        for (int i = 0; i < size - 1; ++i)
        {
            for (int j = 0; j < size - 1; ++j)
            {
                if (mass[m][j + 1] > mass[m][j])
                    std::swap(mass[m][j + 1], mass[m][j]);
            }
        }
    }

Нечётные допили сам.

@Олег Фирамов · 14.11.2017, 22:45 **[ТС]**

nmcf, спасибо

@Олег Фирамов · 15.11.2017, 22:33 **[ТС]**

Добрый вечер!
После сортировки массива через функцию выводит бред. Помогите найти и исправить проблему.
Условие:
Дана матрица (N+15)x(N+15), отсортировать парные рядки в прямом порядке, а непарные в обратном, числа генерировать случайно в интервале [- N-15; N+15].

Код:

C++

#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <iomanip>
#include <stdio.h>
using namespace std;
 
void sort(int **mass, int size)
{
    for (int m = 1; m < size; m += 2)
    {
        for (int i = 0; i < size - 1; ++i)
        {
            for (int j = 0; j < size - 1; ++j)
            {
                if (mass[m][j + 1] > mass[m][j])
                {
                    swap(mass[m][j + 1], mass[m][j]);
                }
 
 
                else
                {
                    if (mass[m][j + 1] < mass[m][j])
                    {
                        swap(mass[m][j + 1], mass[m][j]);
                    }
                }
            }
        }
    }
}
 
 
    
    
 
 
void main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    int N = 0;
    int size = N + 15;
    cout << "Введите число: ";
    cin >> N;
 
    int **mass = new int*[N];
 
    for (int i = 0; i < N; i++) 
    {
 
        mass[i] = new int[N];
 
    }
    cout << "\n";
    cout << " Массив до сортировки: " << endl;
    cout << "\n";
    for (int i = 0; i <N; i++)
    {
 
        for (int j = 0; j < N; j++)
 
        {
            mass[i][j] = -(N)+rand() % ((N) * 2);
            cout << mass[i][j] << "\t";
        }
        cout << endl;
    }
    cout << "\n";
 
    void sort(int mass,int size);
    cout << "Массив после сортировки" << endl;
    for (int i = 0; i < size; i++)
        cout << mass[i] << ' ';
    cin.get();
    cin.get();
    system("PAUSE");
}

MrGluck · 16.11.2017, 01:15

Комментарий модератора

Олег Фирамов, запрещено дублировать темы

@nmcf · 16.11.2017, 08:50

Я же сказал: чётные и нечётные раздельно. Какой else? Этот фрагмент должен быть неизменным.

@Kuzia domovenok · 16.11.2017, 09:01

Не по теме:

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

@Олег Фирамов · 16.11.2017, 20:54 **[ТС]**

nmcf, я и так и так делал, отдельно тоже самое. Просто взял код последней моей попытки.

@nmcf · 16.11.2017, 21:25

Показывай.

@Олег Фирамов · 17.11.2017, 00:04 **[ТС]**

nmcf,

C++

#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <iomanip>
#include <stdio.h>
using namespace std;
 
void sort(int **mass, int size)
{
    for (int m = 1; m < size; m += 2)
    {
        for (int i = 0; i < size - 1; ++i)
        {
            for (int j = 0; j < size - 1; ++j)
            {
                if (mass[m][j + 1] > mass[m][j])
                {
                    swap(mass[m][j + 1], mass[m][j]);
                }
            }
        }
    }
    for (int m = 1; m < size; m += 2)
    {
        for (int i = 1; i < size - 1; ++i)
        {
            for (int j = 0; j < size - 1; ++j)
            {
                if (mass[m][j + 1] < mass[m][j])
                {
                    swap(mass[m][j + 1], mass[m][j]);
                }
            }
        }
    }
}
 
 
void main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    int N = 0;
    int size = N + 15;
    cout << "Введите число: ";
    cin >> N;
 
    int **mass = new int*[N];
 
    for (int i = 0; i < N; i++)
    {
 
        mass[i] = new int[N];
 
    }
    cout << "\n";
    cout << " Массив до сортировки: " << endl;
    cout << "\n";
    for (int i = 0; i < N; i++)
    {
 
        for (int j = 0; j < N; j++)
 
        {
            mass[i][j] = -(N)+rand() % ((N) * 2);
            cout << mass[i][j] << "\t";
        }
        cout << endl;
    }
    cout << "\n";
 
    void sort(int mass, int size);
    cout << "Массив после сортировки" << endl;
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j <= size; j++)
        {
            cout << setw(4) << mass[i][j];
        }
        cout << endl;
    }
    cout << endl;
 
        system("PAUSE");
}

@Kuzia domovenok · 17.11.2017, 05:13

В таком варианте цикл по I вообще лишний

Добавлено через 5 часов 0 минут
хотя вру

@nmcf · 17.11.2017, 08:01

В 23-й цикл с нуля. В 73-й вызов функции оформи как положено.

@Олег Фирамов · 17.11.2017, 18:33 **[ТС]**

nmcf, Все работает, нашел еще другие ошибки. Сейчас столкнулся с другой проблемой. В конце каждой строки выведенного массива лишние цифры. Подскажешь что исправить?

C++

#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <iomanip>
#include <stdio.h>
using namespace std;
 
void sort(int **mass, int size)
{
    for (int m = 1; m < size; m += 2)
    {
        for (int i = 1; i < size - 1; ++i)
        {
            for (int j = 0; j < size - 1; ++j)
            {
                if (mass[m][j + 1] < mass[m][j])
                {
                    swap(mass[m][j + 1], mass[m][j]);
                }
            }
        }
    }
    for (int m = 0; m < size; m += 2)
    {
        for (int i = 0; i < size - 1; ++i)
        {
            for (int j = 0; j < size - 1; ++j)
            {
                if (mass[m][j + 1] > mass[m][j])
                {
                    swap(mass[m][j + 1], mass[m][j]);
                }
            }
        }
    }
}
 
 
void main()
{
    setlocale(LC_ALL, "Russian");
    srand(time(NULL));
 
    int N = 0;
    int size = N + 15;
    cout << "Введите число: ";
    cin >> N;
 
    int **mass = new int*[N];
 
    for (int i = 0; i < size; i++)
    {
 
        mass[i] = new int[N];
 
    }
    cout << "\n";
    cout << " Массив до сортировки: " << endl;
    cout << "\n";
    for (int i = 0; i < size; i++)
    {
 
        for (int j = 0; j < size; j++)
 
        {
            mass[i][j] = -(N)+rand() % ((N) * 2);
            cout << mass[i][j] << "\t";
        }
        cout << endl;
    }
    cout << "\n";
 
    sort(mass, size);
    cout << "Массив после сортировки" << endl;
    cout << "\n";
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j <= size; j++)
        {
            cout << setw(4) << mass[i][j];
        }
        cout << endl;
    }
    cout << endl;
 
    system("PAUSE");
}

@Kuzia domovenok · 17.11.2017, 18:34

Сообщение от Олег Фирамов

C++

for (int j = 0; j <= size; j++) 
{
   cout << setw(4) << mass[i][j]; 
}

строго меньше size, без равно

@nmcf 7787 / 6555 / 2983 Регистрация: 14.04.2014 Сообщений: 28,633
	16.11.2017, 08:50	12
	Я же сказал: чётные и нечётные раздельно. Какой else? Этот фрагмент должен быть неизменным. 0

@nmcf 7787 / 6555 / 2983 Регистрация: 14.04.2014 Сообщений: 28,633
	14.11.2017, 21:42	2
	Рядки в огороде. Каждую группу раздельно сортируй. 0

@Олег Фирамов 0 / 0 / 0 Регистрация: 14.11.2017 Сообщений: 13
	14.11.2017, 21:53 [ТС]	3
	nmcf, можешь конкретней подсказать что убрать и что изменить? 0

@nmcf 7787 / 6555 / 2983 Регистрация: 14.04.2014 Сообщений: 28,633
	14.11.2017, 21:55	4
	Делаешь цикл по нечётным - с шагом 2. В нём сортируешь текущую строку обычным способом. То же с чётными. 0

@Олег Фирамов 0 / 0 / 0 Регистрация: 14.11.2017 Сообщений: 13
	14.11.2017, 22:17 [ТС]	5
	nmcf, Я что-то неправильно делаю. Результат все тот же. И почему вылазит последняя строка цифр, как ее убрать? Миниатюры 0

@nmcf 7787 / 6555 / 2983 Регистрация: 14.04.2014 Сообщений: 28,633
	14.11.2017, 22:22	6
	Показывай. 0

@Олег Фирамов 0 / 0 / 0 Регистрация: 14.11.2017 Сообщений: 13
	14.11.2017, 22:45 [ТС]	9
	nmcf, спасибо 0

@Kuzia domovenok
	16.11.2017, 09:01 #13
	Не по теме: боже, ты даже вставить готовый ответ в программу без приключений не можешь! 0

@Олег Фирамов 0 / 0 / 0 Регистрация: 14.11.2017 Сообщений: 13
	16.11.2017, 20:54 [ТС]	14
	nmcf, я и так и так делал, отдельно тоже самое. Просто взял код последней моей попытки. 0

@nmcf 7787 / 6555 / 2983 Регистрация: 14.04.2014 Сообщений: 28,633
	16.11.2017, 21:25	15
	Показывай. 0

@Kuzia domovenok 4063 / 3317 / 924 Регистрация: 25.03.2012 Сообщений: 12,485 Записей в блоге: 1
	17.11.2017, 05:13	17
	В таком варианте цикл по I вообще лишний Добавлено через 5 часов 0 минут хотя вру 0

@nmcf 7787 / 6555 / 2983 Регистрация: 14.04.2014 Сообщений: 28,633
	17.11.2017, 08:01	18
	В 23-й цикл с нуля. В 73-й вызов функции оформи как положено. 0