JAVA检查两个数字位数是否相同

Question

嗨，我正在尝试解决Udemy练习：编写一个名为hasSharedDigit的方法，其中包含int类型的两个参数。

每个数字应在10（含）-99（含）之间。如果其中一个数字不在范围内，则该方法应返回false。

如果两个数字中都有数字，例如12和23中的2，则该方法应返回true；否则，该方法应返回false。

我一直在得到真实，而有共享数字（9，99）我无法发现为什么..

public class SharedDigit {
public static boolean hasSharedDigit(int number1, int number2){
    
    if((number1 <10 || number1 >99) && (number2 <10 || number2 >99)) {
        return false;
    } 
    
    int numberOneFirstDigit = number1/10;
    int numberTwoFirstDigit = number2/10;
    int numberOneLastDigit = number1%10;
    int numberTwoLastDigit = number2%10;
    
    if(numberOneFirstDigit == numberTwoFirstDigit || numberOneFirstDigit == numberTwoLastDigit || numberOneLastDigit == numberTwoLastDigit) {
        return true;
    } else {
    return false;
    }
}

}

Answer 1

你在这里缺少一个比较。您拥有的是 f1 == f2 || f1 == l2 || l1 == l2，但缺少的是 l1 == f2，例如 12 和 23 会返回 false 与你的代码因为

• 1！=2（f1==f2失败）
• 1！=3（f1==l2失败）
• 2！=3（l1==l2失败）
• 您的代码缺少2==2（l1==f2），因此添加…||NumberOneLastDigit==NumberTwoFinstDigit到比较条件。

正如其他人已经说过的，您的检查也是错误的，因为它要求两个数字都超出范围，所以如果只有一个超出范围，它不会返回false。

为了方便起见，请尝试扭转条件，使两个数字都必须在范围内：

if(!(number1 >= 10 && number1 <=99 && number2 >= 10 && number2 <= 99)) {
  return false;
}

通常，如果您要“命名”条件，那么阅读和维护代码也会更容易：

boolean n1InRange = number1 >= 10 && number1 <=99;
boolean n2InRange = number2 >= 10 && number2 <=99;

if( !(n1InRange && n2InRange) { ... }
//or
if( !n1InRange || !n2InRange ) { ... }

Answer 2

对方的回答和评论解决了你的直接问题。只是为了好玩，我们可以采取完全不同的方法。

这里的想法是将数字视为文本。这样做，我们可以解决您的两个业务规则中的每一个：

• 每个数字应在10（含）-99（含）的范围内。
  • 10到99之间的数字恰好有两位数。一位数或3位数表示超出范围。
  • 通过将字符串中作为第一个数字的每个字符的代码点收集到一个＜code＞Set＜/code＞中，我们可以得到第二个数字的字符串的代码点的交集。如果交叉点的大小大于0，则我们知道数字是共享的

  为了解释下面的代码…codePoint方法生成一个IntStream，这是一个int数字序列，用于在Unicode中为表示输入整数的字符串中的每个字符定义的每个代码点号。装箱调用将这些int原语转换为整数对象。将这些整数对象添加到Set会自动使它们不同，自动清除任何重复的数字。

  请在IdeOne.com上查看此代码的实时运行。

  private boolean twoIntsAreInRangeAndShareDigits ( int n1 , int n2 )
  {
      String n1String = Integer.toString( n1 );
      String n2String = Integer.toString( n2 );
  
      // Check for negative numbers, meaning out-of-range, not 10-99. 
      if ( n1String.concat( n2String ).contains( "-" ) ) { return false; } 
  
      // Check for exactly 2 digits, meaning within range, 10-99.
      if ( ( n1String.length() == 2 ) && ( n2String.length() == 2 ) )
      {   // Numbers are within range. 
          // Check for common digits.
          Set < Integer > n1CodePoints = n1String.codePoints().boxed().collect( Collectors.toSet() );
          Set < Integer > n2CodePoints = n2String.codePoints().boxed().collect( Collectors.toSet() );
          n1CodePoints.retainAll( n2CodePoints );
          boolean sharesDigit = ( n1CodePoints.size() > 0 );
          return sharesDigit;
      } 
      else  // Else 1 or 3+ digits mean numbers are out-of-range.
      { return false; }
  }
  

  我并不是说这种方法更好，只是作为一种思考问题的不同方式很有趣。

Answer 3

如果其中一个数字不在范围内，则该方法应返回false。

取代

if((number1 <10 || number1 >99) && (number2 <10 || number2 >99)) 

跟

if(number1 <10 || number1 >99 || number2 <10 || number2 >99) 

除此之外，您在比较数字的条件组合中遗漏了< code > numberOneLastDigit = = numberTwoFirstDigit ，即组合应该是

if(
    numberOneFirstDigit == numberTwoFirstDigit || 
    numberOneFirstDigit == numberTwoLastDigit  || 
    numberOneLastDigit  == numberTwoFirstDigit || 
    numberOneLastDigit  == numberTwoLastDigit
)