homework

hw1

function printTri(size, isBlank) {
	// validate 
	if (isBlank && size < 3) {
		console.log('Size should mor than 3');
		return false;
	}
	var i;
	var star;
	var blank;
	for(i = 1; i <= size; i++) {
		star = (2*i) - 1;
		blank = size - i;
		if (size == i) {
			isBlank = false;
		}
		console.log(getBlankText(blank) + getStarText(star, isBlank));
	}
}

function getStarText(star, isBlank) {

	var i = 0;
	var starText = '';
	for (i = 0; i < star; i++) {
		if (!isBlank) {
			starText += '*';	
		}
		else {
			if (i === 0 || i == (star - 1)){
				starText += '*';
			}	
			else {
				starText += ' ';
			}
		}
	}
	return starText;
}

function getBlankText(star) {

	var i = 0;
	var starText = '';
	for (i = 0; i < star; i++) {
		starText += ' ';	
	}
	return starText;
}

printTri(5);
printTri(5, true);

เลือก syntax เป็น javascript ด้วย

Comment : code style

• indent เยอะไป ควรใช้ space แทน tab
• ตัวแปรอ่านยากไปหน่อย เช่น star, blank ตอนแรกงงนึกว่าเป็น text จริงๆ คือ integer
• ใน getBlankText() แต่ว่าตั้งชื่อตัวแปรเป็น starText
• บรรทัด 25-35 ลดเหลือแค่นี้ได้

if (!isBlank || i === 0 || i == (star - 1)) {
  starText += '*';
}   
else {
  starText += ' ';
}

• จริงๆ แบบ blank ก็ใส่ input เป็น 1,2 ได้นะ เพราะตอน n = 1,2 มันก็จะวาดเหมือนแบบไม่ blank นั่นแหละ

comment อื่นๆ

โอเคหมดแล้ว เรียบง่าย ฉับไว