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

Как можно сделать программу более компактнее - C++

Восстановить пароль Регистрация
 
KostyaKulakov
Заблокирован
26.07.2012, 09:29     Как можно сделать программу более компактнее #1
решил использовать новый способ в программе. так вот как можно этот код сделать болие компактнее:
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
#include <iostream>
#include <string>
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "Russian");
    char otvet = 'n';
    main:
    double a,b,c;
    string kurs;
    if (otvet == 'Y' || otvet == 'y')
    {
    cout << endl;
    goto test;
    }
    else
    {
    test:
    cout << "Выберите валюту:" <<"\n";
    cout << "1. Доллары" <<"\n";
    cout << "2. Гривны" <<"\n";
    cout << "3. Евро" <<"\n";
    }
    cin >> a;
            if (a == 1)
            {
            kurs = "Долларов";
            }
            else if (a == 2)
            {
            kurs = "Гривнов";
            }
            else if (a == 3)
            {
            kurs = "Евро";
            }
            else
            {
                goto lol;
            }
    cout << endl << "Введите кол-во " << kurs<<": ";
    cin >> b;
                if (a == 1)
                {
                c = b * 31;
                cout << endl << c << " " << "Рублей." <<"\n";
                goto povtor;
                }
                else if (a == 2)
                {
                c = b * 4;
                cout << endl << c << " " << "Рублей." <<"\n";
                goto povtor;
                }
                else if (a == 3)
                {
                c = b * 38;
                cout << endl << c << " " << "Рублей." <<"\n";
                goto povtor;
                }
                else
                {
                lol:
                cout << "LOL!?" <<"\n";
                }
        povtor:
                cout << endl <<"Хотите повторить? - (Y/N) :";
                cin >> otvet;
                if (otvet == 'y' || otvet == 'Y')
                {
                    goto main;
                }
                else if (otvet == 'n' || otvet == 'N')
                {
                    cout << endl << "Спасибо за использование программы :)" << endl ;
                }
    return 0;
}
Добавлено через 11 минут
+ выявилась ошибка. программа не можит сразу сделать так 3,3. только после повтора нормально проходит. работает только 3.3
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
26.07.2012, 09:29     Как можно сделать программу более компактнее
Посмотрите здесь:

как можно более просто написать эту программку(более понятным языком для начинающего) C++
C++ Как можно сделать программу быстрей?
Сделать программу более компактной + Ручной ввод размерности квадратной матрицы. C++
Как можно сделать что бы в консольном окне можно было вызвать контекстное меню как в win cmd? C++
C++ Нужно сделать, чтобы в Edit можно было вводить не более 3-х символов
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Avazart
 Аватар для Avazart
6897 / 5137 / 252
Регистрация: 10.12.2010
Сообщений: 22,578
Записей в блоге: 17
26.07.2012, 09:52     Как можно сделать программу более компактнее #2
1. Не юзай goto, используй ф-ции.
2. Используй switch/case вместо if()
3. "main:" не очень хорошее имя так как есть еще и int main()
4. Думаю стоит объявить константы вместо магических чисел "c = b * 38;"
Intel~lect
 Аватар для Intel~lect
135 / 124 / 2
Регистрация: 03.07.2012
Сообщений: 355
26.07.2012, 09:54     Как можно сделать программу более компактнее #3
KostyaKulakov, Попробуй переписать программу без меток. В этих goto очень трудно разобраться.
KostyaKulakov
Заблокирован
26.07.2012, 10:03  [ТС]     Как можно сделать программу более компактнее #4
Цитата Сообщение от Intel~lect Посмотреть сообщение
KostyaKulakov, Попробуй переписать программу без меток. В этих goto очень трудно разобраться.
я вот и говарю. прочитал про эти гото в книге. решил попробывать на практике. вроде вышло нечего. но код запутанный получается

Добавлено через 46 секунд
Цитата Сообщение от Avazart Посмотреть сообщение
1. Не юзай goto, используй ф-ции.
2. Используй switch/case вместо if()
3. "main:" не очень хорошее имя так как есть еще и int main()
4. Думаю стоит объявить константы вместо магических чисел "c = b * 38;"
всмысле ф-ции?
Schizorb
 Аватар для Schizorb
508 / 460 / 16
Регистрация: 07.04.2012
Сообщений: 865
Записей в блоге: 1
Завершенные тесты: 1
26.07.2012, 10:04     Как можно сделать программу более компактнее #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
#include <iostream>
#include <string>
using namespace std;
 
int main()
{
    char ans;
    
    double val;
    const int currencies = 3;
    string names[currencies] = {"dollar", "hryvnia", "euro"};
    double index[currencies] = {31, 4, 38};
    
    do
    {
        int choose;
        cout << "Choose currency:" <<"\n";
        for(int i = 0; i < currencies; ++i)
            std::cout << i + 1 << ". " << names[i] << "\n";
        
        cin >> choose;
 
        if(choose < 1 || choose > 3)
            std::cout << "Wrong choose!\n";
        else
        {
            cout << endl << "Input " << names[choose - 1] <<" amount: ";
            cin >> val;
            std::cout << val * index[choose - 1] << " rubles" << "\n";
        }
        
        std::cout << "\nTry again? - (Y/N): ";
        std::cin >> ans;
    }
    while(ans == 'y');
 
    return 0;
}
Avazart
 Аватар для Avazart
6897 / 5137 / 252
Регистрация: 10.12.2010
Сообщений: 22,578
Записей в блоге: 17
26.07.2012, 10:09     Как можно сделать программу более компактнее #6
всмысле ф-ции?
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
void test() // ф-ция
 {
    //test:
    cout << "Выберите валюту:" <<"\n";
    cout << "1. Доллары" <<"\n";
    cout << "2. Гривны" <<"\n";
    cout << "3. Евро" <<"\n";
 }
 
//------------------------------------------------------
int main()
{
//....
 if (otvet == 'Y' || otvet == 'y')
  {
     cout << endl;
     //goto test;
     test();
  }
 else  test();
//....
}
Yandex
Объявления
26.07.2012, 10:09     Как можно сделать программу более компактнее
Ответ Создать тему
Опции темы

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