Laravel Undefined offset: 2 in for loop

Question

I want to add table grid dynamically with one color and multiple image but in controller for loop i get undefined offset:2 error

here is my grid html code:

    <div class="form-group">
        <label class="col-sm-2 control-label">Image</label>
        <input type="number" class="form-control" name="imgcolor[]" />
        <div class="field col-sm-10" align="left">
          <input type="file" class="files" name="image[0][]" multiple />
        </div>
    </div>
    <div class="form-group">
        <label class="col-sm-2 control-label">Image</label>
        <input type="number" class="form-control" name="imgcolor[]" />
        <div class="field col-sm-10" align="left">
          <input type="file" class="files" name="image[1][]" multiple />
        </div>
    </div>

in my controller code:

     $input = $request->all();
        $color = $request->input('imgcolor');
        for($i=0; $i <= count($input['imgcolor']); $i++) {
            for($j=0; $j <= count($request->image[$i]); $j++) {  // here Undefined offset: 2
                $data = $request->file('image')[$i][$j];
                $imagename = $data->getClientOriginalName();
                $data->move('uploads/product/', $imagename);
                Image::create([
                    'product_id' => $product->id,
                    'color_id' => $color[$i],
                    'image' => $imagename
                ]);
            }
        }

actually i need to store data in image table like one color multiple images.

What does the form actually submit? It looks like `imgcolor[]` and `image[]` aren't the same length. — Thomas, Jul 16 '19 at 16:21

score 0 · Accepted Answer · answered Jul 16 '19 at 16:17

0

Replace $i = count($input['imgcolor']); with $i < count($input['imgcolor']);.

Same for inner loop with $j counter.

answered Jul 16 '19 at 16:17

u_mulder

54,101
5
48
64

i already tried that. updated my questions thanks – Romi Jul 16 '19 at 16:19
Strictly `<`, not `<=`. – u_mulder Jul 16 '19 at 16:20
Do you understand __why__ it works? – u_mulder Jul 16 '19 at 16:24
explain please?? – Romi Jul 16 '19 at 16:25
1

Because indexes in array are started from `0`. If you have __3__ elements in array their indexes are `0`, `1`, `2`. – u_mulder Jul 16 '19 at 16:26

Laravel Undefined offset: 2 in for loop

1 Answers1