Implement relevant native PHP interfaces about knapsack HOT 4 CLOSED

Good idea. But i decided not to rename it for now, since when you are using it as a function, you don't have to provide full namespace because count() would conflict with the core count() function. We'll see if people complain about it.

Also, implementing the Countable will probably never happen as it doesn't make sense when we already have the method and we don't want to encourage users to call count().

from knapsack.

Further thinking and tinkering:

• Can't implement ArrayAccess as it implies mutability
• Should not implement Countable as users should use the provided method instead
• size() should probably be renamed to count() in next major release as it's more familiar

from knapsack.

size() should probably be renamed to count() in next major release as it's more familiar

You could deprecate it, create count and make size an alias of it.

from knapsack.

No problem with size, just wanted to provide a solution for the BC issue.

from knapsack.