Игра "Камень, Ножницы, Бумага"

[three_cats]

Выкладываю текущий вариант. Кому-то вдруг интересно будет!

#include <iostream>
#include <string>
#include <time.h>
#include <stdio.h>
#include <conio.h>
#include <locale>

using namespace std;

/* Функция преобразует буквы в полное написание */
string rewrite(char c) {
if (c == 'k') return "Камень";       
if (c == 'n') return "Ножницы";       
if (c == 'b') return "Бумага";             
}

/* Здесь вычисляется, кто победил */
int result(char cuser, char ccomp) {
if ((cuser == 'k') && (ccomp == 'b') || (cuser == 'n') && (ccomp == 'k') || (cuser == 'b') && (ccomp == 'n')) return -1;
if ((cuser == 'k') && (ccomp == 'n') || (cuser == 'n') && (ccomp == 'b') || (cuser == 'b') && (ccomp == 'k')) return 1;
if (cuser == ccomp) return 0;
}

int main() {
    
setlocale(LC_ALL, "Russian");    

srand(static_cast<unsigned>(time(NULL)));

int user_wins = 0;
int comp_wins = 0;

while ((user_wins < 10) && (comp_wins < 10)) {
cout << "Камень, ножницы, или бумага?(k/n/b): ";
char cuser;
cin >> cuser;

char ccomp;

int cc = 0;

/* Здесь ход компьютера */
cc = rand() % 3;

if (cc == 0) ccomp = 'k';
if (cc == 1) ccomp = 'n'; 
if (cc == 2) ccomp = 'b';

if (result(cuser,ccomp) == -1) { comp_wins++; cout << rewrite(cuser) << " против " << rewrite(ccomp) << ", Компьютер победил! (" << user_wins << ":" << comp_wins << ")" << endl; } else
if (result(cuser,ccomp) == 0) cout << rewrite(cuser) << " против " << rewrite(ccomp) << ", Ничья! (" << user_wins << ":" << comp_wins << ")" << endl; else
if (result(cuser,ccomp) == 1) { user_wins++; cout << rewrite(cuser) << " против " << rewrite(ccomp) << ", Вы победили! (" << user_wins << ":" << comp_wins << ")" << endl; }
else cerr << "Ошибка! Неверный ввод!" << endl;
}    

if (comp_wins > user_wins) cout << endl << "Компьютер победил!"; else
if (comp_wins < user_wins) cout << endl << "Поздравляем! Вы победили"; else
if (comp_wins = user_wins) cout << endl << "Ничья!";

cout << endl << "Нажмите любую клавишу...";
_getch();
return 0;    
}

Если поменять ввод русских букв на их латинские аналоги, все прекрасно работает.
Собственно, как это исправить(чтобы компилятор полностью понимал русский, не только вывод, но и ввод)?

И еще вопрос: как переписать код, используя константные ссылки в функциях?

[three_cats]

Вопросы остаются открытыми.

[Smitt&Wesson]

Мама родная! Вы что, в Индии учились? Массивы и циклы применять Вас не учили? А группировать данные при помощи операции "||"?

int result(char cuser, char ccomp) {
if (((cuser == 'к') && (ccomp == 'б')) || ((cuser == 'н') && (ccomp == 'к')) || ((cuser == 'б') && (ccomp == 'н'))) return -1;
// С остальными так-е
}

[three_cats]

Smitt&Wesson, так не очень наглядно.

Вы что, в Индии учились?

А что, в Индии плохо учат?

И где нужно применить массивы/циклы?

Smitt&Wesson,

Краткость - сестра таланта.

Но теща гонорара.

Все-таки, прошу ответить на вопросы, если можно, я же не просто так их задал.

[Smitt&Wesson]

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

Smitt&Wesson, так не очень наглядно.

А что, в Индии плохо учат?

Учат, может и хорошо, но платят программистам за количество написанных строк, а не за эффективность кода.

И где нужно применить массивы/циклы?

Да везде, во всех функциях. Могу показать пример.

Smitt&Wesson,

Но теща гонорара.

Эт уж точно .
Да и вот ещё, на счёт структурированности. Сразу бросается в глаза, что если cuser = ccomp, то результат 0.
Его вообще можно оформить как

if(cuser == ccomp) renern 0;

Если ещё поразмышлять, можно и остальные операторы упростить.
Например

string rewrite(char c) {
Switch(c)
{
  case 'к' : return "Камень";
  case 'н' : return "Ножницы";       
  case 'б' : return "Бумага";              
}
}

[three_cats]

платят программистам за количество написанныхстрок, а не за эффективность кода.

Очень познавательно, спасибо. Я об этом всегда только догадывался!

Да везде, во всех функциях. Могу показать пример.

Покажите, если не сложно.

И как сделать, чтобы понимал русский ввод?

И еще, как правильно использовать константные ссылки в функциях? Потому что выводилась всякая чепуха, сопровождаемая звуками.

Спасибо за замечания, учел:

/* Здесь вычисляется, кто победил */
int result(char cuser, char ccomp) {
if ((cuser == 'k') && (ccomp == 'b') || (cuser == 'n') && (ccomp == 'k') || (cuser == 'b') && (ccomp == 'n')) return -1;
if ((cuser == 'k') && (ccomp == 'n') || (cuser == 'n') && (ccomp == 'b') || (cuser == 'b') && (ccomp == 'k')) return 1;
if (cuser == ccomp) return 0;
}

[Smitt&Wesson]

И как сделать, чтобы понимал русский ввод?

Ну, например можно использовать не символ, а код клавиши. Как это зделать, сейчас нет времени думать. Но можно.

[three_cats]

Сообщение от Smitt&Wesson

Ну, например можно использовать не символ, а код клавиши. Как это зделать, сейчас нет времени думать. Но можно.

Вы меня немножко не поняли. Перейдите в эту тему, если есть время...

[Sciv]

Но теща гонорара.

Судя по тому, какой софт внедряется у меня на работе, у большинства его создателей общая жена

[BDA]

Чуть упростил:

#include <iostream>
#include <string>
#include <time.h>
#include <stdio.h>
#include <conio.h>
#include <locale>
#include <stdlib.h>

using namespace std;

/* Функция преобразует буквы в полное написание */
string rewrite(int c)
{
    if (c == 0) return "Камень";
    else if (c == 1) return "Ножницы";
    else return "Бумага";
}

int main()
{
    setlocale(LC_ALL, "Russian");

    srand(static_cast<unsigned>(time(NULL)));

    int user_wins = 0;
    int comp_wins = 0;

    while ((user_wins < 10) && (comp_wins < 10))
    {
        cout << "Камень, ножницы, или бумага?(К/Н/Б): ";
        unsigned char cuser;
        cin >> cuser;
        int cu;
        switch (cuser)
        {
        case 138:
            cu = 0;break;
        case 141:
            cu = 1;break;
        case 129:
            cu = 2;break;
        default:
            cout << "Ошибка" << endl;
            continue;
        }

        int cc = rand() % 3;

        int answer = (cu - cc + 3) % 3;

        if (answer == 1) {
            comp_wins++;
            cout << rewrite(cu) << " против " << rewrite(cc) << ", Компьютер победил! (" << user_wins << ":" << comp_wins << ")" << endl;
        }
        else if (answer == 0) {
            cout << rewrite(cu) << " против " << rewrite(cc) << ", Ничья! (" << user_wins << ":" << comp_wins << ")" << endl;
        }
        else if (answer == 2) {
            user_wins++;
            cout << rewrite(cu) << " против " << rewrite(cc) << ", Вы победили! (" << user_wins << ":" << comp_wins << ")" << endl;
        }
    }

    if (comp_wins > user_wins) cout << endl << "Компьютер победил!";
    else if (comp_wins < user_wins) cout << endl << "Поздравляем! Вы победили";
    else cout << endl << "Ничья!";

    cout << endl << "Нажмите любую клавишу...";
    _getch();
    return 0;
}