スクレイピングで取得した複数のURLをリストに入れるには

Question

from bs4 import BeautifulSoup
import requests

url = ""　　#スクレイピングしたいURL
r = requests.get(url)
soup = BeautifulSoup(r.text,'html.parser')
wrap = soup.find_all(class_="wrap")　#取得したいURLのクラスがwrapでした

for a in wrap:
  print(a["href"])

こちらこコードでページ内のURLを
取得することができましたが

リストに入れたいという事で

from bs4 import BeautifulSoup
import requests

url = ""　　#スクレイピングしたいURL

title_urls_list  = []

for url  in urls:
  r = requests.get(url)
  soup = BeautifulSoup(r.text,'html.parser')
  wrap = soup.find_all(class_="wrap")　#取得したいURLのクラスがwrapでした

for a in wrap:
  title_urls_list.extend([x.text for x in a["href"]]) 

#こちらを試してみましたがAttributeError: 'str' object has no attribute 'text'と出ます。

title_urls_list = []
このりすとに入れて
print(title_urls_list)

で取得したURL達を表示させるには
どういったコード変更すべきでしょうか？
教えていただけると幸いです。
よろしくお願いいたします。

Answer 1

a["href"]はすでにstr型となっていてxもstr型となりますのでtext属性は存在ぜずエラーが起きているようです。

for a in wrap:
  title_urls_list.extend([x.text for x in a["href"]]) 

三項演算子でやりたいのであれは以下のようにするのがいいのかなと思いました。

from bs4 import BeautifulSoup
import requests

url = ""　　#スクレイピングしたいURL

title_urls_list  = []

for url  in urls:
  r = requests.get(url)
  soup = BeautifulSoup(r.text,'html.parser')
  wrap = soup.find_all(class_="wrap")　#取得したいURLのクラスがwrapでした
　　　　title_urls_list.extend([a["href"] for a in wrap]) 

Answer 2

ワンライナーでまとめたいのであれば、

title_urls_list = []
title_urls_list.append([a['href'] for a in wrap])

ですかね

ちなみに　urls　は複数ページ分のURLでしょうか？
だとすると

for url  in urls:

でwrap上書きされてるので、最終的に最後のwrapしか取れないよな気がします。

wrap = []
wrap.extend(soup.find_all('a'))

で一つのリストにまとめるか、もしくは辞書でページ単位にまとめた方がいいと思います。

まとめなくても

for url in urls:
    r = requests.get(url)
    soup = BeautifulSoup(r.text,'html.parser')
    wrap.extend([a['href'] for a in soup.find_all('a')])

でできそうですね。

Answer 3

print(a["href"])

が出来て、これが何故出来ないのか分かりませんが、

for a in wrap:
  title_urls_list.extend([a["href"]]) 

ですね。
普通は、

for a in wrap:
  title_urls_list.append(a["href"]) 

でしょうか。

また、このfor部分は、for url in urls:のループの中に入れるべきでは？
ループの終わったあとに置くと、最後のwrapしか処理されません。