Re: Missing __toString() part

Hello Rasmus, though you are probably right here your idea brought to it's full extend will bring a major BC break. Just consider we had both key and value as a zval then the current behavior would change for quite a lot of situations including not only objects and resources but also booleans, floats and NULL. Right now everybody is used to the restiction that an array key can only be a string or an integer and if not it is either not acceptable as a key at all or is being converted to either one. best regards marcus Saturday, June 3, 2006, 6:30:38 PM, you wrote: > I don't understand why using the object as an index would trigger a > __toString() call. PHP's array indices are not defined to be strings, > so I don't see this as being a string context use and thus __toString() > shouldn't be called. > I also don't see why we need to make a distinction between the key and > the value here if there is any way we can avoid it. > For example, this obviously works: > class foo { > function a() { echo "a"; } > } > $a = new foo(); > $b[1] = $a; > $b[1]->a(); > Why shouldn't this? > $b[$a] = 1; > key($b)->a(); > I have always looked a PHP array as having 2 places to store things. > You can store stuff in the key with the caveat/feature that it must be > unique and you can store whatever you want in the value part. Only > allowing strings and integers in the key seems like an arbitrary > restriction to me. As long as the key is something we can somehow > uniquely identify I think we should aim to support it. > -Rasmus > Ilia Alshanetsky wrote: >> I think this is a potentially missing bit of functionality we definitely >> need to consider including. There is really no technical reason why >> $foo[new ToStringObject] = "bar"; cannot work or a reason why it >> shouldn't as far as functionality goes. >> >> >> Ilia Alshanetsky >> >> >> >> >> On 3-Jun-06, at 7:42 AM, Marcus Boerger wrote: >> >>> Hello guys, >>> >>> the attached patch closes one more __toString() part. It allows >>> to use objects that define __toString as indexes to arrays. What do >>> you guys think about this, should we add it or stay with the old >>> behavior that didn't allow objects as indexes at all. >>> >>> --Best regards, >>> Marcus >>> mailto:mail@marcus-boerger.de >>> <ze2-offset-obj-tostring-20060603.diff.txt> >>> --PHP Internals - PHP Runtime Development Mailing List >>> To unsubscribe, visit: http://www.php.net/unsub.php >> >> --PHP Internals - PHP Runtime Development Mailing List >> To unsubscribe, visit: http://www.php.net/unsub.php Best regards, Marcus