Посоветуйте API

Публичное апи сделал бы так

from json_schema import get_schema
user = get_schema('user')

Так надо еще путь к папке где-то указать

Я для таких задач использую синглетон с именем Environment, который подгружается из своего конфигурационного файла и уже дергается модулями.

Оно вообще работает у тебя?

А в чем проблема указывать «project_dir / 'json_schema'» в json_schema (ну, точнее в get_schema, ну или там по импорту этому продумать нормально)? Плюс ты можешь там это указать как значение переменной, и если надо менять то после импорта пишешь что-то вроде «get_schema.SCHEMAS_PATH = '/foo/bar/1234/'».

ЯННП)

Конечно. А почему нет

из def load_schemas лучше класс сделать, чтобы сократить «лесенку». Да и if p.is_file(): можно заменить на if not p.is_file(): continue

Ну а глобальная переменная зло ( учитывая что ты ее неправильно используешь -надо бы обьявить global внутри функции ). Лучше отдавать выходным аргументом функции. Да и вообще непонятна причина объявления schem снаружи функции в этом случае.

учитывая что ты ее неправильно используешь - надо бы обьявить global внутри функции

Можно и объявить. Но в данном случае никакие линтеры не ругаются (ругаются только если присваивать к самой переменной, без квадратных скобочек)

Да и вообще непонятна причина объявления schem снаружи функции в этом случае

Для прикола) Указываешь тип переменной, и vscode сразу начинает относиться к ней с уважением, правильное автодополнение выдавать. Подчеркивать, если используешь ее неправильно

Лучше отдавать выходным аргументом функции

Кого куда отдавать?

./json_schema/load_schemas.py

import os
import json

SCHEMA_DIR='/tmp/json_schema/'

def load(json_files_dir=SCHEMA_DIR):
    for root, dirs, files in os.walk(json_files_dir):  
        for filename in files:
            with open(filename) <... ну и вот это все с чтением json ...>

main.py

from json_schema import load_schemas

# Получаем данные из файлов в дефолтном /tmp/json_schema/
data = load_schemas.load()

# Получаем данные из файлов в каталоге /foo/bar/1234/
data = load_schemas.load('/foo/bar/1234/')

# А теперь тот же каталог /foo/bar/1234/, только меняем дефолтное значение
load_schemas.SCHEMA_DIR='/foo/bar/1234/'
data = load_schemas.load()

Это ты путь к схемам прям в модуль захардкодил?)

Указываешь тип переменной

Зачем делать какую-то ерунду из нормального языка?!

путь к схемам обычно пишут в конфиг файле, если что. Так что:

import config
schemas_dir = config.dir_schema

Конфиг есть. В общем, решил не разводить больше модулей с внутренним стейтом, а тупо загрузить схемы в конфиг, который уже импортировать в другие модули

https://www.youtube.com/watch?v=WaqB4Jz7vCM

https://www.youtube.com/watch?v=ePW-kVKB4pw

Твоя «глобальная переменная» это переменная модуля, если ты его импортируешь. Второе - каждый раз как ты импортируешь модуль, создаётся новый объект типа «модуль» со своими собственными переменными, не зависящими от других импортов.

# Позже, в других модулях
from json_schema import schema
schema['user']

schema всегда будет пустым словарём.

Я не понял, как ты собрался передавать данные «между модулями»?

Я не понял, как ты собрался передавать данные «между модулями»?

🤷

А, всё. Я-то думал что это я чего-то не понимаю, но оказалось, что это у тебя проблемы с выражением мыслей.

каждый раз как ты импортируешь модуль, создаётся новый объект типа «модуль» со своими собственными переменными, не зависящими от других импортов.

user@home:~$ cat one.py
NO_U = {}

user@home:~$ cat two.py
from one import NO_U
print(NO_U)

user@home:~$ cat main.py 
from one import NO_U
NO_U['Нет ты'] = 'Не понимаешь'
import two

user@home:~$ python3 main.py 
{'Нет ты': 'Не понимаешь'}

Ну вот. Сразу бы так. Спасибо :)

👌

Хватит мыслить категориями nodejs. В python для этого существуют классы.

В ноде они тоже давно существуют

В конце концов сделал так:

from pathlib import Path
from typing import Any, Dict, Union
from json import load

def load_schemas(schemas_dir: Union[Path, str]) -> Dict[str, Any]:
    schema = {}
    for p in Path(schemas_dir).iterdir():
        if not p.is_file(): continue
        with p.open() as f:
            schema[p.stem] = load(f)
        
    return schema

# config.py
from typing import Dict, Any
from pathlib import Path

from json_schema import load_schemas

project_path = Path(__file__).parent
json_schema_path = project_path / 'json_schema'

schemas: Dict[str, Any]

def _init():
    global schemas
    schemas = load_schemas(json_schema_path)

https://stackoverflow.com/questions/19158339/why-are-global-variables-evil?ut...