Fix bugs with changing teachers and subscription

litolax · Oct 23, 2023 · a814e39 · a814e39
1 parent 03a3745
commit a814e39
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/TeachersTimetable/Services/AccountService.cs b/TeachersTimetable/Services/AccountService.cs
@@ -57,8 +57,9 @@ public async Task<bool> ChangeTeacher(User telegramUser, string? teacherName)
                 teacherNames[i] = teacherNames[i].Trim();
             }
 
-            var correctTeacherNames = this._parseService.Teachers.Where(g => teacherNames.Any(teacher =>
-                g.ToLower().Trim().Contains(teacher.ToLower().Trim()))).ToArray();
+            var correctTeacherNames = teacherNames
+                .Select(t => this._parseService.Teachers.FirstOrDefault(teacher =>
+                    teacher.StartsWith(t, StringComparison.InvariantCultureIgnoreCase))).Where(correctTeacher => correctTeacher is not null).ToArray();
 
             if (correctTeacherNames is null || correctTeacherNames.Length == 0)
             {
@@ -131,8 +132,10 @@ public async Task UpdateNotificationsStatus(User telegramUser)
 
             this._botService.SendMessage(new SendMessageArgs(telegramUser.Id,
                 user.Notifications
-                    ? $"Вы успешно подписались на расписание преподавателя {user.Teachers}"
-                    : $"Вы успешно отменили подписку на расписание преподавателя {user.Teachers}")
+                    ? user.Teachers.Length == 1
+                        ? $"Вы успешно подписались на расписание преподавателя {user.Teachers[0]}"
+                        : $"Вы успешно подписались на расписание преподавателей: {Utils.GetTeachersString(user.Teachers)}"
+                    : "Вы успешно отменили подписку")
             {
                 ReplyMarkup = keyboard
             });

diff --git a/TeachersTimetable/Services/CommandsService.cs b/TeachersTimetable/Services/CommandsService.cs
@@ -110,7 +110,7 @@ private async void OnMessageReceive(Message message)
                 case "Сменить преподавателя":
                 {
                     this._botService.SendMessage(new SendMessageArgs(sender.Id,
-                        $"Для оформления подписки на преподавателей отправьте их фамилии.(Максимум - 5. Пример: Кулецкая, Шавейко, Потоцкий, Левонюк, Протасеня)\nВаши выбранные преподаватели: ```{string.Join(", ", (await this._accountService.GetUserById(sender.Id))?.Teachers ?? Array.Empty<string>())}```")
+                        $"Для оформления подписки на преподавателей отправьте их фамилии.(Максимум - 5. Пример: Кулецкая, Шавейко, Потоцкий, Левонюк, Протасеня)\nВаши выбранные преподаватели: ```\n{Utils.GetTeachersString((await this._accountService.GetUserById(sender.Id))?.Teachers ?? Array.Empty<string>())}\n```")
                     {
                         ParseMode = ParseMode.Markdown
                     });
@@ -165,4 +165,4 @@ private async void OnMessageReceive(Message message)
             }
         }
     }
-}
+}
diff --git a/TeachersTimetable/Services/Utils.cs b/TeachersTimetable/Services/Utils.cs
@@ -101,5 +101,5 @@ public static bool IsDateBelongsToInterval(DateTime? date, DateTime?[]? interval
         return null;
     }
 
-    public static string GetTeachersString(string[] teachers) => string.Join(", ", teachers);
+    public static string GetTeachersString(string?[] teachers) => string.Join(", ", teachers);
 }