0

Хочу создать условия для вывода информации о пассажирах:

  1. Не может быть меньше нуля.

  2. Если равняется нулю то автобус пуст.

  3. Не может быть больше вместимости автобуса.

Но не могу установить третье условие, так как после его установки, при проверке двух других, возникает текст который должен быть только при 3 условии.

package programm;

public class Programm {

    public static void main(String[] args) {
       Avtobus bus = new Avtobus("Calculator");

        bus.setName(666);                  //Номер автобуса
        bus.setNumberOfPassengers(50);     //Количество пассажиров
        bus.setCostOfTravel(30);           //Цена поездки
        bus.setNumberOfStops(10);          //Количество Остановок
        bus.setTheCapacityOfTheBus(50);    //Вместимость автобуса


        bus.Stopped();                  //Остановка
        bus.info1();                    //Первая остановка.Осталось остановок            
        bus.info2();                    //Кол-во пассажиров
        bus.NumberOfAvailableSeats();   //Кол-во свободных мест
        bus.info3();                    //Прибыль с одной остановки
        bus.CalculationOfProfit();      //Подсчет конечной прибыли




package newobject;

public class Avtobus {

    private int Name;
    private int NumberOfPassengers;
    private int CostOfTravel;
    private int NumberOfStops;
    private int TheCapacityOfTheBus;
    private String Calculator;


protected Avtobus(){

}    
public Avtobus(String Calculator) {

this.Calculator = Calculator;

on();

}

public Avtobus(int Name, int NumberOfPassengers, int CostOfTravel, int NumberOfStops, int TheCapacityOfTheBus){

this.Name = Name;

this.NumberOfPassengers = NumberOfPassengers;

this.CostOfTravel = CostOfTravel;

this.NumberOfStops = NumberOfStops;

this.TheCapacityOfTheBus = TheCapacityOfTheBus;

}




public int getNumberOfPassengers(){

return NumberOfPassengers;

}

public void setNumberOfPassengers(int nNumberOfPassengers){

if (nNumberOfPassengers<0){

System.out.println("!!Количество пассажиров не может быть меньше нуля!!");

}else{

NumberOfPassengers = nNumberOfPassengers;

}

if(nNumberOfPassengers ==0){

System.out.println("Автобус пуст");    

}else{

NumberOfPassengers = nNumberOfPassengers;    

}

if(nNumberOfPassengers>TheCapacityOfTheBus){

System.out.println("!!Количество пассажиров не может быть больше чем вместимость 
автобуса!! ");       

}

}
12
  • Анатолий, отформатируйте код нормально, вставьте его текстом а не картинками. 27 ноя 2018 в 6:59
  • Если нужен весь код, я его скину. Я новичок, не бросайтесь камнями, буду рад помощи, так как зашел в тупик. 27 ноя 2018 в 6:59
  • Владимир Клыков, сделал 27 ноя 2018 в 7:09
  • @АнатолийМихеенко советую назвать объект автобуса bus, а не avt 27 ноя 2018 в 7:09
  • @АнатолийМихеенко так же метка netbeans тут не причем, у вас же проблема не из за nb, а из за кода. Я удалил эту метку. 27 ноя 2018 в 7:10

3 ответа 3

4

Кроме else есть еще else if:

public void setNumberOfPassengers(int nNumberOfPassengers)
{
  if (nNumberOfPassengers < 0)
  {
    System.out.println("!!Количество пассажиров не может быть меньше нуля!!");
  } 
  else if (nNumberOfPassengers == 0)
  {
    System.out.println("Автобус пуст");    
  }
  else if (nNumberOfPassengers > TheCapacityOfTheBus)
  {
    System.out.println("!!Количество пассажиров не может быть больше чем вместимость автобуса!!");       
  } 
  else 
  {
    NumberOfPassengers = nNumberOfPassengers;
  }
}

Kоличество пассажиров Вы устанавливаете до того, как задаете вместимость:

    bus.setNumberOfPassengers(50);     //Количество пассажиров
    ...
    bus.setTheCapacityOfTheBus(50);    //Вместимость автобуса

а надо - после:

    ...
    bus.setTheCapacityOfTheBus(50);    //Вместимость автобуса
    bus.setNumberOfPassengers(50);     //Количество пассажиров
6
  • Хм, красиво, про else if не знал, не знал что можно их так использовать, это многое упрощает, но проблема в том, что при вводе значения в bus.setNumberOfPassengers к примеру 50 при том, что bus.setTheCapacityOfTheBus является тоже 50 выходит сообщение System.out.println("!!Количество пассажиров не может быть больше чем вместимость автобуса!!"); которого быть не должно, так как проверка проходиться, так как количество пассажиров не больше вместимости автобуса, а она ровна ей 27 ноя 2018 в 7:30
  • @АнатолийМихеенко Это маловероятно. Скорее всего дело в том, что количество пассажиров Вы устанавливаете до того, как задаете вместимость.
    – user176262
    27 ноя 2018 в 7:33
  • Igor, что вы имеете в виду, вы про про установку значений в bus.setNumberOfPassengers(); и bus.setNumberOfPassengers(); ? или про реализацию самих условия и их расположения в коде? 27 ноя 2018 в 7:36
  • Хм, это и вправду помогло, поменять их местами то есть bus.setNumberOfPassengers(); и bus.setNumberOfPassengers(); но меня терзает любопытство, почему это помогло? 27 ноя 2018 в 7:45
  • @АнатолийМихеенко В реальном мире что известно раньше - вместимость автобуса или количество пассажров?
    – user176262
    27 ноя 2018 в 13:46
2

Иногда применение оператора return даже в void методах помогает избежать множественных if else и сделать код с условиями гораздо более читаемым.

public void setNumberOfPassengers(int nNumberOfPassengers){
    // первое негативное условие - метод работать не может
    if (nNumberOfPassengers < 0) {
        System.out.println("!!Количество пассажиров не может быть меньше нуля!!");
        return;
    } 

    // второе негативное условие - метод работать не может
    if (nNumberOfPassengers == 0) {
        System.out.println("Автобус пуст");
        return;    
    }

    // третье негативное условие - метод работать не может
    if (nNumberOfPassengers > TheCapacityOfTheBus) {
        System.out.println("!!Количество пассажиров не может быть больше чем вместимость автобуса!!");
        return;       
    } 

    // если мы не вывалились из метода в одной из предыдущих проверок
    // выполняем основную работу метода
    NumberOfPassengers = nNumberOfPassengers;
    System.out.println("Ты угадал сколько человек может быть в автобусе! Как #%$# ты это сделал?!! оО");
}

Любое условие, дающее однозначный результат работы метода можно снабжать собственным return.

0

Попробую предложить такой дизайн кода:

int busCapacity;
int numberOfPassenger;

public void setNumberOfPassengers(int amount){

    if ((amount < 0)||( busCapacity < amount))
        System.out.println("ERROR");
    else if (amount == 0)
        System.out.println("Bus is empty");
    else
        numberOfPassenger = amount;
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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