logger icon indicating copy to clipboard operation
logger copied to clipboard

The design of the restriction position for "isLoggable" is unreasonable

Open qlang122 opened this issue 1 year ago • 0 comments

The problem of design thinking!

Logger.json() -> LoggerPrinter.json() (json format in here, Time-consuming) -> LoggerPrinter.d() -> LoggerPrinter.log()

Action code:

for (LogAdapter adapter : logAdapters) {
      if (adapter.isLoggable(priority, tag)) {
        adapter.log(priority, tag, message);
      }
}

Why is this design problematic?

Because many users actually do this:

Logger.addLogAdapter(object : AndroidLogAdapter(formatStrategy) {
      override fun isLoggable(priority: Int, tag: String?): Boolean {
            return BuildConfig.DEBUG
      }
})

And that's where the problem lies image

qlang122 avatar Dec 26 '23 00:12 qlang122