Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:70418 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 76381 invoked from network); 26 Nov 2013 17:57:31 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 26 Nov 2013 17:57:31 -0000 Authentication-Results: pb1.pair.com header.from=pjsturgeon@gmail.com; sender-id=pass Authentication-Results: pb1.pair.com smtp.mail=pjsturgeon@gmail.com; spf=pass; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.215.51 as permitted sender) X-PHP-List-Original-Sender: pjsturgeon@gmail.com X-Host-Fingerprint: 209.85.215.51 mail-la0-f51.google.com Received: from [209.85.215.51] ([209.85.215.51:64037] helo=mail-la0-f51.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id D7/A7-39355-A01E4925 for ; Tue, 26 Nov 2013 12:57:30 -0500 Received: by mail-la0-f51.google.com with SMTP id ec20so4525549lab.38 for ; Tue, 26 Nov 2013 09:57:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=MbTA7XlhxLJQrLJbkZjXPYxGVP91YlKB0yyzgvSoXtw=; b=WbfLc86mjVozAvaF8IsRmPONPrVnTwIN6R1r0swThAl2l/IOHYEsODHAWyZjf8l2pO MLm3re9AJ/JvHTf2Gj7euUmgGMF3sdG+z6sx0bQoJ1A3+pxS5QKHI2/0cM0qeaOcP9Ql Dz5UaeQOfGZn6tyfK6esac7JMgD7R5BSxJJpCzVfPKUuDwwhHIyxhUPbLtS51Q+jx6vt YYP/OLnydHbglcwY+lfQj31HRN2gxBFssHfYf9r/B/1MTgBYdNlEv3Tf39ReVOFsrpk+ N7p0AtQZgkH62FvEO5nsaYS8pAexqTKNvWyNG3kQFBfu7BlsyCKy/Zb+PqT6dr31d23R 3DSQ== MIME-Version: 1.0 X-Received: by 10.112.205.6 with SMTP id lc6mr5070291lbc.45.1385488647093; Tue, 26 Nov 2013 09:57:27 -0800 (PST) Received: by 10.114.217.231 with HTTP; Tue, 26 Nov 2013 09:57:27 -0800 (PST) In-Reply-To: References: Date: Tue, 26 Nov 2013 12:57:27 -0500 Message-ID: To: Paul Dragoonis Cc: Chris London , Mats Lindh , "internals@lists.php.net" Content-Type: text/plain; charset=ISO-8859-1 Subject: Re: [PHP-DEV] [Proposal] Modification to ?: functionality From: pjsturgeon@gmail.com (Philip Sturgeon) On Tue, Nov 26, 2013 at 12:41 PM, Paul Dragoonis wrote: > On Tue, Nov 26, 2013 at 4:58 PM, Chris London wrote: > >> On Tue, Nov 26, 2013 at 8:15 AM, Mats Lindh wrote: >> >> > On Tue, Nov 26, 2013 at 3:43 PM, Chris London wrote: >> > >> >> I believe these two statements are functionally equivalent: >> >> >> >> $foo = $foo ? $foo : 'default'; >> >> >> >> $foo = $foo ?: 'default'; >> >> >> > >> > They are. >> > >> > >> > I would like to change it so it also checks for isset() so I propose the >> >> following would be functionally equivalent: >> >> >> >> $foo = isset($foo) && $foo ? $foo : 'default'; >> >> >> >> $foo = $foo ?: 'default'; >> >> >> > > We can't change the behaviour of ?: that ship has already sailed, we'll > just be hurting existing codebases on upgrading and more user education is > required to understand non-visual changes in syntax like this, lets try to > learn from our mistakes by not changing 'behaviour' anymore. > > An alternative syntax that is similar and not ugly would be good since the > ?: behaviour was broken from the start since you still need to run isset() > before running ?: which was the problem we tried to solve in the first > place but it just didn't happen. > > I don't like the @ symbol it's too different from what's already there on > the ternary logic. > > Proposal: $foo = $arr['value'] ?:: ""; > > Thoughts? > >> >> > The would break the assumption that a reference to an uninitialized value >> > would generate a notice, unless explicitly handled in the logic. >> > >> > While I also would like to have something similar to ?: to handle default >> > values for array keys, etc., this would change a fundamental assumption >> > that as been in place for many years now. I'm not sure if that's a BC >> break >> > that would be acceptable this late. An alternative operator may be more >> > suitable. >> > >> >> What do you think about using: >> >> $foo = $foo @: 'default'; >> >> and possibly >> >> $foo @= 'default'; >> >> >> >> > >> > The change proposed has also been discussed several times since the >> > implementation of ?:. See the ifsetor-RFC: >> > >> > https://wiki.php.net/rfc/ifsetor >> > >> > It also contains links to the discussion around the feature back then. >> > >> > --mats >> > >> This seems like a cracking time to suggest: $x ||= $y; The difference? Anything left of the ? SHOULD fail if it is not set, because you are using it in a condition which is dangerous and odd for other types of condition, but adding in this new syntax (similar to Ruby) would essentially evaluate to this: empty($x) ? $y : $x; This gives keeps the behavior of ternary operators working in the same way, but adds what we want: the ability to nicely set defaults for variables and array keys that may not be set.