Fliffy ksenia by FliffyKsenia · Pull Request #34 · IlyaOrlov/PythonCourse2.0_August22

FliffyKsenia · 2022-09-10T09:36:05Z

Часть заданий к лекциям 1, 2, 3 выполнены.

IlyaOrlov

Оставил несколько замечаний.

IlyaOrlov · 2022-09-10T09:43:26Z

@@ -0,0 +1,6 @@
+radius = input("Введите радиус: ")
+def square(r):


Определение функции (стр.2-3) должно быть обособлено двумя пустыми строками и идти после блока импортов.

IlyaOrlov · 2022-09-10T09:43:43Z

+radius = input("Введите радиус: ")
+def square(r):
+    return math.pi * r ** 2
+import math


Импорты должны располагаться в начале файла с кодом.

IlyaOrlov · 2022-09-10T09:45:13Z

@@ -0,0 +1,3 @@
+#input data
+i=input ("Введите данные")


Согласно PEP8 знак присваивания должен быть с обеих сторон обособлен пробелами.

IlyaOrlov

ОК. Но есть ещё замечания.

IlyaOrlov

Хорошо. Осталось несколько старых замечаний, добавилось немного новых)

IlyaOrlov · 2022-09-14T19:02:51Z

+    # проверяем являются введеные данные числом
+    if x.isdigit():
+    # формируем число на основе введенных цифровых данных пользователем
+        chislo = chislo + x


Можно чуть упростить: chislo += x

IlyaOrlov

Осталось кое-что из старых замечаний и добавилось немного новых.

IlyaOrlov · 2022-09-17T14:37:56Z

+otvety = ['Ты сам-то понял, что написал? ', 'Аргументируй ', 'И? ']
+# запрашиваем у пользователя ввод данных
+fraza = input("Введите вопрос: ")
+ind = 0 - 1


Можно просто написать ind = -1

IlyaOrlov · 2022-09-17T14:38:09Z

+fraza = input("Введите вопрос: ")
+ind = 0 - 1
+while fraza.upper() != "ХВАТИТ":
+        ind = ind + 1


А здесь: ind += 1

IlyaOrlov

Добавил пару комментариев. В целом, всё ОК. Надо только местами поправить стиль и оформление.

IlyaOrlov

?

IlyaOrlov · 2022-09-22T16:48:25Z

+    arr[pos1], arr[pos2] = arr[pos2], arr[pos1]
+
+    return arr
+# вызываем функцию драйвера


Что такое функция драйвера?

IlyaOrlov

Вопросы и замечания еще остаются.

IlyaOrlov · 2022-09-30T12:01:27Z

@@ -0,0 +1,9 @@
+predlogenie = 'Вася играл Adagio, когда произведение было написано Andante'
+print("The original string is : " + str(predlogenie))


Зачем строку оборачивать в str?

IlyaOrlov

Добавил новые замечания. И ст старыми тоже надо что-то делать.

Добавлено задание 7.2

IlyaOrlov · 2022-10-09T07:57:28Z

+dict_muzica = {"Adagio" : "медленно", "Andante" : "спокойно"}
+
+res1 = predlogenie
+for e in dict_muzica:


Можно чуть упростить:

for e, v in dict_muzica.items(): res1 = res1.replace(e, v)

Всё-таки dict_muzica[e] - это целая операция, заставляющая интерпретатор искать в dict_muzica элемент с ключом e, чтоб получить его значение. А так мы сразу и ключ, и значение получаем.

IlyaOrlov

Стало гораздо лучше. Но надо исправить и другие замечания по старым заданиям, и замечание по 7.2

IlyaOrlov · 2022-10-09T08:05:57Z

+
+
+#создаем класс Pupil
+class Pupil:


Pupil должен быть наследником Man из 1-го задания. И у него должно быть имя, задаваемое через __init__.

IlyaOrlov · 2022-10-09T08:06:41Z

+
+#определяем метод класса
+    def solve_task(self):
+        time.sleep(vremy_random)


Нехорошо это, когда функция использует глобальные переменные (vremy_random). Лучше всё вычислять здесь.

IlyaOrlov

Просьба исправить ранее оставленные мною замечания. А то вроде как что-то делается, но до main не доходит. Поэтому результата не видно.

IlyaOrlov

Замечания не исправлены.

IlyaOrlov · 2022-10-20T13:42:10Z

+
+
+class Pupil(Man):
+    def __init__(self,  begin_time = 3, end_time = 6):


А где метод solve_task, который должен быть переопределен у Pupil согласно заданию? Как раз строки 12, 13 и вызов super().solve_task() должны быть в этом самом методе? А __init__ Pupil должен унаследовать от Man.

Задание 12.2 - не знаю как закрывать сервер, пока клиент не отключится. Задания 9.2, 12.1, 12.3 - исправлено по замечаниям

IlyaOrlov

Решение lec12.2, конечно, крутое получилось... Всё предусмотрели)
Оставил комментарии по мелочи. Их и старые замечания надо бы поправить.
Еще я вопросы оставлял... если Вы на них отвечали, возможно, Ваши ответы находятся в статусе Pending - если да, то я их не увижу, пока Вы не найдёте и не нажмёте кнопку с надписью Submit.

IlyaOrlov · 2022-10-21T20:02:23Z

+
+
+# создаем класс для создания потока который будет обрабатывать входящие соединения
+class Server_thread(Thread):


Название в CamelCase - это ServerThread и ServerMain

IlyaOrlov · 2022-10-21T20:04:23Z

@@ -0,0 +1,52 @@
+import pickle


Надо поправить несоответствия PEP8 в этом файле: пустые строки между методами, определениями, классами, ограничения по длине строк.

IlyaOrlov · 2022-10-21T20:05:19Z

+# функция востановления данных о людях.
+def restore():
+    # загружаем из файла данные о людях
+    with open('config_file_name', 'rb') as f:


Функции и методы не должны зависеть от глобальных переменных, иначе они теряют гибкость и ухудшается понимание, что с чем связано.

Доделан файл согласно PEP8

IlyaOrlov

Хорошо. Количество замечаний сокращается.

IlyaOrlov

Добавил еще вопросы/замечания.

IlyaOrlov · 2022-10-26T07:32:30Z

+            for j in range(3, i + 1, 2):
+                if (i % j == 0):
+                    break
+                if j * j > i:


Лучше просто в качестве верхней планки в условии цикла указать корень квадратный из i.

IlyaOrlov · 2022-10-26T07:33:09Z

+        lst = []
+    for i in range(start, end + 1, 2):
+
+        if not ((i > 10) and (i % 10 == 5)):


В чем смысл отдельной проверки деления на 10?

IlyaOrlov · 2022-10-26T07:35:08Z

+    print("Запускаем процедуру через потоки")
+    print(f"Найдено {len(thread_pusk())} простых чисела")
+    print("Запускаем через процессы")
+    print(f"Найдено {len(process_pusk())} простых чисела")


Еще хотелось бы увидеть ответы на вопросы к заданию (в виде комментариев).

IlyaOrlov

Есть существенные замечания по решению 14.2

IlyaOrlov · 2022-10-26T19:44:01Z

+        "CREATE TABLE IF NOT EXISTS manufacturers (Manufacturersid INTEGER PRIMARY KEY AUTOINCREMENT, name text)")
+    manufactors = ['adidas', 'nike  ', 'noname']  # вводим список для заполения таблицы производителей
+    for m in manufactors:  # заполняем таблицу производителей в цикле.
+        cursor.execute(f"insert into manufacturers (name) values ('{m}')")


Подстановка данных в запрос средствами форматирования Python - это открытые ворота для SQL-инъекций!
Аналогично в строках 19, 26 и 34.

IlyaOrlov · 2022-10-26T19:45:09Z

+cursor = conn.cursor()
+
+
+def createdb():  # функция создания таблиц и заполнение данными


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

FliffyKsenia and others added 10 commits August 31, 2022 20:26

Test

55a458d

Work on error

6b34936

test commit on pycharm

ff7b2fd

Lec 1 Task 1

fcdc52e

Update Lec 1 Task 1

ce0285e

test commit on new folder

3b91535

test commit on new folder

396cac3

add lec 2

e3bb952

add lec 2

2ca7c52

lec 3

fedbfce

IlyaOrlov requested changes Sep 10, 2022

View reviewed changes

lec 3.5

7634a0e

FliffyKsenia requested a review from IlyaOrlov September 10, 2022 10:42

IlyaOrlov requested changes Sep 10, 2022

View reviewed changes

Comment thread Practice/FliffyKsenia/lec.2.2.py Outdated

Comment thread Practice/FliffyKsenia/lec.3.5.py

Внесены изменения в файлы к 3 лекции. Добавлены решения к 4 лекции

a516114

IlyaOrlov requested changes Sep 14, 2022

View reviewed changes

FliffyKsenia added 3 commits September 16, 2022 23:14

lec 4

f85be75

Задания 4 и 8 к лекции 4

42b0418

лишние скобки убрала

8f7a825

IlyaOrlov requested changes Sep 17, 2022

View reviewed changes

Задания к лекции №4: 5, 6, 7, 8

2060de3

IlyaOrlov requested changes Sep 19, 2022

View reviewed changes

Comment thread Practice/FliffyKsenia/lec.4.5.py Outdated

Comment thread Practice/FliffyKsenia/lec.4.5.py Outdated

Comment thread Practice/FliffyKsenia/lec.4.5.py Outdated

FliffyKsenia added 8 commits September 21, 2022 06:48

Test

e2fcd2f

Work on error

d468e52

test commit on pycharm

ba11471

test commit on new folder

726ac4d

test commit on new folder

521501f

add lec 2

a6544c0

add lec 2

ba0371a

lec 3

33aa7cc

FliffyKsenia added 8 commits September 21, 2022 06:48

lec 3.5

db2c0bd

Внесены изменения в файлы к 3 лекции. Добавлены решения к 4 лекции

97a790b

lec 4

005a55c

Задания 4 и 8 к лекции 4

e32d1a0

лишние скобки убрала

d6628f0

Задания к лекции №4: 5, 6, 7, 8

b9e089c

Merge remote-tracking branch 'origin/FliffyKsenia' into FliffyKsenia

2cdbfe7

лишние скобки убрала

1ae5421

IlyaOrlov requested changes Sep 22, 2022

View reviewed changes

Задания к лекциям: 5, 7. Внесены исправления по заданию 4.5

c6be173

IlyaOrlov requested changes Sep 30, 2022

View reviewed changes

Задания 3.3 5.4 5.5

61af0ac

IlyaOrlov requested changes Oct 6, 2022

View reviewed changes

Comment thread Practice/FliffyKsenia/lec5.4.py Outdated

Comment thread Practice/FliffyKsenia/lec5.5.py Outdated

Внесены исправления к заданиям: 4.5; 5.3; 5.4; 5.5

e6e1dd3

Добавлено задание 7.2

IlyaOrlov reviewed Oct 9, 2022

View reviewed changes

IlyaOrlov requested changes Oct 9, 2022

View reviewed changes

Задания: 6.2; 7.3; 9.2; 9.3; 9.4; полностю задания по 12 лекции

df57394

IlyaOrlov requested changes Oct 17, 2022

View reviewed changes

IlyaOrlov reviewed Oct 17, 2022

View reviewed changes

Comment thread Practice/FliffyKsenia/lec12.1.server.py Outdated

IlyaOrlov requested changes Oct 17, 2022

View reviewed changes

Отредактировала задание 7.2

b0f74ef

IlyaOrlov requested changes Oct 20, 2022

View reviewed changes

Задание 9.1 - не получается убрать тире

2cb6fba

Задание 12.2 - не знаю как закрывать сервер, пока клиент не отключится. Задания 9.2, 12.1, 12.3 - исправлено по замечаниям

IlyaOrlov requested changes Oct 21, 2022

View reviewed changes

Задание 10.2

67c2def

Доделан файл согласно PEP8

IlyaOrlov requested changes Oct 22, 2022

View reviewed changes

Задача 11.1

16b32b6

IlyaOrlov requested changes Oct 26, 2022

View reviewed changes

Задача 14.2

8bdebcd

IlyaOrlov requested changes Oct 26, 2022

View reviewed changes

		@@ -0,0 +1,6 @@
		radius = input("Введите радиус: ")
		def square(r):

		@@ -0,0 +1,3 @@
		#input data
		i=input ("Введите данные")

		@@ -0,0 +1,9 @@
		predlogenie = 'Вася играл Adagio, когда произведение было написано Andante'
		print("The original string is : " + str(predlogenie))



		class Pupil(Man):
		def __init__(self, begin_time = 3, end_time = 6):



		# создаем класс для создания потока который будет обрабатывать входящие соединения
		class Server_thread(Thread):

		cursor = conn.cursor()


		def createdb(): # функция создания таблиц и заполнение данными

Conversation

FliffyKsenia commented Sep 10, 2022

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

IlyaOrlov 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!

Uh oh!

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

IlyaOrlov 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!

IlyaOrlov left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!