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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
DenKG
0 / 0 / 0
Регистрация: 23.12.2015
Сообщений: 303
#1

Как сделать данный код более логичным и читаемым? - C++

07.06.2016, 16:48. Просмотров 175. Ответов 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
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
void sort(COMPANIES *final, int CompNum)//Сортировка
{
    int i=0;
    int colNum;
    cout << "Введите номер стоблца, по которому хотите отсортировать таблицу:\n"
        << "1.Название компании; 2.Количество продуктов; 3.Продажи за год; 4.Часть рынка;" << endl;
    cin >> colNum;
    if (colNum == 1 ) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (strcmp(final[j].NAME, final[j + 1].NAME)>0)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    if (colNum == 2 ) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (final[j].PRODUCTNUMBER > final[j + 1].PRODUCTNUMBER)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    if (colNum == 3) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (final[j].YEARSALESVOLUME > final[j + 1].YEARSALESVOLUME)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    if (colNum == 4) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (final[j].MARKETPERCENT > final[j + 1].MARKETPERCENT)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    ofstream outlink2;
    outlink2.open("C://Users//УЩ//Documents//Курсовая//ConsoleApplication1//outfile.txt", std::ios_base::app);
    outlink2 << "Отсортированная таблица: ";
    outlink2 << "\n+--------------------+--------------------+--------------------+--------------------+\n"
        << "|" << setw(20) << left << "Название компании"
        << "|" << setw(20) << left << "Количество продуктов"
        << "|" << setw(20) << left << "Продажи за год"
        << "|" << setw(20) << left << "Часть рынка %" << "|";
 
    for (int i = 0; i < CompNum; i++) {
 
        outlink2 << "\n+--------------------+--------------------+--------------------+--------------------+\n"
            << "|" << setw(20) << left << final[i].NAME
            << "|" << setw(20) << left << final[i].PRODUCTNUMBER
            << "|" << setw(20) << left << final[i].YEARSALESVOLUME
            << "|" << setw(20) << left << final[i].MARKETPERCENT << "|";
    }
    outlink2 << "\n+--------------------+--------------------+--------------------+--------------------+\n";
    outlink2.close();
    cout << "\n+--------------------+--------------------+--------------------+--------------------+\n"
        << "|" << setw(20) << left << "Название компании"
        << "|" << setw(20) << left << "Количество продуктов"
        << "|" << setw(20) << left << "Продажи за год"
        << "|" << setw(20) << left << "Часть рынка %" << "|";
 
    for (int i = 0; i < CompNum; i++) {
 
        cout << "\n+--------------------+--------------------+--------------------+--------------------+\n"
            << "|" << setw(20) << left << final[i].NAME
            << "|" << setw(20) << left << final[i].PRODUCTNUMBER
            << "|" << setw(20) << left << final[i].YEARSALESVOLUME
            << "|" << setw(20) << left << final[i].MARKETPERCENT << "|";
    }
    cout << "\n+--------------------+--------------------+--------------------+--------------------+\n";
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.06.2016, 16:48     Как сделать данный код более логичным и читаемым?
Посмотрите здесь:

Можно ли данный код сделать короче? - C++
#include&lt;iostream&gt; #include&lt;iomanip&gt; int main() { int r,f,a,b,c,d,x,y; cout&lt;&lt;&quot;Vvedite znachenie r:&quot;; cin&gt;&gt;r; ...

Код более понятней сделать - C++
Помагите сделать код более понятней и читаемым #include &quot;stdafx.h&quot; #include &lt;iostream&gt; using namespace std; template &lt;class Type&gt;...

Как исправить данный код сортировки? - C++
Вот только не надо никаких анализов, вопросов, и т. д. Конкретно мой пример. Программа на некоторых компиляторах не компилируется....

Как написать данный код средствами ООП? - C++
Всем привет! Недавно начал изучать с++ и написал простенькую задачу: Задание: Найти произведение компонент вектора , удовлетворяющих...

Как правильно прописать данный код программы? - C++
#include &lt;iostream&gt; #include &lt;clocale&gt; #include &lt;cmath&gt; #include &lt;cstdlib&gt; using namespace std; double masA={0}; long H(double...

Как работает данный код? И почему не компилируется? - C++
Обьясните пожалуйста как работает данный код, и скажите почему он не компилируется? И есть ли способ считать числа еще быстрее? ...

Как еще короче записать данный код? - C++
#include &lt;ios&gt; main() { system(&quot;for /f \&quot;tokens=1,2,3\&quot; %a in (input.txt) do set /a %a*%b*%c*2 &gt;output.txt&quot;); } подсказка...

Как оптимизировать данный код игры крестики нолики? - C++
Привет всем! Я написал консольную игру &quot;Крестики нолики&quot;. Хочу посоветоваться как можно оптимизировать игру. Вот исходный код: ...

Как можно сделать данный кусок кода меньше и проще? - C++
Задние: Заменить все элементы с четными номерами на значение второго положительного элемента массива. void change() //замена элементов...

Как можно записать код в более компактном виде? - C++
Вывести столбиком след. числа 3,2, 3,2, 3,3, 3,3,........3,9 у меня вышло довольно размазан: #include &quot;iostream&quot; #include &quot;conio.h&quot; ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Hikari
Хитрая блондиночка $)
1447 / 954 / 285
Регистрация: 21.12.2015
Сообщений: 3,798
07.06.2016, 16:52     Как сделать данный код более логичным и читаемым? #2
Ой ой ой... А STL или хотя бы qsort() для сортировки использовать не желаешь?
DenKG
0 / 0 / 0
Регистрация: 23.12.2015
Сообщений: 303
07.06.2016, 18:21  [ТС]     Как сделать данный код более логичным и читаемым? #3
Цитата Сообщение от Hikari Посмотреть сообщение
Ой ой ой... А STL или хотя бы qsort() для сортировки использовать не желаешь?
Нет, я пока такое делать не умею ))
Мне нужно именно запись в файл вывести в другую функцию

Добавлено через 1 час 25 минут
Ну так что, мне никто не поможет?)
Namat
12 / 12 / 5
Регистрация: 02.12.2014
Сообщений: 35
07.06.2016, 18:50     Как сделать данный код более логичным и читаемым? #4
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
void sort(COMPANIES *final, int CompNum)//Сортировка
{
    int i=0;
    int colNum;
    cout << "Введите номер стоблца, по которому хотите отсортировать таблицу:\n"
        << "1.Название компании; 2.Количество продуктов; 3.Продажи за год; 4.Часть рынка;" << endl;
    cin >> colNum;
    if (colNum == 1 ) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (strcmp(final[j].NAME, final[j + 1].NAME)>0)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    if (colNum == 2 ) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (final[j].PRODUCTNUMBER > final[j + 1].PRODUCTNUMBER)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    if (colNum == 3) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (final[j].YEARSALESVOLUME > final[j + 1].YEARSALESVOLUME)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
    if (colNum == 4) {
        for (int i = 0; i < CompNum - 1; i++)
            for (int j = 0; j < CompNum - 1; j++)
            {
                if (final[j].MARKETPERCENT > final[j + 1].MARKETPERCENT)
                {
                    COMPANIES temp = final[j];
                    final[j] = final[j + 1];
                    final[j + 1] = temp;
                }
            }
    }
 ofstream outlink2;
    outlink2.open("C://Users//УЩ//Documents//Курсовая//ConsoleApplication1//outfile.txt", std::ios_base::app);
    outlink2 << "Отсортированная таблица: ";
    print(outlink2);
    outlink2.close();
    print(cout);
}
void print(std::ostream &out){
    out << "\n+--------------------+--------------------+--------------------+--------------------+\n"
            << "|" << setw(20) << left << "Название компании"
            << "|" << setw(20) << left << "Количество продуктов"
            << "|" << setw(20) << left << "Продажи за год"
            << "|" << setw(20) << left << "Часть рынка %" << "|";
     
        for (int i = 0; i < CompNum; i++) {
     
            out << "\n+--------------------+--------------------+--------------------+--------------------+\n"
                << "|" << setw(20) << left << final[i].NAME
                << "|" << setw(20) << left << final[i].PRODUCTNUMBER
                << "|" << setw(20) << left << final[i].YEARSALESVOLUME
                << "|" << setw(20) << left << final[i].MARKETPERCENT << "|";
        }
        out << "\n+--------------------+--------------------+--------------------+--------------------+\n";
}
И вообще неправильно в функции которая сортирует вводить или выводить данные. Данные должны поступать в функцию, сортироваться и возвращаться
DenKG
0 / 0 / 0
Регистрация: 23.12.2015
Сообщений: 303
07.06.2016, 19:11  [ТС]     Как сделать данный код более логичным и читаемым? #5
Namat,
COMPANIES-это у меня структура. Может, в этом проблема?
Миниатюры
Как сделать данный код более логичным и читаемым?  
Namat
12 / 12 / 5
Регистрация: 02.12.2014
Сообщений: 35
07.06.2016, 21:18     Как сделать данный код более логичным и читаемым? #6
Я имел ввиду это:
C++
1
2
3
4
5
6
7
8
9
10
11
12
 int i=0;
    int colNum;
    cout << "Введите номер стоблца, по которому хотите отсортировать таблицу:\n"
        << "1.Название компании; 2.Количество продуктов; 3.Продажи за год; 4.Часть рынка;" << endl;
    cin >> colNum;
sort(final,CompNum);
ofstream outlink2;
    outlink2.open("C://Users//УЩ//Documents//Курсовая//ConsoleApplication1//outfile.txt", std::ios_base::app);
    outlink2 << "Отсортированная таблица: ";
    print(outlink2,final);
    outlink2.close();
    print(cout,final);
В функцию print добавляешь второй аргумент COMPANIES *final. Так же стоит в функции sort делать выбор не через if, а через switch-case
Yandex
Объявления
07.06.2016, 21:18     Как сделать данный код более логичным и читаемым?
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru