brick / money

100%

master: 100%

LAST BUILD ON BRANCH 0.4.2

branch: 0.4.2

CHANGE BRANCH

x

Reset

• 0.4.2
• master
• defaultmoney
• step
• context
• adjustment
• rational
• embedded-context
• 0.1.0
• v0.1
• 0.2.0
• 0.2.1
• 0.2.2
• 0.2.3
• 0.1.1
• 0.2.4
• 0.3.0
• 0.3.1
• 0.3.2
• 0.3.3
• 0.3.4
• 0.4.0
• 0.4.1
• 0.4.3
• 0.4.4
• 0.4.5
• 0.5.0
• MoneyFactory
• 0.5.3
• php74
• fix_allocate
• 0.6.0
• 0.7.0
• php80
• php81
• ci
• 0.7.1
• 0.8.0
• 0.8.1
• 0.9.0
• php84
• 0.10.0

Build # 309

Build Type

push

travis-ci

Committed by BenMorel

Commit Message

Cache Money::formatTo()'s NumberFormatter for improved performance

Calling formatTo() many times spends quite some time in NumberFormatter constructor, that performs I/O, according to the profiler.
Because we usually call formatTo() many times in a row for the same locale, we can get a ~2x speedup with this fix.

The best way to tackle this in the future would be to deprecate the formatTo() method and introduce a MoneyFormatter object, that would keep a local cache of all locale/scale pairs.

Run Details

427 of 427 relevant lines covered (100.0%)

125.91 hits per line