search

bitovi / querystring-parser

MIT License
8 stars 5 forks source link

Feat/backend 78 #33

Closed RoyAyo closed 2 years ago

RoyAyo commented 2 years ago

This commit includes BACKEND-78, expanding on examples It also includes a fix for ISSUE-32

michael-dean-haynie commented 2 years ago

As general feedback - I think it's better to mostly avoid ternary statements (especially nested ones). When you're very familiar with the code it seems neat and compact. But to anyone else I believe it makes it less readable and mentally straining to figure out what's going on.

Accomplishing the task with several simple-to-understand statements is usually better for readability than getting everything done in a single statement.

My humble opinion - happy to discuss :)