java로 web server만들기 과제(4트)

Question

안녕하세요. java로 web server만드는 과제로 며칠전에도 올렸었는데요.여기서 얻은 답변들 덕분에 공부도 많이 되고 무사히 1차 점검 리뷰를 마쳤습니다.그리고나서 1차로 점검 리뷰 후 피드백을 받았는데 그중 하나가 무슨 뜻인지 이해가 잘 안되고 저대로 했을 때 제가 생각하기에는 어떤 효과인지 잘 모르겠습니다. 코드에서 봤을 때 properties.getproperty()를 많이 호출한다고 하셨던 것 같은데 그게 어떤 단점이 있는 건지 좀 더 자세히 알고 싶습니다. 그리고 어떤 식으로 코드를 짜야 효율적인지 알려주시면 좋을 것 같습니다.아래는 피드백입니다.3. 불필요한 properties 반복 load 작업이 확인됩니다. 이보다는 Config 파일 class 를 만들어 프로그램 최초 실행시 1회만 읽어들이도록 구성하는 것이 좋습니다import java.io.*; import java.net.ServerSocket; import java.net.Socket; import java.util.Properties; import java.util.concurrent.ExecutorService; import java.util.concurrent.Executors; import java.util.logging.Level; import java.util.logging.Logger; public class HttpServer { private static final Logger logger = Logger.getLogger(HttpServer.class.getCanonicalName()); private static final int NUM_THREADS = 50; private static final String INDEX_FILE = "index.html"; private final File rootDirectory; private final int port; public HttpServer(File rootDirectory, int port) throws IOException { if (!rootDirectory.isDirectory()) { throw new IOException(rootDirectory + " does not exist as a directory"); } this.rootDirectory = rootDirectory; this.port = port; } public void start() throws IOException { ExecutorService pool = Executors.newFixedThreadPool(NUM_THREADS); try (ServerSocket server = new ServerSocket(port)) { logger.info("Accepting connections on port : " + server.getLocalPort()); while (true) { try { Socket request = server.accept(); Runnable r = new RequestProcessor(rootDirectory, INDEX_FILE, request); pool.submit(r); } catch (IOException ex) { logger.log(Level.WARNING, "Error accepting connection", ex); } } } } public static void main(String[] args) throws IOException { Properties properties = new Properties(); try ( InputStream inputStream = new FileInputStream("src/main/resources/info.properties") ) { properties.load(inputStream); } logger.info("server.HTTP_DOC_ROOT : " + properties.getProperty("server.HTTP_DOC_ROOT.localhost" )); logger.info("server.port : " + properties.getProperty("server.port", "80") ); File docRoot = new File(properties.getProperty("server.HTTP_DOC_ROOT.localhost" )); int port = Integer.parseInt(properties.getProperty("server.port", "80")); try { HttpServer webserver = new HttpServer(docRoot, port); webserver.start(); } catch (IOException ex) { logger.log(Level.SEVERE, "Server could not start", ex); } } }import org.header.Header; import java.io.*; import java.net.Socket; import java.net.URLConnection; import java.nio.charset.StandardCharsets; import java.nio.file.Files; import java.nio.file.Paths; import java.util.Arrays; import java.util.Date; import java.util.List; import java.util.Properties; import java.util.logging.Level; import java.util.logging.Logger; public class RequestProcessor implements Runnable { private final static Logger logger = Logger.getLogger(RequestProcessor.class.getCanonicalName()); private File rootDirectory; private String indexFileName = "index.html"; private final Socket connection; public RequestProcessor(File rootDirectory, String indexFileName, Socket connection) { if (rootDirectory.isFile()) { throw new IllegalArgumentException( "rootDirectory must be a directory, not a file"); } try { rootDirectory = rootDirectory.getCanonicalFile(); } catch (IOException ex) { logger.severe( ex.getLocalizedMessage() ); } this.rootDirectory = rootDirectory; if (indexFileName != null) { this.indexFileName = indexFileName; } this.connection = connection; } @Override public void run() { // for security checks String root = rootDirectory.getPath(); logger.info("RequestProcessor Start!!!!!!!"); try (OutputStream raw = new BufferedOutputStream(connection.getOutputStream()); Writer out = new OutputStreamWriter(raw); InputStream inputStream = connection.getInputStream(); // getInputStream()는 한번만 호출 가능 BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8)); InputStream propertyInput = new FileInputStream("src/main/resources/info.properties"); ) { Properties properties = new Properties(); properties.load(propertyInput); StringBuilder builder = new StringBuilder(); String line = null; String host = ""; while (!(line = reader.readLine()).isEmpty()) { builder.append(line).append(" "); } String headers = builder.toString(); logger.info("headers : " + builder.toString()); String[] headerProperties = headers.split("\n"); Header header = new Header(headerProperties); host = header.getHost(); String method = header.getMethod(); String version = header.getVersion(); try { if (method.equals("GET")) { String fileName = header.getAccessFile(); if (fileName.endsWith("/")) { root = host.indexOf(".") > -1 ? properties.getProperty("server.HTTP_DOC_ROOT." + host.substring(0,3)) : properties.getProperty("server.HTTP_DOC_ROOT.localhost"); fileName += "index.html"; } else { root = host.indexOf(".") > -1 ? properties.getProperty("server.HTTP_DOC_ROOT." + host.substring(0,3)) : properties.getProperty("server.HTTP_DOC_ROOT.localhost"); } String contentType = URLConnection.getFileNameMap().getContentTypeFor(fileName); File theFile = new File(root, fileName.substring(1, fileName.length())); // 허용되지 않은 mime type String fileSubs = fileName.substring(fileName.indexOf(".") + 1, fileName.length()); String[] blockMimeType = properties.getProperty("blockMimeType").split("\,"); if (theFile.canRead() && theFile.getCanonicalPath().startsWith(root)) { logger.info("HTTP/1.0 200 OK!!!!"); byte[] theData = Files.readAllBytes(theFile.toPath()); logger.info("theFile : " + theFile); if (version.startsWith("HTTP/")) { // send a MIME header sendHeader(out, "HTTP/1.0 200 OK", contentType, theData.length); } raw.write(theData); raw.flush(); } // 403 error else if (Arrays.asList(blockMimeType).contains(fileSubs) || !theFile.getCanonicalPath().startsWith(root)) { String body = responseErrBody(host, properties, ".403"); if (version.startsWith("HTTP/")) { sendHeader(out, "HTTP/1.0 403 Forbidden", "text/html; charset=utf-8", body.length()); } out.write(body); out.flush(); } // 404 error else if (!theFile.exists() && theFile.getCanonicalPath().startsWith(root)) { logger.info("HTTP/1.0 404 file not found!!!!"); String body = responseErrBody(host, properties, ".404"); if (version.startsWith("HTTP/")) { // send a MIME header sendHeader(out, "HTTP/1.0 404 File Not Found", "text/html; charset=utf-8", body.length()); } out.write(body); out.flush(); } else { throw new IOException(); } } else { // method does not equal "GET" String body = new StringBuilder(" ").append("Not Implemented ").append(" ") .append("") .append("

HTTP Error 501: Not Implemented

") .append(" ").toString(); if (version.startsWith("HTTP/")) { // send a MIME header sendHeader(out, "HTTP/1.0 501 Not Implemented", "text/html; charset=utf-8", body.length()); } out.write(body); out.flush(); } } // 500 error catch (IOException e){ String body = responseErrBody(host, properties, ".500"); sendHeader(out, "HTTP/1.0 500 Internal Server Error", "text/html; charset=utf-8", body.length()); out.write(body); out.flush(); } } catch (IOException ex) { logger.log(Level.WARNING, "Error talking to " + connection.getRemoteSocketAddress(), ex); } finally { try { connection.close(); } catch (IOException ex) { } } } private void sendHeader(Writer out, String responseCode, String contentType, int length) throws IOException { out.write(responseCode + " "); Date now = new Date(); out.write("Date: " + now + " "); out.write("Server: JHTTP 2.0 "); out.write("Content-length: " + length + " "); out.write("Content-type: " + contentType + " "); out.flush(); } private String responseErrBody(String host, Properties properties, String errCode) throws IOException { String subhost = host.substring(0,host.indexOf(".") > -1 ? host.indexOf(".") : host.length()); String hostDocRoot = "server.HTTP_DOC_ROOT." + subhost; byte[] theData = Files.readAllBytes(Paths.get(properties.getProperty(hostDocRoot) + properties.getProperty("server.HTTP_DOC_ROOT" + errCode))); String body = new String(theData, StandardCharsets.UTF_8); return body; } }

Clickin · Accepted Answer

말 그대로의 의미죠properties를 그때그때 파일 IO 발생시키면서 읽지 말고Map이든 Class 형태든 만들어서 메모리에 올린 다음 읽으면 굳이 매번 파일 IO를 발생시킬 필요가 없죠간단하게 구현하고 싶다면 properties를 다 읽어서 그냥 HashMap에 때려박으면 1단계는 클리어입니다.

mirheeoj · Answer

만약 프로퍼티 설정을 서버 운용중에 실시간으로 변경시킬 필요가 있다면 매번 파일에서 읽어야 할 수도 있습니다. 그런 목적이 없다면, 프로퍼티 부분을 오버라이딩하든 별도 싱글턴을 만들든 해서 레이지하게 한번 읽은건 맵에 저장하고 읽지 않은건 새로 읽어들이고 이런식으로 처리하실 수도 있습니다. 이러면 파일IO가 간헐적으로 일어나는건 단점이 되지만 대신 어떤 프로퍼티를 읽게 될지 잘 모를 경우 유용할 수 있습니다. 미리 다 읽어서 맵에 저장할수도 있고요. 이 경우 프라퍼티 파일이 매우 비대하면서 쓰지 않는 설정이 대부분일 경우 오히려 비효율적이게 될 가능성도 있습니다.결국 "적절히"... 가 되는 거죠.

장독깨기 · Answer

음... getProperty() 호출 할 때 파일 IO 가 발생하는건 아니겠죠.소스코드 보면 main 과 run 에서 각각 속성 파일을 load 하는데요.이걸 메인에서만 로드하고 공유를 하든, 속성 파일을 별개로 만들든 하면 될 거 같습니다.근데, 이 정도 내용을 지적 할 필요는 없어 보이긴 합니다만,굳이 Config 클래스를 별도로 만들어 해보라 하시니,Config 클래스 만들고 load, getProperty 메서드 추가해서 적용해보는 것이 좋겠습니다.그리고, 처음에 힘들어 하시더니 잘 하셨네요. :)

java로 web server만들기 과제(4트)

3개의 답변

#socketprogramming 목록