Упростить код

@anonimious · Регистрация: 17.10.2011

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

Добрый день, подскажите, можно ли упростить код (используя уровень знаний как в оригинале)?

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 <iostream>
#include <time.h>
#include <conio.h>
using namespace std;
void main(){
       srand(unsigned(time(NULL)));
       setlocale(LC_ALL,"rus");
       int size1, size2; char sw; bool f1=false;
       cout<<"Введите размер первого массива "; cin>>size1;
       cout<<"Введите размер второго массива "; cin>>size2;
       int *mas1=new int[size1]; int *mas2=new int[size2];
       int *mas3=new int[size1+size2];
       for(int i=0;i<size1;i++) mas1[i]=rand()%90+10;
       for(int i=0;i<size2;i++) mas2[i]=rand()%90+10; cout<<endl;
       //for(int i=0;i<size1;i++) cout<<mas1[i]<<' '; cout<<endl;
       //for(int i=0;i<size2;i++) cout<<mas2[i]<<' '; cout<<endl<<endl;
       cout<<"Что с ними нужно сделать?\n"<<endl;
       cout<<"a-вывести элементы обоих массивов\n";
       cout<<"b-вывести общие элементы обоих массивов\n";
       cout<<"c-вывести элементы первого массива, которые не входят во второй\n";
       cout<<"d-вывести элементы вторoго массива, которые не входят в первый\n";
       cout<<"e-вывести не общие элементы\n\n";
        sw=getch();
        switch(sw){
            case 'a':{
                for(int i=0;i<size1;i++) cout<<mas1[i]<<' '; cout<<endl;
                for(int i=0;i<size2;i++) cout<<mas2[i]<<' '; cout<<endl; break;}
            case 'b':{
                for(int i=0;i<size1;i++)
                    if(mas1[i]==mas2[i]) cout<<mas1[i]<<endl; f1=true; break;}
            case 'c':{
                for(int i=0;i<size1;i++)
                    for(int j=0;j<size2;j++)
                        if(mas1[i]!=mas2[j]) {cout<<mas1[i]<<' '; break;}
                        cout<<endl; break;}
            case 'd':{
                for(int i=0;i<size2;i++)
                    for(int j=0;j<size2;j++)
                        if(mas2[i]!=mas1[j]) {cout<<mas2[i]<<' '; break;}
                        cout<<endl; break;}
            case 'e':{
                for(int i=0;i<size1;i++)
                    for(int j=0;j<size2;j++)
                        if(mas1[i]!=mas2[j]) {mas3[i]=mas1[i]; break;}
                for(int i=0;i<size2;i++)
                    for(int j=0;j<size2;j++)
                        if(mas2[i]!=mas1[j]) {mas3[i+size1]=mas2[i]; break;}
                for(int i=0;i<(size1+size2);i++)
                    for(int j=0;j<(size1+size2);j++)
                        if(mas3[i]!=mas3[j]) {cout<<mas3[i]<<' '; break;}
                cout<<endl; break;}
            default : {cout<<"Неправильный выбор\n";}}
       delete []mas1; delete []mas2; delete []mas3;
       if(f1) cout<<"Общих элементов нет"<<endl;}

@Mиxaил · 30.10.2011, 15:29

Какую задачу этот код решает ( не хочу углубляться )?

@Deviaphan · 30.10.2011, 15:30

Только если более читабельным сделать. По коду вроде нечего упрощать.
main должно возвращать значение, соответственно быть int, а не void.

@Mиxaил · 30.10.2011, 15:34

Я бы делал через STL ( вектор и множество ). Получилось бы короче...

@Deviaphan · 30.10.2011, 15:36

Сообщение от Mиxaил

Я бы делал через STL

Не вариант, т.к.

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

используя уровень знаний как в оригинале

@anonimious · 30.10.2011, 15:48 **[ТС]**

Mиxaил, описано же

C++
1
2
3
4
5
6
cout<<"Что с ними нужно сделать?\n"<<endl;
           cout<<"a-вывести элементы обоих массивов\n";
           cout<<"b-вывести общие элементы обоих массивов\n";
           cout<<"c-вывести элементы первого массива, которые не входят во второй\n";
           cout<<"d-вывести элементы вторoго массива, которые не входят в первый\n";
           cout<<"e-вывести не общие элементы\n\n";

Deviaphan, работа в VS2010, там так можно

Добавлено через 2 минуты
Deviaphan, Если не трудно, приведите пример более читабельного

Jupiter · 30.10.2011, 15:52

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

работа в VS2010, там так можно

вы выложили код на общественное обозрение, а у меня MinGW и сдесь так нельзя!!!

Добавлено через 3 минуты
пример более читабельного кода

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
switch(sw)
{
case 'a':
    for(int i = 0; i < size1; ++i)
        cout << mas1[i] << '\n';
 
    for(int i = 0; i < size2; ++i)
        cout << mas2[i] << '\n'; 
 
    break;
 
case 'b':
    for(int i = 0; i < size1; ++i)
    {
       if(mas1[i] == mas2[i]) 
           cout << mas1[i] << endl; 
    }
    f1 = true; 
    break;
//...
}

@anonimious · 30.10.2011, 15:53 **[ТС]**

Jupiter, тогда

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 <iostream>
#include <time.h>
#include <conio.h>
using namespace std;
int main(){
       srand(unsigned(time(NULL)));
           setlocale(LC_ALL,"rus");
           int size1, size2; char sw; bool f1=false;
           cout<<"Введите размер первого массива "; cin>>size1;
           cout<<"Введите размер второго массива "; cin>>size2;
           int *mas1=new int[size1]; int *mas2=new int[size2];
           int *mas3=new int[size1+size2];
           for(int i=0;i<size1;i++) mas1[i]=rand()%90+10;
           for(int i=0;i<size2;i++) mas2[i]=rand()%90+10; cout<<endl;
           //for(int i=0;i<size1;i++) cout<<mas1[i]<<' '; cout<<endl;
           //for(int i=0;i<size2;i++) cout<<mas2[i]<<' '; cout<<endl<<endl;
           cout<<"Что с ними нужно сделать?\n"<<endl;
           cout<<"a-вывести элементы обоих массивов\n";
           cout<<"b-вывести общие элементы обоих массивов\n";
           cout<<"c-вывести элементы первого массива, которые не входят во второй\n";
           cout<<"d-вывести элементы вторoго массива, которые не входят в первый\n";
           cout<<"e-вывести не общие элементы\n\n";
                sw=getch();
                switch(sw){
                        case 'a':{
                                for(int i=0;i<size1;i++) cout<<mas1[i]<<' '; cout<<endl;
                                for(int i=0;i<size2;i++) cout<<mas2[i]<<' '; cout<<endl; break;}
                        case 'b':{
                                for(int i=0;i<size1;i++)
                                        if(mas1[i]==mas2[i]) cout<<mas1[i]<<endl; f1=true; break;}
                        case 'c':{
                                for(int i=0;i<size1;i++)
                                        for(int j=0;j<size2;j++)
                                                if(mas1[i]!=mas2[j]) {cout<<mas1[i]<<' '; break;}
                                                cout<<endl; break;}
                        case 'd':{
                                for(int i=0;i<size2;i++)
                                        for(int j=0;j<size2;j++)
                                                if(mas2[i]!=mas1[j]) {cout<<mas2[i]<<' '; break;}
                                                cout<<endl; break;}
                        case 'e':{
                                for(int i=0;i<size1;i++)
                                        for(int j=0;j<size2;j++)
                                                if(mas1[i]!=mas2[j]) {mas3[i]=mas1[i]; break;}
                                for(int i=0;i<size2;i++)
                                        for(int j=0;j<size2;j++)
                                                if(mas2[i]!=mas1[j]) {mas3[i+size1]=mas2[i]; break;}
                                for(int i=0;i<(size1+size2);i++)
                                        for(int j=0;j<(size1+size2);j++)
                                                if(mas3[i]!=mas3[j]) {cout<<mas3[i]<<' '; break;}
                                cout<<endl; break;}
                        default : {cout<<"Неправильный выбор\n";}}
           delete []mas1; delete []mas2; delete []mas3;
           if(f1) cout<<"Общих элементов нет"<<endl; return 0;}

@Deviaphan · 30.10.2011, 16:01

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

там так можно

То что это компилируется не значит, что так можно. Это противоречит стандарту. Программа должна возвращать код ошибки при завершении, для использования её в скриптах и т.п.

@Thinker · 30.10.2011, 16:05

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

Добрый день, подскажите, можно ли упростить код (используя уровень знаний как в оригинале)?

Можно, пропишите отдельно функции инициализации и вывода элементов массива, тогда дублировать циклы не надо. Вообще лучше всю программу на отдельные функции разбить. Зачем все лепить в одной функции.

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	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 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@Mиxaил 542 / 447 / 162 Регистрация: 10.12.2009 Сообщений: 1,857
	30.10.2011, 15:29
	Какую задачу этот код решает ( не хочу углубляться )? 0

@Deviaphan Делаю внезапно и красиво 1313 / 1228 / 72 Регистрация: 22.03.2011 Сообщений: 3,744
	30.10.2011, 15:30
	Только если более читабельным сделать. По коду вроде нечего упрощать. main должно возвращать значение, соответственно быть int, а не void. 0

@Mиxaил 542 / 447 / 162 Регистрация: 10.12.2009 Сообщений: 1,857
	30.10.2011, 15:34
	Я бы делал через STL ( вектор и множество ). Получилось бы короче... 0