Java: Если оператор для обработки суммы вы хотите, в то время как отбрасывать остальные?

Question

Java: Если оператор для обработки суммы вы хотите, в то время как отбрасывать остальные?

1

У меня есть следующий скрипт:

public void addcheese(int addcheese) {
    if (cheeseamount + addcheese < mincapacity) {
        System.out.println("Sorry no more cheese can be removed.");
    } else if (cheeseamount + addcheese <= maxcapacity) {
        if ((cheeseamount + addcheese > 1900) && (cheeseamount + addcheese < 2000)) {
            System.out.println("Warning : The box is nearing it maximum capacity.");
        }
        cheeseamount = cheeseamount + addcheese;
    }
    else {
        System.out.println("This storage box has reached its maximum capacity. You cannot fill it any more");
    }
}

(Примечание: в макспроизводстве конструктора = 2000)

Скрипт работает так, как должен, однако есть проблема в том, что если бы я должен был ввести число, подобное 3400, оно все равно напечатало бы сообщение "Это хранилище достигло максимума...", не добавляя ничего. Как бы отредактировать сценарий, в этом случае добавьте в поле вверх до максимальной емкости и отбросьте лишнюю сумму. Спасибо.

fge 27 дек. 2014, в 03:23

Источник

0

Ваш интервал нечетный по частям, что затрудняет анализ. Это может быть принятым стилем; Я просто никогда не видел это раньше.
Carcigenicate 27 дек. 2014, в 01:34
1

Я не понимаю, что вы хотите. Но если вы заполните коробку до максимальной вместимости и cheeseamount от избытка, что будет с cheeseamount ? Это очень простой ответ, если я правильно понимаю проблему. Вы должны просто иметь возможность использовать простое назначение внутри последнего блока else .
ajb 27 дек. 2014, в 01:34
2

cheeseamount = Math.min(maxcapacity, cheeseamount + addcheese); ?
fge 27 дек. 2014, в 01:35
1

@fge Тебе это даже не нужно. В else вам понадобится только cheeseamount = maxcapacity; ,
Thor84no 27 дек. 2014, в 01:38
0

@ Thor84no но с этим тебе даже не нужно последнее
fge 27 дек. 2014, в 01:39
0

@fge Но тогда вы измените программу сверх того, что он просил. Здесь нет ничего, указывающего на то, что вам по-прежнему не нужно печатать сообщение о том, что оно достигло максимальной емкости.
Thor84no 27 дек. 2014, в 01:40

Показать ещё 4 комментария

Теги:

java

if-statement

storage

3 ответа

Ещё вопросы

Ваш интервал нечетный по частям, что затрудняет анализ. Это может быть принятым стилем; Я просто никогда не видел это раньше.
Я не понимаю, что вы хотите. Но если вы заполните коробку до максимальной вместимости и cheeseamount от избытка, что будет с cheeseamount ? Это очень простой ответ, если я правильно понимаю проблему. Вы должны просто иметь возможность использовать простое назначение внутри последнего блока else .
cheeseamount = Math.min(maxcapacity, cheeseamount + addcheese); ?
@fge Тебе это даже не нужно. В else вам понадобится только cheeseamount = maxcapacity; ,
@ Thor84no но с этим тебе даже не нужно последнее
@fge Но тогда вы измените программу сверх того, что он просил. Здесь нет ничего, указывающего на то, что вам по-прежнему не нужно печатать сообщение о том, что оно достигло максимальной емкости.

fge · Answer 1 · 2014-12-27T00-02-00.000Z

Вот решение:

public void addcheese(int addcheese) 
{
    final int target = cheeseamount + addcheese;

    if (target < mincapacity) {
        System.out.println("Sorry, no cheese can be removed");
        return;
    }

    if (target >= maxcapacity) {
        cheeseamount = maxcapacity;
        System.out.println("Box is full");
        return;
    }

    cheeseamount = target;
    if (cheeseamount > 1900)
        System.out.println("Warning, box is nearing its maximum capacity");
}

Заметки:

оставил упражнение читателю: предупредите о количестве сыра впустую!
соглашения об именах: Java использует camelCase; предпочтительнее это должны быть addCheese, cheeseAmount и т.д.

Zarwan · Answer 2 · 2014-12-26T23-51-00.000Z

Один из способов добиться этого - это изменить ваше выражение else. В заявлении else вы можете просто установить количество сыра до максимального количества, добавляя как можно больше сыра и отбрасывая остальное:

else {
            cheeseamount = 2000;
            System.out.println("This storage box has now reached its maximum capacity.");

       }

Однако здесь вы теряете количество сыра, которое вы отбрасываете. Если вы хотите сохранить этот номер, вы можете просто создать новую переменную:

else {
            int extraCheese = cheeseamount + addcheese - 2000;
            cheeseamount = 2000;
            System.out.println("This storage box has now reached its maximum capacity.");

       }

Замечание: было бы неплохо сохранить максимальную емкость и любые другие магические числа в качестве констант. Магические числа - это в основном случайные числа, которые появляются во всем вашем коде, что любой, кто смотрит на ваш код, не имеет смысла. Создавая константу с описательным именем, вы можете дать понять другим программистам, что представляет собой число, и вы также можете легко изменить его в будущем.

Konstantinos Chalkias · Answer 3 · 2014-12-26T23-27-00.000Z

Добавьте другое, если еще с этим условием:

else if (cheeseamount != maxcapacity && cheeseamount + addcheese > maxcapacity) {
    cheeseamount = maxcapacity;
    System.out.println("Warning : The box just reached it maximum capacity.");
}