IndexErrorを取得する：文字列インデックスが範囲外にある

Question

整数nが与えられている場合、それを2進（2進）に変換します。次に、連続した1の最大数を表す基数10の整数を見つけて、nのバイナリ表現で表示して出力します。 は、ここで私はそれが

#!/bin/python 
import sys 
k=0 
m=0 
i=0 
n = int(raw_input().strip()) 
x=bin(n)[2:] 
while x[i] : 
    if x[i]==1: 
     m=m+1 
    else : 
     if k<m: 
      k=m 
      m=0 
    i+=1 
print(k) 

私は次のようなエラーに

Traceback (most recent call last): 
File "solution.py", line 9, in <module> 
while x[i] : 
IndexError: string index out of range 

Answer 1

あなたのwhileループ刻みiを取得していますために書いたコードですが、それは、文字列xの終わりを越えてしまったときにチェックされません。あなたのテストwhile x[i]は、文字列の終わりをチェックしません。空であれば各文字をチェックするだけです。文字列内の文字は空ではないため（文字列には'0'と'1'しか含まれていないため）、ループはそのまま続きます。文字列'0'はFalseにテストされません。完全に有効な文字列で、Trueと評価されます。あなたがその構造をしたい場合は

、代わりにチェック

while i < len(x): 

に使用しますが、あなたのコードを再構築し、文字列x内の各文字c見

for c in x: 

を使用するより多くのニシキヘビだろう。

Answer 2

あなたの状態はwhile x[i]で、それは決してFalseと評価されないので間違っています。 Cのヌル終了文字列に慣れていますか？

また、あなたはここでintにstrを比較している：

if x[i]==1: 

これを行うための適切な方法がために、各ループを使用することです：

import sys 

k = 0 
m = 0 
i = 0 

n = int(raw_input().strip()) 
x = bin(n)[2:] 

for c in x: 
    if c == '1': 
     m += 1 
    else: 
     if k < m: 
      k = m 
     m = 0 

# check whether the last run is greater than the current maximum 
if k < m: 
    k = m 

print(k)