Godfather-Bots/autoresponder
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Небольшые косметические улучшения

Browse Source 
* Немного изменил логику работы метода, который вставляет параметры в текст ответа
This commit is contained in:
Mark Struchkov 2019-01-14 15:27:56 +03:00
parent 386b2265c6
commit 687954f799
2 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/main/java/org/sadtech/autoresponder/Autoresponder.java
View File

@ -17,7 +17,7 @@ public class Autoresponder {
private PersonService personService; private PersonService personService;
public String answer(@NotNull Integer idPerson, @NotNull String message) { public String answer(@NotNull Integer idPerson, @NotNull String message) {
Person person = addPerson(idPerson); Person person = CheckAndAddPerson(idPerson);
Unit unit; Unit unit;
if (person.getUnit() == null) { if (person.getUnit() == null) {
unit = unitService.nextUnit(unitService.menuUnit(), message); unit = unitService.nextUnit(unitService.menuUnit(), message);
@ -29,15 +29,14 @@ public class Autoresponder {
} }
public String answer(@NotNull Integer idPerson, @NotNull String message, @NotNull List<String> words) { public String answer(@NotNull Integer idPerson, @NotNull String message, @NotNull List<String> words) {
Person person = addPerson(idPerson); String answer = answer(idPerson, message);
Unit unit = unitService.nextUnit(person.getUnit(), message);
InsertWords insertWords = new InsertWords(); InsertWords insertWords = new InsertWords();
insertWords.setInText(unit.getAnswer()); insertWords.setInText(answer);
insertWords.insert(words); insertWords.insert(words);
return insertWords.getOutText(); return insertWords.getOutText();
} }
private Person addPerson(Integer idPerson) { private Person CheckAndAddPerson(Integer idPerson) {
Person person; Person person;
if (personService.checkPerson(idPerson)) { if (personService.checkPerson(idPerson)) {
person = personService.getPersonById(idPerson); person = personService.getPersonById(idPerson);

2
src/test/java/org/sadtech/autoresponder/AutoresponderTest.java
View File

@ -85,6 +85,7 @@ public class AutoresponderTest {
@Test @Test
public void NoAnswer() { public void NoAnswer() {
person.setUnit(null);
String test = autoresponder.answer(person.getId(), "Привет это срабатывания"); String test = autoresponder.answer(person.getId(), "Привет это срабатывания");
Assert.assertNull(test); Assert.assertNull(test);
} }
@ -125,6 +126,5 @@ public class AutoresponderTest {
public void answerNoPerson() { public void answerNoPerson() {
Assert.assertEquals(autoresponder.answer(100, "Привет это тест срабатывания"), "Здравствуйте, вы написали в нашу компанию!"); Assert.assertEquals(autoresponder.answer(100, "Привет это тест срабатывания"), "Здравствуйте, вы написали в нашу компанию!");
Assert.assertEquals(autoresponder.answer(100, "Привет это тест срабатывания"), "Ответ с {0} параметрами!"); Assert.assertEquals(autoresponder.answer(100, "Привет это тест срабатывания"), "Ответ с {0} параметрами!");
} }
} }