[코드 리뷰] 8주차 async / parallel, watalfall

구체적인 변수명

• 가독성을 위해 좀 더 구체적으로 변수명 짓기
• var 사용 지양

export default function parallel(tasks, finalCallback) {
  var results = [];
  ❌ var count = 0;

↓

✅ const taskCount = 0;

---------------------------------------------------

❌countNumber -> ✅increaseCount

-----------------------------------------------------


❌ function callback(tasksIndex, countNumber)  {
    tasks[tasksIndex](function (value) {
    results[tasksIndex] = value;


✅ function callback(index, countNumber) {
    tasks[index](function (result) {
    results[index] = result;

• tasks[tasksIndex]의 task가 반복되기 때문에 아래와 같이 정리하는 것이 더 깔끔하고 명료.
• callback 인자는 비동기처리에 대한 결과 값이 인자로 오기 때문에 result라고 표기하는 것이 더 옳음.
  
  

boolean 간략하게 표기

export default function parallel(tasks, finalCallback) {
  const results = [];
  const count = 0;

  ❌ if (tasks.length === 0) {
  
  ✅ if (!tasks.length) {

한 줄 처리, early return

 if (tasks.length === 0) {
    return finalCallback(results);
  }


↓

 if (tasks.length === 0) return finalCallback(results);