1

Имеется текстовый файл, в каждой строке которого описано событие. Описание состоит из: даты, некоторого текста и страны. Мы парсим этот файл, так чтобы каждая строка преобразовывалась в такой класс.

public class Holidays implements Comparable<Holidays> {
    private Date date;
    private String name;
    private String country;

    public Holidays(Date date, String name, String country) {
        this.date = date;
        this.name = name;
        this.country = country;
    }

Потом объекты Holidays мы пихаем в ThreeMap.

date и name всегда уникальны, country часто повторяется.

Вопрос: как сделать так,чтобы при повторении страны не создавался новый объект String country для каждого Holidays, а использовался уже имеющийся(если такой уже есть).

4
  • 2
    Выглядит как экономия на спичках.
    – Nofate
    22 сен 2017 в 17:42
  • Пример естественно абстрагирован.
    – YourFrend
    22 сен 2017 в 18:05
  • Это плохо, так как может привести к проблеме XY
    – Nofate
    22 сен 2017 в 18:24
  • Реализовал как в вашем ответе, все отлично. На будущее учту.
    – YourFrend
    22 сен 2017 в 20:40

2 ответа 2

3

Выглядит как экономия на спичках, но, если вы уверены, что это вам необходимо (отпрофилировали приложение, убедились, что это - узкое место) вы можете сделать свой кэш строк:

public class StringPool {
    private final Map <String, String> map = new ConcurrentHashMap<>();

    public String intern(String s) {
        String exist = map.putIfAbsent (s, s);
        return (exist == null) ? s : exist;
    }
}

countriesPool = new StringPool();

и прогонять все считанные имена стран через него:

country = countriesPoll.intern(country);
// ... 
Holidays h = new Holidays(date, name, country);

PS. Сразу выкиньте морально устаревший класс Date и воспользуйтесь классами из пакета java.time (для начала посмотрите LocalDateTime).

3
  • А не проще ли использовать джавовский пул строк? 22 сен 2017 в 18:02
  • 1
    Проще, но не лучше ) В докладе Алексея Шипилёва по ссылке из ответа @SergeyGornostaev рассказано, почему.
    – Nofate
    22 сен 2017 в 18:03
  • Разобрался, спасибо.
    – YourFrend
    22 сен 2017 в 18:09
2

Можно написать свой интернер. Вариант из доклада "Катехизис java.lang.String" Шипилёва:

public class CHMInterner<T> {
    private final Map<T, T> map;

    public CHMInterner() {
        map = new ConcurentHashMap<>();
    }

    public T intern(T t) {
        T exist = map.putIfAbsent(t, t);
        return (exist == null) ? t : exist;
    }
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.