« Return to Thread: set.empty() method

Re: set.empty() method

by Brendan Eich-2 :: Rate this Message:

| View in Thread

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@...
> <mailto:khs4473@...>> 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@... <mailto:michael@...>> 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@... <mailto:erights@...>> 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@... <mailto:adam@...>> wrote:
>         >>
>         >> I'd agree with using clear() and isEmpty() with their
>         respective actions.
>         >>
>         >> On Wed, Feb 15, 2012 at 00:37, Brendan Eich
>         <brendan@... <mailto:brendan@...>> 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@... <mailto:es-discuss@...>
>         >> >> https://mail.mozilla.org/listinfo/es-discuss
>         >> >>
>         >> > _______________________________________________
>         >> > es-discuss mailing list
>         >> > es-discuss@... <mailto:es-discuss@...>
>         >> > 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@... <mailto:es-discuss@...>
>         > https://mail.mozilla.org/listinfo/es-discuss
>         >
>         _______________________________________________
>         es-discuss mailing list
>         es-discuss@... <mailto:es-discuss@...>
>         https://mail.mozilla.org/listinfo/es-discuss
>
>
>
>     _______________________________________________
>     es-discuss mailing list
>     es-discuss@... <mailto:es-discuss@...>
>     https://mail.mozilla.org/listinfo/es-discuss
>
>
> _______________________________________________
> es-discuss mailing list
> es-discuss@...
> https://mail.mozilla.org/listinfo/es-discuss
_______________________________________________
es-discuss mailing list
es-discuss@...
https://mail.mozilla.org/listinfo/es-discuss

 « Return to Thread: set.empty() method