อาร์กิวเมนต์ไม่ถูกต้องที่ให้มาสำหรับ foreach()

มันมักจะเกิดขึ้นกับฉันในการจัดการข้อมูลที่สามารถเป็นได้ทั้งอาเรย์หรือตัวแปรโมฆะและฟีดบางอย่างforeachกับข้อมูลเหล่านี้

$values = get_values();

foreach ($values as $value){
  ...
}

เมื่อคุณป้อนข้อมูล foreach ด้วยข้อมูลที่ไม่ใช่อาร์เรย์คุณจะได้รับคำเตือน:

คำเตือน: มีการระบุอาร์กิวเมนต์ที่ไม่ถูกต้องสำหรับ foreach () ใน [... ]

สมมติว่ามันเป็นไปไม่ได้ที่จะ refactor get_values()ฟังก์ชั่นเพื่อส่งกลับอาร์เรย์เสมอ (ความเข้ากันได้แบบย้อนหลังไม่มีซอร์สโค้ดไม่ว่าด้วยเหตุผลใดก็ตาม) ฉันสงสัยว่าวิธีไหนที่สะอาดและมีประสิทธิภาพที่สุดเพื่อหลีกเลี่ยงคำเตือนเหล่านี้:

• กำลังส่ง$valuesไปยังอาร์เรย์
• กำลังเริ่มต้น$valuesกับอาร์เรย์
• ห่อforeachด้วยif
• อื่น ๆ (โปรดแนะนำ)

โดยส่วนตัวฉันคิดว่านี่เป็นสิ่งที่สะอาดที่สุด - ไม่แน่ใจว่ามันมีประสิทธิภาพมากที่สุดหรือไม่!

if (is_array($values) || is_object($values))
{
    foreach ($values as $value)
    {
        ...
    }
}

เหตุผลสำหรับการตั้งค่าของฉันคือมันไม่ได้จัดสรรอาเรย์ที่ว่างเปล่าเมื่อคุณไม่มีอะไรจะเริ่มด้วย

แล้วอันนี้ละ? ทำความสะอาดมากและทั้งหมดในบรรทัดเดียว

foreach ((array) $items as $item) {
 // ...
 }

ฉันมักจะใช้โครงสร้างคล้ายกับนี้:

/**
 * Determine if a variable is iterable. i.e. can be used to loop over.
 *
 * @return bool
 */
function is_iterable($var)
{
    return $var !== null 
        && (is_array($var) 
            || $var instanceof Traversable 
            || $var instanceof Iterator 
            || $var instanceof IteratorAggregate
            );
}

$values = get_values();

if (is_iterable($values))
{
    foreach ($values as $value)
    {
        // do stuff...
    }
}

โปรดทราบว่ารุ่นนี้ไม่ได้ทดสอบมันพิมพ์ลงใน SO โดยตรงจากหน่วยความจำ

แก้ไข:เพิ่มทะลุตรวจสอบ

โปรดอย่าพึ่งพาการส่งเป็นวิธีการแก้แม้ว่าผู้อื่นจะแนะนำว่านี่เป็นตัวเลือกที่ถูกต้องเพื่อป้องกันข้อผิดพลาด แต่อาจทำให้เกิดข้อผิดพลาดอีกอย่างหนึ่งได้

ระวัง:ถ้าคุณคาดหวังว่าจะได้อาร์เรย์ในรูปแบบเฉพาะกลับมาสิ่งนี้อาจทำให้คุณล้มเหลว ต้องการการตรวจสอบเพิ่มเติมสำหรับสิ่งนั้น

เช่นการหล่อแบบบูลไปยังอาร์เรย์(array)boolจะไม่ส่งผลในอาร์เรย์ว่างเปล่า แต่อาร์เรย์ที่มีองค์ประกอบหนึ่งที่มีค่าบูลีนเป็น int: หรือ[0=>0][0=>1]

ฉันเขียนข้อสอบด่วนเพื่อนำเสนอปัญหานี้ (นี่คือการทดสอบสำรองในกรณีที่ URL การทดสอบแรกล้มเหลว)

รวมการทดสอบสำหรับ: null, false, trueเป็นclassนักและarrayundefined

ทดสอบอินพุตของคุณก่อนใช้ใน foreach ทุกครั้ง ข้อเสนอแนะ:

1. การตรวจสอบประเภทอย่างรวดเร็ว :$array = is_array($var) or is_object($var) ? $var : [] ;
2. พิมพ์ Array hintingในวิธีการก่อนที่จะใช้ foreach และระบุประเภทการคืนสินค้า
3. ล้อมหน้าไว้ด้วยถ้า
4. ใช้try{}catch(){}บล็อก
5. การออกแบบรหัส / การทดสอบที่เหมาะสมก่อนการผลิตออกมา
6. ในการทดสอบอาร์เรย์กับรูปแบบที่เหมาะสมที่คุณสามารถใช้array_key_existsคีย์ที่เฉพาะเจาะจงหรือทดสอบความลึกของอาร์เรย์ (เมื่อมันเป็นหนึ่ง!)
7. แยกวิธีผู้ช่วยเหลือของคุณลงในเนมสเปซส่วนกลางเสมอเพื่อลดรหัสที่ซ้ำกัน

ลองสิ่งนี้:

//Force array
$dataArr = is_array($dataArr) ? $dataArr : array($dataArr);
foreach ($dataArr as $val) {
  echo $val;
}

;)

$values = get_values();

foreach ((array) $values as $value){
  ...
}

ปัญหาเป็นโมฆะเสมอและการหล่อนั้นเป็นวิธีการแก้ปัญหาการทำความสะอาด

ก่อนอื่นตัวแปรทุกตัวจะต้องเริ่มต้น เสมอ.
การคัดเลือกนักแสดงไม่ใช่ตัวเลือก
ถ้า get_values ​​(); สามารถกลับตัวแปรประเภทที่แตกต่างกันค่านี้จะต้องตรวจสอบแน่นอน

ส่วนขยายที่กระชับยิ่งขึ้นของรหัสของ@ Kris

function secure_iterable($var)
{
    return is_iterable($var) ? $var : array();
}

foreach (secure_iterable($values) as $value)
{
     //do stuff...
}

โดยเฉพาะอย่างยิ่งสำหรับการใช้โค้ดเทมเพลต

<?php foreach (secure_iterable($values) as $value): ?>
    ...
<?php endforeach; ?>

หากคุณใช้ php7 และคุณต้องการจัดการข้อผิดพลาดที่ไม่ได้กำหนดเท่านั้นนี่คือ IMHO ที่สะอาดที่สุด

$array = [1,2,3,4];
foreach ( $array ?? [] as $item ) {
  echo $item;
}

foreach ($arr ? $arr : [] as $elem) {
    // Does something 
}

นี่ไม่ได้ตรวจสอบว่าเป็นอาร์เรย์หรือไม่ แต่ข้ามการวนลูปถ้าตัวแปรนั้นเป็นโมฆะหรืออาร์เรย์ว่างเปล่า

ฉันไม่แน่ใจว่าเป็นกรณีนี้หรือไม่ แต่ปัญหานี้ดูเหมือนจะเกิดขึ้นหลายครั้งเมื่อย้ายข้อมูลเว็บไซต์ wordpress หรือย้ายเว็บไซต์ไดนามิกโดยทั่วไป หากเป็นกรณีนี้ตรวจสอบให้แน่ใจว่าโฮสติ้งที่คุณกำลังโยกย้ายไปใช้เป็นเวอร์ชัน PHP เดียวกันกับที่ไซต์เก่าของคุณใช้

หากคุณไม่ได้ย้ายเว็บไซต์ของคุณและนี่เป็นเพียงปัญหาที่เกิดขึ้นแล้วลองอัปเดตเป็น PHP 5 ซึ่งจะดูแลปัญหาเหล่านี้บางอย่าง อาจดูเหมือนเป็นวิธีโง่ ๆ แต่หลอกลวงให้ฉัน

กรณีพิเศษสำหรับการแจ้งเตือนนี้เกิดขึ้นหากคุณตั้งค่าอาร์เรย์เป็นโมฆะภายในลูป foreach

if (is_array($values))
{
    foreach ($values as $value)
    {
        $values = null;//WARNING!!!
    }
}

วิธีการแก้ปัญหานี้:

$type = gettype($your_iteratable);
$types = array(
    'array',
    'object'
);

if (in_array($type, $types)) {
    // foreach code comes here
}

คำเตือนข้อโต้แย้งที่ไม่ถูกต้องให้มาสำหรับการforeach()แสดงทวีต /wp-content/plugins/display-tweets-phpไปที่ จากนั้นใส่รหัสนี้ในหมายเลขบรรทัด 591 มันจะทำงานได้อย่างสมบูรณ์

if (is_array($tweets)) {
    foreach ($tweets as $tweet) 
    {
        ...
    }
}

ดูเหมือนว่าจะมีความสัมพันธ์กับสภาพแวดล้อม:

ฉันมีข้อผิดพลาด "อาร์กิวเมนต์ไม่ถูกต้องมา foreach ()" เฉพาะในสภาพแวดล้อม dev แต่ไม่ใช่ใน prod (ฉันทำงานบนเซิร์ฟเวอร์ไม่ใช่ localhost)

แม้จะมีข้อผิดพลาด var_dump ระบุว่ามีอาร์เรย์ที่ดี (ในทั้งสองกรณีแอปและ dev)

if (is_array($array))รอบforeach ($array as $subarray)แก้ปัญหา

ขออภัยที่ฉันไม่สามารถอธิบายสาเหตุได้ แต่เนื่องจากฉันใช้เวลาพอสมควรในการหาวิธีแก้ปัญหาฉันจึงคิดว่าการแบ่งปันสิ่งนี้ดีขึ้นเป็นการสังเกต

ใช้ฟังก์ชั่น is_array เมื่อคุณจะผ่านอาร์เรย์เพื่อ foreach loop

if (is_array($your_variable)) {
  foreach ($your_variable as $item) {
   //your code
}
}

สิ่งที่เกี่ยวกับการกำหนดอาร์เรย์ว่างเปล่าเป็นทางเลือกถ้าget_value()ว่างเปล่า?
ฉันไม่สามารถคิดถึงวิธีที่สั้นที่สุด

$values = get_values() ?: [];

foreach ($values as $value){
  ...
}

ฉันจะใช้การรวมกันของempty, issetและis_arrayเป็น

$array = ['dog', 'cat', 'lion'];

if (!empty($array) && isset($array) && is_array($array) {
    //loop
    foreach ($array as $values) {
        echo $values; 
    }
}

ฉันจะทำสิ่งเดียวกันกับแอนดี้ แต่ฉันจะใช้ฟังก์ชั่น 'ว่าง'

ชอบมาก:

if(empty($yourArray))
{echo"<p>There's nothing in the array.....</p>";}
else
{
foreach ($yourArray as $current_array_item)
  {
    //do something with the current array item here
  } 
}