공부 저장소/Node.js

[생활코딩 Node.js] 8. 홈 페이지 만들기

tipsygypsy 2022. 7. 31. 20:50

홈페이지 구현

쿼리스트링 id값이 없는 루트(/) 주소로 접속했을 때 띄울 홈 화면 내용을 조건문을 이용하여 만들어 준다.

그냥 따라하면 조건문 하나만 중첩해 추가하면 되는 쉬운 챕터였는데, 코드가 너무 보기싫게 길다고 느껴져서 효율화를 시키다가 삽질을 거하게 했다.

먼저 강의에서 가르쳐 준 소스코드.

var http = require('http');
var fs = require('fs');
var url = require('url');

var app = http.createServer(function(request,response){
    var _url = request.url;
    var queryData = url.parse(_url, true).query;
    var pathname = url.parse(_url, true).pathname;
    if(pathname === '/'){
      if(queryData.id === undefined){
        fs.readFile(`data/${queryData.id}`, 'utf8', function(err, description){
          var title = 'Welcome';
          var description = 'Hello, Node.js';
          var template = `
          <!doctype html>
          <html>
          <head>
            <title>WEB1 - ${title}</title>
            <meta charset="utf-8">
          </head>
          <body>
            <h1><a href="/">WEB</a></h1>
            <ul>
              <li><a href="/?id=HTML">HTML</a></li>
              <li><a href="/?id=CSS">CSS</a></li>
              <li><a href="/?id=JavaScript">JavaScript</a></li>
            </ul>
            <h2>${title}</h2>
            <p>${description}</p>
          </body>
          </html>
          `;
          response.writeHead(200);
          response.end(template);
        });
      } else {
        fs.readFile(`data/${queryData.id}`, 'utf8', function(err, description){
          var title = queryData.id;
          var template = `
          <!doctype html>
          <html>
          <head>
            <title>WEB1 - ${title}</title>
            <meta charset="utf-8">
          </head>
          <body>
            <h1><a href="/">WEB</a></h1>
            <ul>
              <li><a href="/?id=HTML">HTML</a></li>
              <li><a href="/?id=CSS">CSS</a></li>
              <li><a href="/?id=JavaScript">JavaScript</a></li>
            </ul>
            <h2>${title}</h2>
            <p>${description}</p>
          </body>
          </html>
          `;
          response.writeHead(200);
          response.end(template);
        });
      }
    } else {
      response.writeHead(404);
      response.end('Not found');
    }
});
app.listen(3000);

근데 이게 너무 길어서 읽기 불편했던 나머지, ‘템플릿은 중복인데 그냥 밖으로 빼고 조건문 안에서 title이랑 description만 정의해 주면 되는 거 아닌가?’ 라고 생각해 고친 내 코드가 이것.

var http = require('http');
var fs = require('fs');
var url = require('url'); //url모듈 사용

var app = http.createServer(function(request,response){
  var url_add = request.url;
  var queryData = url.parse(url_add, true).query;
  var pathname = url.parse(url_add, true).pathname;

  var template = `
  <!doctype html>
  <html>
  <head>
  <title>WEB1 - ${title}</title>
  <meta charset="utf-8">
  </head>
  <body>
  <h1><a href="/">WEB</a></h1>
  <ol>
  <li><a href="/?id=HTML">HTML</a></li>
  <li><a href="/?id=CSS">CSS</a></li>
  <li><a href="/?id=JavaScript">JavaScript</a></li>
  </ol>
  <h2>${title}</h2>
  <p>${description}</p>
  </body>
  </html>
  `;

  if(pathname === '/'){
    if(queryData.id === undefined){
      fs.readFile(`data/${queryData.id}`, 'utf-8', function(err, description){
        var title = 'Welcome';
        var description = 'Hello, Node.js';
        response.writeHead(200);
        response.end(template);
      });
    } else {
      fs.readFile(`data/${queryData.id}`, 'utf-8', function(err, description){
        var title = queryData.id;
        response.writeHead(200);
        response.end(template);
      });
    }
  } else {
    // 존재하지 않는 pathname일 경우 에러처리
    response.writeHead(404);
    response.end('Not found');
  }

});
app.listen(5000);

돌려보니 title이 없다고 자꾸 에러가 났다. 정의 다 해 줬는데? 하고 단계별로 로그를 잔뜩 찍어 보니 아예 첫 번째 if문에 들어가기도 전에 에러 발생. 한참만에 깨달았는데 template 안의 변수들이 정의되는 순간에 없기 때문에 발생한 에러였다.

여기서 두 번째로 ‘그럼 일단 디폴트 값으로 아무렇게나 title이랑 description 변수를 처음에 생성해주면 에러는 안 날 테고, if문 안에서 값이 바뀌니까 조건별로 원하는 값으로 출력되겠지’ 라고 생각하여 var template 바로 윗 줄에 두 라인을 추가.

var title = 'defalt';
var description = 'defalt';

…했더니 과연 에러는 안 나는데 모든 페이지의 타이틀과 내용이 defalt라고 출력되는 사태가 발생했다. template literal 을 정의할 때 그 안에 있는 변수의 내용도 그 시점의 내용으로 받아서 고정이 된다는.. 변수의 시점 문제를 의도치 않게 깨닫게 되었다. 결국 템플릿은 각각의 if문 안으로 돌려놓기로 했다. 쓸데없는 중복이라고 생각했는데 다 이유가 있었다.