Переменные Java Passing и возвращаемое значение

Question

У меня возникли проблемы с передачей некоторых переменных с помощью метода, а затем с тем, чтобы этот метод возвращал значение. Метод checkValue должен рассматривать каждый элемент массива в массиве orderSplit, и если с ними происходит ошибка, верните сообщение об ошибке, если он не вернет пустой errorMessage. Но на данный момент программа, похоже, вообще не выполняет этот метод. Какие-либо предложения?

Вот пример моего кода:

public class Foo { 
    public static void main(String args[]) { 
     BufferedReader br = new BufferedReader(new InputStreamReader(System.in)); 
     String order = null; 
     try { 
      order = br.readLine(); 
     } catch (IOException exc) { 
      System.out.println("ERROR: A Problem has occured"); 
     } 

     String[] orderSplit = null; 
     orderSplit = order.split(" "); 

     String errorMessage = ""; 
      checkValue(orderSplit, errorMessage); 
     if (errorMessage == "") { 
      System.out.println("SUCCESS"); 
     } 

    } 
    public static String checkValue(String[] orderSplit, String errorMessage) { 
      // check the ordersplit values 
      return errorMessage; 
    } 
} 

Answer 1

Вы не присвоит результат этого метода ни к чему. Измените свой код на:

String errorMessage = checkValue(orderSplit); 

И не то, чтобы метод checkValue не нуждался в сообщении об ошибке в качестве аргумента. Он будет создавать один сам по себе и вернуть его вызывающему.

Кроме того, назначение нулевой переменной вы переназначить сразу после того, как, например:

String[] orderSplit = null; 
orderSplit = order.split(" "); 

ненужно. Вам просто нужно

String[] orderSplit = order.split(" "); 

И вы никогда не должны сравнивать строки с ==. == проверяет, соответствуют ли две переменные одному и тому же объекту String. Вы должны использовать equals() метод, который проверяет, если две строки содержат ту же самую последовательность символов:

if (errorMessage.equals("")) { 

Answer 2

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

String errorMessage = ""; 
checkValue(orderSplit, errorMessage); 

в

String errorMessage = ""; 
errorMessage = checkValue(orderSplit, errorMessage); 

Удачи

Answer 3

Код выше просто присваивает "" переменной errorMessage и возвращает ссылку на строку "". Это само по себе ничего не делает.

Я предполагаю, что у вас есть код, который присваивает значение errorMessage в методе checkVlaue. Есть некоторые оговорки к этому. Строки неизменны в java. В вашей сигнатуре сообщения передается значение, содержащее ссылку на содержимое errorMessage во время разговора по адресу checkValue. Если вы попытаетесь назначить другое значение этой переменной внутри checkValue и вернуть ее, вы фактически возвращаете значение, которое является ссылкой на другой строковый объект. После того, как вы вернетесь к вызывающему методу, печать errorMessage напечатает "", потому что это строковый объект, который по-прежнему указывает на метод вызова errorMessage. Если вы изменили свой вызов на следующее:

String errorMessage = checkValue(orderSplit); 

Вы сейчас назначая эталонное значение, которое возвращается из checkedValue to ErrorMessage in the calling method. This will now print whatever the result of checkValue` был.

Чистота должна помнить, что даже если большинство вещей в java являются объектными ссылками (указателями), все сигнатуры методов фактически передаются по значению. Когда вы повторно назначаете, вы не изменяете то, на что указывает указатель, а скорее назначают новый указатель на переменную все вместе.

Answer 4

java.lang.String является неизменным. Итак, что бы вы ни делали с errorMessage внутри вашего метода, не будет видно вне метода, поскольку по существу вы создаете новые объекты String.

Вы должны действительно проверить возвращаемое значение метода, как предлагают другие ответы.