`array_push($arr, ...$map)` should support non-numeric keys
Aperta
Nessuno ha ancora preso questa issue.
Category: Arrays
Extension: standard
Feature
Status: Needs Triage
- Lingua principale
- C
- Stelle
- 40.4k
- Fork
- 8.1k
- Merge medio
- 2g 13h
- PR unite (30g)
- 96
Descrizione
Description
The following code:
<?php
function array_push_cust(&$arr, ...$args) {
foreach ($args as $k => $v) {
if (is_int($k)) {
$arr[] = $v;
} else {
$arr[$k] = $v;
}
}
}
$arr = [1, 'a' => 2];
$arr2 = [3, 'b' => 4];
$res = $arr;
array_push_cust($res, ...$arr2);
print_r($res);
$res = $arr;
array_push($res, ...$arr2);
print_r($res);
Resulted in this output:
Array
(
[0] => 1
[a] => 2
[1] => 3
[b] => 4
)
Fatal error: Uncaught ArgumentCountError: array_push() does not accept unknown named parameters in /in/0p5Sv:21
Stack trace:
#0 /in/0p5Sv(21): array_push(Array, 3, b: 4)
#1 {main}
thrown in /in/0p5Sv on line 21
But I expected this output instead:
Array
(
[0] => 1
[a] => 2
[1] => 3
[b] => 4
)
Array
(
[0] => 1
[a] => 2
[1] => 3
[b] => 4
)
PHP Version
PHP 8.0 and higher
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con la riproduzione collegata di 3v4l e conferma il comportamento di PHP 8.0+ per gli array decompressi con chiavi stringa. Traccia il punto di ingresso di array_push e la copertura esistente per la gestione degli argomenti; il lavoro è completato quando l’esempio conserva sia le chiavi numeriche sia quelle non numeriche senza l’errore relativo ai parametri denominati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- php
- Ambito
- backend
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 45/100