json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0) 循环时

Question

import requests
from bs4 import BeautifulSoup as bs4
from threading import Thread
import json
import time
from datetime import datetime
import _thread

webhook = ("https://hooks.slack.com/services/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")

s = requests.session()

payload={"channel": "#bot-monitor", "username": "Astro-bot", "text": "<https://www.astro-bot.io/astro/buyme-asdf-j4wrj-ypzkf|Astro Bot Restock! Click to Purchase>", "icon_emoji": ":rocket:"}


response = s.get('https://www.astro-bot.io/api/commerce/inventory/stock/?crumb=BQXy6KNmMGWENWE3YWQzMTc3MDU5NzE1ODdkNDNiM2RmZTEyNjg4&itemId=5b0dc9172b6a283905dabe08')
data = json.loads(response.text)
stock = data['results'][0]['qtyInStock']

def checkstock():
    response = s.get('https://www.astro-bot.io/api/commerce/inventory/stock/?crumb=BQXy6KNmMGWENWE3YWQzMTc3MDU5NzE1ODdkNDNiM2RmZTEyNjg4&itemId=5b0dc9172b6a283905dabe08')
    data = json.loads(response.text)
    stock = data['results'][0]['qtyInStock']
    print(stock)


def post():
    global stock
    if stock in [0]:
        print("Out of stock! LOL!")
        time.sleep(3)
        checkstock()

    else:
        if stock > (0):
        s.post(webhook, json=payload)


def main():
    checkstock()
    post()



if __name__ == '__main__':
    main()

代码运行一次，然后当它再次检查库存时，我得到：

从 None 提高 JSONDecodeError("Expecting value", s, err.value) json.decoder.JSONDecodeError：预期值：第 1 行第 1 列（字符 0）。

请提出建议。

Answer 1

我努力保持礼貌，但代码的 sn-p 是这样一个反模式（全局、重复代码、函数外代码、不良流控制）的集合，因此几乎没有动力进行调试。你越早、越深入地重构代码，对你的项目就越好。

话虽如此，我认为较短的代码可以完成请求的任务，它没有JSONDecodeError：

import requests
import time

ASTRO_URL = 'https://www.astro-bot.io/api/commerce/inventory/stock/?crumb=BQXy6KNmMGWENWE3YWQzMTc3MDU5NzE1ODdkNDNiM2RmZTEyNjg4&itemId=5b0dc9172b6a283905dabe08'
WEBHOOK_URL = 'https://hooks.slack.com/services/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'

def qty_in_stock(url):
    data = requests.get(url).json()
    return data['results'][0]['qtyInStock']    

def do_something():
    pass

def main():
    stock = qty_in_stock(ASTRO_URL)
    if stock == 0:
        print('out ot stock!')
        time.sleep(3)
        stock = qty_in_stock(ASTRO_URL)
        print(stock)
    else:
        do_something()

if __name__ == '__main__':
    main()