memory issue when trying to create an array

Question

I have a form my company uses to create some files on the fly based on the input. I have a text area$fileString and they can enter file names followed by a hard return. I take that and create an array, $list. What i'm trying to do is if they enter in the text area two file names like this:

item1

item2

that it will create an array that contains 6 values, not two, so like this:

$genList = array(item1_f, item1_b, item1_i, item2_f, item2_b, item2_i);

I am getting this error though when I run my code:

Fatal error: Allowed memory size of 268435456 bytes exhausted (tried to allocate 24 bytes) in /home/dcolor/public_html/dev/create.php on line 18

Line 18 is:

array_push($genList, $list[0] . "_f", $list[0] . "_b", $list[0] . "_i");

What am I doing wrong here? Code below

    $fileString = $_POST['fileList'];
    $unique = $_POST['unique'];
    $size = $_POST['size'];
    $generateArray = $_POST['generateArray'];
    $list = explode("\r\n",$fileString);

    if ($generateArray == "yes") {      
        if ($size == "5x7inimpos") {
            $genList = array();
            while (!empty($list)) {             
                array_push($genList, $list[0] . "_f", $list[0] . "_b", $list[0] . "_i");                    
            }
        }
        print_r($genlist);
    }

My first guess is that while(!empty($list)) {} is an infinite loop. — Terry Seidler
– Terry Seidler, Commented Sep 17, 2012 at 16:03

moonwave99 · Accepted Answer · 2012-09-17 16:05:25Z

3

while (!empty($list)) {             
        array_push($genList, $list[0] . "_f", $list[0] . "_b", $list[0] . "_i");                    
}

$list never gets emptied, you never get out of the while loop.

You just want to iterate over the list:

foreach($list as $element)
{

    array_push($genList, $element . "_f", $element . "_b", $element . "_i");                    

}

answered Sep 17, 2012 at 16:05

moonwave99

22.9k3 gold badges48 silver badges71 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Daniel M · Accepted Answer · 2012-09-17 16:06:28Z

2

The problem is

while (!empty($list)) {             
    array_push($genList, $list[0] . "_f", $list[0] . "_b", $list[0] . "_i");                    
}

Because $list will never be empty and thus causing an endless loop. PHP tries to push more and more elements into $genList until the memory limit exceeds.

You probably want to remove the element in the loop.

answered Sep 17, 2012 at 16:06

Daniel M

3,38922 silver badges30 bronze badges

2 Comments

thindery Over a year ago

I don't understand why the $list will never be empty. If the original text area only had 4 file name values, $list would contain 4 array values. It would go empty after the 4th go through, correct?

Terry Seidler Over a year ago

No, because in the While loop you are not removing anything from list. You are reading the value in $list[0] an infinite amount of times :). In this case you could just iterate over the list like in moonwave99's answer.

jimp · Accepted Answer · 2012-09-17 16:05:16Z

1

This looks like an infinite loop to me. Why would $list ever become empty?

while (!empty($list)) {             
    array_push($genList, $list[0] . "_f", $list[0] . "_b", $list[0] . "_i");                    
}

Therefore, you will allocate memory with array_push until you hit your [256MB] memory limit.

answered Sep 17, 2012 at 16:05

jimp

17.6k3 gold badges30 silver badges39 bronze badges

Collectives™ on Stack Overflow

memory issue when trying to create an array

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related