Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:90383
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 74.125.82.52 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <568FFE13.2010304@gmail.com>
References: <CADZXHp6UA4iH6196vN_U+rLSu1Tw+EazQod2LVgwJ7kkUCTd7Q@mail.gmail.com>
	<CA+kxMuQkV=OP2Wt9V6DnebMoBgy6aUf3NmZTDPo_yXXAG5N=Yw@mail.gmail.com>
	<568FDA4B.60005@gmail.com>
	<CA+kxMuRc=ZZuZqY3hrR=14hBrLfSNJ8E8hUbOvp6xe-eyzGU5Q@mail.gmail.com>
	<568FFE13.2010304@gmail.com>
Date: Fri, 8 Jan 2016 11:29:19 -0700
Message-ID: <CAObuZdsrQ-AVmFZBOe5QJUAAkg3CfSW22ZqBBsmF6WYd5xRVhQ@mail.gmail.com>
To: Stanislav Malyshev <smalyshev@gmail.com>
Cc: Dan Ackroyd <danack@basereality.com>, Rowan Collins <rowan.collins@gmail.com>, 
	"internals@lists.php.net" <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=089e014935f4f9c0cb0528d6c3dd
Subject: Re: [PHP-DEV] Deprecation of the Error Control Operator (@ symbol)
From: derokorian@gmail.com (Ryan Pallas)

--089e014935f4f9c0cb0528d6c3dd
Content-Type: text/plain; charset=UTF-8

On Fri, Jan 8, 2016 at 11:21 AM, Stanislav Malyshev <smalyshev@gmail.com>
wrote:

> Hi!
>
> > $counts[$item] ?? $counts[$item] = 1 ?? $counts[$item]++;
>
> This looks completely unreadable and I do not think it works:
> https://3v4l.org/PlAHH
> Seems to leave $counts as 1 always. But even if it didn't, I would never
> pass that on code review - it is very hard to understand what's going on
> and it is incredibly easy to break it. Doing such stuff is definitely
> not the solution, if you hate @, then if() is way better, at least it's
> clear.
>
> > $item = new StdClass;
> > @$counts[$item] = "This is a bad use of error suppression.";
> >
> > There is no insertion, because arrays don't support object keys. The
> > operation completly failed and so there should be an exception.
>
> No, there should not. If somebody tries to count garbage, I just don't
> want to count it. I don't care about it, it's garbage.


I agree with Dan on this one. Trying to use an invalid type as a key should
produce something. Your argument that its garbage and you don't care may be
valid for a specific use case, but it may also be a bug in another case.
IE: it wasn't supposed to be an object, and the wrong variable was used,
but tracking down that error if nothing is produced when doing something
defined as invalid will be hard. If you don't want to count garbage, then
you should use if()

if (!is_garbage($key)) @$count[$key]++;


>
> > Currently that code fails silently.
>
> And that is exactly what I want it to do. Sometimes silent failure is
> *good*, I don't want to know about every little thing that could go
> wrong, I just want what's right to be done.
> Of course, your intent may be the opposite - but that's exactly the
> point, there's more than one use case.
> --
> Stas Malyshev
> smalyshev@gmail.com
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--089e014935f4f9c0cb0528d6c3dd--