Re: [問題] 圖片垂直翻轉

看板C_and_CPP (C/C++)作者pnpncat (meow)時間13年前 (2012/07/13 22:14)推噓1(1推 0噓 10→)

留言11則, 4人參與討論串2/2 (看更多)

: 程式碼(Code)：(請善用置底文網頁, 記得排版) : HDC turnHDC(int op, HDC* dest, HDC* src) : { : RECT rc; : GetClipBox(*src, &rc); : switch (op) { : case V_TURN: : for (int i = 0; i < rc.right; i++) : for(int j = 0; j < rc.bottom; j++) : SetPixel(*dest, i, j, GetPixel(*src,i, : rc.bottom-j)); : break; : case H_TURN: : break; : } : } : 補充說明(Supplement)：這團code有諸多問題第一風格詭異...... 諸如op使用int型態而不用enum switch裡面的 case V_TURN: 明明是label卻縮排八格等等........ 第二迴圈外橫內直....... 圖檔的儲存都是先跑橫向再換行的你應該外直內橫也就是 for (int j = 0; j < bottom; ++j) { for(int i = 0; i < right; ++i) { //blablabla......... } } 這樣寫運算時才會符合記憶體中的儲存方式否則圖檔如果太大電腦做cache不方便可能會拖慢效能第三你對函式的呼叫者莫名的有信心...... 誰知道函式的呼叫者會不會傻傻的將同樣的指標當作dest和src一起丟進去呢？這個預期沒有道理呀當你過了兩個月再回來用自己寫的這個函數你還記得這個函數的參數需要不犯這個錯才能正確運作嗎？前兩點就算了但第三點應該從函數設計上就杜絕這種問題的發生才對假如換成讓函數直接傳回一個指標指向一張翻轉過的圖而不要讓dest當作一個參數不就可以徹底解決這個問題了嗎？又或者如果你有理由必須用目前的方法來設計函數那你就應該要作相應的檢查例如 if (src == dest) { //作特殊處理 } //作正常處理總之當你需要將一個東西寫成函數多半就不會只用一次也常常不會只以一種方法使用他如果你希望函數只在某種限制下被使用你就應該明確的將這種限制寫進code裡面讓別人不能不按規矩來才對......^^" -- 直接閱讀《琴劍六記》 http://gs.cathargraph.com/p/list.html 　　《琴劍六記》Facebook專頁 https://www.facebook.com/GSannals -- ※ 發信站: 批踢踢實業坊(ptt.cc) ◆ From: 27.105.22.143