Adding object to a list with a for loop?

Question

I've done my research on stack overflow and could not find an answer that works out for me even though similar questions have been posted in the past. My issue is that I want to add string values that I am getting from a for loop that goes through a list and matches to a regular expression. I want to add them to a combined list of values, I am using a clean function that works on the submission of a form but when the data prints into my terminal it comes out like this:

   ['#example']
   ['#test']

when I want those values to print like this:

   ['#example', '#test',]

Here's my code in my views.py

      def clean(self):
        data = self.cleaned_data
        regex = re.compile("\B#\w\w+")
        tweeters = data['tweets']
        split_tweets = tweeters.split()
        for x in split_tweets:
           if re.search(regex, x):
            master_list = [x]
            print master_list

John La Rooy · Accepted Answer · 2015-08-27 22:37:34Z

2

You need to append to a list, otherwise you are just throwing away all the previous values

  def clean(self):
      data = self.cleaned_data
      regex = re.compile("\B#\w\w+")
      tweeters = data['tweets']
      split_tweets = tweeters.split()
      master_list = []
      for x in split_tweets:
          if re.search(regex, x):
              master_list.append(x)
      print master_list

You can also use a list comprehension here instead

  def clean(self):
      data = self.cleaned_data
      regex = re.compile("\B#\w\w+")
      tweeters = data['tweets']
      split_tweets = tweeters.split()
      master_list = [x for x in split_tweets if re.search(regex, x)]
      print master_list

answered Aug 27, 2015 at 22:37

John La Rooy

306k54 gold badges378 silver badges513 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

TigerhawkT3 · Accepted Answer · 2015-08-27 22:36:34Z

1

You're doing some strange things with that assignment. Why do you keep reassigning a list? Initialize the list before the loop, append() to it inside the loop, and print the whole thing after the loop.

master_list = []
for x in split_tweets:
   if re.search(regex, x):
    master_list.append(x)
print master_list

answered Aug 27, 2015 at 22:36

TigerhawkT3

49.5k6 gold badges66 silver badges101 bronze badges

Comments

YOBA · Accepted Answer · 2015-08-27 22:36:26Z

1

Do this instead:

  def clean(self):
    data = self.cleaned_data
    regex = re.compile("\B#\w\w+")
    tweeters = data['tweets']
    split_tweets = tweeters.split()
    master_list = []
    for x in split_tweets:
       if re.search(regex, x):
        master_list.append(x)
    print master_list

You need to append elements to the list instead of affecting a new "one element list" [x] to master_list, it get overridden by the next value

answered Aug 27, 2015 at 22:36

YOBA

2,8071 gold badge16 silver badges30 bronze badges

Collectives™ on Stack Overflow

Adding object to a list with a for loop?

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related