Игра "Кубики" - упростить код

@BeardVis · Регистрация: 22.07.2017

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

Всем привет! Нашел задачку:
Написать игру «Кубики». Пользователь и компьютер по очереди бросают 2 кубика. Победитель – тот, у кого по результатам 3х бросков сумма больше. Предусмотреть красивый интерфейс игры.
Написал) Все работает так как и хотел, но может есть какие-то замечания, или возможность упростить. Особенно придираюсь к лесенке else if, чет другого не придумал.

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
93
94
95
#include <iostream>
#include <conio.h>
#include <cstdlib>
#include <ctime>
 
using namespace std;
 
int main (){
    srand(time(0));
    int n, one, two, co=0, ol, k=0, var, sum1=0, sum2=0;
    bool kto=true;
    cout << "Privet davaii sigraem v kosti.\nPobedit tot u kogo suma 3 broskov budet bolshe\n\n";
    while (k<3)
    {
        cout << "=========================\n1 - Brositj kubik\n2 - Vijti\nViberi variant: ";
        cin >> var;
        if (var==1)
        {
            for (int j=0; j<=1; j++)
                {
                    one=1+(rand()%6); 
                    two=1+(rand()%6);
                    cout << "=========================\n";
                    if (kto) cout << "\tTvoi kubiki\n";
                    if (!kto) cout << "\tMoi kubiki\n";
                    cout << "=========================\n";
                    for (int i=1; i<=5; i++)
                    {
                            if (i==1) n=1;
                            else if (i==2 && one==1) n=2;
                            else if (i==2 && one==3 || i==2 && one==2) n=3;
                            else if (i==2 && one==4 || i==2 && one==5 || i==2 && one==6) n=4;
                            else if (i==3 && one==3 || i==3 && one==1 || i==3 && one==5) n=5;
                            else if (i==3 && one==2 || i==3 && one==4) n=2;
                            else if (i==3 && one==6) n=4;
                            else if (i==4 && one==1) n=2;
                            else if (i==4 && one==3 || i==4 && one==2) n=6;
                            else if (i==4 && one==4 || i==4 && one==5 || i==4 && one==6) n=4;
                            else if (i==5) n=7;
                        switch (n){
                            case 1:
                            cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                            break;
                            case 2:
                            cout << (char)186 << "       " << (char)186;
                            break;
                            case 3:
                            cout << (char)186 << " o     " << (char)186;
                            break;
                            case 4:
                            cout << (char)186 << " o   o " << (char)186;
                            break;
                            case 5:
                            cout << (char)186 << "   o   " << (char)186;
                            break;
                            case 6:
                            cout << (char)186 << "     o " << (char)186;
                            break;
                            case 7:
                            cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                            break;
                        }
                        if (co%2==0) {
                            cout << "\t";
                            i--;
                            ol=one;
                            one=two;}
                        else {
                            cout << endl;
                            one=ol; }
                    co++;
                    }
                    if (kto){
                        sum1=sum1+one+two;
                        kto=false;
                        continue;
                    }
                    if(!kto){
                        sum2=sum2+one+two;
                        kto=true;
                    }
                }
            }
            else if (var==2) return 0;
            else {
                cout << "\n!!! Viberi variant 1 ili 2 !!!\n";
                continue;}
                k++;
    }
    cout << "\nTvoj rezultat = " << sum1 << endl << "Moj rezultat = " << sum2 << endl;
    if (sum1>sum2)
        cout << "\n\tTi pobedil ;)";
    else cout << "\n\tTi proigral :(";
    getch ();
}

@GbaLog- · 22.07.2017, 21:55

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

Предусмотреть красивый интерфейс игры.

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

Написал)

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

Privet davaii sigraem v kosti.\nPobedit tot u kogo suma 3 broskov budet bolshe

это красивый интерфейс? )

@BeardVis · 22.07.2017, 22:00 **[ТС]**

Ну... Для меня да, я только недавно начал заниматься. Зато кубики классные вышли

@Antikl · 23.07.2017, 04:58

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

Зато кубики классные вышли

Как совет что можно сделать: ну разложи на функции, поставь Коментарии, по ровняй, можно разделить на файлы.

@GbaLog- · 23.07.2017, 10:48

для начала варнинги исправьте хотя бы.

Code
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
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
a.cpp: In function 'int main()':
a.cpp:9:15: warning: conversion to 'unsigned int' from 'time_t {aka long int}' may change the sign of the result [-Wsign-conversion]
     srand(time(0));
           ~~~~^~~
a.cpp:31:43: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==2 && one==3 || i==2 && one==2) n=3;
                                      ~~~~~^~~~~~~~~
a.cpp:32:43: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==2 && one==4 || i==2 && one==5 || i==2 && one==6) n=4;
                                      ~~~~~^~~~~~~~~
a.cpp:32:79: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==2 && one==4 || i==2 && one==5 || i==2 && one==6) n=4;
                                                                          ~~~~~^~~~~~~~~
a.cpp:33:43: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==3 && one==3 || i==3 && one==1 || i==3 && one==5) n=5;
                                      ~~~~~^~~~~~~~~
a.cpp:33:79: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==3 && one==3 || i==3 && one==1 || i==3 && one==5) n=5;
                                                                          ~~~~~^~~~~~~~~
a.cpp:34:43: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==3 && one==2 || i==3 && one==4) n=2;
                                      ~~~~~^~~~~~~~~
a.cpp:37:43: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==4 && one==3 || i==4 && one==2) n=6;
                                      ~~~~~^~~~~~~~~
a.cpp:38:43: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==4 && one==4 || i==4 && one==5 || i==4 && one==6) n=4;
                                      ~~~~~^~~~~~~~~
a.cpp:38:79: warning: suggest parentheses around '&&' within '||' [-Wparentheses]
                             else if (i==4 && one==4 || i==4 && one==5 || i==4 && one==6) n=4;
                                                                          ~~~~~^~~~~~~~~
a.cpp:42:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                           ^~~
a.cpp:42:56: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                        ^~~
a.cpp:42:70: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                      ^~~
a.cpp:42:83: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                                   ^~~
a.cpp:42:96: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                                                ^~~
a.cpp:42:109: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                                                             ^~~
a.cpp:42:122: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                                                                          ^~~
a.cpp:42:135: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                                                                                       ^~~
a.cpp:42:148: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)201 << (char)205  << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)187;
                                                                                                                                                    ^~~
a.cpp:45:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << "       " << (char)186;
                                           ^~~
a.cpp:45:69: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << "       " << (char)186;
                                                                     ^~~
a.cpp:48:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << " o     " << (char)186;
                                           ^~~
a.cpp:48:69: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << " o     " << (char)186;
                                                                     ^~~
a.cpp:51:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << " o   o " << (char)186;
                                           ^~~
a.cpp:51:69: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << " o   o " << (char)186;
                                                                     ^~~
a.cpp:54:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << "   o   " << (char)186;
                                           ^~~
a.cpp:54:69: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << "   o   " << (char)186;
                                                                     ^~~
a.cpp:57:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << "     o " << (char)186;
                                           ^~~
a.cpp:57:69: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)186 << "     o " << (char)186;
                                                                     ^~~
a.cpp:60:43: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                           ^~~
a.cpp:60:56: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                        ^~~
a.cpp:60:69: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                     ^~~
a.cpp:60:82: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                                  ^~~
a.cpp:60:95: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                                               ^~~
a.cpp:60:108: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                                                            ^~~
a.cpp:60:121: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                                                                         ^~~
a.cpp:60:134: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                                                                                      ^~~
a.cpp:60:147: warning: use of old-style cast [-Wold-style-cast]
                             cout << (char)200 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)205 << (char)188;
                                                                                                                                                   ^~~
a.cpp:79:34: warning: 'ol' may be used uninitialized in this function [-Wmaybe-uninitialized]
                         sum2=sum2+one+two;
                              ~~~~^~~~
a.cpp:40:25: warning: 'n' may be used uninitialized in this function [-Wmaybe-uninitialized]
                         switch (n){
                         ^~~~~~

@DevAlone · 23.07.2017, 20:09

Здесь проще переписать заново)
Вот тебе "красивый" интерфейс, остальное доделай сам:

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
#include <iostream>
 
const size_t LINES_PER_DICE = 5;
const size_t DICES_COUNT = 6;
 
const char* dicePictures[DICES_COUNT][LINES_PER_DICE] = {
    { "-------",
        "-     -",
        "-  .  -",
        "-     -",
        "-------" },
    { "-------",
        "- .   -",
        "-     -",
        "-   . -",
        "-------" },
    { "-------",
        "-  .  -",
        "-  .  -",
        "-  .  -",
        "-------" },
    { "-------",
        "- . . -",
        "-     -",
        "- . . -",
        "-------" },
    { "-------",
        "- . . -",
        "-  .  -",
        "- . . -",
        "-------" },
    { "-------",
        "- ... -",
        "-     -",
        "- ... -",
        "-------" },
};
 
void drawDices(unsigned char value1, unsigned char value2)
{
    if (value1 < 1 || value1 > 6
        || value2 < 1 || value2 > 6)
        return;
 
    // построчно рисует два кубика
    for (size_t i = 0; i < LINES_PER_DICE; i++) {
        std::cout << dicePictures[value1 - 1][i] << " " << dicePictures[value2 - 1][i] << std::endl;
    }
}
 
int main()
{
    srand(time(0));
 
    while (true) {
 
        drawDices(rand() % 6 + 1, rand() % 6 + 1);
 
        std::cin.get();
    }
}

Новые блоги и статьи Все статьи Все блоги /
делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .

@BeardVis 1 / 1 / 0 Регистрация: 22.07.2017 Сообщений: 57
	22.07.2017, 22:00 [ТС]
	Ну... Для меня да, я только недавно начал заниматься. Зато кубики классные вышли 0