Stop Using Pseudo-Types

(f2r.github.io)

24 points | by speckx 6 days ago ago

8 comments

  • pwdisswordfishy a day ago ago

    Why? The article fails to explain.

    > The introduction of first-class callables simplifies callback handling. You no longer have an excuse to define your callbacks like this:

    > <?php

    > $data = array_map(trim(...), [' x', 'z ']);

    “First-class callables” is that exact syntax.

    • donatj a day ago ago

      Exactly. The article kind of circles around the idea that it's bad, but never really lands on why.

      My experience is with modern annotations, callable and iterable are pretty powerful, convenient, and safe.

      - https://phpstan.org/writing-php-code/phpdoc-types#callables

      - https://phpstan.org/writing-php-code/phpdoc-types#iterables

    • joe_hoyle a day ago ago

      The problem is that first class callables actually makes a new object wrapper for each reference, so trim(…) != trim(…). (It can be true in some cases, it depends if the memory is freed for the first reference).

      • pwdisswordfishy a day ago ago

        And how is Closure::fromCallable('trim') recommended by the article any better in that respect?

            $ php8.4 -r "var_dump(Closure::fromCallable('trim') === Closure::fromCallable('trim'));"
            bool(false)
        • joe_hoyle a day ago ago

          It is not better, this is a problem with the first-class callable syntax altogether.

  • nilslindemann a day ago ago

    Stop using languages where strings are callable :-p

    • pwdisswordfishy a day ago ago

      I think the point of introducing the (...) syntax that for some reason this article apparently denigrates (if I even understand it correctly) is to eventually deprecate that.

  • boycothn 2 days ago ago

    [dead]