DRY & KISS Principles

What issue with the KISS principle is demonstrated in this utility function implementation?

function formatData(data, options = {}) {
  const {
    dateFormat = 'yyyy-MM-dd',
    numberFormat = 'en-US',
    currency = 'USD',
    timezone = 'UTC',
    precision = 2,
    useGrouping = true,
    transformNulls = true,
    transformUndefined = true,
    customTransforms = {}
  } = options;

  const transformed = {};
  for (const [key, value] of Object.entries(data)) {
    if (value instanceof Date) {
      transformed[key] = formatDate(value, dateFormat, timezone);
    } else if (typeof value === 'number') {
      if (key.toLowerCase().includes('price') || key.toLowerCase().includes('amount')) {
        transformed[key] = formatCurrency(value, currency, numberFormat, precision, useGrouping);
      } else {
        transformed[key] = formatNumber(value, numberFormat, precision, useGrouping);
      }
    } else if (value === null && transformNulls) {
      transformed[key] = '';
    } else if (value === undefined && transformUndefined) {
      transformed[key] = '';
    } else if (customTransforms[key]) {
      transformed[key] = customTransforms[key](value);
    } else {
      transformed[key] = value;
    }
  }
  return transformed;
}

The function name is not descriptive enough
The function doesn't handle all cases
The function has too many parameters and complex logic
This code violates the KISS principle by implementing an overly complex function with too many options and branching logic. A better approach would be to break this down into smaller, focused functions: const formatters = { date: (value, options) => formatDate(value, options.format, options.timezone), currency: (value, options) => formatCurrency(value, options), number: (value, options) => formatNumber(value, options) }; function formatField(value, type, options = {}) { if (value == null) return options.transformNull ? '' : value; return formatters[type]?.(value, options) ?? value; } This approach: 1. Separates concerns into smaller, focused functions 2. Makes the code easier to understand and maintain 3. Reduces complexity in the main function 4. Makes it easier to add new formatters
The function doesn't validate inputs

Next Question (13/20)