ウェブスクレイピング 「href属性」と「alt属性」の取得について

Question

前提・実現したいこと

特定のサイトから「href属性」と「alt属性」含めた情報ををウェブスクレイピングで取得したい。
具体的には、下記の情報を取得します。
「パチンコORスロット,機種ID（href属性の一部）,NEW!（alt属性）,機種名、台数」
このうち、「機種ID（href属性の一部）」と「NEW!（alt属性）」以外は取得できていますが、
下記問題が発生しており、正確な情報を取得できておりません。
ご教授の程よろしくお願いいたします。

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

①href属性の取得
→href属性を取得後、一番後ろの数字を任意で取得しましたが、結果を見ると同じ数字になっている。
②alt属性の取得
→NEWの情報のある機種は情報を取得し、そうでない場合は「N」としていますが、結果を見ると全てNEWになっている。

該当コード

import requests
from bs4 import BeautifulSoup
from pprint import pprint
url = 'https://p-town.dmm.com/shops/tokyo/12670'
r = requests.get(url)
soup = BeautifulSoup(r.text, 'lxml')

data = {}
for ul in soup.select('ul[class="list-machinesettings"]'):
    machine_type = 'パチ'
    if 'パチ' in ul.h4.text:
        machine_type = 'パチンコ'
    elif 'スロ' in ul.h4.text:
        machine_type = 'スロット'

    machines = []
    for img in soup.select('i img'):
        new_machine = img.attrs.get('alt', 'N')
    for a in ul.select('a[class="link"]'):
        if 'href' in a.attrs:
            machine_url = a.attrs['href']
            machine_id = machine_url.rsplit('/', 1)[1]
        else:
            machine_url = '機種ID無'
    for li in ul.select('li[class="item"]'):
        name = li.select_one('div[class="text"]').get_text(strip=True)
        num = li.select_one('div[class="number"]').get_text(strip=True)
        machines.append([machine_type,machine_id,new_machine,name, num])
    data[machine_type] = machines

pprint(data)

試したこと

過去の履歴やQiitaを検索しましたが、どうしても上記2点の問題の解決には至りませんでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

おもな原因は、HTML階層の走査順に誤りがある（NEWや機種IDを全て取得後に機種名などを取りにいっている）からです。
修正するとしたら以下のような感じになるかと思います。

    # 略
        machine_type = 'パチンコ'
    elif 'スロ' in ul.h4.text:
        machine_type = 'スロット'

    machines = []
    for li in ul.select('li[class="item"]'):

        new_machine = 'N' # 初期化する！
        for img in li.select('i img'):
            new_machine = img.attrs.get('alt', 'N')

        for a in li.select('a[class="link"]'):
            if 'href' in a.attrs:
                machine_url = a.attrs['href']
                machine_id = machine_url.rsplit('/', 1)[1]
            else:
                machine_url = '機種ID無'

        name = li.select_one('div[class="text"]').get_text(strip=True)
        # 略

Comments

[nasu0922]

'li[class="item"]'を先頭にコードを書き直すのですね。大変参考になりました。
アドバイス通り修正し問題なく動作いたしました。
ご教授いただき誠にありがとうございました。