素数と完璧な正方形チェッカーPython

Question

私は自分の素数と完全な正方形のチェッカーをPythonを使って書こうとしています これは素数であれば 'Foo'を出力し、それはここでどちら が私のコードである場合には完全な方形、プリント「FooBarのは、」：何とか

def FooBar(): 
    prime = True 
    perfSqr = False 
    for target in range(100,100001): 
     for num in range(1,target+1): 
      if target % num == 0 and num != target: 
       prime = False 

      if target // num == num and target % num == 0: 
       perfSqr = True 

    if prime is True: 
     print 'Foo' 
    elif perfSqr is True: 
     print 'Bar' 
    else: 
     print 'FooBar' 

if __name__ == '__main__': 
    FooBar() 

、私はそれがすべてで動作して取得することはできません、誰も私にいくつかのヒントを与えることができますか？

Answer 1

最初は、numを1からtarget + 1に変更します。 num % 1は常に0なので、素数は決して印刷されません。

次に、primeとperfSqrフラグを繰り返し設定しないでください。また、外側のループの内側でprintステートメントを移動する必要があるため、すべての繰り返しを印刷します。

これは期待通りに動作します：

def FooBar(): 
    for target in range(100, 100001): 
     prime = True 
     perfSqr = False 
     for num in range(2, target + 1): 
      if target % num == 0 and num != target : 
       prime = False 

      if target // num == num and target % num == 0: 
       perfSqr = True 

     if prime or perfSqr: 
      print(num, end=', ') 
      if prime: 
       print('Foo', end=', ') 
      elif perfSqr: 
       print('Bar', end=' ') 
      print('\n') 

Answer 2

1）あなたのインデントは、あなたが望むように思えるオフである、それはあなたが行う）ループ

2のための第1の内側にあるので、ブロックをインデントする場合最初のforループの中でprimeとperfSqrをリセットしないので、一度設定されるとリセットされません。

3）内側のforループ範囲は1から始まり、2から始める必要があります。そうでない場合は、すべての数字がプライムになります。

は、これはあなたが

def FooBar(): 
for target in range(100,100001): 
    prime = True 
    perfSqr = False 
    for num in range(2,target+1): 
     if target % num == 0 and num != target: 
      prime = False 

     if target // num == num: 
      perfSqr = True 

    if prime is True: 
     print 'Foo' 
    elif perfSqr is True: 
     print 'Bar' 
    else: 
     print 'FooBar' 



if __name__ == '__main__': 
    FooBar()