２つのリストに共通する数字を重複なくリストに入れたいです。

Question

Python3の基礎学習の復習のために２つのリストの数字で共通する数字を重複なくリストにいれるコードを作成しております。
重複が発生してはいますが、共通する数字をリストに格納するところまでは作成できたのですが、
重複する場合の処理がうまく実装できず、詰まっております。
自分のコードに足りない物を教えて頂ければ幸いです。
よろしくお願いします。

def intersection(n1,n2):
    i = 0
    list = []
    for i in n1:
        for j in n2:
            if j in range(i):
                list.append(j)
                i += 1
    
    return list


【実行結果】
入力①
intersection([1,2,4],[2,2,3,5])
⇩
[2, 2, 3, 5]

入力②
intersection([1,2,4],[2,3,5])
⇩
[2, 3, 5]

Comments

[aokikenichi]

例示の【実行結果】について重複は無視するとしても
「２つのリストの数字で共通する数字」
になっていないと思いますが。

[meg_]

> 共通する数字をリストに格納するところまでは作成できた
出来てませんね。
> [1,2,4],[2,2,3,5]
上記の重複数字は > [2, 2, 3, 5] ではなく [2]では？

[amaturePy]

失礼しました！再度コードを確認します！
ご指摘ありがとうございます！
変に修正してしまったかもしれません！

[amaturePy]

確認したところ、元々のコードが間違っておりました！
なんで自分のコードが上記の結果になるかも踏まえて再度考えてみます。

Answer 1

色々ありそうですが、n2の要素だけforで回してn1に入っているか判定、その集計先をsetにしとくという手段で書いてみました。（※setは重複を自動でなくしてくれます）

def intersection(n1,n2):
    d_set = set()
    for j in n2:
        if j in n1:
            d_set.add(j)        
    return list(d_set)
intersection([1,2,4],[2,2,3,5])
# [2]

追記

set使わないver

def intersection(n1,n2):
    i = 0
    lst = []
    for j in n2:
        if (j in n1) and not(j in lst):
            lst.append(j)
    return lst
intersection([1,2,4],[2,2,3,5])
# [2]

LouiS0616さんのコード、これが最もスマート

def intersection(n1,n2):
    return list(set(n1)&set(n2))
intersection([1,2,4],[2,2,3,5])
# [2]

Comments

[amaturePy]

失礼しました！元々の自分のコードが全く自分の予想に即する挙動をしておりませんでした！
再度自分のコードを見直します！
set知らなかったです。調べてみます。
頂いたコードを実行したところ２が重複してしまっているようです。
結果→[2, 2, 3, 5]

[LouiS0616]

setを使って良いなら、return list(set(n1) & set(n2)) で済む気もしますが。

[jeanbiego]

>amaturePy
あれ？　こちらだと[2]と出ます。
一度、そのままコピーでやってみてください。

[jeanbiego]

>amaturePyさん
あ、なんか最初間違えて違うコード貼り付けちゃったからですかね？
今解答欄にあるやつで試してみてください。

[jeanbiego]

>LouiS0616さん
あ、ほんと仰る通りですね。元のコードに手加えるだけで思考停止してました。

[amaturePy]

失礼しました！エディタの方を開き直した所、確かにおっしゃる通りの動作を確認できました！自分のコードは無駄にインクリメントなど入れていて、そもそものロジックを理解できてないのが情けないです。
ありがとうございました！