CesiumGS / CesiumGS/3d-tiles-validator

Streamline the validation of arrays of objects

Open
#281 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.