Упорядочить массив в порядке убывания суммы цифр элементов

@mark12552 · Регистрация: 05.10.2013

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

Вот мой код, но он не работает:

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
#include <iostream> 
using namespace std; 
int sumcifr (int a)
{
    int b,s;
    s=0;
    while (a>0)
    s=s+(a%10);
    a/=10;
    return sumcifr;
}
 
int main() 
{ 
int n,i,b,j,s; 
cout<<" n="; 
cin>>n; 
float y[n]; 
for (i=0;i<n;i++)
{ 
cout<<"\n Y["<<i<<"]="; 
cin>>y[i]; 
} 
 
for(j=1;j<n;j++) 
for(i=0;i<n-j;i++) 
if (sumcifr (y[i])<sumcifr (y[j])) 
{ b=y[i]; 
y[i]=y[i+1]; 
y[i+1]=b; 
}
for (i=0;i<n;i++) 
cout<<y[i]<<endl; 
return 0; }

Что не так? Помогите исправить.

@Тамика · 29.01.2014, 18:54

Первое, что бросилось в глаза -

C++
1
2
3
4
5
6
7
8
9
int sumcifr (int a)
{
    int b,s;
    s=0;
    while (a>0)
    s=s+(a%10);
    a/=10;
    return sumcifr;
}

Что Вы возвращаете? Наверное Вам нужно s вернуть.

Добавлено через 2 минуты
И зачем в этой функции переменная b? Она нигде не используется.

@Mger · 29.01.2014, 20:01

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
#include <iostream>
#include <time.h>
#include <iomanip>
using namespace std;
void Sort(int mas[], int n)
{
    for (int i = 0; i < n-1; i++)
    {
        for (int j = 0; j < n-1; j++)
        {
            if (mas[j]>mas[j + 1])
            {
                swap(mas[j], mas[j + 1]);
            }
        }
    }
}
void Print(int mas[], int n)
{
    for (int a = 0; a < n; a++)
    {
        cout << mas[a] << " ";
    }
    cout << endl;
}
int sumcifr(int t)
{
    int d, q = 0;
    while (t>0)
    {
        d = t % 10;
        t /= 10;
        q += +d;
    }
    return q;
}
int main()
{
    setlocale(LC_ALL,"rus");
    srand((unsigned)time(NULL));
    const int n = 25;
    int arr[n];
    for (int i = 0; i < n; i++)
    {
        arr[i] = rand()+1000;
    }
    for (int i = 0; i < n; i++)
    {
        cout<<"MAS "<<i<<" [" << arr[i]<<"]" << " "<<endl;
    }
    int max[n];
    for (int i = 0; i < n; i++)
    {
        max[i] = sumcifr(arr[i]);
    }
    cout << "Сумма цифр элементов массива до сортировки:"<<endl;
    Print(max, n);
    cout << endl;
    Sort(max, n);
    cout << "Сумма цифр элементов массива после сортировки:" << endl;
    Print(max, n);
    system("pause > void");
}

@mark12552 · 29.01.2014, 20:09 **[ТС]**

Mger, Спасибо большое, но вы можете исправит мой код?

@Mger · 29.01.2014, 20:52

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

Mger, Спасибо большое, но вы можете исправит мой код?

У вас тут почти все неправильно

Добавлено через 13 минут
Во первых вы создали статистический массив его значение должна быть константной
пример:

C++
1
2
3
4
5
6
7
8
9
10
11
12
int const y=5;
int mas[y];либо int mas[5] или любое число то есть его значение должна быть заранее известной 
второе функция 
int sumcifr (int a)
{
    int b,s;
    s=0;
    while (a>0)
    s=s+(a%10);
    a/=10;
    return sumcifr;
}

вы тут должны возвращать переменную "s" а не имя функции
ну вообщем почитайте про массивы и функции

@mark12552 · 30.01.2014, 09:32 **[ТС]**

Mger, Ну вот я исправил, что еще нужно изменить?

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
#include <iostream> 
using namespace std; 
int sumcifr (int a)
{
    int s;
    s=0;
    while (a>0)
    s=s+(a%10);
    a/=10;
    return s;
}
 
int main() 
{ 
int *y,n,i,b,j,s; 
cout<<" n="; 
cin>>n; 
y=new int [n]; 
for (i=0;i<n;i++)
{ 
cout<<"\n Y["<<i<<"]="; 
cin>>y[i]; 
} 
 
for(j=1;j<n;j++) 
for(i=0;i<n-j;i++) 
if (sumcifr (y[i])<sumcifr (y[j])) 
{ b=y[i]; 
y[i]=y[i+1]; 
y[i+1]=b; 
}
for (i=0;i<n;i++) 
cout<<y[i]<<endl; 
return 0; }

@Mger · 30.01.2014, 14:31

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
#include <iostream> 
using namespace std;
int sumcifr(int a)
{
    int d, q = 0;
    while (a>0)
    {
        d = a % 10;
        a /= 10;
        q += +d;
    }
    return q;
}
 
int main()
{
    setlocale(LC_ALL, "rus");
    int *y, n, i, b, j, s;
    cout << " n=";
    cin >> n;
    y = new int[n];
    for (i = 0; i<n; i++)
    {
        cout << "\n Y[" << i << "]=";
        cin>>y[i];
    }
    cout << endl;
    int *mas = new int[n];
    for (int i = 0; i < n; i++)
    {
        mas[i] = sumcifr(y[i]);
    }
    cout << endl;
    for (int i = 0; i < n; i++)
    {
        cout << "Сумма цифр Y[" << i << "]=" << mas[i] << endl;
    }
    for (j = 1; j<n; j++)
    for (i = 0; i<n; i++)
    if (mas[j]<mas[i])
    {
        swap(mas[j], mas[i]);
    }
    for (int i = 0; i < n; i++)
    {
        cout << "Сумма цифр Y[" << i << "]после сортировки =" << mas[i] << endl;
    }
    system("pause>>void");
}

1. у вас неправильная функция я исправил!
2. тут лучше создать еще 1 массив и в нем записать сумму цифр элементов массива ! а так все правильно

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@mark12552 0 / 0 / 0 Регистрация: 05.10.2013 Сообщений: 23
	29.01.2014, 20:09 [ТС]
	Mger, Спасибо большое, но вы можете исправит мой код? 0