Skip to content

Commit

Permalink
feat: #98 Use the overrridden Excel sheet names also from json file i…
Browse files Browse the repository at this point in the history
…nstead of hard-coding in the exporters.
  • Loading branch information
dmitry-weirdo committed Nov 28, 2024
1 parent 0c172a1 commit 65d831d
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 93 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,22 +16,4 @@ public class FrequencyVocabularyTopExporter extends NonStandardVocabularyTopExpo
public int dictionaryId() {
return NonStandardDictionary.FREQUENCY_VOCABULARY.id;
}

@Override
public String topByBestSpeedExcelSheetName() {
// return "Топ по рекорду в «Частотном словаре»"; // 36 chars
return "Топ по рекорду в «Частотном»"; // 28 chars
}

@Override
public String topByRacesCountExcelSheetName() {
// return "Топ по пробегу в «Частотном словаре»"; // 36 chars
return "Топ по пробегу в «Частотном»"; // 28 chars
}

@Override
public String topByHaulExcelSheetName() {
// return "Топ по времени в «Частотном словаре»"; // 36 chars
return "Топ по времени в «Частотном»"; // 28 chars
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -16,22 +16,4 @@ public class MiniMarathonTopExporter extends NonStandardVocabularyTopExporterDef
public int dictionaryId() {
return NonStandardDictionary.MINI_MARATHON.id;
}

@Override
public String topByBestSpeedExcelSheetName() {
// return "Топ по рекорду в «Мини-марафоне, 800 знаков»"; // 44 chars
return "Рекорды в «Мини-марафоне»"; // 25 chars
}

@Override
public String topByRacesCountExcelSheetName() {
// return "Топ по пробегу в «Мини-марафоне, 800 знаков»"; // 44 chars
return "Пробег в «Мини-марафоне»"; // 24 chars
}

@Override
public String topByHaulExcelSheetName() {
// return "Топ по времени в «Мини-марафоне, 800 знаков»"; // 44 chars
return "Время в «Мини-марафоне»"; // 23 chars
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@

import java.util.Objects;

// todo: this object is basically the same as impelemented by NonStandardVocabularyTopExporterGenerator
public interface NonStandardVocabularyTopExporter extends VocabularyTopExporter {

@Override
Expand Down Expand Up @@ -81,7 +82,11 @@ default String topByBestSpeedAdditionalHeader() {
}
@Override
default String topByBestSpeedExcelSheetName() {
return VocabularyTopUtils.topByBestSpeedExcelSheetName(vocabulary());
// same logic as in NonStandardVocabularyTopExporterGenerator
return Objects.requireNonNullElseGet(
vocabularyTopData().topByBestSpeedExcelSheetName,
() -> VocabularyTopUtils.topByBestSpeedExcelSheetName(vocabulary())
);
}

@Override
Expand All @@ -98,7 +103,11 @@ default String topByRacesCountAdditionalHeader() {
}
@Override
default String topByRacesCountExcelSheetName() {
return VocabularyTopUtils.topByRacesCountIn(vocabulary()); // 30 chars :)
// same logic as in NonStandardVocabularyTopExporterGenerator
return Objects.requireNonNullElseGet(
vocabularyTopData().topByRacesCountExcelSheetName,
() -> VocabularyTopUtils.topByRacesCountIn(vocabulary())
);
}

@Override
Expand All @@ -115,7 +124,11 @@ default String topByHaulAdditionalHeader() {
}
@Override
default String topByHaulExcelSheetName() {
return VocabularyTopUtils.topByHaulIn(vocabulary());
// same logic as in NonStandardVocabularyTopExporterGenerator
return Objects.requireNonNullElseGet(
vocabularyTopData().topByHaulExcelSheetName,
() -> VocabularyTopUtils.topByHaulIn(vocabulary())
);
}

// override compared to default to not compare "voc-" 2 times, since vocabularyCode() already contains "voc-"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,22 +16,4 @@ public class NormalInEnglishTopExporter extends NonStandardVocabularyTopExporter
public int dictionaryId() {
return NonStandardDictionary.NORMAL_IN_ENGLISH.id;
}

@Override
public String topByBestSpeedExcelSheetName() {
// return "Топ по рекорду в «Обычном in English»"; // 37 chars
return "Рекорды в «Обычном in English»"; // 30 chars
}

@Override
public String topByRacesCountExcelSheetName() {
// return "Топ по пробегу в «Обычном in English»"; // 37 chars
return "Пробег в «Обычном in English»"; // 29 chars
}

@Override
public String topByHaulExcelSheetName() {
// return "Топ по времени в «Обычном in English»"; // 37 chars
return "Время в «Обычном in English»"; // 28 chars
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -16,22 +16,4 @@ public class ShortTextsTopExporter extends NonStandardVocabularyTopExporterDefau
public int dictionaryId() {
return NonStandardDictionary.SHORT_TEXTS.id;
}

@Override
public String topByBestSpeedExcelSheetName() {
// return "Топ по рекорду в «Коротких текстах»"; // 35 chars
return "Рекорды в «Коротких текстах»"; // 28 chars
}

@Override
public String topByRacesCountExcelSheetName() {
// return "Топ по пробегу в «Коротких текстах»"; // 35 chars
return "Пробег в «Коротких текстах»"; // 27 chars
}

@Override
public String topByHaulExcelSheetName() {
// return "Топ по времени в «Коротких текстах»"; // 35 chars
return "Время в «Коротких текстах»"; // 26 chars
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -16,22 +16,4 @@ public class TrainingIndexFingersTopExporter extends NonStandardVocabularyTopExp
public int dictionaryId() {
return NonStandardDictionary.TRAINING_INDEX_FINGERS.id;
}

@Override
public String topByBestSpeedExcelSheetName() {
// return "Топ по рекорду в «Тренируем указательные»"; // 41 chars
return "Рекорды в «Тренируем указ-е»"; // 28 chars
}

@Override
public String topByRacesCountExcelSheetName() {
// return "Топ по пробегу в «Тренируем указательные»"; // 41 chars
return "Пробег в «Тренируем указ-е»"; // 27 chars
}

@Override
public String topByHaulExcelSheetName() {
// return "Топ по времени в «Тренируем указательные»"; // 41 chars
return "Время в «Тренируем указ-е»"; // 26 chars
}
}

0 comments on commit 65d831d

Please sign in to comment.