Ваша проблема - ваши блоки выбора else / if.Каждый раз вы позволяете этому продолжаться, даже если выбор прошел.Вы также не проверяете должным образом.В каждой паре if / else, которую вы используете, вы проверяете
if(user X && pc Y) ...
else if (pc Z) ...
. Это будет оценка одного из двух блоков, если пользователь играет X, а ПК Y, ИЛИ, если ПК играет z.Вы должны настроить его так:
if(user X)
if(pc Y) ...
else if (pc Z) ...
Или
if(user X && pc Y) ...
else if(user X && pc Z) ...
Я предлагаю первое.Это немного более эффективно, потому что он проверяет пользователя X только один раз.Это не даст половины заметной разницы.
Вы должны сделать это так:
if(user plays paper)
if(pc plays rock)
...
else if (pc plays scissors)
...
else if(user plays rock)
if(pc plays scissors)
...
else if (pc plays paper)
...
else if(user plays scissors)
if(pc plays rock)
...
else if (pc plays paper)
...