Re: [問題] 請教有無方法簡化我的code

[loveme00835 (最愛朴素妍)]

※ 引述《chessjim (jim)》之銘言：
:         ( *[1m *[m 為色碼，可以按 Ctrl+V 預覽會顯示的顏色 )
:              ( 未必需要依照此格式，文章條理清楚即可 )
: 遇到的問題： (題意請描述清楚)
: 我寫了一段code
: 發現code中重複的地方很多
: 不知道有沒有方法簡化它
: 希望得到的正確結果：
: 能夠簡化我case1,case2,case3中的code
: 因為重複的地方很多
: 我的code片段如下
: //pre:level,type分別控制三角形層級與種類
: //post:印出三角形
: void printTriangle(int level,int type){
:      int line,cnt;
:      switch(type){
:          case 1:
:              for(line=1;line<=level;line++){
:                  for(cnt=1;cnt<=line;cnt++){
:                      printf("1");
:                  }//end for-cnt
:                  printf("\n");
:              }//end for-line
:          break;
:          case 2:
:              for(line=1;line<=level;line++){
:                  for(cnt=1;cnt<=line;cnt++){
:                      printf("%d",cnt);
:                  }//end for-cnt
:                  printf("\n");
:              }//end for-line
:          break;
:          case 3:
:              for(line=1;line<=level;line++){
:                  for(cnt=1;cnt<=line;cnt++){
:                      printf("%d",line);
:                  }//end for-cnt
:                  printf("\n");
:              }//end for-line
:          break;
:          default:
:              printf("你輸入了錯誤的三角形種類!!");
:          break;
:      }//enf switch
: }//end
出現type code了, 首先需要做的就是把 int type這個參數
換成列舉型態, 提升可讀性並且也提供靜態型別檢查, 不用
呼叫之後才用default去檢查合法與否(雖然我還想不到這三
種三角形該怎麼敘述他).

   http://codepad.org/MCGDiB5G


裡面三個巢狀迴圈可以分開寫成不同函式, 並且使用清楚的
名稱去命名它們 :

   http://codepad.org/IL2Kb3by

再來就是 switch 的問題, 如果新增了一種三角形, 這邊是
一定要改的, 所以可以用函數指標陣列, 利用列舉值當索引
來選擇呼叫特定的函式, 這樣要新增也比較方便 :

   http://codepad.org/whCDsD8u


以上, 簡化多少不是問題, 能不能讓你未來能少打一點程式
碼, 以及程式好不好加入新功能, 才是最重要的.

--
     ◢████ ◢█    ◢██◣ ◢█   ◢███  ◢█    T-ara版怎麼去
     ████◤ ██  ◢██◣█ ██   ████  ██        s ~> T-ara
     █／███ ██  ██  ██ █／█ ◢███  █／█          歡迎您的光臨
     ████◤ ██  ██  ██ ██◤ ███◤  ██◤  恩靜、智妍、孝敏
     █／███ ██  █／██◤ ██   █／██  ██       素妍、居麗、寶藍
     ████◤ █◤  ◥██◤   █◤   ████◤█◤            ψmakigoto123

--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 140.121.197.115
※ 編輯: loveme00835     來自: 140.121.197.115      (10/05 10:08)
推 stupid0319:三個巢狀迴圈可以寫成一個函式吧
推 lgen7604:強者的見解就是不一樣  獲益良多
推 xatier:推love大
→ loveme00835:三個"不同功能"的部分寫在一起, 這是很常見的"責任分
→ loveme00835:配不均" 的行為, 會造成擴充不易, 且會有if switch的
→ loveme00835:結構重複出現
推 holymars:同一樓，我也覺得巢狀迴圈應該在switch-case的外面
→ holymars:這三個三角形「不同」的部份只有印出來的東西不一樣
→ holymars:你在抽取函式的時侯應該把同功能的部份保留，只抽取不同
→ holymars:功能的部份　不然會造成redundant code反而維護不易
→ loveme00835:可是如果要改動這裡印出的數字的規則, 迴圈裏面就會變
→ loveme00835:的更複雜
→ loveme00835:在改程式碼的時候也沒辦法分開來做, 假如是別的使用者
→ loveme00835:勢必要動到你的原始碼
推 holymars:不會更複雜啊　只是把三個function合成一個來寫
→ holymars:假如你有第四種三角形　而且它的形狀真的不一樣而不是只
→ holymars:有印出來的東西不一樣的話，才需要再寫另一個函式...
→ loveme00835:我就是說樓上這種情形, 加上一個倒三角就變成兩個函式
推 holymars:就算有倒三角的需求，也應該要把三個正三角的寫成一個函
→ holymars:式啊...
→ loveme00835:然後client端因為相依於這樣的改變, 選擇的地方又要作
→ loveme00835:改動, 你著重在邏輯簡化, 我著重在擴充上 0.0
推 chessjim:學到很多重要觀念,大謝
推 holymars:選擇的地方不用改動吧　是printTriangle要傳兩個參數進去
→ holymars:啊…我不是說你的printTriangle　我是說三個合成一個之後
→ holymars:要傳兩個參數　然後該函式在迴圈裡判斷要印什麼東西
→ loveme00835:加上倒三角之後, 本來從三選一變四選一的地方啦~
推 holymars:對啊　那裡不用動啊= = 是你的三個函式要合成一個啊
→ loveme00835:吼~ :( 我知道你說三個合成一個的地方, 就是上篇推文
→ loveme00835:中的程式碼, 我是指加上新三角之後, 就要做兩種函式的
→ loveme00835:切換, 加入新的正三角數字規則, 就要改變本來的三合一
→ loveme00835:函式, 然後切換函式的地方會變得複雜
推 holymars:不用啊= = 本來三合一的函式不用改　再加一個新函式就好
→ holymars:一樣在你選函式的地方加上另一個case呼叫新的函式而已
→ loveme00835:就是加入新的函式, 就要if(type= 1 ~ 3 )呼叫三合一
→ loveme00835:if(case=3) 呼叫新函式, 但是如果新的三角形是使用者
→ loveme00835:自己想加的話, 他就還要改到這個印出的函式, 或是還要
→ loveme00835:再包一層, 不是很麻煩嗎?
→ loveme00835:第 2 個 if 打錯 =  =
推 holymars:不用改印出的函式　只要再加一個新的函式就好....
→ holymars:原本的三個函式大部份的功能相同就該寫成一個　新加的函
→ holymars:式如果沒辦法共用原本的功能就再寫另一個而已　這邏輯沒
→ holymars:有很複雜吧..
→ loveme00835:我知道你說的阿=  =, 想想寫選擇的函式跟加新三角形的
→ loveme00835:作者是不同人的情況, 為了讓他的三角形你的選擇函式要
→ loveme00835:多一個case, 然後多一堆稀奇古怪的圖形, 就更多case,
→ loveme00835:這樣就顯示不出來當初三合一的好處了
推 holymars:那不就是你的printTriangle函式在做的事嗎XD 我沒有反對
→ holymars:這塊啊
→ holymars:我說的是你的printOne Two Three應該要合成一個
→ loveme00835:可以合阿~ 只不過不這樣分很清楚的話比較不能突顯這
→ loveme00835:個方法的用途
推 stupid0319:寫法本來就很多,像遊戲外掛被封鎖,換個寫法又是活龍
→ loveme00835:只不過我習慣上還是會分開寫啦, 再加上函式名稱說明
→ loveme00835:各種不同三角形的特徵, 如果這邊用的是繼承 + 多型,很
→ loveme00835:容易看出分開寫優點
→ loveme00835:另外發現一點: 三合一函式相依於 type 的實際值, 假如
→ loveme00835:這個數值分佈有需要變動, 三合一的部份就要改寫
※ 編輯: loveme00835     來自: 140.121.197.115      (10/05 13:38)