作業通過，關於 academy 那題有些小建議

Hi Flora! 作業通過囉，以下是一些建議（不需在此次作業修改）

最後用 nameMapping 的方式滿好的，前面的實作有點複雜化了建議可以照這樣的方式實作

const colleges = {
  art: ["中國文學系", "外國語文學系", /*...*/],
  science: ["數學系", "物理學系", /*...*/],
  // ... 其他學院 ...
  d: ["創新領域學士學位學程"]
};

const calculateGroupedValues = (labels, values) => {
  // 宣告一個函式來處理 mapping
  const result = {
    art: 0, science: 0, // ... 其他學院初始值 ...
    d: 0
  };

  labels.forEach((label, index) => {
    Object.keys(colleges).forEach((key) => {
      if (colleges[key].includes(label)) {
        result[key] += values[index];
      }
    });
  });

  return result;
};

// 應用函式的方式如下
const groupedValues = calculateGroupedValues(labels, values);

後續就可以接到你原本的作法了！ btw，ChatGPT 也可以用來協助程式優化可以把程式碼貼給他詢問有什麼優化上的建議

非常能夠理解改動的原因，我自己在寫也有意識到要放進object做處理比較合理，謝謝ChouChou用心的檢視作業，下次我會多用心在程式碼的易讀性上~期待之後的見面

Sincerely, Flora

ChouChouHu @.***> 於 2023年9月11日週一下午5:34寫道：

Hi Flora! 作業通過囉，以下是一些建議（不需在此次作業修改）

最後用 nameMapping 的方式滿好的，前面的實作有點複雜化了建議可以照這樣的方式實作

const colleges = { art: ["中國文學系", "外國語文學系", /.../], science: ["數學系", "物理學系", /.../], // ... 其他學院 ... d: ["創新領域學士學位學程"] };

const calculateGroupedValues = (labels, values) => { // 宣告一個函式來處理 mapping const result = { art: 0, science: 0, // ... 其他學院初始值 ... d: 0 };

labels.forEach((label, index) => { Object.keys(colleges).forEach((key) => { if (colleges[key].includes(label)) { result[key] += values[index]; } }); });

return result; };

// 應用函式的方式如下 const groupedValues = calculateGroupedValues(labels, values);

後續就可以接到你原本的作法了！ btw，ChatGPT 也可以用來協助程式優化可以把程式碼貼給他詢問有什麼優化上的建議

— Reply to this email directly, view it on GitHub https://github.com/flora29058486/Appwork-front-end-HW/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFVGBYVZAS3G62MN27CERTXZ3LJDANCNFSM6AAAAAA4TAYNQA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

flora29058486 / Appwork-front-end-HW

作業通過，關於 academy 那題有些小建議 #1