이 Java 코드에 프로그래밍상의 결함이 있습니까?

Question

import java.lang.*; 
import java.io.*; 

class test 
{ 
    public static void main(String[] a) throws Exception 
    { 
     int i; 
     String[] str = new String[]{"javac","example.java"}; 
     String[] str1 = new String[]{"java","example"}; 
     Runtime r = Runtime.getRuntime(); 
     Process p = null; 
     Process p1 = null; 
     p=r.exec(str); 
     p1=r.exec(str1); 
     InputStreamReader reader = new InputStreamReader (p1.getInputStream()); 
     BufferedReader br = new BufferedReader(reader); 
     FileWriter fw = new FileWriter("this.txt",true); 
     char[] c = new char[1]; 
     while ((i=br.read())!=-1) 
     { 
      c[0]  = (char) i ; 
      fw.write(c); 
      c = new char[1]; 
     } 
     fw.close(); 
    } 
} 

이것은 런타임 클래스를 사용하는 간단한 프로그램입니다. '프로세스'코드를 종료해야합니까?

미리 감사드립니다.

Answer 1

1. 프로세스 객체를 null로 초기화 할 이유가 없습니다.

그냥 할 : 당신이 그것에 인 동안, 더 나은 변수 이름은

이 도움이 될

Process p = r.exec(str); 
Process p1 = r.exec(str1); 

. 한 번에 한 글자 이상 읽고 쓰는 것으로 성능을 향상시킬 수 있습니다 :

3. 플랫폼 기본값을 사용하는 대신 명시 적으로 인코딩을 지정하고자 할 수 있습니다.

InputStreamReader reader = new InputStreamReader (p1.getInputStream(), inputCharsetName); 
BufferedReader br = new BufferedReader(reader); 
FileOutputStream fos = new FileOutputStream("this.txt", true); 
Writer writer = new OutputStreamWriter(fos, outputCharsetName); 

Answer 2

입력 스트림은 절대로 닫히지 않으며, finally 블록에서 작성기를 닫는 것이 좋습니다.

Answer 3

'프로세스'코드의 종료가 필요합니까?

나는 프로세스에 달려 있다고 생각합니다. 당신이에 기대하는 것처럼

p.destroy(); 
p1.destroy(); 

Answer 4

여러분의 프로그램은 방법을 작동하지 않습니다

당신은 쉽게 종료 할 수 있습니다 (? 그리고 당신은이 프로그램이하는 때 그들을 종료 싶어). 즉. example.java의 컴파일이 완료되기 전에 example.class를 실행하려고 시도합니다.

Answer 5

컴파일 프로세스를 완료하려면 Process.waitFor()를 사용해야합니다. 그렇지 않으면 컴파일에 시간이 너무 오래 걸리기 때문에 존재하지 않는 클래스 파일을 실행하려고 할 가능성이 큽니다.

버퍼가 너무 작습니다. char[] c = new char[1]; // lolz array with exactly one element과 같은 작업을 할 때마다 적어도 두 번보아야합니다.

Answer 6

먼저 두 번째 프로세스를 실행하기 전에 컴파일이 완료 될 때까지 기다려야합니다. WAITFOR() 메소드를 사용하여

int compilationResult = -1; 
try { 
    compilationresult = p.waitFor(); 
} 
catch (InterruptedException ie) { 
    ... 
} 

그런 다음 코드의 나머지 부분을 계속할지 여부를 결정 compilationResult (0의 값이 성공적으로 컴파일을 표시)를 확인.

두 번째로, 출력 처리 후 p1 프로세스가 완료되면 프로세스가 완료 될 것으로 기대되는지 여부에 따라 프로세스가 완료 될 때까지 기다리거나 직접 종료해야합니다.

프로세스가 정상적으로 완료 될 때까지 기다리려면 try/catch 블록에 p1.waitFor()을 입력하여 위와 유사한 코드 블록을 사용하십시오.

자신을 죽일하려면 사용

이 과정이 완료 확실 후

p1.destroy(); 

, 당신은 다음 p1.exitValue()과 종료 값을 확인 할 수 있습니다. 왜 이것에 대해 조심해야할까요? 처리가 올바르게 실행될 때까지는 수집하고 쓰는 출력을 this.txt으로 신뢰하면 안됩니다.