AtCoder Beginner Contest 122 B問題が解けない

Question

前提・実現したいこと

Python3 (3.4.3)でABC122のB問題を解いています。(https://atcoder.jp/contests/abc122/tasks/abc122_b)
以下のソースコードの様な形での実装を試みたのですが、上手くいきませんでした。

該当のソースコード

S = input()
ACGT = ["A","C","G","T"]

ans = 0 
ans_lis = [0]

for i in range(len(S)):
    if S[i] in ACGT:
        ans = ans + 1 
    elif S[i] not in ACGT:
        ans_lis.append(ans)
        ans = 0 

print(max(ans_lis))

結果

試したこと

この問題を見た時に大まかに以下の方針を立てて実装しました。

<指定された文字列を一文字ずつ読み込む> → <読み込んだ文字がACGT 文字列に該当する場合は変数ansに+1する>
→ <読み込んだ文字がACGT 文字列に該当しない場合は、現時点での変数ansの値をリストに代入した後に、ansを初期化する>
<max()でリスト内に入ってる最大の数字を出力し、最も長い ACGT 文字列 の長さを求める>

しかし、結果を見ると何問か<WA>があり、自分のコードではどのようなパターンに対応できないか分かりません。
直接的な回答でなくても、実装漏れしている例などを教えて頂ければ幸いです。

Answer 1

一番最後に最長のACGT文字列が現れるとき、ans_lisにその結果が保持されません。
ループ脱出後に、余分なようでも ans_lis.append(ans) とすれば解消すると思います。

その他、気になること

いわゆるfor-eachを活用する
for i in range(len(S)): より for ch in S: の方が良いでしょう。
本問では各文字のインデックスを意識する必要は無い筈です。

else節を活用する
条件の真偽で処理を分けたいだけなら、if-elseを使うのが自然です。

if ch in ACGT:
    ...
else:
    ...

文字列の包含判定
実は ch in ["A","C","G","T"] ではなく ch in "ACGT" で充分です。

Comments

[Danrussia]

回答ありがとうございます。以下の用に訂正して、AC取れました。ありがとうございます

S = input()

ans = 0
ans_lis = [0]

for i in range(len(S)):
if S[i] in "ACGT":
ans = ans + 1
else:
ans_lis.append(ans)
ans = 0

ans_lis.append(ans)
print(max(ans_lis))