30

If the code was to work properly then whenever someone types something in the chat they get 5 experience and that information gets put into a .json file, but instead what happens is whenever someone types something into the chat it gives me this error.

on_message users = json.dumps(f) 
TypeError: Object of type TextIOWrapper is not JSON serializable

Here is the code that I am using:

import discord
from discord.ext import commands
from discord.ext.commands import Bot
import asyncio
import json
from json import dumps, loads, JSONEncoder, JSONDecoder
import os

client = commands.Bot(command_prefix='^')
os.chdir(r'C:\Users\quiny\Desktop\sauce')

@client.event
async def on_ready():
    print ("Ready when you are xd")
    print ("I am running on " + client.user.name)
    print ("With the ID: " + client.user.id)

@client.event
async def on_member_join(member):
    with open('users.json', 'r') as f: 
        users = json.dumps(f)

    await update_data(users, member)

    with open('users.json', 'w') as f:
        json.loads("users, f")

@client.event
async def on_message(message):
    with open('users.json', 'r') as f:
        users = json.dumps(f)

    await update_data(users, message.author)
    await add_experience(users, message.author, 5)
    await level_up(users, message.author, message.channel)

    with open('users.json', 'w') as f:
        json.loads("users, f")

async def update_data(users, user):
    if not user.id in users:
        users[user.id] = {}
        users[user.id]['experience'] = 0
        users[user.id]['level'] = 1

async def add_experience(users, user, exp):
    users[user.id]['experience'] += exp

async def level_up(users, user, channel):
    experience = users[user.id]['experience']
    lvl_start = users[user.id]['level']
    lvl_end = int(experience ** (1/4))

    if lvl_start < lvl_end:
        await client.send_message(channel, '{} has achieved a slightly higher 
level of {}, yay'.format(user.mention, lvl_end))
        users[user.id]['level'] = lvl_end
Taku
  • 31,927
  • 11
  • 74
  • 85
kybt
  • 319
  • 1
  • 3
  • 8
  • 22
    FWIW: I got that very error because I accidentally swapped the arguments to `json.dump()`, e.g. I used `json.dump(fp, obj)` instead of the proper `json.dump(obj, fp)`. – jlh Dec 29 '18 at 18:35

2 Answers2

32

I had this error and I used the json.dump the wrong way around:

Correct way:

with open("Jello.json", "w") as json_file:
    json.dump(object_to_be_saved, json_file)

Wrong way:

I did it the other way around json.dump(json_file, object_to_be_saved)... Thanks Python.

Nagabhushan S N
  • 6,407
  • 8
  • 44
  • 87
Ben Butterworth
  • 22,056
  • 10
  • 114
  • 167
20

You have your loads and dumps backwards, and you should be using load and dump instead (the s suffix means those functions work on strings.). load from a file, dump to a file

users = {}

@client.event
async def on_message(message):
    # No need to load the dictionary, our copy is the most correct
    await update_data(users, message.author)
    await add_experience(users, message.author, 5)
    await level_up(users, message.author, message.channel)
    with open('users.json', 'w') as f:
        json.dump(users, f)

@client.event
async def on_ready():
    print ("Ready when you are xd")
    print ("I am running on " + client.user.name)
    print ("With the ID: " + client.user.id)
    # Load the json just once, when the bot starts
    global users
    with open('users.json') as f:
        try:
            users = json.load(f)
        except:
            users = {}
Patrick Haugh
  • 59,226
  • 13
  • 88
  • 96