diff --git a/web/src/pages/SvodPage/SummaryTable.jsx b/web/src/pages/SvodPage/SummaryTable.jsx index c00da85..0939371 100644 --- a/web/src/pages/SvodPage/SummaryTable.jsx +++ b/web/src/pages/SvodPage/SummaryTable.jsx @@ -8,6 +8,7 @@ import { buildRowHierarchy } from '../../components/RealtimeTable/utils/rowUtils import { createFixedRowVirtualizerOptions, getRootFontSizePx } from '../../components/RealtimeTable/utils/virtualizerOptions'; import DenominationSelect from '../../components/common/DenominationSelect'; import useDenomination from '../../components/common/useDenomination'; +import { formatSummaryAmount } from './summaryFormat'; const tableSize = (rem) => rem * 16; // Высота строки задана в rem явно (строка и ячейка), виртуализатор layout не читает. @@ -78,10 +79,9 @@ const scaleColumns = (columns, factor, denomination) => Cell: ({ cell }) => { const value = cell.getValue(); if (value === null || value === undefined || value === '') return originalCell ? originalCell({ cell }) : '—'; - const numeric = Number(value); - if (!Number.isFinite(numeric)) return originalCell ? originalCell({ cell }) : value; - if (numeric === 0) return '-'; - return new Intl.NumberFormat('ru-RU', { maximumFractionDigits: 2 }).format(numeric * factor); + const formatted = formatSummaryAmount(value, factor); + if (formatted !== null) return formatted; + return originalCell ? originalCell({ cell }) : value; }, } : {}), diff --git a/web/src/pages/SvodPage/summaryFormat.js b/web/src/pages/SvodPage/summaryFormat.js new file mode 100644 index 0000000..7038a50 --- /dev/null +++ b/web/src/pages/SvodPage/summaryFormat.js @@ -0,0 +1,10 @@ +const amountFormatter = new Intl.NumberFormat('ru-RU', { maximumFractionDigits: 2 }); + +// Возвращает null, если значение не число: вызывающий отдаёт исходное значение. +export const formatSummaryAmount = (value, factor) => { + if (value === null || value === undefined || value === '') return null; + const numeric = Number(value); + if (!Number.isFinite(numeric)) return null; + if (numeric === 0) return '-'; + return amountFormatter.format(numeric * factor); +}; diff --git a/web/src/pages/SvodPage/summaryFormat.test.js b/web/src/pages/SvodPage/summaryFormat.test.js new file mode 100644 index 0000000..668d68a --- /dev/null +++ b/web/src/pages/SvodPage/summaryFormat.test.js @@ -0,0 +1,37 @@ +import assert from 'node:assert/strict'; +import test from 'node:test'; + +const NBSP = ' '; + +test('formatSummaryAmount: форматирование и множители', async () => { + const { formatSummaryAmount } = await import('./summaryFormat.js'); + assert.equal(formatSummaryAmount(0, 1), '-'); + assert.equal(formatSummaryAmount('0', 1e-3), '-'); + assert.equal(formatSummaryAmount(1234.567, 1), `1${NBSP}234,57`); + assert.equal(formatSummaryAmount(5_000_000, 1e-3), `5${NBSP}000`); + assert.equal(formatSummaryAmount(5_000_000, 1e-6), '5'); + assert.equal(formatSummaryAmount('1500', 1), `1${NBSP}500`); +}); + +test('formatSummaryAmount: нечисловые значения возвращают null', async () => { + const { formatSummaryAmount } = await import('./summaryFormat.js'); + for (const v of [null, undefined, '', 'abc', Number.NaN, Number.POSITIVE_INFINITY]) { + assert.equal(formatSummaryAmount(v, 1), null); + } +}); + +test('Intl.NumberFormat создаётся один раз на модуль', async () => { + const Original = Intl.NumberFormat; + let created = 0; + Intl.NumberFormat = function CountingNumberFormat(...args) { + created += 1; + return new Original(...args); + }; + try { + const { formatSummaryAmount } = await import('./summaryFormat.js?fresh'); + for (let i = 1; i <= 50; i += 1) formatSummaryAmount(i * 100, 1e-3); + assert.equal(created, 1); + } finally { + Intl.NumberFormat = Original; + } +});