задача: отсортировать строки квадратной целочисленной квадратной матрицы по убыванию количества одинаковых элементов в каждой строке.
Имеется код
C++ |
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
| #include <iostream>
#define n 2 //размер матрицы
void create_mas();
void see_mas();
void sort_mas();
void create_mas();
int m[n][n];
void main()
{
create_mas(); //создаем массив
system("cls");
std::cout<<"first mass"<<std::endl;
see_mas(); //показываем его начальный
std::cout<<std::endl<<std::endl<<"now sort it"<<std::endl;
sort_mas(); //сортируем
see_mas(); //показываем отсортированый
std::cout<<std::endl<<std::endl;
system("pause");//пауза
}
void create_mas()
{
for(int i=0;i<n;i++) //строки
for(int j=0;j<n;j++)//столбцы
m[i][j]=rand()%99; //рандом от нуля
}
void see_mas()
{
for(int i=0;i<n;i++) //строки
{
std::cout<<std::endl;//новая строка
for(int j=0;j<n;j++)//столбцы
std::cout<<m[i][j]<<" ";//елемент+пробел
}
}
void sort_mas()
{
int need,temp,c1,c2;//флаг, временная, количество одинаковых в паре строк
do{
need=0; //обнуляем
c1=c2=0;//обнуляем
for(int s=0;s<n-1;s++){ //перебрать столбцы попарно
//поиск одинаковых в одной строке строке
for(int i=0;i<n-1;i++){ // идем с 0 по n - 2 элемент
for(int j=i+1;j<n;j++){ // идем с i + 1 по n - 1 элемент
if(m[i][s]==m[j][s])c1++; // если равны i и j элементы увеличиваем счетчик //для первой строки
if(m[i][s+1]==m[j][s+1])c2++;}} // если равны i и j элементы увеличиваем счетчик //для второй строки
//если с2 больше с1 поменять местами
if(c2>c1){
for(int c=0;c<n;c++) //меняем местами по столбцам(по 2 штуки(верхний с нижним))
{ //через пузырек
temp=m[c][s];
m[c][s]=m[c][s+1];
m[c][s+1]=temp;
need=1;
}
}
}
}while(need);
} |
|
в результате получаю исходный массив, почему?
Добавлено через 1 час 20 минут
то есть оно работает, но как то не так....
если ввести матрицу
2 2 1
1 2 1
0 0 1
то получаю матрицу
1 2 2
1 1 2
1 0 0
почему то сортирует вообще не как должно....
Добавлено через 1 час 2 минуты
в общем проблема была в том, что я неправильно представлял себе структуру матрицы(m[a][b] а-столбцы, b- строки, я думал наоборот....)
вот правильный код, может кому-то пригодится:
C++ |
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
| #include <iostream>
#define n 2
void create_mas();
void see_mas();
void sort_mas();
void null_m1();
void see_m1();
void create1_mas();
int m[n][n];
void main()
{
//create_mas(); //создаем массив
create1_mas();
system("cls");
std::cout<<"first mass"<<std::endl;
see_mas(); //показываем его начальный
std::cout<<std::endl<<std::endl<<"now sort it"<<std::endl;
sort_mas(); //сортируем
see_mas(); //показываем отсортированый
std::cout<<std::endl<<std::endl;
system("pause");//пауза
}
void create_mas()
{
for(int i=0;i<n;i++) //строки
for(int j=0;j<n;j++)//столбцы
m[i][j]=rand()%3; //рандом от нуля
}
void create1_mas()
{
for(int i=0;i<n;i++) //строки
{
for(int j=0;j<n;j++){
std::cin>>m[i][j];}}
}
void see_mas()
{
for(int i=0;i<n;i++) //строки
{
std::cout<<std::endl;//новая строка
for(int j=0;j<n;j++)//столбцы
std::cout<<m[i][j]<<" ";//елемент+пробел
}
}
void sort_mas()
{
int need,temp,c1,c2;//флаг, временная, количество одинаковых в паре строк
do{
need=0; //обнуляем
for(int s=0;s<n-1;s++){ //перебрать столбцы попарно
//поиск одинаковых в одной строке строке
c1=c2=0;//обнуляем
for(int i=0;i<n-1;i++){ // идем с 0 по n - 2 элемент
for(int j=i+1;j<n;j++){ // идем с i + 1 по n - 1 элемент
if(m[s][i]==m[s][j])c1++; // если равны i и j элементы увеличиваем счетчик //для первой строки
if(m[s+1][i]==m[s+1][j])c2++;}} // если равны i и j элементы увеличиваем счетчик //для второй строки
//если с2 больше с1 поменять местами
if(c1<c2){
for(int c=0;c<n;c++) //меняем местами по столбцам(по 2 штуки(верхний с нижним))
{ //через пузырек
temp=m[s][c];
m[s][c]=m[s+1][c];
m[s+1][c]=temp;
need=1;
}
std::cout<<"now do"<<std::endl;
}
std::cout<<"c1="<<c1<<"c2="<<c2<<std::endl<<std::endl;
}
}while(need);
} |
|