2016-12-08 10 views
-1
Does my method look correct? I have a function called: 

int型searchInventory(CONST構造体項目項目[]、CONST int型のsku_item、 のconst int型のサイズ)この関数は、タイプ項目のアレイのアドレスを受け取りこのインベントリ配列に対して線形検索を実行するにはどうすればよいでしょうか?見つかった場合は一致する項目のインデックス、見つからなければ-1を返します。

(項目[])、所望のアイテムのSKU番号の整数、および配列のサイズを表す整数サイズ。フラグを正しく使用するかどうかはわかりません。配列を通じて、私は希望のsku番号と返品の項目を検索しなければならない場合は、一致する項目のインデックス。 見つからない場合は、-1を返す必要があります。このサイトの新機能では、皆さんに必要な情報を私が手伝ってくれたかどうかはわかりません。まったく分かっていれば、私に助言してください。

for (i = 0; i < size; i++) 
{ 
    if (sku_item == items[i]._sku) 

     return i; 
    else 
    { 
     return -1; 
    } 

} 
+1

for-loopの後に移動します。 (そしてelseブロックを削除する) – BLUEPIXY

答えて

4
for (i = 0; i < size; i++) { 
    if (sku_item == items[i]._sku) { 
     return i; 
    } 
} 
return -1; 

あなたが発見されない項目を宣言する前に、配列全体を反復する必要があります。したがって、return -1をループ外に移動してください。

関連する問題