[Croudflup]

Ucze sie #python z kursu pewnego i taki kod gość pokazał na wyciaganie maili/numerów tel ale za nic nie chce mi lecieć, syntax error mi wchodzi choćbym do schowka wrzucał nie wiem ile numerów. Najpierw krzaczy na pętli - po skomentowaniu nadal i tak ciągle, gdziekolwiek później. Ktoś wie o co chodzi? pyperclipa mam na kompie już.
Kod:
#! python3
import re, pyperclip
# Create a regex for phone numbers
phoneRegex= re.compile(r'''
#111-222-2222, 222-1111, [phone], 555-0000 ext. 12345, ext. 12345, x12345

((\d\d\d)|(\(\d\d\d)))? #area code (optional)
(\s|-) #separator
\d\d\d #3 dig
- #separator
\d\d\d\d #last 4 dig
(((ext(\.)?\s)|x) #extension word-part (optional)
(\d{2,5}))? #extension number-part (optional)
)
''' ,re.X)

#todo: create a regex for email adresses
emailRegex=re.compile(r'''
#[e-mail]

[a-zA-Z0-9.+]+ #name part
@ #@ symbol
[a-zA-Z0-9.+]+ #domain name part
''' re.VERBOSE)
#todo: get the text off the clipboard
text=pyperclip.paste()
#todo: extract the email/phone from the text
extractedPhone= phoneRegex.findall(text)
extractedEmail= emailRegex.findall(text)

allPhoneNumbers=[]
for phoneNumber in extractedPhone:
allPhoneNumbers.append(phoneNumber[0])

print(extractedPhone)
print(extractedEmail)
print(allPhoneNumbers)
#copy the extracted email/phone to clipboard
results='\n'.join(allPhoneNumbers) +'\n'+'\n'.join(extractedEmail)
pyperclip.copy(results)

[C.....t]

@Croudflup: po pierwsze, to weź wrzuć kod na jakiś pastebin, albo coś podobnego, a drugie, to daj co konkretnie się wywala, jakiś komunikat błędu chociażby, bo inaczej to wróżenie z fusów.

[Croudflup]

@CytrynowySorbet:
[pastbin]
(https://pastebin.com/JyCfR2Qz) błąd to zwykły syntax error w okienku. jak wywalę pętlę to ten sam błąd ale na exctracted przy print(ex.)

[Croudflup]

@CytrynowySorbet: https://pastebin.com/EkjhDi8N nie wiem czemu mi to kasują

[s.....3]

@Croudflup: wywaliłem twoje regexy i wpisałem swoje. Teraz sobie szukaj błędów, w swoim kodzie:
https://pastebin.com/L1M9E9Yc

jeśli chodzi o numery to wpisałem coś przykładowego, ale dla adresów email, powinno działać należycie

edit: nie masz przecinka przed re.VERBOSE w 24 linijce

[Croudflup]

@stranger13: odpaliłem działa jutro sprawdzę dzięki !

[mrq]

@Croudflup: pierwszy lepszy interpreter pythona online zwraca taki błąd:

File "main.py", line 24
''' re.VERBOSE)
^
SyntaxError: invalid syntax

Chodzi o przecinek?

[Croudflup]

@mrq: wrzuciłem przecinek potem bo też wywaliło to i wywalił mi błąd że pyperclipa nie ma ten interpreter ( ͡° ͜ʖ ͡°)

[Malcolmpl]

@stranger13 @Croudflup Ile jeszcze lat musi minąć zanim ludzie nauczą się, że żaden regex nie ogarnia poprawnie adresu email?

[Croudflup]

@Malcolmpl: ja się dopiero uczę (⌐ ͡■ ͜ʖ ͡■)

[mrq]

wrzuciłem przecinek potem bo też wywaliło to i wywalił mi błąd że pyperclipa nie ma ten interpreter

@Croudflup:

Mi tez taki błąd zwracał tez założyłem że online'owy interpreter możne tego nie mieć. Oprócz tego, to może być gdzieś jakaś literówka. Trzeba szukać rozwiązania. Od błędu do błędu ;).

[Croudflup]

@mrq: wywaliłem kilka rzeczy zrestarttowalem sporo błędów i linijek z nimi wyskoczyło polatalem to i lecę dalej xd

[mrq]

@Croudflup: W sensie upraszczasz kod? Też dobre rozwiązanie.

[Croudflup]

@mrq: noms nie mam na razie chyba skilla jeszcze do jakiegoś konstruktywnego uproszczenia ale staram się xd