Files
Forge-Engine/static/game/js/play/offline/compute-operation.js
T
williamandClaude Sonnet 5 b2e933f322
Build and deploy / test-python (push) Successful in 11m12s
Build and deploy / test-js (push) Successful in 53s
Build and deploy / lint-python (push) Successful in 3m56s
Build and deploy / lint-js (push) Successful in 3m1s
Build and deploy / build-and-push (push) Skipped
Build and deploy / deploy (push) Skipped
Build and deploy / sonarqube (push) Failing after 3m58s
Reorganisation game/document : renommage screens->game_engine + sous-dossiers game/ dans routes, scripts, static, templates, tests
Prepare la scission a venir entre l'editeur Jeu 2D et le futur editeur
Support de formation (voir docs/plan/PLAN.md), sans toucher a
l'architecture en couches existante :

- screens/ renomme en game_engine/ (nom clair pour le moteur du jeu 2D,
  avant l'arrivee d'un second "moteur" cote document) : ~85 imports
  corriges, contrat import-linter mis a jour, meme forme de couches.
- routes/, scripts/, static/, templates/, tests/ : tout ce qui est
  propre au jeu 2D deplace dans un sous-dossier game/ de chacun
  (routes/game/, static/game/, templates/game/, tests/game/,
  scripts/game/) ; ce qui est partage par le site (auth, onboarding,
  dashboard, uploads, db/) reste a la racine de chaque dossier. Un
  sous-dossier document/ (vide) cree dans chacun pour le futur chantier.
- styles/ volontairement inchange : les 3 fichiers sources sont
  concatenes en un seul static/style.css charge par tout le site,
  scinder leur CONTENU (editeur vs partage) serait un refactor CSS
  distinct, pas un deplacement mecanique.
- Chaine d'export SCORM (publish/build_scorm_package.py) mise a jour en
  profondeur : copie des assets, URLs d'icones relatives a
  static/style.css (qui ne bouge pas), manifeste, wrapper SCORM.
- Deux regressions d'un sweep de renommage anterieur corrigees au passage
  (screens.js/screens/scene-objects incorrectement convertis en
  game_engine.js/game_engine/scene-objects dans des commentaires).
- Effet de bord Windows decouvert et corrige : git mv + Path.write_text
  convertissent des fichiers en CRLF (core.autocrlf=true) - ~189 fichiers
  normalises en LF.
- .eslintrc.json/package.json : uniquement les chemins de glob mis a jour
  (static/game/js/...) ; la preparation eslint-plugin-unicorn du lot 7
  reste volontairement non committee (package-lock.json restaure a la
  version precedente).

Verifications : ruff, mypy --strict (391 fichiers), vulture, bandit,
lint-imports tous verts ; 591/591 tests Python, 276/276 tests JS ;
demarrage serveur + requetes HTTP manuelles confirmant que les assets
deplaces repondent en 200 au nouvel emplacement et 404 a l'ancien.

SKIP=djlint : backlog H021 (styles inline) deja documente comme dette
assumee dans CODE_QUALITY.md section 6, aucun template touche par ce
commit au-dela d'un deplacement de fichier.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-09-19 12:27:53 +02:00

103 lines
5.1 KiB
JavaScript

// Export Web/SCORM (voir le plan "port complet du runtime jouable côté
// navigateur") — port fidèle de game_engine/data_actions/compute_operation.py,
// utilisé UNIQUEMENT quand window.FORGE_OFFLINE est vrai (paquet exporté,
// aucun serveur disponible) — voir apply-actions.js, les deux seuls
// endroits qui en ont besoin. Toute divergence avec la version Python doit
// être volontaire et commentée (ex. modulo/int()).
// int() Python tronque vers zéro — Math.trunc fait exactement ça
// (contrairement à Math.floor, qui arrondit vers -l'infini).
function forgePyInt(n) {
return Math.trunc(n);
}
// Le signe du résultat de "%" suit le DIVISEUR en Python, le DIVIDENDE en
// JS — sans ce correctif, "modulo" donnerait un résultat différent de
// compute_operation.py dès qu'une des deux valeurs est négative.
function forgePyMod(a, b) {
return ((a % b) + b) % b;
}
function computeDefinirNombre(rawValue, round) {
const n = Number.parseFloat(rawValue);
return Number.isNaN(n) ? 0 : round(n);
}
// Partagée entre "incrementer" et "decrementer" (même calcul, signe
// inversé) — extrait de forgeComputeNewValue (lot 4 "modernisation JS", S3776).
function computeIncrementerOuDecrementer(operation, current, rawValue, round) {
let amount = (rawValue === null || rawValue === undefined || rawValue === '') ? 0 : Number.parseFloat(rawValue);
if (Number.isNaN(amount)) amount = 0;
if (operation === 'decrementer') amount = -amount;
return round((current || 0) + amount);
}
function computeMultiplier(current, rawValue, round) {
let factor = (rawValue === null || rawValue === undefined || rawValue === '') ? 1 : Number.parseFloat(rawValue);
if (Number.isNaN(factor)) factor = 1;
return round((current || 0) * factor);
}
// Partagée entre "diviser" et "modulo" — extrait de forgeComputeNewValue.
function computeDiviserOuModulo(operation, current, rawValue, round) {
let divisor = (rawValue === null || rawValue === undefined || rawValue === '') ? 0 : Number.parseFloat(rawValue);
if (Number.isNaN(divisor)) divisor = 0;
if (divisor === 0) return current;
const result = operation === 'diviser' ? (current || 0) / divisor : forgePyMod(current || 0, divisor);
return round(result);
}
// Partagée entre "minimum" et "maximum" — extrait de forgeComputeNewValue.
function computeMinimumOuMaximum(operation, current, rawValue, round) {
const bound = Number.parseFloat(rawValue);
if (Number.isNaN(bound)) return current;
const res = operation === 'minimum' ? Math.max(current || 0, bound) : Math.min(current || 0, bound);
return round(res);
}
function computeAlea(rawValue, isDecimal) {
const parts = String(rawValue || '').split(',');
let lo = Number.parseFloat(parts[0]); let
hi = Number.parseFloat(parts[1]);
if (Number.isNaN(lo)) lo = 0;
if (Number.isNaN(hi)) hi = 0;
if (lo > hi) { const tmp = lo; lo = hi; hi = tmp; }
if (isDecimal) return Math.random() * (hi - lo) + lo; // NOSONAR S2245 - tirage de jeu, jamais crypto
const iLo = Math.trunc(lo); const
iHi = Math.trunc(hi);
return Math.floor(Math.random() * (iHi - iLo + 1)) + iLo; // NOSONAR S2245 - tirage de jeu, jamais crypto
}
// Dispatch par nom d'opération — chaque fonction ci-dessus reçoit déjà
// tout ce dont elle a besoin (current/rawValue/round/isDecimal), ce
// dispatcher n'a donc plus qu'une seule décision (présence d'un
// handler), au lieu d'empiler ici les conditions de chaque branche (même
// méthode que runActionNode, static/game/js/play/actions.js).
const OPERATION_HANDLERS = {
definir_texte: (current, rawValue) => rawValue || '',
definir_nombre: (current, rawValue, round) => computeDefinirNombre(rawValue, round),
definir_bool_vrai: () => 1,
definir_bool_faux: () => 0,
basculer_bool: (current) => (current ? 0 : 1),
incrementer: (current, rawValue, round) => computeIncrementerOuDecrementer('incrementer', current, rawValue, round),
decrementer: (current, rawValue, round) => computeIncrementerOuDecrementer('decrementer', current, rawValue, round),
multiplier: (current, rawValue, round) => computeMultiplier(current, rawValue, round),
diviser: (current, rawValue, round) => computeDiviserOuModulo('diviser', current, rawValue, round),
modulo: (current, rawValue, round) => computeDiviserOuModulo('modulo', current, rawValue, round),
minimum: (current, rawValue, round) => computeMinimumOuMaximum('minimum', current, rawValue, round),
maximum: (current, rawValue, round) => computeMinimumOuMaximum('maximum', current, rawValue, round),
alea: (current, rawValue, round, isDecimal) => computeAlea(rawValue, isDecimal),
};
function forgeComputeNewValue(operation, current, rawValue, isDecimal) {
const round = function (v) { return isDecimal ? v : forgePyInt(v); };
const handler = OPERATION_HANDLERS[operation];
if (!handler) throw new Error(`opération inconnue : ${operation}`);
return handler(current, rawValue, round, isDecimal);
}
// static/game/js/play/__tests__/ (node:test) — même patron que conditions.js.
if (typeof module !== 'undefined' && module.exports) {
module.exports = { forgeComputeNewValue, forgePyInt, forgePyMod };
}