開發平台(Platform): (Ex: Win10, Linux, ...)
windows
編譯器(Ex: GCC, clang, VC++...)+目標環境(跟開發平台不同的話需列出)
GCC
額外使用到的函數庫(Library Used): (Ex: OpenGL, ...)
NO
問題(Question)：
此段程式有甚麼risks 跟 如何改進他
程式碼(Code)：(請善用置底文網頁, 記得排版，禁止使用圖檔)
void clean_buffer(int A[], unsigned int number){
unsigned int i;
if(number > 0){
for(i = 0; i <= number - 1; i++){
A[i] = 0;
}
}
}
這題是去年在NV面試時遇到的考題
考官有提到number 跟 array A[] 的size是一樣的
但答案是跟 overflow 或是 踩到未定義記憶體位址有關係
實在是忘記當初的答案 最近想到這題實在是太想知道答案
無奈chatgpt沒辦法給出正確的回答 上來請教各位大大 再麻煩幫忙解惑了

number - 1 遇到 number==0 會 overflow

code看起來好像沒問題，不過可以簡化，if判斷拿掉，for的條件改成i < number看起來比較乾淨

考題應該沒有if

number==0直接就不會進迴圈了吧

考題有if 這是我直接複製下來的我也記得他好像是說number的地方要改

for裡面的條件改成 i < number 應該連if都不用包了

int 比 uint 還是要轉型，那就有可能不是你預期的比較結果阿抱歉看錯

邏輯上沒什麼問題 難道他是希望你把number和i的型別換成size_t ?

實際跑了一下，好像 clean_buffer(int* A, unsigned比較像正常的寫法

該不會是說int不一定對齊記憶體大小，要用unsigned long吧有可能就是K大講的那個，要size_t去對到CPU定址能力。

可能要檢查 A != NULL 吧

答案應該是unsigned int有可能無法表達std::size_t所有的數，可以用T (&A)[N]https://godbolt.org/z/GMxdPcrYzint* A比較正常的說法聽起來怪怪的，因為本來就一樣https://eel.is/c++draft/dcl.fct#5.sentence-3

沒說這題是考c還是c++啊

會不會是你記錯題目? 考題裡的i是不是int而不是你記得的unsigned int? 如果i的型別是int的話會有問題但我想想又有矛盾，如果是因為i的型別為int而造成的overflow，那那個number會很大，而這個number又是array size理論上應該沒這麼大的array才對。。。如果array是動態配置的可能可以，但如果array是動態配置function參數寫int []我會想砍人

是我的話會用 memset;不過搞不好是禁用?

我覺得這題應該要問Caller

認同一樓說的，number==0會變無窮迴圈

認同上面兩樓 問caller

如果是說未定義記憶體位置有說陣列長度為多少嗎number-1的索引是否超過原本陣列的大小

number過大會造成A位址回卷，因為A不是對齊bit0（猜

認同一樓，unsigned int的-1是-1的補數，通常非常非常大XD

number == UINT_MAX 會變無窮迴圈

外面有number>0，overflow的情況裡面不會出現吧

你會不會其實不小心把答案跟題目弄反了？一般用 unsigned 可能寫成 for (unsigned i; i < n; ++i)，沒意識到n 可能是 0？

會計算 n+1次 n-1 很浪費效能？還是char*？for buffer？

number 為 0 就已經被 if(number > 0) 擋掉了，不會出事啊至於 number 過大，原Po已經說「考官有提到number 跟array A[] 的size是一樣的」，你要搞出一個 stack 宣告得出來但是 unsigned int 存不下的陣列大小？因為是 int 陣列，似乎需要 > 16GB 的 stack 大小呢XD在這之前，這個陣列要怎麼宣告出來XD

喔喔，那個if被我無視了，那應該不是這問題

為什麼一定要放stack XD

看不出來，就算是alignment的問題，array的記憶體位置也會shift到對的位置。目前看得比較會是i <= number - 1的編譯順序會不會不一樣

看起來最有可能的是caller給的size會超過uint吧，清不乾淨

我有點好奇是否有因為optimization 省略if的情況

最佳化的話可能要編譯器能 (1) 把迴圈化簡成 memset 之後(2) 這個函數被 inline (3) if 的條件在上層函數已滿足不過 (3) 這個條件記得不容易滿足 (技術上&實際程式上都是)

了解，仔細想caller給的size超過unsigned是有可能的，以c標準而言，unsigned是有可能16bit的。

會不會是number給的size是byte,也就是 sizeof (unsigned int)計算byte，但你是用unsigned int去做index所以會爆掉？int A[i]會位移爆掉？

會不會跟32bit記憶體定址有關?