Week2 #15
Week2 #15
Conversation
ad3002
left a comment
ad3002
left a comment
There was a problem hiding this comment.
Все круто! Местами код слишком сложный, можно упростить.
Со сложным кодом две проблемы 1) если ты в команде, то все должны понимать что код делает 2) ты должен быть уверен что если в команде только ты, что ты через пару лет будешь понимать, что ты написал
| *$py.class | ||
|
|
||
| # WebStorm | ||
| .idea/ |
There was a problem hiding this comment.
Удали папку .idea из репозитория, но оставь у себя. Пойми как это сделать.
There was a problem hiding this comment.
Мне тогда надо глобальный гитигнор отредаяить?
| return names | ||
| pivot = names[random.randint(0, len(names) - 1)] | ||
|
|
||
| middle = [i for i in names if i == pivot] |
There was a problem hiding this comment.
ну вот точно хочется переменную name, а не i, тут же строки
There was a problem hiding this comment.
Согласен, но она же безликая по идее)
| a = list(set(a1) | set(a2)) | ||
|
|
||
| for bypass in range(1, len(a)): | ||
| for k in range(len(a) - bypass): |
There was a problem hiding this comment.
почему k? глобальная идея в том, что переменные которые одинаковые по сути называются одинаково у всех, код должен читаться без комментариев исходя из названий переменных
There was a problem hiding this comment.
Ну, да, логично
Взял свой старый код сюда
| nonlocal transition | ||
| if number: | ||
| string = "{} {}" | ||
| hundreds = {0: '', |
There was a problem hiding this comment.
это константы, их лучше задать заглавными буквами
There was a problem hiding this comment.
Это не противоречит пеп8 кэмэл-стайлу?
| ```python | ||
| # merge sort practise | ||
| def merge(a1, a2): | ||
| sorted_arr = [None for _ in range(len(a1) + len(a2))] |
There was a problem hiding this comment.
использование None вместо 0 здесь оправдано если у тебя где то будет проверка x is None
| ```python | ||
| def create_phone_number(n): | ||
| string = "({}{}{}) {}{}{}-{}{}{}{}" | ||
| return string.format(*n) |
| file = input() | ||
|
|
||
| with open(fr"D:\Downloads\{file}.txt") as f_read, open("new.fasta", 'w') as f_write: | ||
| for line in f_read: |
There was a problem hiding this comment.
лучше while, в цикле for не ожидается что будет меняться состояние f_read, например для словарей бы ошибку выкинуло, тут мало кода и видно где, а если бы этот ридлайн был бы где-то в глубинах?
| threshold = int(f_read.readline()) | ||
|
|
||
| a = f_read.readlines() | ||
| qualities = [None for _ in range(len(a) // 4)] |
|
|
||
|
|
||
| def get_incorrect_phred(file): | ||
| with open(rf"D:\Downloads\{file}.txt") as f_read: |
| with open(rf"D:\Downloads\{file}.txt") as f_read, open(rf"filtered{file}.fastaq", 'w') as f_write: | ||
| cut_off = int(f_read.readline()) | ||
| lines = iter(f_read.readlines()) | ||
| while lines: |
There was a problem hiding this comment.
мне нравится
line = fh.readline()
while line:
...
line = fh.readline()Должно работать!
2 participants
No description provided.