search

eshcherbin / save-the-moment

SPbAU educational Android project
0 stars 0 forks source link

Implemented filtering by tags #16

Closed eshcherbin closed 7 years ago

eshcherbin commented 7 years ago

В список моментов добавлена фильтрация по тегам.

eshcherbin commented 7 years ago

Кажется, группы в моём регулярном выражении работают не так, как хотелось бы. Буду исправлять.

eshcherbin commented 7 years ago

И всё же, текущая реализация вроде бы делает всё, как надо: сплитит по разделителю и запускает Loader с полученными тегами. Проверка полученных тегов на валидность производится, чтобы лишний раз не обращаться к БД.

krinkinmu commented 7 years ago

Какое отношение проверка на валидность имеет к обращениям к БД? Тег - это просто строка, без разделителей, я вообще не понимаю, какую валидность вы проверяете?

eshcherbin commented 7 years ago

Мы не хотим, чтобы тегом была произвольная строка, а только строка, состоящая из латиницы, кириллицы, цифр и символов # и -. Запускать Loader каждый раз при изменении текста может быть неэффективно, поэтому перед этим я проверяю, что все теги удовлетворяют вышеперечисленным требованиям.

krinkinmu commented 7 years ago
  1. И я уже спрашивал, откуда появилось такое странное ограничение
  2. Вы уже упоминали, что добавили кирллицу
  3. Вы для каждого поддерживаемого языка собираетесь менять регулярное выражение?
eshcherbin commented 7 years ago

Хорошо, никаких ограничений на теги.