Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Python Regular Expressions in Python Introduction to Regular Expressions Email Groups

Rabih Atallah
Rabih Atallah
3,405 Points
Posted by Rabih Atallah
Rabih Atallah
3,405 Points

code error

Please what is wrong with my code?

emails.py 
import re

string = '''Love, Kenneth, kenneth+challenge@teamtreehouse.com, 555-555-5555, @kennethlove
Chalkley, Andrew, andrew@teamtreehouse.co.uk, 555-555-5556, @chalkers
McFarland, Dave, dave.mcfarland@teamtreehouse.com, 555-555-5557, @davemcfarland
Kesten, Joy, joy@teamtreehouse.com, 555-555-5558, @joykesten'''
contacts = re.search(r'''
^(?P<email>[\w.+]*@[\w.]*)\t?
(?P<phone>\d{3}-\d{3}-\d{4})$''', string, re.X|re.M)

1 Answer

Chris Freeman
MOD
Chris Freeman
Treehouse Moderator 68,227 Points
Chris Freeman
Chris Freeman
Treehouse Moderator 68,227 Points

I found 5 errors

  • remove leading '^'. Using this will only match beginning of line. Failed due to user name
  • change both "" to "+". Using "" means 0 or more. This might give false positive
  • add missing comma. Need to "consume" the comma between email and phone number
  • change \t to \s. The whitespace between the fields is a space not a tab
  • remove trailing "$". Using this will only match end of line. Failed due to twitter field.
contacts = re.search(r'''
(?P<email>[\w.+]+@[\w.]+),\s? # <-- remove leading '^', change both  "*" to "+", add missing comma, and change \t to \s.
(?P<phone>\d{3}-\d{3}-\d{4})''', string, re.X|re.M) # <-- remove trailing "$"
Rabih Atallah
Rabih Atallah
3,405 Points
Rabih Atallah
Rabih Atallah
3,405 Points

I tried it this way and it's still not working. The return is none

Rabih Atallah
Rabih Atallah
3,405 Points
Rabih Atallah
Rabih Atallah
3,405 Points

Sorry Chris It DID work!