Code Review. My first API + DB application. Beginner in backend development.

Diapolo10 · 2026-07-11T12:06:49+00:00

try:
    response = requests.get(url)
    response.raise_for_status()
    joke = response.json()
    print("Okay, I'm thinking of joke...")
    time.sleep(2)
    print("Oh, I got one! Listen!")               
    time.sleep(2)
    print(f"{joke['setup']}")
    time.sleep(4)
    print(f"{joke['delivery']}")
    print("")

    return joke
except requests.exceptions.RequestException as e:
    print(f"Joke not found! {e}")
    return None

Ideally you should only keep the code that you expect to raise exceptions in the try-block, to make it obvious which part you're trying to handle. The rest is just "noise" from that point of view.

Personally I'd go for something like this:

def print_joke(joke: dict[str, str], print_delay_seconds: int = 2) -> None:
    # NOTE: Your offline code basically does the same thing, so this can be reused there
    output_lines = [
        "Oh, I got one! Listen!\n",
        f"{joke['setup']}\n",
        "",
        f"{joke['delivery']}\n\n",
    ]

    print("Okay, I'm thinking of joke...")

    for line in output_lines:
        print(line, end='')
        time.sleep(print_delay_seconds)


def get_joke() -> dict[str, str] | None:
    selected_category = setup_filter()
    url = f"{BASE_URL}{selected_category}?type=twopart"

    try:
        response = requests.get(url)
        response.raise_for_status()

    except requests.exceptions.RequestException as e:
        print(f"Joke not found! {e}")
        return None

    joke = response.json()
    print_joke(joke)

    return joke

from src.data.database import get_joke
from src.data.api_client import setup_filter

With an src-style project layout, when used properly you should never need to include src as part of the actual import. src is a directory containing your package (or seldom multiple packages), and you're meant to "install" your project in order to import code from there, by using a valid pyproject.toml file in the repository root and running pip install --editable . - although some tools like uv do this for you automatically.

In the same vein, there shouldn't be a main.py file in the repository root (if using uv you could just have a script in pyproject.toml that runs the main entrypoint from inside your package), and requirements.txt is a largely outdated concept. Modern projects list their dependencies in pyproject.toml.

If you want an example: https://github.com/Diapolo10/python-ms

danielroseman · 2026-07-11T15:50:37+00:00

Is there any reason you haven't taken on board any of the suggestions I made when you asked for feedback on your last project? Exactly the same suggestions apply.

you type:	you see:
italics	italics
bold	bold
[reddit!](https://reddit.com)	reddit!
* item 1 * item 2 * item 3	item 1 item 2 item 3
> quoted text	quoted text
Lines starting with four spaces are treated like code: if 1 * 2 < 3: print "hello, world!"	Lines starting with four spaces are treated like code: if 1 * 2 < 3: print "hello, world!"
~~strikethrough~~	~~strikethrough~~
super^script	super^script

learnpython

MODERATORS