Suggestions for overlapping functionality between Python Endpoints [closed]

Question

Closed. This question is opinion-based. It is not currently accepting answers.

Want to improve this question? Because this question may lead to opinionated discussion, debate, and answers, it has been closed. You may edit the question if you feel you can improve it so that it requires answers that include facts and citations or a detailed explanation of the proposed solution. If edited, the question will be reviewed and might be reopened.

Closed 9 months ago.

Improve this question

Currently I'm working on FastAPI endpoints in Python which basically handle two types of entities:

/run/entityA
/run/entityB

Now based on the type of entity, some of the arguments change but a lot of the processing of other arguments overlap.

Current approach:

@router.post("/run/entityA")
def _run_a(args_a):
    # Common processing
    # Entity A related processing
    # Common processing

@router.post("/run/entityB")
def _run_a(args_b):
    # Common processing
    # Entity B related processing
    # Common processing

Previous Approach

Previously, I kept one set of args, keeping the entity related args as Optional in FastAPI and doing an if check inside the function:

@router.post("/run")
def _run(args):
   # Common Processing
   if args.entityA_arg:
   # Entity A related processing
   if args.entityB_arg:
   # Entity B related processing
   # Common processing

I see downsides in both. What are some suggestions to make the code cleaner?

maybe common processing put in separate function and execute it in /run/entityA and /run/entityB — furas
– furas, Commented Feb 25 at 4:49
@Shubham give a path parameter answer, which is greate, and for your information in FastAPI the order of path operation is matter — timothy jeong
– timothy jeong, Commented Feb 25 at 14:12

Shubham · Accepted Answer · 2025-02-25 09:51:25Z

1

@router.post("/run/{entity}") 
def _run_(entity:str):
 if entity = a: 
  #for a
 else:
  #for b

You can try this beautiful way.

answered Feb 25 at 9:51

Shubham

1031 silver badge7 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Suggestions for overlapping functionality between Python Endpoints [closed]

1 Answer 1

Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Related