Fatal error uncaught error call to undefined function each

22

votes

Answer

Solution:

1. For your first two example cases, you could usekey() andcurrent() to assign the values you need.

   $ar = $o->me;   // reset isn't necessary, since you just created the array
   $typ = key($ar);
   $val = current($ar);
   

2. $out = array('me' => array(), 'mytype' => 2, '_php_class' => null);
   $expected = [key($out), current($out)];
   

   In those cases, you can usenext() to advance the cursor afterward, but it may not be necessary if the rest of your code doesn’t depend on that.

3. For the third case, I’d suggest just using aforeach() loop instead and assigning$kv inside the loop.

   foreach ($broken as $k => $v) {
        $kv = [$k, $v];
   }
   

4. For the fourth case, it looks like the key is disregarded inlist(), so you can assign the current value.

   $this->result = current($this->cache_data);
   

   Like the first two cases, it may be necessary to advance the cursor withnext() depending on how the rest of your code interacts with$this->cache_data.

5. Fifth can be replaced with afor() loop.

   reset($array);
   for ($i = 0; $i < 30; $i++) {
       $id = key($array);
       $item = current($array);
       // code
       next($array);
   }
   

303

votes

Answer

Solution:

I found a way to fix it and thought to share the information. Here are also other cases about how to upgrade each() loops to foreach().

Case 1: Missing $value

reset($array);
while (list($key, ) = each($array)) {

Update to:

foreach(array_keys($array) as $key) {

Case 2: Missing $key

reset($array);
while (list(, $value) = each($array)) {

Update to:

foreach($array as $value) {

Case 3: Not missing anything

reset($array);
while (list($key, $value) = each($array)) {

Update to:

foreach($array as $key => $value) {

800

votes

Answer

Solution:

you could create your owneach() function using key(), current() and next(). then replace your calls with that function, like this:

<?php
function myEach(&$arr) {
    $key = key($arr);
    $result = ($key === null) ? false : [$key, current($arr), 'key' => $key, 'value' => current($arr)];
    next($arr);
    return $result;
}

1.

$ar = $o->me;
reset($ar);
list($typ, $val) = myEach($ar);

2.

$out = array('me' => array(), 'mytype' => 2, '_php_class' => null);
$expected = myEach($out);

3.

for(reset($broken);$kv = myEach($broken);) {...}

265

votes

Answer

Solution:

reset($array);
while (list($key, $value) = each($array)) {

UPDATE

reset($array);
foreach($array as $key => $value) {

704

votes

Answer

Solution:

Here are some ways to do it:

The standardforeach loop (very readable):

foreach($this->contents as list($products_id)) {
    $total_items += $this->get_quantity($products_id);
}

Or, reducing:

$total_items = array_reduce($this->contents, function($acc, $item) {
    return $acc + $this->get_quantity($products_id[0]);
});

Or, in a functional expression:

$total_items = array_sum(array_map([$this, 'get_quantity'],
                         array_column($this->contents, 0)));

None of these methods needreset($this->contents); preceding it.

895

votes

Answer

Solution:

The way you most definitely shouldn’t do is put the function «back into php» by adding it to the auto_prepend_file setting in php.ini

auto_prepend_file = "/var/www/php/auto_prepend.php"

Then make the file and enter in the function with an function_exists wrapper.

<?php
/**
 * Adds the depreciated each() function back into 7.2
 */
if (!function_exists('each')) {
    function each($arr) {
        $key = key($arr);
        $result = ($key === null) ? false : [$key, current($arr), 'key' => $key, 'value' => current($arr)];
        next($arr);
        return $result;
    }
}

This essentially declares the function before your php application runs. When your application tries to run the each function it’ll use your version instead.

This is absolutely not the way you should be approaching this problem, especially in production! However you’re a developer with time constraints and you just want to try arbitrary frameworks for your next project and they haven’t been updated to work on your local development server without winding back your php version.

When you’ve committed to a code base for your project please go ahead and implement the changes in the accepted answer because they do work.

I used Wee Zel’s emulation of the each function

775

votes

Answer

Solution:

To expand on Petro Mäntylä excellent correct answer for Case 3:

Here is a full example of a «Case 3» situation, because I find full examples far more informative that one line code fragments:

This is genuine code from a 3rd party old code base (TCPDF)

DEPRECATED:

while (list($id, $name) = each($attr_array)) {
      $dom[$key]['attribute'][$name] = $attr_array[$id];
      ...              
      ...             
   }

FIXED:

 // while (list($id, $name) = each($attr_array)) {
 foreach($attr_array as $feKey => $feRow){
    // $dom[$key]['attribute'][$name] = $attr_array[$id];
    $dom[$key]['attribute'][$feRow] = $attr_array[$feKey];
    ...
    ...
    }
 unset($feKey,$feRow);

547

votes

Answer

Solution:

Replace this code

while (list($_key,$_resourceTypeNode) = each($GLOBALS['config']['ResourceType'])) {
//             if ($_resourceTypeNode['name'] === $resourceTypeName) {
//                 $this->_resourceTypeConfigCache[$resourceTypeName] = new CKFinder_Connector_Core_ResourceTypeConfig($_resourceTypeNode);

//                 return $this->_resourceTypeConfigCache[$resourceTypeName];
//             }
//         }

with this one

foreach ($GLOBALS['config']['ResourceType'] as $key => $_resourceTypeNode) {
            if (isset($_resourceTypeNode['name'])) {
                if ($_resourceTypeNode['name'] === $resourceTypeName) {
                    $this->_resourceTypeConfigCache[$resourceTypeName] = new CKFinder_Connector_Core_ResourceTypeConfig($_resourceTypeNode);
                    
                    return $this->_resourceTypeConfigCache[$resourceTypeName];
                }
            }
        }

863

votes

Answer

Solution:

 //  while (list($products_id, ) = each($this->contents)) {
   //  $total_items += $this->get_quantity($products_id);
 // }

Update To :

foreach(array_keys($this->contents) as $products_id) {
  $total_items += $this->get_quantity($products_id);
}

Other Condition:

foreach($this->contents as $key =>$value) {
  $total_items += $this->get_quantity($products_id);
}

71

votes

Answer

Solution:

What about using this function?

function array_fetch(array $a) {
   $element = current($a);
   next($a);
   return $element;
}

    // function.php
<?php
    namespace fooNamespace {
        function foo() {
            return "Calling foo"
        }
    }
?>
    // main.php
include function.php
<?php
    echo fooNamespacefoo();
?>

<?php
    class foo {
        function barFn() {
          echo "Hello foo!"
        }
        function bar() {
            barFn();
            // interpreter is confused which instance's function is called
            $this->barFn();
        }
    }
    $a = new foo();
    $a->bar();
?>

• Verify that the file exists. Find the PHP file in which the function definition is written.

• Verify that the file has been included using the require (or include) statement for the above file on the page. Check that the path in the require/include is correct.

• Verify that the file name is spelled correctly in the require statement.

• Print/echo a word in the included file to ascertain if it has been properly included.

• Defined a separate function at the end of the file and call it.

• Check functions are closed properly. (Trace the braces)

• If you are calling methods of a class, make sure $this-> is written.

Shraddha is a JavaScript nerd that utilises it for everything from experimenting to assisting individuals and businesses with day-to-day operations and business growth. She is a writer, chef, and computer programmer. As a senior MEAN/MERN stack developer and project manager with more than 4 years of experience in this sector, she now handles multiple projects. She has been producing technical writing for at least a year and a half. She enjoys coming up with fresh, innovative ideas.

LinkedIn