Skip to content

задание1 к лекции1#32

Open
AlexaEltarenko wants to merge 27 commits into
mainfrom
KanatevaBranch
Open

задание1 к лекции1#32
AlexaEltarenko wants to merge 27 commits into
mainfrom
KanatevaBranch

Conversation

@AlexaEltarenko
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть замечания.
Также лучше называть файлы python, используя английские буквы, чтоб не было проблем с их запуском в различных системах.

Comment thread Practice/KanatevaA/задание №1 к лекции 1.txt Outdated
Comment thread Practice/KanatevaA/задание №1.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хорошо, но есть некоторые замечания.
Кстати, старый PullRequest #10, наверное, стоит закрыть.


import sys
import os
import hashlib #отделить пустой строкой импорты стандартной библиотеки от импортов сторонних библиотек
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Так-то здесь все импорты из стандартной библиотеки.


def parse_arguments():
parser = argparse.ArgumentParser()
subparsers = parser.add_subparsers(dest='subcommand', help='subcommand help') #пробел = пробел
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Очень хорошо, что Вы обратили на это внимание! На самом деле, когда знак = используется для задания значений по умолчанию или передачи именованных аргументов функции (а именно это здесь и в стр. 49 и 52 происходит, мы это ещё пройдём) пробелами он не выделяется. Это единственное исключение из общего правила.

@@ -0,0 +1,7 @@
import math

def sguare(r):
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хорошо. Но к старым замечаниям добавилось немножко новых.

@@ -0,0 +1,4 @@
t = int (input("введие время:"))
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Согласно PEP8 пробелов между именем функции и скобкой быть не должно.

Comment thread Practice/KanatevaA/Lek3Task4.py Outdated
Comment thread Practice/KanatevaA/Lek3Task4.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

# а вместо чисел, кратных пяти — слово Buzz.
# Если число кратно пятнадцати, то программа должна выводить слово FizzBuzz.

for a in range(0,101):
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему начинаем с 0?

# Если число кратно пятнадцати, то программа должна выводить слово FizzBuzz.

for a in range(0,101):
if a % 3 == 0:
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для числа кратного 15 напечатается и Fizz, и Buzz, и FizzBuzz, а должно только FizzBuzz.
Кстати, что-то кроме этих слов печатается (имеются в виду числа не кратные ни 3, ни 5)?

print(a)
i = 0
while i < len(a):
print(a[i])
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Надо ещё и индекс числа печатать в соответствии с заданием.

# Если пользователь ввел не числовой символ, вывести предупреждение и запросить новый символ.

a = input("введите число:")
while not a.isdecimal(): #можно и через моржовы оператор while not (a := input("введите число:")).isdecimal():
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно и даже нужно.


a = input("введите число:")
while not a.isdecimal(): #можно и через моржовы оператор while not (a := input("введите число:")).isdecimal():
if a == "stop" or a == "Stop" or a == "STOP":
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно проще: if a.lower() == "stop"

a = input("введите число:")
else:
a = int(a)
print(f"Квадрат числа:{a*a}")
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Только там не квадрат числа требовался, а формирование числа из вводимых пользователем цифр.

if a == "хватит":
break
else:
a = input("ваш вопрос?")
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Строки 15 и 16 стоит поменять местами. Иначе первый вопрос (из стр.9) алгоритм игнорирует.

Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Замечания остаются.

Comment thread Practice/KanatevaA/Lek4Task6.py Outdated
Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Остались старые замечания, и ещё вопрос появился.

print(f"наибольшее число {b}")
return b

fun(30, 10)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вопрос на понимание: можно ли в этом решении обойтись одним print для вывода наибольшего значения, используя возможность возврата значения из функции?

Copy link
Copy Markdown
Owner

@IlyaOrlov IlyaOrlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем сюда-то присылать контрольную? Я же попросил: в телеграм или на почту.


def myfun(l):
len(l)
return len(l)
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет. Надо написать СВОЮ реализацию len. В задании написано: Стандартную функцию len использовать нельзя!

# print(myfun(fjflg))
def myfun(a):
for i in a[::-1]:
return a #исправила return что-бы фраза выводилась певернутая целиком
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Перевернутая не выводится, однако.


def myfun(a):
s = 0
for s in a if s == a
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это вообще нерабочий код!

@AlexaEltarenko
Copy link
Copy Markdown
Collaborator Author

AlexaEltarenko commented Sep 23, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants