First clear solution for Stressful Subject by wyf.141060 - python coding challenges

Enable Javascript in your browser and then refresh this page, for a much enhanced experience.

First solution in Clear category for Stressful Subject by wyf.141060

def pure(word):
    ret = []
    [ret.append(letter) for letter in word.lower() if letter not in [''.join(ret)[-1:], '!', '-', '.']]
    return ''.join(ret)


def is_stressful(subj):
    return True in [pure(word) in ["help", "asap", "urgent"] for word in subj.split()] \
           or subj.isupper() \
           or subj[-3:] == '!!!'

Dec. 15, 2018

Comments:

ayubutrym on April 23, 2019, 4:06 p.m.
In Python the construction `True in [...]` can be written as `any(...)` What is the reason to complicate it like `''.join(ret)[-1:]` instead of `(ret[-1] if ret else '')` or writing line 3 as a conventional `for` loop where you can reference the latest added letter directly? If you really like functional style consider using `itertools.groupby` like: <pre class='brush: python'>from itertools import groupby def pure(word): return ''.join(k for k, g in groupby(filter(lambda c: c not in '!-.', word.lower()))) </pre>

wyf.141060 on April 23, 2019, 4:49 p.m.
Yeah, itertools.groupby is a powerful tool, i'm very happy to know it. I'm agree with you on the any() expression and reference ret[-1] directly. Now, i feel not good when i see the code which was wrote before several months. At that time, i was a fresh pythoner. Luckly, i learn a lot from Best Sloutions and other's advice, and I'll write more pythonic code. Thank you and anyone who guides others!

SendGrid

0 %

PyCharm The Python IDE for Professional Developers. Enjoy productive Python, web and scientific development with PyCharm. Download it now!