[tor-bugs] #24419 [Metrics/Onionoo]: Improve getter names for boolean fields

Tor Bug Tracker & Wiki blackhole at torproject.org
Sat Oct 13 12:56:35 UTC 2018


#24419: Improve getter names for boolean fields
-----------------------------+------------------------------
 Reporter:  karsten          |          Owner:  metrics-team
     Type:  enhancement      |         Status:  new
 Priority:  Medium           |      Milestone:
Component:  Metrics/Onionoo  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:                   |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:                   |        Sponsor:
-----------------------------+------------------------------

Comment (by efgyirfe784):

 Please see https://gitlab.com/743zpnpGUq27GgR/onionoo/tree/24419-isXY for
 a proposed solution to this ticket.

 Please note:

 1. I did not change
 src/test/org/torproject/metrics/onionoo/updater/DummyStatusEntry's
 "getUnmeasured()" to "isUnmeasured()" because the interface
 org.torproject.descriptor.NetworkStatusEntry requires it, and I figured
 org.torproject.descriptor is outside the scope of this ticket. Note
 however that many docs classes had a "getMeasured()" method that is now
 "isMeasured()", which differs slightly from NetworkStatusEntry's
 "getUnmeasured()".

 2. Several classes in org.torproject.metrics.onionoo.docs (like
 SummaryDocument) have a field called "isRelay" which has a getter method
 named "isRelay()". I think this makes sense because of what the field
 represents, but some folks prefer not to have field names repeated as
 method names. I didn't change this, just an FYI.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/24419#comment:1>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list