Jak używać tej wartości logicznej w instrukcji if?

Question

private String getWhoozitYs(){ 
    StringBuffer sb = new StringBuffer(); 
    boolean stop = generator.nextBoolean(); 
    if(stop = true) 
    { 
     sb.append("y"); 
     getWhoozitYs(); 
    } 
    return sb.toString(); 
} 

To jest fragment kodu dla projektu, który robię w kursie programowania. Problem, który mam, polega na tym, że po zadeklarowaniu Boolean Stop i próbie przypisania mu losowo wygenerowanej wartości Boolean, nie mogę użyć go w instrukcji if, aby określić, czy powinienem dodać więcej y do StringBuffer, czy nie. Mam generator losowy wewnątrz konstruktora, więc ta część nie stanowi problemu. Zakładałem, że skoro deklarowałem wartość boolean poza instrukcją if, mógłbym użyć jej wewnątrz, ale nie wydaje się, żeby tak było. Prawdziwe pytanie brzmi: jak mogę użyć losowo określonej wartości boolowskiej w instrukcji if.

Answer 1

if(stop = true) powinno być if(stop == true), lub po prostu (lepiej!) if(stop).

Właściwie jest to dobra okazja, aby zobaczyć powód dlaczego zawsze używać if(something) jeśli chcesz zobaczyć, czy to true zamiast pisać if(something == true) (złego stylu!).

Wykonując stop = true, przypisujesz true do stop i nie można tego porównać.

Dlaczego więc kod poniżej instrukcji if został wykonany?

Zobacz JLS - 15.26. Assignment Operators:

W czasie wykonywania wynikiem wyrażenia przypisania jest wartość zmienna po nastąpieniu cesji. Wynik wyrażenia przypisania przypisania nie jest zmienną.

Tak więc, ponieważ napisałeś stop = true, spełniasz warunek if.

Answer 2

Od stop jest logiczna można zmienić tę część do:

//... 
if(stop) // Or to: if (stop == true) 
{ 
    sb.append("y"); 
    getWhoozitYs(); 
} 
return sb.toString(); 
//... 

Answer 3

Problemem tutaj jest

jeśli (stop = true) to nie przypisanie Porównanie.

Spróbuj if (stop == true)

również spojrzeć na Top Ten Errors Java Programmers Make.

Answer 4

if(stop == true) 

lub

if(stop) 

= służy do przeniesienia.

== służy do sprawdzania stanu.

if(stop = true) 

Przypisuje wartość true, aby zatrzymać i ocenia, czy (prawda). Więc zawsze będzie wykonywał kod wewnątrz, jeśli ponieważ stop zawsze będzie przypisany z wartością true.

Answer 5

Spróbuj tego: -

private String getWhoozitYs(){ 
    StringBuffer sb = new StringBuffer(); 
    boolean stop = generator.nextBoolean(); 
    if(stop) 
    { 
     sb.append("y"); 
     getWhoozitYs(); 
    } 
    return sb.toString(); 
} 

Answer 6

= dla przypisania

zapisu

if(stop){ 
    //your code 
} 

lub

if(stop == true){ 
    //your code 
} 

Answer 7

dodatkowo można po prostu napisać

if(stop) 
{ 
     sb.append("y"); 
     getWhoozitYs(); 
} 

Answer 8

Właściwie cała podejście byłoby czystsze, gdybyś miał tylko użyć jednej instancji StringBuffer, zamiast tworzyć jedną w każdym wywołaniu rekurencyjnym ... pójdę do:

private String getWhoozitYs(){ 
    StringBuffer sb = new StringBuffer(); 
    while (generator.nextBoolean()) { 
     sb.append("y"); 
    } 

    return sb.toString(); 
}