[問題] 程式碼大量重複卻不知如何解決

[woody3724 (woody)]

給一個字串，若遇到字母，則依照字母順序將它改成 k 個字母後的字母，且不改變大小寫

若超過 Z/z 則回頭從A/a算起  且 k 必定大於 0 且小於等於 9

附上26個字母讓大家方便查詢

A B C D E F G H I J K L M N O P Q R S T U V W X Y Z

例如字串為 x3W4jO/a   且 k = 6    則改成 d3C4pU/G

我的寫法:https://i.imgur.com/XBIdN7s.jpg

但是第 3~7 行 與 9~13 行重複性太高了，根本只差在第5行的 'Z' 與第11行的 'z'

因此我改成 https://i.imgur.com/T4YPD3L.jpg

但明顯是錯的，原因在23~26行的註解

不知道各位有沒有漂亮的寫法呢?  謝謝

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 68.181.88.39
※ 文章網址: https://www.ptt.cc/bbs/Programming/M.1508307377.A.359.html
推 eight0: 我覺得這不是很大量
→ Neisseria: 與其糾結在這幾行程式碼上還不如把
→ Neisseria: 公開函式/方法設計一下比較實在
→ MOONRAKER: 還以為多大量
→ MOONRAKER: 等你上班會看到一個block重覆十幾次
→ MOONRAKER: 每一個block可能一百行 那也沒多大
→ MOONRAKER: 這麼簡單的事情上面搞最佳化就像小學生
→ MOONRAKER: 比誰寫字比較快一樣
→ kisaraki1976: 即使是小學生比寫字比快，也是可以
→ kisaraki1976: 練到基本功的。給個提示，去看看ASC
→ kisaraki1976: II表中字母的排列順序，你再思考一
→ kisaraki1976: 下條件判斷式怎麼合併怎麼寫。
推 yvb: 圖2列22前算個z, 列23就可改為 if(s[i] > z)
→ MOONRAKER: 年紀太小做推理訓練不會有好的成果。
→ MOONRAKER: 清楚比簡短重要 對新手如此 老手也如此
→ bcew: 在23行加上用s[i]的值區分大小寫就能有正確
→ iFEELing: 怎麼覺得好像在密碼學看過這東西...
→ iFEELing: 凱撒密碼 ?
→ adrianshum: 抽個function 就好了。像 char shiftC
→ adrianshum: har(char c, int charsToShift, char
→ adrianshum:  lowerBound, char upperBound)
→ adrianshum: 不在lower & upper bound 中間的就直
→ adrianshum: 接回傳原字
推 vi000246: 你可以去買本clean code 或重構的書
推 Killercat: 別用-=, 有個運算元叫做"%"
→ Killercat: 用%會精簡很多
→ F04E: 推薦讀一下 如何撰寫0錯誤程式 這本書