// bad
const createArticle = (title, author, date, content) => { }
createArticle('震驚，一男子竟偷偷干這事', 'zhangnan', '2020/06/29', '某天深夜，我喝多了點(diǎn)酒...')

// good
const createArticle = ({title, author, date, content}) => { }
createArticle({
 title: '震驚，一男子竟偷偷干這事'，
 author: 'zhangnan',
 date: '2020/06/29',
 content: '某天深夜，我喝多了點(diǎn)酒...'
})

// bad 代碼擠成一堆,很難理清
// 男生女生的通知方式還有所不同，后期如果要改動(dòng)女生的通知方式，很難保證不會(huì)影響到男生
// 因?yàn)榇蠹叶紝懺谕粋€(gè)函數(shù)里

const notifyStudents = (studentList) => {
 studentList.forEach(student => {
  if (student.gender === 'male') {
   const sender1 = new SmsSender({ carrier: '電信' });
   sender1.init();
   sender1.sendTo(student) 
  } else {
   const sender2 = new SmsSender({ carrier: '聯(lián)通' });
   sender2.init();
   sender2.sendTo(student);
   
   const sender3 = new EmailSender({ type: 'QQ郵箱' });
   sender3.connect();
   sender3.sendTo(student)
  }
 })
}


// good 函數(shù)拆分，各司其職，清晰明了
// 雖然看起來(lái)代碼量多了一點(diǎn)點(diǎn)
// 但是分工明確，互不影響
const initSmsSender = (carrier) => {
 const sender = new SmsSender({ carrier });
 sender.init();
}

const initEmailSender = (type) => {
 const sender = new EmailSender({ type });
 sender.connect();
}

const notifyMales = (studentList) => {
 const smsSender = initSmsSender('電信');
 const maleList = studentList.filter(student => student.gender === 'male');
 
 maleList.forEach(male => smsSender.sendTo(male));
}

const notifyFemales = (studentList) => {
 const smsSender = initSmsSender('聯(lián)通');
 const emailSender = initEmailSender('QQ郵箱');
 
 const femaleList = studentList.filter(student => student.gender === 'female');
 
 femaleList.forEach(female => {
  smsSender.sendTo(female);
  emailSender.sendTo(female);
 })
}

// bad
const shouldIBuyThisPhone = (phone) => {
 const {price, year, brand} = phone;
 if (price > 5000 && year === new Date.getFullYear() && brand === 'huawei') {
  // 馬上剁手
 }
}

// good
const isHuaweiFlagShipThisYear = ({ price, year, brand }) => {
 const HIGH_PRICE = 5000;
 return price > HIGH_PRICE && year === new Date.getFullYear() && brand === 'huawei'
}

const shouldIBuyThisPhone = (phone) => {
 if (isHuaweiFlagShipThisYear(phone)) {
  // 馬上剁手
 }
}

// bad 需要判斷標(biāo)志變量，同時(shí)做出不同的相應(yīng)動(dòng)作
const print = (person) => {
 if (person === 'A') {
  device.print({ 
   page: 1, 
   color: 'gray', 
   orientation: 'landscape' 
   
  })
 }
 
 else if (person === 'B') {
  device.print({ 
   page: 1, 
   color: 'colorful', 
   orientation: 'vertical' 
  })
 }
 
 else if (person === 'C') {
  device.print({ 
   page: 2, 
   color: 'colorful' ,
   orientation: 'landscape'
  })
 }
 
 ......
 
}


// good
const print = (config) => {
 device.print(config)
}