OS: OS X 10.10.4
Python:2.7.10
virtualenv環境

python初心者です。
とあるグルメサイトのクローラーを勉強として作成しています。
URLの末尾の数字をループして代入することによって特定地域のお店の名前、電話番号を書き出すという構成を考えて書いてみました。

coding=UTF-8

from bs4 import BeautifulSoup
import re
import urllib2
import sys

for num in range(48065,48070):
mise = "http://tabelog.com/tokyo/A1307/A130701/131" +str(num) +"/"

url = urllib2.urlopen(mise).read().decode('utf-8')
soup = BeautifulSoup(url, "html.parser")

name = soup.find_all(class_="display-name") [0].string
tel = soup.find_all(class_="tel-main")[0].string

print(soup.prettify())

print name
print tel

これの出力結果は

まんま屋
非公開
浜焼太郎十条店
03-5948-5583
Traceback (most recent call last):
File "soup.py", line 14, in <module>
tel = soup.find_all(class_="tel-main")[0].string
IndexError: list index out of range

となり、tel-mainから配列が取得できないものはエラーになってしまい終了してしまいます。
これを「取得できなければ次へいく」という仕様にするにはどのようにすればよいでしょうか？
初歩的ですみませんがご回答お願いします。

また、最終的には地域を指定するURL部分もfor文で囲み、全国すべての店の情報を書き出せるようにするのが目標です。

行動規範の内容に同意します

回答1件

ベストアンサー

find_all は見つからなかった時に、None になるみたいなのでそれで判定してみました。
他にもっと良いやり方があるかもしれませんが、どうでしょうか？

python
1w = soup.find_all(class_="tel-main")
2if w is not None:
3    tel = ''
4else:
5    tel = w[0].string

参考: PythonとBeautiful Soupでスクレイピング - Qiita
参考: PythonではNoneの比較は==ではなくisを使う - こんにちはこんにちはmonmonです！

-- 追記
調べてみたら、None ではなく空の配列になってました。
これでどうでしょうか？
今度はちゃんと確認しましたので大丈夫かと思います！

python
1w = soup.find_all(class_="tel-main")
2if len(w) == 0:
3    tel = ''
4else:
5    tel = w[0].string

投稿2015/07/16 09:36

編集2015/07/16 11:38

ryunix

総合スコア1656

4mmsod

2015/07/16 10:55

ご回答ありがとうございます。この場合ですと if w is None tel = ' ' として"not"はいらないのではないですか？どちらにせよ、ありがとうございます！

4mmsod

2015/07/16 10:57

ryunixさんの回答を参考にして出力したところ、次はまんま屋非公開浜焼太郎十条店 03-5948-5583 Traceback (most recent call last): File "soup.py", line 18, in <module> tel = w[0].string IndexError: list index out of range となり、配列エラーがでるところで止まるようになってしまいました・・・。難しいですね。