目的の結果が取れない

Question

#目的の結果が取り出せない
関数を作り、if _name以下の結果の通りに出力したいのですが、print(list(remove_all_before([1, 2, 3, 4, 5], 3)))の結果が[2,3,4,5]と出てきて上手くできません。この場合 [3,4,5]と出力されれば成功なのですが・・

質問が上手く伝わるかわかりませんが、どなたか分かる方原因を教えていただきたいです。

#コード
def remove_all_before(items: list, border: int) -> Iterable:
i = 0
while items[i] < border:
del items[i]
i = i + 1
return items

if name == 'main':
print("Example:")
print(list(remove_all_before([1, 2, 3, 4, 5], 3)))
assert list(remove_all_before([1, 2, 3, 4, 5], 3)) == [3, 4, 5]
assert list(remove_all_before([1, 1, 2, 2, 3, 3], 2)) == [2, 2, 3, 3]
assert list(remove_all_before([1, 1, 2, 4, 2, 3, 4], 2)) == [2, 4, 2, 3, 4]
assert list(remove_all_before([1, 1, 5, 6, 7], 2)) == [1, 1, 5, 6, 7]
assert list(remove_all_before([], 0)) == []
assert list(remove_all_before([7, 7, 7, 7, 7, 7, 7, 7, 7], 7)) == [7, 7, 7, 7, 7, 7, 7, 7, 7]
print("Coding complete? Click 'Check' to earn cool rewards!")

Answer 1

iをインクリメントする必要はありません。
delするたびに要素が前に詰められるので、常に先頭要素だけ見れば充分です。

---

追記
ただしこの方法では、[1, 1, 5, 6, 7], 2 のようなケースに対応できません。
根本的に方法を変えるか、あるいは『要素が含まれていない場合』に特別に分岐すれば良いです。

コードの貼り方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

特にPythonの場合、インデントが崩れるとコードの意味が変わってしまいます。

---

また、どこかのサイトの問題を解いているのであれば、引用元を示すべきです。

Comments

[masamuneyuya]

delの性質を理解しておりませんでした。丁寧に教えていただき有り難うございます。

[masamuneyuya]

確かにその通りですね。追記部分を参考にして、もう一度考えてみます。

Answer 2

i = 0
while items[i] < border:

これ1回しか回らないのでループしてる意味ないですよ。