php / php/php-src

Inconsistent behaviour with AppendIterator and empty generators

Đang mở
#11,591 7 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Bug Category: Generators Extension: spl Status: Verified
Ngôn ngữ chính
C
Star
40.4k
Fork
8.1k
Merge trung bình
2 ngày 13 giờ
Pull request đã merge (30 ngày)
96

Mô tả

Description

I've encountered inconsistent behaviour in the handling of empty generators with AppendIterator. Please can you confirm whether this is a bug or some weird expected behaviour.

Scenario 1

When iterating over a generator that yields nothing, foreach has no items to iterate over so var_dump isn't called, as expected.

<?php

function yieldEmpty()
{
    yield from [];
}

foreach (yieldEmpty() as $item) {
    var_dump($item);
}

Scenario 2

When attaching a generator yielding items and a generator yielding nothing and attaching both to an AppendIterator, foreach iterates over the items from the former and the three values are output, as expected.

<?php

function yieldRows()
{
    yield from ['one', 'two', 'three'];
}

function yieldEmpty()
{
    yield from [];
}

$iterator = new AppendIterator();
$iterator->append(yieldRows());
$iterator->append(yieldEmpty());

foreach ($iterator as $item) {
    var_dump($item);
}

Outputs (as expected):

string(3) "one"
string(3) "two"
string(5) "three"

Scenario 3

When attaching ONLY a generator yielding nothing and attaching it to an AppendIterator, foreach unexpectedly fails with an exception.

<?php

function yieldEmpty()
{
    yield from [];
}

$iterator = new AppendIterator();
$iterator->append(yieldEmpty());

foreach ($iterator as $item) {
    var_dump($item);
}

Expected output (following on from scenario 1 and 2):

string(3) "one"
string(3) "two"
string(5) "three"

Actual output:

PHP Fatal error:  Uncaught Exception: Cannot traverse an already closed generator in /Users/richard.coupland/test3.php:11
Stack trace:
#0 /Users/richard.coupland/test3.php(11): AppendIterator->rewind()
#1 {main}
  thrown in /Users/richard.coupland/test3.php on line 11

Fatal error: Uncaught Exception: Cannot traverse an already closed generator in /Users/richard.coupland/test3.php:11
Stack trace:
#0 /Users/richard.coupland/test3.php(11): AppendIterator->rewind()
#1 {main}
  thrown in /Users/richard.coupland/test3.php on line 11
PHP Version

8.2.7

Operating System

No response

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu bằng cách tái hiện ba kịch bản AppendIterator với các generator yieldEmpty và yieldRows trên PHP 8.2.7, tập trung vào lỗi xảy ra trong foreach và AppendIterator->rewind(). Xác định xem hành vi khác biệt là có chủ đích hay là một bug, sau đó thêm hoặc cập nhật phạm vi kiểm thử hồi quy để trường hợp generator rỗng có hành vi được xác định và vượt qua kiểm thử.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
php
Lĩnh vực
backend
Loại issue
Lỗi
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Sôi nổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
55/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.