CesiumGS / CesiumGS/3d-tiles-validator
Streamline the validation of arrays of objects
- Dominant language
- TypeScript
- Stars
- 474
- Forks
- 156
- PR merge metrics
- No merged PRs in 30d
Description
This is a high-level (but low-priority) design issue, aiming at a possible refactoring, only to make the code a bit nicer.
Right now, there are many places where certain _arrays_ are validated, and the pattern that frequently appears is (in pseudocode):
```
const theArray = someObject.theArray;
if (defined(theArray)) {
if (!BasicValidator.validateArray(...)) {
result = false;
} else {
for (let i=0; i
Contributor guide
No contributing guide indexed for this repository
Research direction
No source files or tests are named. Start by locating BasicValidator and the existing array-validation call sites, then compare the repeated checks and element validators. Done should mean a settled design and a consistent refactoring that preserves the current validation rules.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100