プレイヤーがアイテムを使用した時、 アイテム欄からアイテムを消す処理を書きたいです

Question

実現したいこと

unityの、ビジュアルスタジオでスクリプトを書いています。
プレイヤーがＥキーを押してアイテムを使用した時、
アイテム欄からアイテムを消す処理を書きたいです

プレイヤーが拾ったアイテムを、Item型のitemlstの中に追加しています。
リストの中に追加されたアイテムのスプライトが、
アイテム欄（slots[]）の中に順番に表示されていきます

アイテム欄は三つしかないので、
itemlstも三つまでしか入らないようにしました。

slots[]は、ヒエラルキーにある３つのアイテム欄に個別に取り付けられた、複数あるSloteスクリプトを
配列の中に入れています。

リストの中からリムーブしたアイテムのスプライトを、
slots[]（アイテム欄）から消したいです。

public class InventryUI : MonoBehaviour
｛

public void UpdateUI()
{

 slots = WheponSlotPanel.GetComponentsInChildren<Slot>();
 
 for (int i = 0; i < slots.Length; i++)  
 {

   //もし、拾ったアイテムの数よりもiが小さければ
     if (i < Inventry.instance.itemList.Count)
     {
         slots[i].AddItem(Inventry.instance.itemList[i]);
         
     }
 }

}

Item[] itemlst = Inventry.instance.itemList.ToArray();

for (int k = 0; k < itemlst.Length; k++)
{

    Slot slotsUI= slots[k];

if (Input.GetKeyDown(KeyCode.E))
{
   
    Inventry.instance.itemList.RemoveAt(k);
    
    
    slotsUI.ClearItem(slotsInItem);
    
}

｝

public class Slot : MonoBehaviour
{

public void AddItem(Item itemAdd)
{
item = itemAdd;
icon.sprite = itemAdd.itemedateicons;

}

public void ClearItem(Item item)
{
slotsItam = item;
slotsItam = null;

icon.sprite = null;

}

｝

アイテム欄は、三つ並んでおり、拾ったアイテム（リストの中に入っている）を左から順番に表示します
リムーブしたアイテムのインデックスと、同じインデックスのｓloteｓ[]のスプライトを非表示
にしようと思いました。

しかし

Inventry.instance.itemList.RemoveAt(k);

の一行から

ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index

というエラーが発生しました。

ｃ#です。
どのように修正したら良いのかわかりません。
是非、ご教授お願いします。

Answer 1

Logを出力するなどして消す処理が複数回実行されているのか確認してみてください。
複数回実行されているとすれば
k=0の時、Inventry.instance.itemListの数=3、itemList.RemoveAt(0);
k=1の時、Inventry.instance.itemListの数=2、itemList.RemoveAt(1);
k=2の時、Inventry.instance.itemListの数=1、itemList.RemoveAt(2);
という風に動作するかと思います。
このように動くとk=2の時にRemoveAtで指定するIndexがitemListの数を超えてしまうためArgumentOutOfRangeExceptionが発生します。

こういったListの削除の場合はループは大きい数字から小さい数字へとループすれば連続して消されても
k=2の時、Inventry.instance.itemListの数=3、itemList.RemoveAt(2);
k=1の時、Inventry.instance.itemListの数=2、itemList.RemoveAt(1);
k=0の時、Inventry.instance.itemListの数=1、itemList.RemoveAt(0);
と動作するのでArgumentOutOfRangeExceptionは発生しません。

ただ1回のUpdateUIの呼び出しで複数回消す処理が走るのは正しいのでしょうか？

Comments

[kontiwa]

回答ありがとうございます。
参考にし書き換えてみたところArgumentOutOfRangeExceptionのエラーを修正する事ができました。
しかし、指摘して頂いた通り複数回処理が走っているからなのか、例えばslots[２]のスプライトだけを消したいのに、slots[0]やｓloteｓ[１]などの全てのアイテムの表示が消えてしまいます。
（リストからは[２]しか消えていません）
どのように書き換えれば良いのか、何かアドバイスがあれば教えていただけると幸いです。

[YAmaGNZ]

アイテムを使用するというのはどのような動作を考えているのでしょうか？
インベントリを開いて、使用するアイテムを選択しEキーで使用するという動作でしょうか？
それともアイテムを拾った順に使用するという動作でしょうか？
そういった動作仕様で考え方が異なるかと思います。

どちらにしろループで消すのではなく消したいアイテムのインデックスを求めてそのインデックスの物だけを消すという動作になるかと思います。

[kontiwa]

使用するアイテムを選択し、使用するという動作です。
消したいアイテムのインデックスを求める方法で、再度書き換えて見ようと思います。
初めてアイテム欄のスクリプトを書くので戸惑っていました。
親切に教えて頂き、本当にありがとうございます。