-1

В чем архитектурные недостатки данного кода и как можно их исправить?

import java.util.Scanner;

public class Main{
    public static void main(String ars[]){
        int l = 0;
        Scanner scn = new Scanner(System.in);
        System.out.println("Выберите всадника апокалипсиса");
        System.out.println("1 - желтый");
        System.out.println("2 - черный");
        System.out.println("3 - красный");
        System.out.println("4 - оранжевый");
        System.out.println("5 - синий");

        int j = scn.nextInt();
        switch(j){
        case 1:
            j = 1;
            System.out.println("Вы выбрали первого всадника");
            break;
        case 2:
            j = 2;
            System.out.println("Вы выбрали второгое всадника");
            break;
        case 3:
            j = 3;
            System.out.println("Вы выбрали третьего всадника");
            break;
        case 4:
            j = 4;
            System.out.println("Вы выбрали четвертого всадника");
            break;
        case 5:
            j = 5;
            System.out.println("Вы выбрали пятого всадника");
            break;
            default:
                System.out.println("Выберите всадников из списка!!");
        }
        int a = (int)(Math.random()*4+1);
            if (a == 1){
            System.out.println("Сегодня - Победил жёлтый всадник");
        }else if(a == 2){
            System.out.println("Сегодня - Победил черный всадник");
        }else if(a == 3){
            System.out.println("Сегодня - Победил красный всадник");
        }else if(a == 4){
            System.out.println("Сегодня - Победил оранжевый всадник");
        }else if(a == 5){
            System.out.println("Сегодня - Победил синий всадник");
        }   
            System.out.println(a);
            System.out.println(j);

            if (a==j){
            System.out.println("Вы победили, вы молодец!");
            }else
                System.out.println("Вы проиграли. Ничего страшного, бывает");
    }
}

10
  • убери j = 2; вот эти присвоения 16 сен 2015 в 18:12
  • Поясните, в чем возникла проблема? Напишите информативный заголовок. В текущей формулировке этот вопрос будет удален 16 сен 2015 в 18:13
  • да, не очень понятно, что должно быть и в чем проблема, только в "чистоте" кода? 16 сен 2015 в 18:13
  • @DenisZinkovskiy, думаю да, тут речь о чистоте кода. А закрывать не надо.
    – Qwertiy
    16 сен 2015 в 18:17
  • 2
    Автор, я вам немного подправил заголовок и текст.
    – Nick Volynkin
    16 сен 2015 в 19:05

1 ответ 1

1

Код плохо обозрим, это да.

Я бы сделал так: поместил бы все сообщения, относящиеся к конкретному всаднику, в отдельную структуру данных. И обращался бы через неё.

Тогда код бы выглядел как-то так:

System.out.println("Выберите всадника апокалипсиса");
for (Knight k : knights)
    System.out.println(k.Texts.Prompt);

Knight chosenKnight = null;
while (chosenKnight == null)
{
    int j = scn.nextInt();
    if (j < 1 || j > knights.size())
    {
        System.out.println("Выберите всадников из списка!!");
        continue;
    }
    chosenKnight = knights[j - 1];
}

System.out.println(chosenKnight.Texts.ChoiceConfirmation);

int a = (int)(Math.random() * knights.size());
Knight winningKnight = knights[a];
System.out.println(winningKnight.Texts.WinnerAnnouncement);

if (chosenKnight == winningKnight)
    System.out.println("Вы победили крч, красавец чувак!!");
else
    System.out.println("Вы просрали свои деньги крч");

(Впрочем, остальные тексты я бы тоже куда-то выгрузил.)

4
  • System.out.println("Пацан к успеху шел, не получилось, не фартануло.");
    – Nick Volynkin
    16 сен 2015 в 18:57
  • Там ведь j имелась в виду?
    – Nick Volynkin
    16 сен 2015 в 18:59
  • 2
    Всадников апокалипсиса четыре штуки, а не 5. Чума, Война, Голод и Смерть. Миф возникший из суровых реалий средневековой Европы.
    – igumnov
    16 сен 2015 в 19:42
  • @igumnov: Точно! Подправил код, чтобы он был совместим с этим.
    – VladD
    16 сен 2015 в 20:52

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.