api: add search filter by class schedule and department #234
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -31,6 +31,7 @@ def create_class(teachers: list, classroom: str, schedule: str, | |
| return Class.objects.create(teachers=teachers, classroom=classroom, schedule=schedule, | ||
| days=days, _class=_class, special_dates=special_dates, discipline=discipline) | ||
|
|
||
|
|
||
| @handle_cache_before_delete | ||
| def delete_classes_from_discipline(discipline: Discipline) -> QuerySet: | ||
| """Deleta todas as turmas de uma disciplina.""" | ||
|
|
@@ -70,11 +71,21 @@ def filter_disciplines_by_teacher(name: str) -> QuerySet: | |
| return search_disciplines | ||
|
|
||
|
|
||
| def filter_disciplines_by_schedule_and_department_code(schedule: str, department_code: str) -> QuerySet: | ||
| """Filtra as disciplinas pelo horário.""" | ||
| return Discipline.objects.filter(Q(classes__schedule__icontains=schedule) & Q(department__code=department_code)).distinct("id") | ||
|
Comment on lines
+74
to
+76
There was a problem hiding this comment. Não entendi muito bem o uso do There was a problem hiding this comment. Olá, Caio! A função Algo parecido é feito na busca por professor, uma vez que encontramos disciplinas com turmas com o professor especificado, nós filtramos, dentre as turmas daquela disciplina, somente as turmas daquele professor. Essa filtragem de turmas é feita dentro serializer após obter os objetos disciplines em questão. class DisciplineSerializer(DisciplineSerializerSchedule):
classes = serializers.SerializerMethodField()
def get_classes(self, discipline: Discipline):
teacher_name = self.context.get('teacher_name')
schedule = self.context.get('schedule')
classes = discipline.classes.all() if hasattr(
discipline, 'classes') else Class.objects.none()
if teacher_name:
classes = dbh.filter_classes_by_teacher(
name=teacher_name, classes=classes)
if schedule:
classes = dbh.filter_classes_by_schedule(
schedule=schedule, classes=classes)
return ClassSerializer(classes, many=True).dataNão conheço uma maneira de filtrar os objetos O que acha? There was a problem hiding this comment.
Entendi. Se já temos um sistema dessa forma, vamos manter então (pelo menos por enquanto).
Por enquanto, vamos manter do jeito em que está mesmo. Para utilizar uma query em SQL, temos que verificar as opções que o framework entrega pra gente (se é anti SQL Injection e tudo mais). Para essa thread, só basta a resolução do problema reportado que é o mesmo do |
||
|
|
||
|
|
||
| def filter_disciplines_by_code(code: str, disciplines: Discipline = Discipline.objects) -> QuerySet: | ||
| """Filtra as disciplinas pelo código.""" | ||
| return disciplines.filter(code__icontains=code) | ||
|
|
||
|
|
||
| def filter_classes_by_schedule(schedule: str, classes: BaseManager[Class] = Class.objects) -> QuerySet: | ||
| """Filtra as turmas pelo horário.""" | ||
| return classes.filter(schedule__icontains=schedule) | ||
|
Comment on lines
+84
to
+86
There was a problem hiding this comment. A proposta por realizar um filtro através do horário é boa, mas acho que não é eficiente utilizando essa função. Pelo o que eu entendi, ela vai encontrar padrões que contém trechos diretos. Por exemplo: Imagine que temos uma lista de matérias distintas, com os seguintes horários: schedules = ['24M34', '46M34']Se eu realizo uma query para receber matérias que possuem o horário Essa pesquisa é funcional, mas se limita a uma EDIT: A função icontains é equivalente: SELECT classes WHERE schedule ILIKE '%4M34%';Se a query fosse desse tipo (abaixo), o problema seria resolvido: SELECT classes WHERE schedule ILIKE '%4%M%3%4%';Ou seja: Com a query There was a problem hiding this comment. Excelente observação, Caio! Concordo com as sugestões e irei modificar o código para implementá-las. |
||
|
|
||
|
|
||
| def filter_disciplines_by_year_and_period(year: str, period: str, disciplines: Discipline = Discipline.objects) -> QuerySet: | ||
| """Filtra as disciplinas pelo ano e período.""" | ||
| return disciplines.filter(department__year=year, department__period=period) | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eu acredito que nesse caso, as mensagens de erro devem ser mais descritivas.
Ao realizar uma pesquisa, tive certa dificuldade em saber o que estava fazendo de errado.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Olá, modificarei as mensagens para torná-las mais descritivas.