Телеграмм чат группы python

Ivan (antisleep)

А что, импорты это какая-то проблема?

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

15:09пожаловаться #1

for _ in range(len(ls)): -> внешний цикл
И lss.index(data['tiker']
lss.index() это не бесплатная функция )

Anatoly Kashtanov in Python — вакансии и аналитика

15:09пожаловаться #2

да чего вы разошлись выше товарищ написал, самый оптимальный код. ИМХО. Никаких collections и appendoFF.

15:09пожаловаться #3

Влад Никандров

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

Это заблуждение
Язык тебе позволяет с нуля написать свою либу для работы с датами
Ты что будешь либу писать вместо datetime?

15:10пожаловаться #4

🌼S

Anatoly Kashtanov

да чего вы разошлись выше товарищ написал, самый оптимальный код. ИМХО. Никаких collections и appendoFF.

Где?

Anatoly Kashtanov in Python — вакансии и аналитика

15:10пожаловаться #5

Где?

def calculate(data):
lss = {i['tiker'] for i in data}
res = [{'tiker': i, 'summ': sum([int(j['value']) for j in filter(lambda x: x['tiker'] == i, data)])} for i in lss]
return res

15:11пожаловаться #6

Где?

Вот то первое решение с filter, sum и list comprehension)))

15:11пожаловаться #7

Что совсем не оптимально

15:11пожаловаться #8

ВН

Ivan (antisleep)

если потребуется (если на то пошло я тот тип который сначала читает спецификации))))

15:12пожаловаться #9

🌼S

Anatoly Kashtanov

def calculate(data):
lss = {i['tiker'] for i in data}
res = [{'tiker': i, 'summ': sum([int(j['value']) for j in filter(lambda x: x['tiker'] == i, data)])} for i in lss]
return res

ужасно

15:12пожаловаться #10

ВН

ужасно

ФП стайл он такой)))

15:12пожаловаться #11

🌼S

Влад Никандров

ФП стайл он такой)))

Да какой фп стайл)

15:13пожаловаться #12

ВН

Да какой фп стайл)

полу фп стайл)))

Anatoly Kashtanov in Python — вакансии и аналитика

15:13пожаловаться #13

Чего ужасного-то? Я конечно еще тот питонист,но код понятный, без подключения сторонних либ, короток, нет aapend pop insert. Думаю очень быстрый. Что не так с ним?

Michael V in Python — вакансии и аналитика

15:14пожаловаться #14

Вадим Смирнов

да

плавали... я когда в сбере работал, там айтишники были практически 100% мужики, но вообще банк состоял из женщин процентов на 90. "Тяжелая" жизнь 😊

15:14пожаловаться #15

🌼S

from collections import defaultdict
data = defaultdict(int)

for t in ls:
    data[t['tiker']] += int(t['value'])


[{‘tiker’: t, ‘summ’: v} for t, v in data.items()]

15:17пожаловаться #16

🌼S

O(2N) и нормально читается

15:17пожаловаться #17

🌼S

Anatoly Kashtanov

фильтры, лябды, сравнения, компрехеншены двойной вложенности

15:19пожаловаться #18

ВН

from collections import defaultdict
data = defaultdict(int)

for t in ls:
    data[t['tiker']] += int(t['value'])


[{‘tiker’: t, ‘summ’: v} for t, v in data.items()]

ладна победил) Но рефактор кода это как бы сл стадии)

Michael V in Python — вакансии и аналитика

15:22пожаловаться #19

вопрос стиля, зачем нужна вся эта экспрессивность, если ей никогда не пользуешься? С другой стороны перегибать тоже не стоит. Я в этом чате когда-то FizzBuzz в одну строчку на питоне писал в качестве примера как не надо делать 😊