python3で辞書を使った文字変換をする方法

Question

標準入力で得られた文字列をある規則に従って変更する暗号的なプログラムを書いています。
辞書のキーを取得し、それが標準入力のリストの文字と一致しない場合はそのまま、する場合は辞書の値を表示するものです。

lis = list(input_lines)
dic ={  "A":"4",
        "E":"3",
        "G":"6",
        "I":"1",
        "O":"0", 
        "S":"5", 
        "Z":"2" }
#print(lis)
#k = dic.keys()
print(k)
for moji in lis:
    if dic != moji:
        print(moji)
    else:
        print(dic[moji])

このコードで実行するとキーがその標準入力にあっても標準入力のまま出力されます。
理由が分かりません。
回答の程よろしくお願い致します。

Answer 1

問題設定がやや不明瞭ですが、str#translate の利用も検討できそうです。

from string import ascii_uppercase

table = str.maketrans(
    'AEGIOSZ', '4361052'
)

src = ascii_uppercase
print(src)

dst = src.translate(table)
print(dst)

実行結果 Wandbox

ABCDEFGHIJKLMNOPQRSTUVWXYZ
4BCD3F6H1JKLMN0PQR5TUVWXY2

Comments

[bonji]

回答ありがとうございます。

Answer 2

産駒として get() を使う例を示します。

lis = list(input())
dic = {
    "A":"4",
    "E":"3",
    "G":"6",
    "I":"1",
    "O":"0",
    "S":"5",
    "Z":"2"
}

for moji in lis:
    tr = dic.get(moji)
    if tr is None:
        tr = moji
    print(moji, " -> ", tr)

追記：

コメントでの指摘を受けて...

transed = [dic.get(moji, moji) for moji in lis]
print(lis, " -> ", transed)

Comments

[LouiS0616]

dict#getは第二引数にデフォルト値を与えられるので、それを利用すると行数が減りそうですね。

[katoy]

ああ、そうでした！

[bonji]

回答ありがとうございます。おかげで上手くいきました。

Answer 3

if dic != mojiの部分が違います。辞書と文字なので常にFalseになってしまいます。
kを定義しているので、 if not moji in kのようにするとよいです。

Comments

[LouiS0616]

横から失礼します。
not a in b でももちろん誤りではないのですが、Pythonでは a not in b の方が好まれます。

[Udomomo]

そうでした。ありがとうございます。