New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

スクレイピングの新しいサンプルを追加 #21

Merged
merged 6 commits into from Oct 23, 2017

Conversation

Projects
None yet
2 participants
@laughk
Collaborator

laughk commented Oct 21, 2017

コピペですぐに使えるサンプルを1つ追加します。

スクレイピングの新しいサンプルを追加
コピペですぐに使えるサンプルを1つ追加します。

@laughk laughk force-pushed the laughk:add-scraping-sample branch from ce6a783 to fc4048c Oct 21, 2017

@takanory

一通りコメントしました。対応したら merge しちゃってくれていいです

def main():
# connpass の PyNumon#6 のイベント参加者・申込者一覧のURL
url = 'https://python-nyumon.connpass.com/event/62147/participation/#participants'

This comment has been minimized.

@takanory

takanory Oct 23, 2017

Member

#participants はなくてもよさそう

Matthias
```
### その他

This comment has been minimized.

@takanory

takanory Oct 23, 2017

Member

その他はまるっと削除でいいんじゃないかな

This comment has been minimized.

@laughk

laughk Oct 23, 2017

Collaborator

あとで削除PR出せばいいかなと思ってましたが、そういうことでしたら消しちゃいます!

# 参加者枠を示す文字に "講師・メンター枠" が含まれるものを取り出す
if '講師・メンター枠' in participant_type:
menters_table = participation_table

This comment has been minimized.

@takanory

takanory Oct 23, 2017

Member
  • 英単語的には mentor なので直しといたほうが良いかと
  • if で見つけたら brake で抜けていんじゃね?

This comment has been minimized.

@laughk

laughk Oct 23, 2017

Collaborator

英単語的には mentor なので直しといたほうが良いかと

単純に間違ってしまったました。直します!

if で見つけたら brake で抜けていんじゃね?

確かにそのほうがよさそうですね。ここも反映します!

# menter_names は List
menter_names = menters_table.find_all(class_='display_name')
# 取り出した 講師・メンター枠の要素から純粋な名前だけを取り出す(不純なものを取り除く)

This comment has been minimized.

@takanory

takanory Oct 23, 2017

Member

不純っていうか、前後の無駄な空白や改行を取り除く。みたいな表現でよいのでは

This comment has been minimized.

@laughk

laughk Oct 23, 2017

Collaborator

確かに素直に表現したほうがいいですね。更新します!

@laughk

This comment has been minimized.

Collaborator

laughk commented Oct 23, 2017

@takanory レビューありがとうございます!
コメント頂いたところの対応が終わったのでマージします!

@laughk laughk merged commit 6a04da7 into pynyumon:master Oct 23, 2017

@laughk laughk deleted the laughk:add-scraping-sample branch Oct 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment