Почему переменная roof не меняет своего значения?

Question

[Alex]

import java.util.Scanner;

public class House {
	
Scanner in = new Scanner(System.in);

int chooseRoof(int roof) {
	for( ; ; ) {
		System.out.println("Choose your roof material:");
		System.out.println("Default roof - 45 \t Tile roof - 95 \t Slate roof - 126");
		System.out.println("Enter a number of the choosen material:");
		String input = in.nextLine();
		
		// checking input, if input is empty, warn user about it
		// and ask to try again, after that - continue loop.
						
		if(input.isEmpty()) {
			System.out.println("You don't enter anything.");
			System.out.println();
			continue;
		}
		
		// checking input, if input have any characters, warn user about it
		// and ask to try again, after that - continue loop.
		
		if(!input.matches("[-+]?\\d+")){
			System.out.println("The number must consists only numerals.");
			System.out.println();
			continue;
		}
		
		if (input.equals("45") || input.equals("95") || input.equals("126")) {
			roof = Integer.parseInt(input);
			return roof;
		}
		else {
			System.out.println("Your enter wrong number. Pls choose the material and "
					+ "use his number.");
			System.out.println();
			continue;
		}
	}
	
}

public class Building {
	public static void main(String args[]) {
        int roof = 45;
        House home = new House();
        home.chooseRoof(roof); // Переменная roof должна поменять свое значение
        System.out.println(roof); // Но когда мы ее выводим, видим старое значение,
                                             // почему? Как это исправить?
        }
}

Comments

[MaxLich]

А с чего бы она поменяла значение? Вообще-то, во все методы переменные передаются по значению.

[nubus4000]

int roof = home.chooseRoof();
В методе chooseRoof в самом начале:
int roof = 0;
И избавиться от лишнего параметра. Это примитив, он не меняется по ссылке, как объект.

[MaxLich]

Исправить можно так:

int roof = 45;
        House home = new House();
        roof = home.chooseRoof(roof); 
        System.out.println(roof);

Но код всё равно какой-то бессмысленный и неправильный.

[Alex]

MaxLich, если не трудно, можете указать что именно неправильно и бессмысленно? Второй месяц пытаюсь что-то соображать на Java. Буду рад любым советам, рекомендациям.

[MaxLich]

Alex, почитайте какую-то хорошую книгу по джаве для начинающих. Просто это основы, тут базовые знания нужны. Так просто ничего не объяснишь.

[MaxLich]

Alex, Этот кусок кода, например, можно заменить на свитч:

if (input.equals("45") || input.equals("95") || input.equals("126")) {
      roof = Integer.parseInt(input);
      return roof;
    }

И это плохо - харкодить. Данные должны быть отделены от кода и выполняемых команд. Также здесь применён антипатерн "магические числа". Непонятно, что означают эти числа и что такое руф. Если это какие-то конкретные значения, фиксированные, то лучше применить перечисления.

[nubus4000]

Alex, Набросал быстренько, но не проверял. Код тоже не идеальный. Вообще крышу надо выносить как отдельный класс (и для каждой крыши делать своих наследников со своими параметрами, а класс Roof просто будет абстрактным, с дефолтными методами и параметрами), или делать мапу с доступными крышами и работать с ней (если у крыш будет только название и числа). Да вообще много чего можно напридумывать. Есть книга, называется Объектно-ориентированный анализ и проектирование (от Head First), советую прочитать.

import java.util.Scanner;

public class Work
{
  public static void main(String[] args)
  {
   House house = new House(getRoofNumber());
   System.out.println(house.getRoof());
  }
  
  public static int getRoofNumber()
  {
    Scanner s = new Scanner(System.in);
    int numberOfRoof = 0;
    
    System.out.println("Choose your roof material:\n"+
                      "1)Default roof\n" + "2)Tile roof\n" + "3)Slate roof");
    
    while(true) 
    {
    if (s.hasNextInt())
    {
      numberOfRoof = s.nextInt();
      break;
    }
    else
    {
      System.out.println("Try again");
    }	
    }
    	return numberOfRoof;
    } 
  }

class House
{
  private static final int DEFAULT_ROOF_NUMBER = 1;
  private static final int TILE_ROOF_NUMBER = 2;
  private static final int SLATE_ROOF_NUMBER = 3;
  
  private Roof roof;
  
  public House()
  {
  }
  
  public House(Roof roof)
  {
    this.roof = roof;
  }
  
  public House(int numOfRoof)
  {
    switch(numOfRoof)
    {
      case DEFAULT_ROOF_NUMBER:
      	 this.roof = Roof.DEFAULT_ROOF;
      	 break;
     
      case TILE_ROOF_NUMBER:
      	this.roof = Roof.TILE_ROOF;
      	break;
      
      case SLATE_ROOF_NUMBER:
        this.roof = Roof.SLATE_ROOF;
      	break;
      
      default:
      	throw new RoofNotFoundException("Root of number " + numOfRoof + " not found");
    }
  }
  
  public Roof getRoof()
  {
    return roof;
  }
  
  public void setRoof(Roof roof)
  {
    this.roof = roof;
  }

}

enum Roof 
{
	  DEFAULT_ROOF, TILE_ROOF, SLATE_ROOF
}

class RoofNotFoundException extends RuntimeException
{
  public RoofNotFoundException (String message)
  {
    super(message);
  } 
}

[Alex]

nubus4000, https://github.com/Cocaine4ik/Java/tree/master/Gam...
Тут полный код программы, которую я писал на тот момент. Из знаний тогда было только 4 проработанные главы книги Г. Шилдта. Сейчас знаю немного больше, может как нибудь перепишу.

[Alex]

MaxLich, читаю Г. Шилдта, решаю задачки отсюда cppstudio.com/cat/285

Answer 1

[D3lphi]

Как в java примитивные типы передаются? По значению или по ссылке? По значению, конечно. А это значит, что изменение значения этой переменной в методе chooseRoof() никак не повлияет на значение переменной в методе main().

Comments

[Денис Загаевский]

Непримитивные типы передаются по ссылке, и, тем не менее, изменение параметра метода не влияет на значение переменной в вызывающем методе.

[D3lphi]

Денис Загаевский, Вы хотите сказать, что:

class A { // Чем не непримитивный тип?
    public int x;
}

class B {
    public static void main() {
         A a = new A();
         a.x = 3;
         change(a);
         System.out.println(a.x); // Выведет 3, по-вашему?
    }

    public static void change(A a) {
         a.x = 5;
    }
}

Под "непримитивными" вы понимаете классы-обертки Integer, Double и тд?

[Денис Загаевский]

D3lphi, ты исказил пример. Правильной аналогией было

public static void change(A a) {
         a = new A(5);
    }

[D3lphi]

Денис Загаевский, Вы же в этом случае вообще присваиваете переменной ссылку на другой объект. С чего переменная в вызывающем методе должна изменить свое значение?

[Денис Загаевский]

D3lphi, вот и я думаю - с чего? А ты аргументируешь невозможность изменения переменной в вызывающем методе тем, что передаётся примитив, а значит - по значению.

Как в java примитивные типы передаются? По значению или по ссылке? По значению, конечно. А это значит, что изменение значения этой переменной в методе chooseRoof() никак не повлияет на значение переменной в методе main().

[D3lphi]

Денис Загаевский,

А ты аргументируешь невозможность изменения переменной в вызывающем методе тем, что передаётся примитив, а значит - по значению.

И? Это не является истиной? Примитивы передаются по значению. Значение копируется в вызываемый метод. Если же передается ссылочный тип (непримитивный), то в метод передается ссылка на объект.

P.S А мы друзья с вами или что? Давно на "ты" перешли?

[Денис Загаевский]

D3lphi,

то в метод передается ссылка на объект.

Продолжай рассуждения, не стесняйся.