в целом норм, разве что упростить вот этот кусок. есть вариант return None сразу, но тогда менее читабельно.
оставь так, проверь по пепу (у тебя смысловые части слиплись)
Тернарник добавь, единственное что.
А так я обычно вот так делаю для читаемости :
If statement :
--code
Blank line
Else:
--code
Blank line
Return