有没有办法简化这段代码,使用 javascript 为我网页上的特定断点创建 css 变量?

Is there a way to simplify this code creating css variables with javascript for specific breakpoints on my webpage?

提问人:Jonas Q. 提问时间:11/3/2023 更新时间:11/3/2023 访问量:53

问:

我正在使用这一段 javascript 代码创建一个包含 css 变量的字符串。 它帮助我专门为网页上的每个断点设置大小。

这是我的代码:

const sizes = {
  xs: 0.9,
  md: 2,
  lg: 2.5,
  xl: 4
};

function getCssVar(size) {
  var varStr = "";

  const defaultBPs = ["xs", "sm", "md", "lg", "xl"];

  defaultBPs.forEach(bp => {
    const bpi = defaultBPs.indexOf(bp);
    var nextHigherIndex = defaultBPs.indexOf(Object.keys(size)[getNextHighestIndex(Object.keys(size).map(bp => (defaultBPs.indexOf(bp))), defaultBPs.indexOf(bp))]);

    if (getNextHighestIndex(Object.keys(size).map(bp => (defaultBPs.indexOf(bp))), defaultBPs.indexOf(bp)) === Object.keys(size).length) {
      nextHigherIndex = defaultBPs.length;
    }

    if (size[bp]) {
      defaultBPs.forEach(bp2 => {
        const bp2i = defaultBPs.indexOf(bp2);
        const generateVar = " --size-" + bp2 + ":" + size[bp] + ";";

        if (bp2i >= bpi && bp2i < nextHigherIndex) {
          varStr += generateVar;
        }
        
      })
    };

  });
  return varStr;
}

function getNextHighestIndex(arr, value) {
  var i = arr.length;
  while (arr[--i] > value);
  return ++i; 
}


console.log(getCssVar(sizes));

输出如下:

 --size-xs:0.9; --size-sm:0.9; --size-md:2; --size-lg:2.5; --size-xl:4;

从技术上讲,这是正确的答案,并且此代码已经有效,但我仍然想知道是否有任何方法可以改进我的代码。

我希望它更短、更紧凑,但输出相同。

任何帮助将不胜感激。 谢谢。

JavaScript 函数 断点 简化

评论

0赞 Nina Scholz 11/3/2023
如果你已经有一个索引,你为什么要使用 ?indexOfforEach
3赞 Barmar 11/3/2023
如果代码有效,并且您正在寻找有关改进它的建议,那么代码审查是合适的地方。但请看 codereview.meta.stackexchange.com/questions/5777/......首先。
0赞 Wyck 11/3/2023
“我希望它更短、更紧凑,但输出相同。”哦,天哪。你问了Code Golf。:/

答:

1赞 tarkh 11/3/2023 #1

我不知道我是否正确,但将这种逻辑视为一种选择......检查内联注释

//
// Main function
const getCssVar = (s, last = 0) => {
  // Possible modes in proper sequence
  const modes = ["xs", "sm", "md", "lg", "xl", "xxl"];
  // Loop modes and check if value has been provided for current mode
  // If no value provided, put last (0 or any passed overrider)
  // And update last with new value
  const res = {};
  for(const m of modes) last = res[`--size-${m}`] = s[m] || last;
  // Transform res object to any needed string
  const str = Object.keys(res).map(key => `${[key, res[key]].join(':')};`).join('');
  // Return result
  return str;
}

//
// Tests
console.log("Test 1:", getCssVar({
  md: 2,
  lg: 2.5,
  xl: 4
}, 1.33)); // here we override last (lower) value

console.log("Test 2:", getCssVar({
  md: 2,
  xl: 4
}));

console.log("Test 3:", getCssVar({
  xs: 0.9,
  md: 2,
}));