[問題] 面試遇到readByte

[glitzie (GZ)]

  面試遇到的問題, 我當時整個人傻掉
  他要我舉出下列程式隱藏的問題

  bool ProcessEvilInput::onDecode(SkStream* stream, SkBitmap* bm,
                                  SkBitmap::Config pref, Mode mode)
  ...
     int width = readByte(buf, 6 + i*16);
     int height = readByte(buf, 7 + i*16);
     if (stream->read((void*)buf, length) != length){
         return false;
     }
     int offset = read4Bytes(buf, 18 + i*16);
     int bitCount = read2Bytes(buf, offset+14);
     switch (bitCount)
     {
      case 1:
      case 4:
             c = SkBitmap::kIndex8_config;
             break;
      case 8:
      case 24:
      case 32:
             c = SkBitmap::kARGB_8888_config;
             break;
      default:
             RETURN_ERROR(("Image with %d not supported\n", bitCount));
             continue;
      }
  ...
  }



--
※ 發信站: 批踢踢實業坊(ptt.cc) 
◆ From: 128.220.159.20
→ stosto:..............
推 purpose:現在是什麼狀況？
推 aecho:隱藏的問題1，八成是沒文件沒註解吧。
推 ledia:變數怎麼來的 ? 我猜想考 input buf check 吧
推 purpose:"Image with %d not supported\n" 我猜是文法不夠正確
→ purpose:i*16，有可能 Exception
→ firejox:continue?
推 abzxcx:這個是.............................................冏
推 aecho:那個c應該是color space之類的，灰階和彩色~
→ aecho:然後看到6+i*16這種東西....沒說明是什麼會很想揍人
→ aecho:看起來像是讀bitmap的東西，那bitCount的case本身可能有問題
→ aecho:而且印象中，灰階和彩色的header大小是不一樣的。
推 aecho:呃…我喜歡文件和註解 Q__Q
推 aecho:http://goo.gl/iqVSe , http://goo.gl/ncPsQ
→ aecho:沒文件的情形下，我認為這題考官要開放可以google =w=
→ aecho:kARGB_8888我覺得應該是case 32可以用而己…
→ aecho:可是連SkBitmap的文件也沒有，所以…天曉得…
→ aecho:wiki的資料，color depth有1,4,8,16,24,32,48,64
→ aecho:所以case有漏掉幾種…另外就是調色盤的問題
→ aecho:以前處理的時候，有的圖不是ARGB的放法~記得是要看調色盤
推 aecho:呃…說錯了…那時從twain拿黑白的圖時…
→ aecho:有黑底或白底的設定…然後需要改動調色盤的值。
推 purpose:不知道是原問題真的只給這樣，還是原PO有刪掉
推 aecho:想著想著頭很痛…決定去休息了…今天明明就是假日啊。休息去
推 aecho:洗完澡後，又想到隱藏性的問題了… QwQ
→ aecho:首先，這題八成是給有待在業界的人考的，剛畢業天曉得DIB
→ aecho:再來，考這種問題應該是想找及時的戰力。
推 aecho:而且這家公司應該不小，這題可以刷掉一堆面試者。
推 aecho:會有面試者太多的煩惱，應該公司不會太小吧…
→ aecho:嗯，換個角度思考後，頭果然比較不會痛了…呵呵呵
推 yoco315:最大的隱藏問題就是這份考卷很爛 XD
推 stosto:混C跟C++的CODE...該繼續看嗎?
推 stosto:抱歉看錯~
→ stosto:忽略上面推文XDDD
推 VictorTom:總覺得資訊很不足Orz 話說, default後那個continue是合
→ VictorTom:語法的嗎?_?
→ james732:我的直覺是buf的長度夠不夠塞資料 XD
→ a5480277:這看起來找的領域不是那麼大眾化啦 比較專精於某方面
→ a5480277:考慮到資訊的不足 也不大可能每一項都背起來 應該可以用
→ a5480277:推理的方式講解 像是猜測它為何這樣想(像是讀標頭檔內容)
→ a5480277:整個程式乍看之下 就是在讀標頭檔 判斷輸入的圖片格式
→ a5480277:由於資訊不足 就把可能發生錯誤的地方點出來講
→ a5480277:像是i跟c buff continue 還有第一個if是否該調到最上面
推 justben:會讀不到東西吧 void*轉型
→ justben:buff GG?
→ firejox:如果void*轉型讀不到 那fread....?
推 stosto:buff不是指標卻存位址? 最近剛好有看到這種寫法
→ stosto:然後就是用void*硬轉