if (condition) {
  action();
}

// 코드의 로직이 명확하게 종속적으로 나뉘는 코드를 쓸 때, 
// 종속된 code block은 주인 code block보다 두 칸 들여쓰기 합니다.

if (condition) {
action();
}

// 아래 코드의 들여쓰기를 수젖ㅇ하여 GOOD으로 옮기기
transmogrify({
   a: {
    b: function(){
    }
}});
// code block이 바뀌고 해당 code block에 맞춰 줄을 바꿀 때 2칸 들여쓰기 규칙을 지켜야 합니다.
// 속성 a 안에 속성 b가 있는데, 마치 같은 범주에 있는 속성으로 보입니다. 또한 함수와 객체의 구분이 명확하지 않습니다.

// GOOD
let interestRate = 0.2;
// 이자율임을 쉽게 알 수 있습니다. 어떤 금융상품의 이자율인지 표기할 수 있으면 더 좋겠네요.

// BAD
let numA = 0.2;
// 단지 숫자라는 것 의미 외 다른 의미가 없어 무슨 변수인지 추정해야 합니다.

// GOOD
let animals = ['cat', 'dog', 'fish'];
// 변수가 가지는 의미를 명확하게 표현하고 있습니다.

// BAD
let targetInputs = ['cat', 'dog', 'fish'];
// 변수가 가지는 구조를 표현하고 있는 변수명으로, animals가 더 낫습니다.

// GOOD
let animals = ['cat', 'dog', 'fish'];
// 하나의 동물이 아니라, 여러 마리의 동물이 있는 데이터임이 명시적으로 표현되어 있습니다.

// BAD
let animalList = ['cat', 'dog', 'fish'];
// animals로 표현되는 것이 의미를 조금 더 잘 표현합니다.

let animal = ['cat', 'dog', 'fish'];
// 하나의 동물인지, 여러 마리의 동물 리스트인지 알 수 없습니다.

// GOOD
let isDog = true;
let 강아지입니다 = true;

// GOOD
let countBlocks = function() {
  // 벽돌의 개수를 세는 함수임을 쉽게 알 수 있습니다.
}

let countWaterBlocksBetweenTowers = function() {
  // 타워 사이의 벽돌의 개수를 세는 함수임을 쉽게 알 수 있습니다.
}

// BAD
let waterBlocks = function() {
  // 변수명으로는 역할이 파악이 되지 않기 때문에, 아래의 주석이 없으면 무슨 역할을 하는지 파악하기 어렵습니다.
  // 이 함수는 벽돌의 개수를 세는 함수입니다. 타워 사이의 벽돌을 세는 것을 의미합니다.
}

class Animal(){
  // ES6 부터 사용 가능한 class가 할당되는 변수는 첫 글자를 대문자로 적습니다.
}

function Animal() {
  // class 생성자 함수가 될 함수임으로 대문자로 적습니다.
}

const MAX_ITEMS_IN_QUEUE = 100;

// GOOD
for (key in object) {
  alert(key);
}
// for문의 작동 범위를 눈으로 확인 가능합니다.

// BAD
for (key in object)
  alert(key);
// for문의 작동 범위를 눈으로 확인할 수 없습니다.

// GOOD
let dog = 'dog';
let cat = 'cat';

// BAD
let dog = 'dog';
let cat = "cat";
// 띄어쓰기와 tab과 같이 "", '', 혼용은 최악의 코드 스타일입니다.

// GOOD
let multilineText = `this is line one
this is line two
this is line three`;
// 엔터가 명확하게 보입니다.


// BAD
let multilineText = 'this is line one' + '\n' + 'this is line two' + '\n' + 'this is line three';
// 엔터가 명확하게 보이지 않습니다.

// GOOD
alert('hi');

// BAD
alert('hi')

// GOOD
if (condition) {
  response();
}

// BAD
if (condition) {
  response();
};

// GOOD
let greet = function () {
  alert('hi');
};

// BAD
let greet = function () {
  alert('hi');
}

// GOOD
if (0 === '') {
  alert('looks like they\'re equal');
}
// 엄격한 동치 연산은 `0`과 `''`을 다르다고 평가합니다.


// BAD
if (0 == '') {
  alert('looks like they\'re equal');
}
// 엄격하지 않은 동치 연산은 `0`과 `''`을 같다고 평가합니다.

// GOOD
if (actual !== expected) {
  console.log('FAILED ' + testName + ': Expected ' + expected + ', but got ' + actual);
} else {
  console.log('passed');
}

// BAD
return (actual === expected) ? 'passed' : 'FAILED ['+ testName + '] Expected "'+expected+'",but got '+'"'+actual+'"';

// GOOD
if (!isEqual) {


// BAD
if (! isEqual) {

// GOOD
function square(n) {
  return n * n;
}

// BAD
function square(n) {
  let squaredN = n * n;
  return squaredN;
}
// 변수를 선언하지 않아도 됩니다.

// GOOD
if(equalSizes && equalValues) {
  // 사이즈가 같고, 값이 같은 경우
} else {
}

// BAD
if(!equalSizes || !equalValues) {
  // 사이즈가 같지 않거나 값이 값지 않은 경우
} else {
}
// 이 조건문은 불필요하게 조건에 대해 고민을 해야 합니다.

// GOOD
return charSet.size > text.length;

// BAD
if(charSet.size < text.length) {
  return false;
}
return true;
// 비교문은 무조건 true 혹은 false로 평가되기 때문에 장황하게 조건문을 작성하지 않습니다.

// GOOD
function square(n) {
  return n * n;
}

function assertEqual(actual, expected, testName) {
  // compare actual and expected
}

// BAD
function square(n) {
  return n * n;
}



function assertEqual(actual, expected, testName) {
  // compare actual and expected
}

// GOOD
alert('I chose to put no visual padding around this string');
// 괄호와 따옴표 사이 공백이 없습니다.

alert( 'I chose to put visual padding around this string' );
// 괄호와 따옴표 사이 공백이 일관되게 있습니다.

// BAD
alert( 'I only put visual padding on one side of this string');
// 한 쪽에는 공백이 있고, 한 쪽에는 공백이 있습니다.

// GOOD
let firstItem = getFirst();
let secondItem = getSecond();
let thirteenthItem = getThirteenth();

// BAD
let firstItem      = getFirst();
let secondItem     = getSecond();
let thirteenthItem = getThirteenth();
// 불필요한 들여쓰기가 너무 많아졌습니다.

// GOOD
assertEqual(Math.pow(3, 2), 9, 'Math.pow squares properly');

// BAD
assertEqual(Math.pow(3,2),9,'Math.pow squares properly');

// GOOD
'Failed [' + testName + ']'...

if(actual === expected){
  // action
} else {
  // alternate action
}

// BAD
'Failed ['+testName+']'...

if(actual===expected){
  // action
}else{
  // alternate action
}

// GOOD
let camelCased = 'Used in javascript';

const MAX_ITEMS_IN_QUEUE = 100;
// JavaScript에서 'snake_case'을 사용하는 경우는, 상수 이름을 지을 때입니다.

// BAD
let snake_cased = 'Used in other languages';