[p.....k]

#naukaprogramowania #python

wreszcie sam coś zrobiłem :) program ma robić akronim ze słów, czyli skrót z pierwszych liter.

import re


def abbreviate(words):

,,,,filtered = re.findall(r'[A-Z]+\'?[A-Z]?', words.upper())

,,,,acronym = ''.join([chr[0] for chr in filtered])

,,,,return acronym

[p.....k]

@LazyInitializationException: nie stąd się biorą dzieci

[asterix61]

@ponuryrolnik: Problem w tym, że Twój kod jest powolny, ciężki i.. nie działa

>>> abbreviate("United States of America")

'USOA'

A wystarczyło odpowiednio pobawić się tekstem

[p.....k]

@asterix61: według założeń - działa. jak to powolny? w sensie, w 1 linijce ma być?

[p.....k]

@asterix61: tylko, że test jest taki np:

def test_lowercase_words(self):

self.assertEqual(abbreviate("Ruby on Rails"), "ROR")
czyli wyszłoby RR i nie przeszło

[asterix61]

@ponuryrolnik: Regexy są powolne, lepiej zrobić wiele operacji na stringu niż puścić jednego regexa.
Ok, czyli kwestia przypadku wchodzi w grę, chyba że są jakieś zasady, które słowa wchodzą w skład, a które nie.
Aby przejść ten test, wystarczy usunąć if z mojego przykładu. Dawno w projekcie nie korzystałem z regexa, zawsze lepszym rozwiązaniem okazywało się kilka prostych operacji na stringu :)

[p.....k]

"The Road _Not_ Taken"

@asterix61: bez ifa to też nie przejdzie

ale co to znaczy regexy są

[asterix61]

@ponuryrolnik:

regex 3.3365409990001353
join split 0.8575375489999715

To oznacza, że są wolniejsze od zwykłych operacji. W tym przypadku ponad 4 razy

[M.....o]

@ponuryrolnik: dla samego sportu - przejdzie to test case'y?

"".join([next((char.upper() for char in word if char.isalpha()), "") for word in "ruby _on_ 142 Rails".split()])

[p.....k]

@MakuZo:

 def test_punctuation_without_whitespace(self):
 self.assertEqual(abbreviate("Complementary metal-oxide semiconductor"), "CMOS")

@asterix61: no w sumie jak jakaś operacja ma się powtarzać tysiące razy to ma sens.

[maad]

@asterix61: Python w ogóle jest powolny

[KevinMalone]

@ponuryrolnik:
https://stackoverflow.com/questions/26214328/why-python-regex-is-so-slow

[asterix61]

@maad: Nigdy nie mówiłem, że jest demonem prędkości. Ale świadomy programista nie pozwoli spowolnić go jeszcze bardziej :)

[M.....o]

@ponuryrolnik: Dzięki, a sprawdzisz jeszcze to? Obiecuję, ostatni raz ( ͡° ͜ʖ ͡°)

"".join(word[0] for word in "".join(char if char.isalpha() else " " for char in string.upper()).split())

[p.....k]

@MakuZo: "Halley's Comet" tu nie przechodzi xD niby takie łatwe, już się śmiał typ, że dzieci w przedszkolu robią...

https://pastebin.com/5xSywByN

tu masz testy jak chcesz się bawić

[p.....k]

@asterix61: i jak, rozgryzłeś to bez regexa? tu masz testy: https://pastebin.com/5xSywByN

[asterix61]

@ponuryrolnik: https://pastebin.com/K1N7Uf7a done, roboty na jakąś minute :D

[p.....k]

@asterix61: ctrl-c było bo jest taka solucja [w[0] for w in words.replace, a ty nawet tego w nie zmieniłeś xD

[asterix61]

@ponuryrolnik: wtf XD dłużej bym szukał rozwiązania, niż to pisał. To jak czepianie się, że ktoś zmienną nazwał 'name' XD

[p.....k]

@asterix61: wcześniej użyłeś x, a pierwsza solucja na exercism bez regexa właśnie używa w... napisz program wyliczający % szanse, że kłamiesz