set.empty() method

Adam Shannon adam at ashannon.us
Tue Feb 14 22:54:15 PST 2012


deleteAll() seems confusing because (even though sets aren't designed
like this) it seems like deleteAll(key) would delete all entries that
are mapped from key.

On Wed, Feb 15, 2012 at 00:47, Mark S. Miller <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> 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> 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
>> >> https://mail.mozilla.org/listinfo/es-discuss
>> >>
>> > _______________________________________________
>> > es-discuss mailing list
>> > 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



-- 
Adam Shannon
Developer
University of Northern Iowa
Sophomore -- Computer Science B.S. & Mathematics
http://ashannon.us


More information about the es-discuss mailing list