allPrimes空のタプルを印刷します。Python

Question

このコードを持っていて、その中にすべての素数を入れたタプルを印刷することになっています。しかし代わりに、それは空のタプルを印刷しているだけです...

誰かが私にその理由を教えてもらえますか？ 私もタプルを使用する必要があります。ここで

def isPrime(number): 
for i in range(2,int(number**(0.5))+1): 
    if number % i == 0: 
     return False 
    else: 
     return True 


def allPrimes(number): 
    tup=() 
    for i in range(1,number): 
     if isPrime(i) == True: 
      tup += (i,) 
    print(tup) 

allPrimes(26) 

あなたisPrime機能が動作しないためです正しいコード

def isPrime(number): 
    if number < 2: 
     return False 
    for i in range(2, int(number ** (0.5)) + 1): 
     if number % i == 0: 
      return False 
    return True 



def allPrimes(number): 
    tup=() 
    for i in range(1,number): 
     if isPrime(i) == True: 
      tup += (i,) 
     print(tup) 

allPrimes(26) 
out[1]: (2, 3, 5, 7, 11, 13, 17, 19, 23) 

Answer 1

isPrime()関数は1から始まります。すべての整数は1で割り切れるので、常にFalseを返します。代わりに2で始まります。

def isPrime(number): 
    if number < 2: 
     return False 
    for i in range(2, int(number ** (0.5)) + 1): 
     if number % i == 0: 
      return False 
    return True 

また、あなたのallPrimesはおそらくリストではなく、タプルを使用する必要があり、あなたはisPrime(i) == Trueするのではなく、単にisPrime(i)を使用することができますが、それはそれがあるように動作します。

Answer 2

です。

number % 1、つまり '1で除算したときの剰余'は常に整数でゼロになります。

Answer 3

あなたのコード内のいくつかの問題があります

1）isPrimeリターンでは、あなたは、関数スコープ

3の外tupを印刷している）間違ったライン上のTrue

2）あなたはありません1の場合（isPrime）

4）tupleを使用して素数を格納する場合、listはベットされていますter;はるかに効率的です。

5）Pythonで関数名にsnake caseを使用します。 number % i != 0:数が素数である場合、あなたは `番号％のi == 0の場合、反復の終了時にブレークをTrueを返す必要があるため、最初の反復で

def is_prime(number): 
    if number < 2: 
     return False 

    for i in range(2,int(number**(0.5))+1): 
     if number % i == 0: 
      return False 

    return True 

def all_primes(number): 
    my_primes = [] 
    for i in range(1,number): 
     if is_prime(i): 
      my_primes.append(i) 
    return my_primes 

if __name__ == "__main__": 
    print all_primes(40) 

Answer 4

：変更を加える

：

def isPrime(number): 
    res = True 
    if number < 2: 
     res = False 
    else: 
     for i in range(2,int(number**0.5)+1): 
      if number % i == 0: 
       res = False 
       break 
    return res