Shalapugin Dmitry by dimamushalapugin · Pull Request #35 · learnpythonru/basic_exercises

dimamushalapugin · 2023-05-17T11:19:32Z

No description provided.

krepysh

Обрати внимание на то как ты называешь переменные и функции. Имена должны быть понятными и объяснять что в переменной лежит или что функция делает.
По списком лучше итерироваться сразу по объектем, а не по индексу по длине.
Не используй i,j,k, etc как имена переменных в циклах, кроме как для индексов.

krepysh · 2023-05-18T14:31:20Z

for_challenges.py

 names = ['Оля', 'Петя', 'Вася', 'Маша']
-# ???
+for i in is_male:
+    if is_male[i] == True:


c True никогда не нужно сравнивать. просто if something:

krepysh · 2023-05-18T14:32:14Z

for_challenges.py

 }
 names = ['Оля', 'Петя', 'Вася', 'Маша']
-# ???
+for i in is_male:


Позапускай код, он кажется не делает то что нужно. Я бы ожидал здесь увидеть другой for.

krepysh · 2023-05-18T14:34:03Z

for_challenges.py

-# ???
+a = 0
+print(f'Всего {len(groups)} группы.')
+for i in groups:


for group in groups:

krepysh · 2023-05-18T14:34:27Z

for_challenges.py

-# ???
+a = 0
+print(f'Всего {len(groups)} группы.')
+for i in groups:


Suggested change

for i in groups:

for group in groups:

krepysh · 2023-05-18T14:35:14Z

for_challenges.py

+a = 0
+print(f'Всего {len(groups)} группы.')
+for i in groups:
+    a += 1


В питоне при итерировании списка принято использовать сразу элементы списка, а не индексы. Не используй индексы без острой необходимости.

krepysh · 2023-05-18T15:01:43Z

for_dict_challenges_bonus.py

    return messages


+def zadacha(messages):


Разбей код на несколько максимально независимых функций, вынеси print наружу.

Назови переменные понятными именами: ab, ac, ac, i, j, k - плохие названия для переменных, потому что не показывают что в них лежит и как будут использоваться.

Код не разбит

krepysh · 2023-05-18T15:03:54Z

string_challenges.py

 # Вывести количество букв "а" в слове
 word = 'Архангельск'
-# ???
+print(len([j for j in ''.join(word) if j.lower() == 'а']))


переусложнено - join тут вообще не нужен.
используй методы lower и count у str

krepysh · 2023-05-18T15:05:14Z

string_challenges.py

 # Вывести количество гласных букв в слове
 word = 'Архангельск'
-# ???
+print(len([j for j in ''.join(word) if j.lower() not in ['а', 'е', 'у', 'ы', 'о', 'э', 'я', 'и', 'ю']]))


вынеси список в перменную vowels
join не нужен

krepysh · 2023-05-18T15:06:28Z

string_challenges.py

 sentence = 'Мы приехали в гости'
-# ???
+for i in ''.join(sentence).split():
+    print(i[0])


join не нужен
вместо i подошло бы имя переменной word

krepysh · 2023-05-18T15:06:47Z

string_challenges.py

 # Вывести усреднённую длину слова в предложении
 sentence = 'Мы приехали в гости'
-# ???
+q = []


krepysh

Much better!
Есть только замечание про поразбивать функции на функции попроще. Можешь потренироваться использовать в пайчарм рефакторинг:
Выдели курсором код который хочешь вынести в отдельную функцию, нажми правой кнопкой мыши - выбери Refactoring, используй Extract method функцию, Должна хорошо тут сработать.

krepysh · 2023-05-19T12:24:54Z

for_challenges.py

 names = ['Оля', 'Петя', 'Вася', 'Маша']
-# ???
+for name in names:
+    if name == 'Петя' or name == 'Вася':


здесь надо использовать is_male.
А что если мы хотим больше имен добавить в программу. Должно быть одно место в программе ответственное за вычисление пола по имени.

krepysh · 2023-05-19T12:25:33Z

for_challenges.py

-# ???
+counter_1 = 0
+for group in groups:
+    counter_1 += 1


Используй enumerate, вместо того что бы считать индекс руками

krepysh · 2023-05-19T12:25:56Z

for_challenges.py

 ]
-# ???
+count_groups = 0
+for group in groups:


krepysh · 2023-05-19T12:27:44Z

for_dict_challenges.py

-
+for klass in school:
+    print(
+        f"Класс {klass['class']}: мальчики {[is_male[men_person['first_name']] for men_person in klass['students']].count(True)} "


Вынеси функцию расчета числа мальчиков и девочек в отдельную функцию, так что бы можно было ее переиспользовать в следующих задачах.

krepysh · 2023-05-19T12:28:17Z

for_dict_challenges_bonus.py

    return messages


+def zadacha(messages):


Код не разбит

krepysh · 2023-05-19T12:28:40Z

string_challenges.py

 # Вывести усреднённую длину слова в предложении
 sentence = 'Мы приехали в гости'
-# ???
+spisok = []


Suggested change

spisok = []

words = []

krepysh

Почти отлично.
Поправь эту штуку, что бы у тебя klass передавался аргументом явно. Очень плохая практика лазить неявно к чему-то вне твоей локальной области видимости.

krepysh · 2023-05-25T05:25:12Z

for_challenges.py

 ]
-# ???
+
+for group in enumerate(groups, start=1):


распакуй же сразу:

Suggested change

for group in enumerate(groups, start=1):

for group_number, group in enumerate(groups, start=1):

И в остальных местах тоже. Эта техника называется list unpacking, кода ты паре переменных присваиваешь значение листа.

krepysh · 2023-05-25T05:26:56Z

for_dict_challenges.py


+count = 0
+for klass in school_students:
+    most_common_name = Counter(name['first_name'] for name in school_students[count]).most_common(1)


school_students[count] это klass. Лучше использовать klass, и избавиться от count совсем.

krepysh · 2023-05-25T05:29:29Z

for_dict_challenges.py

+    return [is_male[men_person['first_name']] for men_person in klass['students']].count(True)
+
+
+def womens_count():


klass должен передаваться явно как аргумент (можно еще и назвать его по другому, что бы точно не было сомнений.

dimamushalapugin added 5 commits May 15, 2023 10:48

Done

884c8cc

Done

93c3aa6

Done

e7901d1

Done

206062b

Done

280ba49

krepysh suggested changes May 18, 2023

View reviewed changes

dimamushalapugin added 4 commits May 19, 2023 11:26

Done

222cfb6

Done

4f67043

Done

e2f6375

Done

1e9a453

krepysh suggested changes May 19, 2023

View reviewed changes

dimamushalapugin added 3 commits May 20, 2023 12:39

Done

70f736d

Done

a7e2911

Please do code review

e022ad4

krepysh suggested changes May 25, 2023

View reviewed changes

	for group in enumerate(groups, start=1):
	for group_number, group in enumerate(groups, start=1):

		return [is_male[men_person['first_name']] for men_person in klass['students']].count(True)


		def womens_count():

Conversation

dimamushalapugin commented May 17, 2023

Uh oh!

krepysh left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

krepysh left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

krepysh left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants