Exec

[michi2]

Ich habe folgende funktion zum kompilieren eines (java) Quellcodes geschrieben.
Leider spackt sie aber noch ein wenig...
Wie kann ich diese Verbessern (fehler beheben, Performence u. anderes)?

  final class Exec
  {
   String E_ausgabe  = "";
   String E_Befehl  = "";
   Process p2;
   boolean b1 = false;

  class Prozess extends Thread
     {
     public void run()
        {
        try
           {
           p2 = Runtime.getRuntime().exec(E_Befehl );
           }
        catch(java.io.IOException e){}
        }
     }

  class Lesen extends Thread
  {
     public void run()
	{
    try
      {
    Thread.sleep(5000);
       }
    catch (InterruptedException e){}
    p2.destroy();
    BufferedReader in = new BufferedReader(new InputStreamReader(p2.getInputStream()) );	//Ausgaben des Compilers lesen
    String s, s1;
    try
      {
    while((s = in.readLine()) != null )
	{
	E_ausgabe   = E_ausgabe  + (s + "\n");
	}


    BufferedReader in2 = new BufferedReader(new InputStreamReader(p2.getErrorStream()) );	//Fehler des Compilers lesen
    while((s1 = in2.readLine()) != null)
	{
	int i = s1.indexOf("error");
	int l = s1.indexOf("errors");
	if(i == -1 && l == -1)
	  {
	  E_ausgabe   = E_ausgabe  + (s1 + "\n");
	  }
	 else
	  {
	  E_ausgabe   = E_ausgabe  + (s1.substring(0, i));
	  if(l != -1)
	    {
		E_ausgabe   = E_ausgabe  + ("Fehler");
		E_ausgabe   = E_ausgabe  + (s1.substring(i + 6, s1.length()));
	    }
	  else
	    {
		E_ausgabe   = E_ausgabe  + ("Fehler");
		E_ausgabe   = E_ausgabe  + (s1.substring(i + 5, s1.length()));
	    }
	    E_ausgabe   = E_ausgabe  + ("\n");
	  }
	}//Ende 'while'

      }	//Ende 'try'
    catch (java.io.IOException IOEx){System.out.println("Fehler");}
    b1 = true;
        }	//Ende 'run'
     }	//Ende 'Lesen'


  public  void compile(String m_dateiname,String compiler) throws IOException
	{
    if(System.getProperty("os.name").indexOf("Windows") > -1)
	{
	E_ausgabe   = E_ausgabe  + "cmd /c ";
	E_Befehl        = E_Befehl        + "cmd /c ";
	}
    E_ausgabe   = E_ausgabe  + ( compiler + " " + "\"" + m_dateiname+ "\"");
    E_Befehl        = E_Befehl        + ( compiler + " " + "\"" + m_dateiname+ "\"");
    E_ausgabe   = E_ausgabe  + (":\n");
    Prozess pr = new Prozess();
    pr.start();
    Lesen l = new Lesen();
    l.start();
    while(!b1)
       {
          try
             {
             Thread.sleep(500);
	}
          catch (InterruptedException e){}
        }

  }

  public void interpret(String m_dateiname_Name, String m_dateiname_Path,String interpreter) throws IOException
  {
    String t_dateiname_Name = m_dateiname_Name.substring(0,m_dateiname_Name.length() - 5);
    if(System.getProperty("os.name").indexOf("Windows") > -1)
	{
	E_ausgabe   = E_ausgabe  + "cmd /c ";
	E_Befehl        = E_Befehl        + "cmd /c ";
	}
    E_ausgabe   = E_ausgabe  + (interpreter + " -classpath " + "\"" + m_dateiname_Path.substring(0,m_dateiname_Path.length() - 1) + "\"" + " " + t_dateiname_Name );
    E_Befehl        = E_Befehl        + (interpreter + " -classpath " + "\"" + m_dateiname_Path.substring(0,m_dateiname_Path.length() - 1) + "\"" + " " + t_dateiname_Name );
    E_ausgabe   = E_ausgabe  + (":\n");
    Prozess pr = new Prozess();
    pr.start();
    Lesen l = new Lesen();
    l.start();
    while(!b1)
	    {
          try
	    {
             Thread.sleep(500);
	  }
          catch (InterruptedException e){}
	}
  }

     String getString()
      {       
     return E_ausgabe ;
      }
}

Danke!!!

 

[The_S]

Hilfreich wäre zu wissen, wo genau die Fehler liegen (also was nicht geht), dann kann man auch leichter helfen, ohne dass man erstmal alles ausprobieren muss .

 

[michi2]

Allso, oft gehts - wenn jedoch der Compiler sehr viele Fehler berichtet beendet er sich aus irgend einem Grund nicht.
Dann macht natürlich auch die Methode nicht weiter.
Darum hab ich das mit den ganzen Threats gemacht um den Prozess nach 5 Sekunden zu beenden. (Zeile 27 u. 30)
Das funktioniert aber nicht!

 

[Bleiglanz]

völlig hanebüchen

du kannst den Java-Compiler auch direkt aus dem Code heraus aufrufen (ohne den Umweg über exec), dann kriegst du das ganze etwas "plattformunabhängiger" hin

 

[KSG9|sebastian]

package de.test;

import java.io.File;
import java.lang.reflect.Method;
import java.util.ArrayList;
import java.util.List;

public class CompilerTest {
	File destDir;
	String classPath;
	boolean debug;
	public boolean compile(File[] sources) 
	   { 
	      List args = new ArrayList(); 

	      if(destDir != null) { 
	         args.add("-d"); 
	         args.add(destDir.getPath()); 
	      } 

	      if(classPath != null) { 
	         args.add("-classpath"); 
	         args.add(classPath); 
	      } 

	      if(debug) 
	         args.add("-g"); 

	      //if(deprecation) 
	         args.add("-deprecation"); 

	         if(! System.getProperty("java.vm.version").startsWith("1.3")) { 
	             args.add("-source"); 
	             args.add("1.4"); 
	         } 

	         for(int i = 0; i < sources.length; i++) 
	             args.add(sources[i].getPath()); 

	         int length = args.size(); 
	         String[] params = new String[length]; 
	         args.toArray(params); 

	         Class compiler = null; 
	         Method compileMethod = null; 

	         try { 
	             compiler = Class.forName("com.sun.tools.javac.Main"); 

	             if (compiler == null) 
	                 return false; 

	             Class[] p = new Class[] {String[].class}; 

	             compileMethod = compiler.getMethod("compile", p); 

	         } catch (ClassNotFoundException e) { 
	                 return false; 
	         } catch (NoSuchMethodException e) { 
	             
	             return false; 
	         } 

	 

	         try { 
	             Object objResult = null; 
	             Object[] arguments = new Object[] { params }; 
	             objResult = compileMethod.invoke(compiler.newInstance(), 
	arguments); 

	            return true;
	         } catch (Exception e) { 
	             e.printStackTrace(); 
	             return false; 
	         }
	   }

}