Написал код к задаче, где нужно написать функцию, которая будет искать совпадения с заданным словарем, флаг функции letter_case
отвечает за то, нужно ли сравнивать регистр символов или Python
и python
это одно и то же слово.
Но мне вообще не нравится, что для поднятого флага я продублировал такой же кусок кода, уверен что можно добавить логику, чтобы это вместить компактно, но я не могу до неё додуматься...
И хотел бы по возможности услышать замечания о коде: как я мог его упростить или выбрать легче решение, соответствует ли мой код общепринятым правилам названия переменных и PEP-8
!
articles_dict = [
{
"title": "Endless ocean waters.",
"author": "Jhon Stark",
"year": 2019,
},
{
"title": "Oceans of other planets are full of silver",
"author": "Artur Clark",
"year": 2020,
},
{
"title": "An ocean that cannot be crossed.",
"author": "Silver Name",
"year": 2021,
},
{
"title": "The ocean that you love.",
"author": "Golden Gun",
"year": 2021,
},
]
def find_articles(text_filter, letter_case=False):
new_d = []
if letter_case:
for d in articles_dict:
for key_d, val in d.items():
if str(val).find(text_filter) != -1:
new_d.append(d)
return new_d
else:
text_filter = text_filter.lower()
for d in articles_dict:
for key_d, val in d.items():
if str(val).lower().find(text_filter) != -1:
new_d.append(d)
return new_d