set.empty() method
Brendan Eich
brendan at mozilla.org
Wed Feb 15 10:18:46 PST 2012
I agree and find the add/remove pairing to be winning compared to
add/delete. It dodges the ES3 issue. I'd go for pithy over verb-noun-y
any day, so clear > deleteAll. Please, y'all, save us from Java-esque
logorrhea!
/be
Dean Landolt wrote:
>
>
> On Wed, Feb 15, 2012 at 8:58 AM, Kevin Smith <khs4473 at gmail.com
> <mailto:khs4473 at gmail.com>> wrote:
>
> Not to backtrack the conversation, but I'm not convinced that
> "delete" should be used to remove an element. Noone's going to
> ask me, "how do I delete an element from the set?". They're going
> to ask me, "how do I remove an element?".
>
>
> That's a good point. Just like noone's going to ask "how do I put an
> element".
>
> add : remove :: put : delete
>
> Also, I'm currently suspicious of any parallels between deleting
> a property and removing an element from a set. The whole point of
> Set and Map is to provide a collection abstraction independent of
> javascript's object model (so as to end the confusion that arises
> from their conflation), correct?
>
> This is a good argument in favor of "clear" as well. You never really
> want to "clear" a typical object, it only really makes sense in the
> context of collections.
>
> Plus, syntax-ignorant tokenizers (like the highlighter on the
> proposal page) are going to make delete look pretty weird : )
>
> khs
>
>
> On Wed, Feb 15, 2012 at 8:07 AM, Michael A. Smith
> <michael at smith-li.com <mailto:michael at smith-li.com>> wrote:
>
> +1 on clear() as it's pithy, understandable, and behaves the
> same in
> Java and Python.
>
> What real benefit comes from drawing this connection between
> delete()
> and deleteAll()? I suspect the everyday programmer won't care,
> and the
> ones who do will look it up.
>
> -Michael A. Smith
>
> On Wed, Feb 15, 2012 at 1:47 AM, Mark S. Miller
> <erights at google.com <mailto:erights at google.com>> wrote:
> > clear() is ok. Also, java.util.Map and java.util.Set use
> clear() so it would
> > also be familiar to many people.
> >
> > Perhaps deleteAll() would be more mnemonic, as its
> relationship with
> > delete() would be obvious?
> >
> >
> > On Tue, Feb 14, 2012 at 10:39 PM, Adam Shannon
> <adam at ashannon.us <mailto:adam at ashannon.us>> wrote:
> >>
> >> I'd agree with using clear() and isEmpty() with their
> respective actions.
> >>
> >> On Wed, Feb 15, 2012 at 00:37, Brendan Eich
> <brendan at mozilla.org <mailto:brendan at mozilla.org>> wrote:
> >> > Good idea, but I suggest an unambiguous verb instead of an
> >> > adjective-or-verb: clear.
> >> >
> >> > "empty" is often used for the predicate, in naming
> conventions that
> >> > eschew
> >> > "isEmpty" and "emptyp" patterns.
> >> >
> >> > /be
> >> >
> >> >
> >> > Peter Michaux wrote:
> >> >>
> >> >> If some piece of code needs to empty a set, it would be
> good to do
> >> >> that in a single call
> >> >>
> >> >> set.empty();
> >> >>
> >> >> Otherwise we might be left doing the following which
> could be very
> >> >> inefficient.
> >> >>
> >> >> set.forEach(function(element) {
> >> >> set['delete'](element);
> >> >> });
> >> >>
> >> >> Peter
> >> >> _______________________________________________
> >> >> es-discuss mailing list
> >> >> es-discuss at mozilla.org <mailto:es-discuss at mozilla.org>
> >> >> https://mail.mozilla.org/listinfo/es-discuss
> >> >>
> >> > _______________________________________________
> >> > es-discuss mailing list
> >> > es-discuss at mozilla.org <mailto:es-discuss at mozilla.org>
> >> > https://mail.mozilla.org/listinfo/es-discuss
> >>
> >>
> >>
> >> --
> >> Adam Shannon
> >> Developer
> >> University of Northern Iowa
> >> Sophomore -- Computer Science B.S. & Mathematics
> >> http://ashannon.us
> >
> >
> >
> >
> > --
> > Cheers,
> > --MarkM
> >
> > _______________________________________________
> > es-discuss mailing list
> > es-discuss at mozilla.org <mailto:es-discuss at mozilla.org>
> > https://mail.mozilla.org/listinfo/es-discuss
> >
> _______________________________________________
> es-discuss mailing list
> es-discuss at mozilla.org <mailto:es-discuss at mozilla.org>
> https://mail.mozilla.org/listinfo/es-discuss
>
>
>
> _______________________________________________
> es-discuss mailing list
> es-discuss at mozilla.org <mailto:es-discuss at mozilla.org>
> https://mail.mozilla.org/listinfo/es-discuss
>
>
> _______________________________________________
> es-discuss mailing list
> es-discuss at mozilla.org
> https://mail.mozilla.org/listinfo/es-discuss
More information about the es-discuss
mailing list