Skip to content

Commit

Permalink
IOS-3375 Remove redundant formatters copies
Browse files Browse the repository at this point in the history
  • Loading branch information
joe-pusya committed Oct 16, 2024
1 parent e102a36 commit 43d2ce8
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 41 deletions.
10 changes: 10 additions & 0 deletions Anytype/Sources/Helpers/Formatters/NumberFormatter+Extension.swift
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
import Foundation

extension NumberFormatter {
static let `default`: NumberFormatter = {
let formatter = NumberFormatter()
formatter.numberStyle = .decimal
formatter.groupingSeparator = ""
return formatter
}()
}
15 changes: 2 additions & 13 deletions Anytype/Sources/Models/Relations/RelationsBuilder.swift
Original file line number Diff line number Diff line change
Expand Up @@ -18,20 +18,9 @@ final class RelationsBuilder: RelationsBuilderProtocol {

// MARK: - Private variables

private let dateFormatter: DateFormatter = {
let dateFormatter = DateFormatter()
dateFormatter.dateStyle = .medium
dateFormatter.timeStyle = .none
dateFormatter.doesRelativeDateFormatting = true
return dateFormatter
}()
private let dateFormatter = DateFormatter.relationDateFormatter

private let numberFormatter: NumberFormatter = {
let formatter = NumberFormatter()
formatter.numberStyle = .decimal
formatter.groupingSeparator = ""
return formatter
}()
private let numberFormatter = NumberFormatter.default

// MARK: - Internal functions

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,7 @@ final class TextRelationEditingService: TextRelationEditingServiceProtocol {
@Injected(\.relationsService)
private var service: any RelationsServiceProtocol

private let numberFormatter: NumberFormatter = {
let formatter = NumberFormatter()
formatter.numberStyle = .decimal
formatter.groupingSeparator = ""
return formatter
}()
private let numberFormatter = NumberFormatter.default

// MARK: - TextRelationDetailsServiceProtocol

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,8 @@ import UIKit
final class RelationsSectionBuilder {

// MARK: - Private variables

private let dateFormatter: DateFormatter = {
let dateFormatter = DateFormatter()
dateFormatter.dateStyle = .medium
dateFormatter.timeStyle = .none
dateFormatter.doesRelativeDateFormatting = true
return dateFormatter
}()

private let dateFormatter = DateFormatter.relationDateFormatter

// MARK: - Internal functions

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,7 @@ final class RelationFilterBuilder {
return dateFormatter
}()

private let numberFormatter: NumberFormatter = {
let formatter = NumberFormatter()
formatter.numberStyle = .decimal
formatter.groupingSeparator = ""
return formatter
}()
private let numberFormatter = NumberFormatter.default

func relation(
detailsStorage: ObjectDetailsStorage,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,6 @@ final class SetFiltersTextViewModel: ObservableObject {

private var keyboardListenerHelper: KeyboardEventsListnerHelper?

private static let numberFormatter: NumberFormatter = {
let formatter = NumberFormatter()
formatter.numberStyle = .decimal
formatter.groupingSeparator = ""
return formatter
}()

init(filter: SetFilter, onApplyText: @escaping (String) -> Void) {
self.input = Self.initialValue(from: filter)
self.keyboardType = Self.keybordType(for: filter)
Expand All @@ -40,7 +33,7 @@ final class SetFiltersTextViewModel: ObservableObject {
switch filter.conditionType {
case .number:
if let doubleValue = filter.filter.value.safeDoubleValue {
return numberFormatter.string(from: NSNumber(floatLiteral: doubleValue)) ?? ""
return NumberFormatter.default.string(from: NSNumber(floatLiteral: doubleValue)) ?? ""
} else {
return ""
}
Expand Down

0 comments on commit 43d2ce8

Please sign in to comment.