2017-11-17 7 views
-2

こんにちは、このコードでは、メルセンヌ素数の数は4であるが、8を出力するという問題があります。メルセンヌ素数が1から1000までの数3,7,31,127ですが、15、63、255、511も出力されます。理由はわかりません。 100個の数字ごとにプライム数を出力する方法もありますか?おかげプログラムメルセンヌの数が間違って表示されます

public static void main(String[] args) { 
    System.out.println("Prime numbers between 1 and 1000 are\n"); 
    String message = ""; 
    String message2 = ""; 
    // loop numbers from 1 to 1,000, printing only the primes 
      int counter2 = 0; 
      int counter = 0; 
      for(int number s= 2; number <= 1000; number ++){    
       if(isPrime(number)) { 
        counter++;      
        if(counter % 10 == 0) { 
        System.out.println(number); 
        } 
        else 
        System.out.print(number + " "); 
       } 
       }    
     for(int number2 = 2; number2 <= 1000; number2 ++){   
      if(isMersenne(number2)){    
       counter2++; 
       System.out.print(number2 + "*"); 
       }    
      } 
      System.out.print("\n\n The number of primes between 1 and 1000 are " + counter); 
      System.out.print("\n The number of Mersennes is " + counter2);     
} 

public static boolean isPrime(int number){ 
      // initialize the boolean variable prime to true 
      boolean isPrime = true; 
      for(int divisor = 2; divisor <= number /2; divisor++) { 
      if(number % divisor == 0) { 
       isPrime = false; 
      }  
      } 
      return isPrime; 
     } 

public static boolean isMersenne(int number){   
       // declare and initialize variable powTwo to 2 
       int powTwo = 2; 
       boolean mersenne = false; 
        while(powTwo <= number){ 
          powTwo *= 2; 
         if(powTwo - number == 1)     
         // if the prime is one less than a power of 2, return true 
         mersenne = true; 
        }    
     return mersenne;  
      } // end method isMersenne 
     } 

これはあなたの問題はisMersenneだけの数が2の累乗より1小さいかどうかチェックしているということである私のプログラムの出力

Prime numbers between 1 and 1000 are 

2 3 5 7 11 13 17 19 23 29 
31 37 41 43 47 53 59 61 67 71 
73 79 83 89 97 101 103 107 109 113 
127 131 137 139 149 151 157 163 167 173 
179 181 191 193 197 199 211 223 227 229 
233 239 241 251 257 263 269 271 277 281 
283 293 307 311 313 317 331 337 347 349 
353 359 367 373 379 383 389 397 401 409 
419 421 431 433 439 443 449 457 461 463 
467 479 487 491 499 503 509 521 523 541 
547 557 563 569 571 577 587 593 599 601 
607 613 617 619 631 641 643 647 653 659 
661 673 677 683 691 701 709 719 727 733 
739 743 751 757 761 769 773 787 797 809 
811 821 823 827 829 839 853 857 859 863 
877 881 883 887 907 911 919 929 937 941 
947 953 967 971 977 983 991 997 3*7*15*31*63*127*255*511* 

The number of primes between 1 and 1000 are 168 
The number of Mersennes is 8 

答えて

2

です。それがプライムであることもチェックしていません。

isMersenneメソッドを変更して、isPrimeを呼び出すようにする必要があります。その番号が2の累乗数よりも小さいかどうかを確認します。 isPrimeがfalseを返す場合は、isMersenneもfalseを返す必要があります。

+0

ありがとうございました。 –

関連する問題