Форум программистов, компьютерный форум CyberForum.ru

Оптимизация задачи вывода количество нужных партий и людей - C++

Восстановить пароль Регистрация
 
Никита_123
0 / 0 / 0
Регистрация: 08.07.2016
Сообщений: 2
08.07.2016, 12:14     Оптимизация задачи вывода количество нужных партий и людей #1
Добрый день. Есть простая задача, она реализована, но пока реализована криво.

Из-за недостатка знаний, реализовал по следующему принципу:
1. Создаётся двумерный динамический массив с двумя столбцами (номер партии и вводимое количество людей)
2. Вводим число партий
3. Вводим через пробел номер партии и число людей
4. Общая таблица выводится на экран
5. Выводится на экран количество нужных партий и людей.

Хочу узнать мнение опытных людей по следующим вопросам:
1) Как лучше оформить для упрощения читаемости кода
2) Как можно упростить сам код, чтобы он быстрее работал

Адрес пояснения задачи:
http://acm.timus.ru/problem.aspx?space=1&num=1025

Минусы программы: не реализованы проверки на ввод в двух местах
Необходимо вручную вводить номер партии
Довольно сложно реализовано
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
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
#include <iostream>
#include <conio.h>
 
using namespace std;
using std::endl;
using std::cout;
using std::cin;
 
int main()
{
    system("chcp 1251"); 
    
    cout << "Смерть Демократии" << endl;
    
    short n = 0;
    short m = 2; //т.к. в 1-м ст. партии, во 2-м люди
    cout << "Введите количество партий: " << endl; cin >> n;
    
    //!!!!!Сделать проверку на ввод числа!!!!!!
    
    short **Massiv = new short* [n]; // Выделение памяти под массив указателей 
    //для строк
    
    for(short i = 0; i < n; i++){
        Massiv[i] = new short [m]; // выделение памяти под массив столбцов 
        
    }
    
    short a=0;
    for(short i = 0; i < n; i++){
        a=i+1;//используется для проверки ввода
        
        cout << "Введите через пробел номер партии и их количество в партии" << endl;
        for(short j = 0; j < m; j++){
            
            cin >> Massiv[i][j];
            
            //!!!!!сделать проверку на ввод!!!!!
            
            
        }
    }
    
    cout << "_________________________________" << endl;
    // ===Вывод на экран=== //
    for(short i = 0; i < n; i++){
        
        cout << "Партия/Кол-во людей в партии " << (i+1) << " [ ";
        
        for(short j = 0; j < m; j++){
            
            if(j == m-1){
                cout << Massiv[i][j] << ']' << endl;
                break;
            }else 
                cout << Massiv[i][j] << ',';
        }
        
    }
    //===
    cout << "_________________________________" << endl;
    
    if(n >= 2){
        cout << "Количество партий, которое должно вас поддержать: " << ((n/2)+1) << endl;
    }else{
        cout << "Вас должна поддержать одна партия" << endl;
    }
    
    short summ = 0; //для записывания суммы
    for(short i = 0; i < ((n/2)+1); i++){
        cout << "Количество людей, которое должно вас поддержать в партии: " << i+1 << endl; 
        
        for(short j = 0; j < m; j++){
            summ += Massiv[i][j];
        }
        summ = ((summ - (i+1))/2) +1;
        cout << " равно " << summ << endl;
        summ = 0;
    }
    
    
    // ===Освобождение памяти=== //
    for(short i = 0; i < n; i++)
    {
        delete [] Massiv[i]; // Освобождение памяти по столбцам
    }
    delete [] Massiv; // Освобождение памяти по строкам
    // ===
    
    _getch();
    return 0;
}
//Заранее Спасибо
Лучшие ответы (1)
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
avgoor
562 / 352 / 83
Регистрация: 05.12.2015
Сообщений: 1,137
08.07.2016, 15:32     Оптимизация задачи вывода количество нужных партий и людей #2
Никита_123, Вы решили какую-то другую задачу, потому что:
1) Вам задан четкий формат ввода и вывода: Если на вход поступит "3\n1 5 7" - на выходе должно быть "6". У вас на выходе будет "Смерть демократии", т.е. ваша программа не пройдет ни один тест.
2) На выходе должно быть минимальное число людей, распределенных по округам, а не количество партий.
3) Про проверку ввода в условии задачи - ни слова, т.е. ее делать не надо.

Алгоритм решения такой:
Вводим n - число групп.
Заводим массив/вектор длиной n;
в цикле вводим в него количество людей в каждой группе.
сортируем (нам важно начало массива длиной n/2+1, т.е. можно применить partial_sort)
Заводим переменную sum=0;
в цикле по части массива длиной n/2+1 прибавляем к ней: sum+=arr[i]/2+1;
Выводим sum.
Все.
Никита_123
0 / 0 / 0
Регистрация: 08.07.2016
Сообщений: 2
08.07.2016, 15:42  [ТС]     Оптимизация задачи вывода количество нужных партий и людей #3
М, т.е. можно обойтись одномерным динамическим массивом?
Кстати да, зря поставил двумерный.

Так можно немного изменить условие, тем более, что оно учебное. Будет выводиться больше информации в консоль, это не критично. Про проверку это важно в любом случае, по моему к этому стоит привыкать.
А само оформление, не смотря на смысл программы, сам стиль оформления?
avgoor
562 / 352 / 83
Регистрация: 05.12.2015
Сообщений: 1,137
08.07.2016, 16:10     Оптимизация задачи вывода количество нужных партий и людей #4
Цитата Сообщение от Никита_123 Посмотреть сообщение
Будет выводиться больше информации в консоль, это не критично
Критично. Проверять вас будет не человек, а машина. И если она ждет число - на выходе должно быть число.
Представтье:
Ваш вывод: Введите число групп:
Ввод: "Всего групп десять"
Как на это отреагирует ваш cin>>n?
Цитата Сообщение от Никита_123 Посмотреть сообщение
Про проверку это важно в любом случае, по моему к этому стоит привыкать
Стоит привыкать к выполнению ТЗ.
Mr.X
Эксперт С++
 Аватар для Mr.X
2802 / 1578 / 247
Регистрация: 03.05.2010
Сообщений: 3,666
08.07.2016, 18:37     Оптимизация задачи вывода количество нужных партий и людей #5
Сообщение было отмечено автором темы, экспертом или модератором как ответ
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
#include <algorithm>
#include <iostream>
#include <numeric>
#include <vector>
///////////////////////////////////////////////////////////////////////////////
typedef std::vector     < int   >   T_group_sizes;
///////////////////////////////////////////////////////////////////////////////
template< typename  T >
T   more_than_half( T   t )
{
    return  t / 2   +   1;
}
///////////////////////////////////////////////////////////////////////////////
int     main()
{
    std::ios::sync_with_stdio( false );
    int     groups_total{};
    std::cin    >>  groups_total;
 
    T_group_sizes   group_sizes( groups_total );
 
    for( auto   &   group_size  :   group_sizes )
    {
        std::cin    >>  group_size;
    }//for
 
    auto    middle_it   =       group_sizes.begin   ()
 
                            +   more_than_half      (
                                                        group_sizes.size()
                                                    );
 
    std::partial_sort
        (
            group_sizes.begin   (),
            middle_it,
            group_sizes.end     ()
        );
 
    std::cout   <<  std::accumulate
                        (
                            group_sizes.begin(),
                            middle_it,
                            int(),
 
                            []  (
                                    auto    L,
                                    auto    R
                                )
                            {
                                return  L   +   more_than_half(R);
                            }
                        );
}
Yandex
Объявления
08.07.2016, 18:37     Оптимизация задачи вывода количество нужных партий и людей
Ответ Создать тему
Опции темы

Текущее время: 09:18. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru