画像ファイルの一括保存

Question

画像ファイルの一括保存

ひとつのページにある画像ファイルを一括保存したいのですが、ローカルファイルに作成したディレクトリに1つしか保存することができません。

発生している問題・エラーメッセージ

エラーメッセージは出ていません

該当のソースコード

#! python3
# 画像サイトのダウンローダー
# 検索結果の画像10枚をダウンロードする

import requests, bs4, os

# 保存先の作成
url = 'http://gensun.org/pid/138658'
os.makedirs('gazo', exist_ok=True)

res = requests.get(url)
res.raise_for_status()

soup = bs4.BeautifulSoup(res.text)

# jogを抜き出す
gazo_elem = soup.select('.imagebox img')    
if gazo_elem == []:
    print('画像が見つかりませんでした。')
else:
    for i in range(10):
        gazo_url = gazo_elem[0].get('src')  
        # 画像をダウンロードする
        print('画像をダウンロード中{}...'.format(gazo_url))
        res = requests.get(gazo_url)
        res.raise_for_status()

            #画像を./gazoに保存する
        gazo_file = open(os.path.join('gazo',os.path.basename(gazo_url)),'wb')
        for chunk in res.iter_content(100000):
            gazo_file.write(chunk)
        gazo_file.close()


print('完了')

Answer 1

ループ内で毎回0番目の画像のURLを取り出しているためです。

gazo_url = gazo_elem[0].get('src')  

書き換えるとしたらこんな感じでしょうか。

# 中略
else:
    for elem in gazo_elem[:10]:
        gazo_url = elem.get('src')  
        # 画像をダウンロードする
        print('画像をダウンロード中{}...'.format(gazo_url))
        res = requests.get(gazo_url)
        res.raise_for_status()

            #画像を./gazoに保存する
        gazo_file = open(os.path.join('gazo',os.path.basename(gazo_url)),'wb')
        for chunk in res.iter_content(100000):
            gazo_file.write(chunk)
        gazo_file.close()

Comments

[kolobokkule]

原因がわかりとてもすっきりしました。ありがとうございます。
しかしいただいたものを試してみると、エラーメッセージ
Traceback (most recent call last):
File "C:/Users/A14-0325/PycharmProjects/test/downloader.py", line 22, in <module>
gazo_url = gazo_elem.get('src')
AttributeError: 'list' object has no attribute 'get'

が出てしまいました。
gazo_urlがリストとして成り立っていない？ということでしょうか？
まだ学びはじめなもので、この点お聞かせいただけると嬉しいです。

[hayataka2049]

gazo_url = elem.get('src')

[kolobokkule]

解決いたしました。
ありがとうございました！