Java домашнее задание. Проблема с булева логика

Question

Принимая свой первый класс Java, и я застрял на глупые. Я делаю проект палиндрома. Эта логика кажется хорошей. В любом случае отображается True. Что я делаю не так?

вызов:

boolean result = check(input); 

или в самом методе:

public static void display(boolean result, String palindrome) 
{ 
    if (result = true) 
    { 
     JOptionPane.showMessageDialog(null, palindrome 
       + " is a palindrome."); 
    } else 
     JOptionPane.showMessageDialog(null, palindrome 
       + " is not a palindrome."); 

} 

Вот весь код: импорт javax.swing.JOptionPane;

public class Palindrome 
{ 

public static void main(String args[]) 

{ 
    // declare variables 

    // call methods 
    String input = retrieveInput(); 
    boolean result = check(input); 
    display(result = false, input); 
    finish(); 
} 

// Accepts and validates input 
public static String retrieveInput() 
{ 
    // declare variables 
    int length; 
    String palindrome = null; 
    boolean done = false; 
    while (!done) 
    { 
     try 
     { 
      // user input 
      palindrome = JOptionPane.showInputDialog(null, 
        "Please enter a 5 digit integer:"); 
      length = palindrome.length(); 

      // data validation 
      if (length != 5) 
      { 
       throw new NumberFormatException(); 
      } else 
       done = true; 
     } 

     catch (NumberFormatException e) 
     { 
      JOptionPane.showMessageDialog(null, 
        "Error. Please enter a 5 digit integer", "Error", 
        JOptionPane.INFORMATION_MESSAGE); 
     } 
    } 
    return palindrome; 
} 

public static Boolean check(String palindrome) 
{ 
    // determine if palindrome 

    int left = 0; 
    int right = palindrome.length() - 1; 

    while (left < right) 
    { 
     if (palindrome.charAt(left) != palindrome.charAt(right)) 
      return false; 

     left++; 
     right--; 
    } 

    return true; 

} 

// The output method displays commission and sales 
public static void display(boolean result, String palindrome) 
{ 
    if (result = true) 
    { 
     JOptionPane.showMessageDialog(null, palindrome 
       + " is a palindrome."); 
    } else 
     JOptionPane.showMessageDialog(null, palindrome 
       + " is not a palindrome."); 

} 

// finish() method exits program 
public static void finish() 
{ 
    System.exit(0); 
} 

} 

Answer 1

== используется при сравнении значений.

= - оператор присваивания, а не оператор сравнения.

так попробовать:

if (result == true) 

или

if (result) 

Ваш полный код после редактирования:

public static void main(String args[]) 
{ 
    String input = retrieveInput(); 
    boolean result = check(input); 
    display(result, input); // change result = false 
    finish(); 
} 

Answer 2

В этой строке

if (result = true) 

вы присваиваете значение true переменной result и это значение (true) используется в качестве условия для if, не проверяя, если result равно true. Другими словами, ваш if всегда будет оценивать свое состояние true.

Вы нуждались бы, чтобы использовать == оператор

if (result == true) 

или просто

if (result) 

Answer 3

if (result = true) 

наборы result к истинным и оценивает его (опять же, как true). Использование:

if(result==true) 

или

if(result) 

вместо этого. Первый является синтаксисом для большинства сопоставлений значений, а последний просто работает для булевых.