Re: [問題] 新手for＋判斷式問題

[outshaker (out)]

※ 引述《yvmi (鞭策自己社社長)》之銘言：
: 這是一個在 codewars 裡面的小練習：
: 目標是做一個函式判斷一個字串裡面是否有包含所有英文字母。
: 我的作法是把所有的英文轉成 ascii 碼，建成一個 list
: 然後將 list 中非英文的 ascii 碼以外的刪除。
: 但是實測時發現有部分數值在 97-122 以外的值並不會被刪除。
: 比方說如果投入的 s = 'abcdef!!!!'
:                             ^^→這兩碼驚嘆號會被刪除，後面兩碼不會。
: 如果是在字母的中間則是連續第二個字元不會被刪除。
: 請問這個判斷式的問題在哪裡？
: 感謝大家。
: def is_pangram(s):
:     s = s.lower()
:     s_list = list(s)
:     ascii_i = []
:     for i in s_list:
:         ascii_i.append(ord(i))
:     for j in ascii_i:
:         if j<97 or j>122:
:             ascii_i.remove(j)

我把你的資料改成 s="abc!@#$" 跑過同樣的程式碼變成 "abc@$"
這表示!和#有被程式讀取到，但@和$應該是跳過沒有被讀取到
推測原因是你使用remove刪除了資料影響原本的讀取

一般這種loop是按照順序(或稱index)去讀資料的
讀到第4個字元，發現是!，移除掉該元素，清單變成"abc@#$"
                                               123456
下一次抓的是第5個字元，是#，移除該元素，清單變成"abc@$"
                                                 12345
下一次抓第6個字元，這時候清單已經抓不到字元了，於是退出loop

要檢查程式碼運作其實很簡單，加一段print(ascii_i) 就可以知道資料會怎麼變化
然後再加一些推理就可以猜出原因

最後真的不建議在迴圈裡面增加或刪除資料
改變清單的大小，程式會有很多不預期的結果

原PO推文已經有建議用set或counter了，都是現成且方便的解決方案
這邊我也給一個做法，先把字母挑出來做成一張表，然後檢查a-z是否都有出現在裡面

def test(s):
    t = []
    for c in s:
        if 'a' <= c <= 'z' or 'A' <= c <= 'Z':
            t.append(c.lower()) # 把字母加進清單

    a = "abcdefghijklmnopqrstuvwxyz"
    return all([x in t for x in a])

all() 是用來檢查所有條件都成立的
[x in t for x in a] 是縮寫，等同於以下內容：
list_ = []
for x in a: # x 代表a-z的字母
    list_.append(x in t) # 檢查字母是否在清單內

然後把字母加入清單的部分可以用 filter() 的寫法，有興趣你再找相關資料

以上

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 118.167.195.220 (臺灣)
※ 文章網址: https://www.ptt.cc/bbs/Python/M.1582639685.A.464.html
→ outshaker: 打這麼久沒有稿費QQ
推 Schottky: 沒板主的板，發文不會給 P 幣
了解，謝謝解惑
推 cuteSquirrel: push
謝謝
推 bibo9901: 不知道有什麼好推的 槽點一堆
小弟第一次在這發文，學藝不精還請指教
→ jiyu520: 其實可以用 copy or deepcopy
請問是複製哪個地方？不太明白
※ 編輯: outshaker (118.167.195.220 臺灣), 02/26/2020 01:21:17
→ uranusjr: 其實你可以 if c in string.ascii_letters:
→ outshaker: 當初想說用原生的功能打就好了 還是謝謝你的建議