Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
11 / 11 / 7
Регистрация: 23.12.2015
Сообщений: 950

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

07.06.2016, 16:48. Показов 831. Ответов 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";
}
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
07.06.2016, 16:48
Ответы с готовыми решениями:

Как сделать данный код нормальным?
В общем написал код ... но он оказался уж очень галимым((( вот он using System; namespace laba2 { class Program { ...

Как линк специфического формата сделать читаемым моим браузером?
link /assets/course14/contacts.png браузер mazila firefox 38 а то он его не видит как бы

Как сделать данный код с помощью OUT\REF ???
нужно этот же код, сделать через OUT и REF (методы процедуры) как это сделать ? using System; using System.Collections.Generic; ...

5
Хитрая блондиночка $)
 Аватар для Hikari
1472 / 988 / 399
Регистрация: 21.12.2015
Сообщений: 3,785
07.06.2016, 16:52
Ой ой ой... А STL или хотя бы qsort() для сортировки использовать не желаешь?
0
11 / 11 / 7
Регистрация: 23.12.2015
Сообщений: 950
07.06.2016, 18:21  [ТС]
Цитата Сообщение от Hikari Посмотреть сообщение
Ой ой ой... А STL или хотя бы qsort() для сортировки использовать не желаешь?
Нет, я пока такое делать не умею ))
Мне нужно именно запись в файл вывести в другую функцию

Добавлено через 1 час 25 минут
Ну так что, мне никто не поможет?)
0
12 / 12 / 5
Регистрация: 02.12.2014
Сообщений: 35
07.06.2016, 18:50
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";
}
И вообще неправильно в функции которая сортирует вводить или выводить данные. Данные должны поступать в функцию, сортироваться и возвращаться
0
11 / 11 / 7
Регистрация: 23.12.2015
Сообщений: 950
07.06.2016, 19:11  [ТС]
Namat,
COMPANIES-это у меня структура. Может, в этом проблема?
Миниатюры
Как сделать данный код более логичным и читаемым?  
0
12 / 12 / 5
Регистрация: 02.12.2014
Сообщений: 35
07.06.2016, 21:18
Я имел ввиду это:
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
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
07.06.2016, 21:18
Помогаю со студенческими работами здесь

Как сделать код более лаконичным?
Есть вот такой вот кусочек кода, хотелось бы сделать его покороче и покрасивее. async public static void Sttory() { ...

Как сделать код более компактным?
$(document).ready(function(){ var sumNum = parseInt(document.getElementById(&quot;digit1&quot;).innerText) +...

Как сделать код более правильным?
Здравствуйте. Сделал сот такую штуку для сайта https://jsfiddle.net/yx608nb3/1/ . Правда, получился костыль. Подскажите, как можно этот код...

Подскажите как сделать код более аккуратным
На одном из своих сайтов использую таймер, задача была сделать таймер цикличным, чтобы каждые три дня он обновлялся заново, я решил сделать...

Подскажите, как сделать код более универсальным?
просьба помочь сократить код правильнее сказать переделать в универсальный! ... var elem = document.getElementById('q6'); var...


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

Или воспользуйтесь поиском по форуму:
6
Ответ Создать тему
Новые блоги и статьи
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 . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru