奇数抽取的Java方法

Question

编写一个方法提取tOddDigits（），从正数n中提取奇数，并将奇数顺序组合成一个新数字。新数字返回给调用方法。如果输入数字n不包含任何奇数，则返回-1。例如，如果n=1234567，则返回1357；如果n=28，则返回-1。

测试用例：

(1) n : 12345; 
(2) n : 54123; 
(3) n : 246; 
(4) n : -12 (give an error message)

预期产出：

(1) oddDigits = 135; 
(2) oddDigits = 513; 
(3) oddDigits = -1; 
(4) oddDigits = Error input!!

这是我所做的方法，但输出是

(1) oddDigits = 135; 
(2) oddDigits = 513; 
(3) oddDigits = -1; 
(4) oddDigits = -1

最后的输出应该是错误输入！

public static long extractOddDigits(long n){


String output = "";
if(n < 0) // check if negative
{   
    output = "Error Input!!";

}
if(n % 2 == 0){ //check even

    output = "-1";
}

while(n > 0) { 
   int left = (int) (n % 10);
   if(left % 2 != 0)    
     output = left + output;
     n /= 10;
}
System.out.println("oddDigit = " + output);
 }

如何检查n是否为负数，然后转到错误输入？

Answer 1

问题是：

当n=-12时，您正在分配输出"Error Input！！"

然后由于也是n%2==0，您正在将输出覆盖到“-1”

另外，你的逻辑——如果n可以被2整除，它就不会有奇数，这是不正确的

Answer 2

首先，仅仅因为数字是偶数，并不意味着它不包含任何奇数。

所以：

if(n % 2 == 0){ //check even

    output = "-1";
}

是错误的。您也不需要键入cast the（n%10）；to int。

您可以保留一个布尔变量funOdd，它跟踪到目前为止我们是否找到了任何奇数位，最后如果它仍然为假，我们可以简单地打印-1。

所以正确的代码是：

int n = 246;
String output="";
boolean foundOdd=false;
if(n < 0) // check if negative
  output = "Error Input!!";    
else
{
  while(n > 0)
    { 
      int left =  (n % 10);
      if(left % 2 != 0)    
      {output = left + output; foundOdd = true;}
      n /= 10;
    }
}
if(foundOdd)
  System.out.println("oddDigit = " + output);
else System.out.println(-1);

Answer 3

您的问题是，前两个if可能会为输出赋值，但不是以独占方式，您可以找到一个负数、偶数和奇数。因此，您必须确保代码隔离了每个选项。

此外，您的要求是，该方法必须返回数字，而不仅仅是打印数字，因此您的方法应该返回long，而不是void，但我会把它作为您作业的一部分；）

public static void extractOddDigits(long n){

  String output = "";
  if(n < 0) {   
    output = "Error Input!!";

  } else {

    while(n > 0) { 
      int left = (int) (n % 10);
      if (left % 2 != 0) {
        output = left + output;
      }
      n /= 10;
    }
  }
  if (output.equals("") {
    output="-1";
  }
  System.out.println("oddDigit = " + output);
}