pagr0m / courses-fogstream- Goto Github PK

https://github.com/PaGr0m/Fogstream/blob/e21b400bccfe730251574dfcd7cf025a37014f99/Practice%203/Files_6.py#L9

После импорта код разделяется двумя пустыми строками

Файл необходимо закрыть сразу, как ты считал с него данные, зачем тащить закрытие сессии через весь код?

https://github.com/PaGr0m/Fogstream/blob/e21b400bccfe730251574dfcd7cf025a37014f99/Practice%203/MyDir.py#L11

две пустые строки после импорта

1. https://github.com/PaGr0m/Fogstream/blob/54904089353bda18ac6e7a27ddd1031266d37ff6/Practice_9/mysite/discography/models.py#L8

Вообще, сортировка на стороне модели не очень хорошая забава. Пример на практике был просто показательным

2. Поля musician, album и genre не совсем корректно отражают суть. Скорее это поле "НАзвание" в соотвестующих моделях

1. https://github.com/PaGr0m/Fogstream/blob/05875ad862facf64064803e7e2b8b345f079f31b/Final%201/Parentheses.py#L10 регулярка быстрее работать будет
2. https://github.com/PaGr0m/Fogstream/blob/05875ad862facf64064803e7e2b8b345f079f31b/Final%201/Parentheses.py#L35
   самое крутое решение -- использовать словарь
3. https://github.com/PaGr0m/Fogstream/blob/05875ad862facf64064803e7e2b8b345f079f31b/Final%201/Parentheses.py#L53
   Словарь был бы круче
4. https://github.com/PaGr0m/Fogstream/blob/05875ad862facf64064803e7e2b8b345f079f31b/Final%201/Parentheses.py#L38
   https://github.com/PaGr0m/Fogstream/blob/05875ad862facf64064803e7e2b8b345f079f31b/Final%201/Parentheses.py#L46

ЗААААЧЕЕЕЕМ?! можно было просто stack.index(stak.pop()) == close_parenthesis.index(symbol)

1. https://github.com/PaGr0m/Fogstream/blob/dd6161478f54a7c8b50c8f2bf05f2537da54cea9/Practice_9/mysite/discography/views.py#L11

А почему нельзя было использовать обращение по связанным полям?

2. https://github.com/PaGr0m/Fogstream/blob/dd6161478f54a7c8b50c8f2bf05f2537da54cea9/Practice_9/mysite/discography/views.py#L28

для чего форма используется?

3. https://github.com/PaGr0m/Fogstream/blob/dd6161478f54a7c8b50c8f2bf05f2537da54cea9/Practice_9/mysite/discography/views.py#L49

Это же поле есть в любом инстансе модели музыкант. ЗАчем отдельно прокидывать?

4. https://github.com/PaGr0m/Fogstream/blob/dd6161478f54a7c8b50c8f2bf05f2537da54cea9/Practice_9/mysite/discography/views.py#L55

Для чего называть шаблон с названием урла?