Ant Problem mit classpath

[absoluteBeginner]

Hallo,
bin Neustarter mit Ant und habe ein Problem mit einem eigenen Task. Mein Build findet den Task nicht, so dass ich beim Run immer eine Nullpointer-Exception bekomme. Kann jemand helfen?

Hier ist mein Build.xml:

<project name="EigenerTask" default="use" basedir =".">
	
	<property name = "sourcedir" value="."/>
	<property name="classesdir" value="classes"/>

	
	<target name="clean" description="Delete all generated files">
	    <delete dir="${classesdir}" failonerror="false"/>
	    <delete file="${ant.project.name}.jar"/>
	</target>

	
	<target name="compile" description="Compiles the Task">
	    <mkdir dir="${classesdir}"/>
	    <javac srcdir="${sourcedir}" destdir="${classesdir}"/>
	</target>

	
	<target name="jar" description="JARs the Task" depends="compile">
	    <jar destfile="${ant.project.name}.jar" basedir="${classesdir}"/>
	</target>
	
	
	<target name="use" description="Use the Task" depends="jar">
	    <taskdef name="dateiLaenge" classname="DateiLaenge" classpath="${ant.project.name}.jar" />
		<dateiLaenge/>
		<dateiLaenge filename = "/Users/anjawegner/Documents/workspace/EigenerTask/src/calc.java"/>
		<daeiLaenge/>
	</target>

</project>	
<project name="EigenerTask" default="use" basedir =".">
	
	<property name = "sourcedir" value="."/>
	<property name="classesdir" value="classes"/>

	
	<target name="clean" description="Delete all generated files">
	    <delete dir="${classesdir}" failonerror="false"/>
	    <delete file="${ant.project.name}.jar"/>
	</target>

	
	<target name="compile" description="Compiles the Task">
	    <mkdir dir="${classesdir}"/>
	    <javac srcdir="${sourcedir}" destdir="${classesdir}"/>
	</target>

	
	<target name="jar" description="JARs the Task" depends="compile">
	    <jar destfile="${ant.project.name}.jar" basedir="${classesdir}"/>
	</target>
	
	
	<target name="use" description="Use the Task" depends="jar">
	    <taskdef name="dateiLaenge" classname="DateiLaenge" classpath="${ant.project.name}.jar" />
		<dateiLaenge/>
		<dateiLaenge filename = "/Users/anjawegner/Documents/workspace/EigenerTask/src/calc.java"/>
		<daeiLaenge/>
	</target>

</project>

..und hier der die Datei zum Task:

import org.apache.tools.ant.BuildException;
import org.apache.tools.ant.Task;
import org.apache.tools.ant.Project;


import java.io.*;

public class DateiLaenge extends Task {

	private String filename;
	private Project project;

    public void setProject(Project proj) {
        project = proj;
    }
    
	public void setFilename(String filename) {
		this.filename = filename;
	}

	public void execute() throws BuildException {
		File myFHandle = new File(filename);
		String laenge = String.valueOf(myFHandle.length());

		if (!myFHandle.isDirectory()){
			System.out.println("Hallo Task");
			String message = project.getProperty(laenge);
        	project.log("Dateilaenge: " + message + ".", Project.MSG_INFO);
		} else
			project.log("Datei ist Verzeichnis. ", Project.MSG_INFO);	
	}
}

 

[fastjack]

Hallo,

diese Aufrufe

[XML]<dateiLaenge/>[/XML]

setzen ja keinen filename, der aber in

 File myFHandle = new File(filename);

verwendet wird. Das würd ich erstmal abprüfen.

Kompilier mal so, um die Zeilennummern in den Exceptions zu erhalten:

[XML]<javac srcdir="${sourcedir}" destdir="${classesdir}" debug="on"/>[/XML] debug auf on setzen.

 

[fastjack]

Außerdem brauchst Du noch einen Default-Konstruktor :

public DateiLaenge() {
    super();
}

und zudem kannst Du das Project-Attribut entfernen. SetProject braucht Du auch nicht. Das Project erhälst Du bereits beim erben von Task, Zugriff erfolgt mit this.project.

Da die Tasks beankonform (leerer public Konstruktor, getter (getXXX, aber bei boolean auch isXXX) und setter-Methoden (setXXX) für alles, was zugreifbar sein soll) sein sollen, würde ich noch den Getter für Filename einbauen:

    public String getFilename() {
    	  return this.filename;
    }

 

[absoluteBeginner]

Hallo,

danke für die schnelle Antwortt(sorry, bei mir hat´s etwas länger gedauert

Habe damit zumindest mal den Taskzugriff hinbekommen, es hakt aber anscheinend noch bei der Rückgabe der gesetzten Property. Habe noch etwas "rumprobiert", Code sieht jetzt so aus:

[XML]<project name="EigenerTask" default="use" basedir =".">

<property name = "sourcedir" value="."/>
<property name="classesdir" value="classes"/>


<target name="clean" description="Delete all generated files">
<delete dir="${classesdir}" failonerror="false"/>
<delete file="${ant.project.name}.jar"/>
</target>


<target name="compile" description="Compiles the Task">
<mkdir dir="${classesdir}"/>
<javac srcdir="${sourcedir}" destdir="${classesdir}" debug="on"/>
</target>


<target name="jar" description="JARs the Task" depends="compile">
<jar destfile="${ant.project.name}.jar" basedir="${classesdir}"/>
</target>


<target name="use" description="Use the Task" depends="jar">
<taskdef name="dateiLaenge" classname="DateiLaenge" classpath="${ant.project.name}.jar" />
<dateiLaenge filename = "/Users/anjawegner/Documents/workspace/EigenerTask/src/Calc.java"/>
<echo message="Property myFile = ${myFile}!"/>
</target>

</project> [/XML]

import java.io.*;

public class DateiLaenge extends Task {

	private String filename;
	private Project project;
	
	public DateiLaenge() {
	    super();
	}
	
	public String getFilename() {
        return this.filename;
	}
    
	public void setFilename(String filename) {
		this.filename = filename;
	}

	public void execute() throws BuildException {
		File myFHandle = new File(filename);
		String laenge = String.valueOf(myFHandle.length());
		
		System.out.println("Laenge " + laenge);

		
		if (!myFHandle.isDirectory()){
			System.out.println("Hallo Task");
			project.setUserProperty("Laenge myFile", laenge);
		} else
			project.setUserProperty("Laenge myFile", "Datei ist Verzeichnis!");	
	}
}

Der Task wird aufgrufen, Dateilänge wird auch ermittelt, trotzdem bekomme ich für Zeile 26 im Build eine Nullpointer-Exception...

 

[fastjack]

private Project project;

kannst Du auch rausnehmen.

Stattdessen mußt Du das Project benutzen, das Du bereits ererbt hast, von Task

if (!myFHandle.isDirectory()){
    System.out.println("Hallo Task");
    this.project.setUserProperty("Laenge myFile", laenge);
} else {
    this.project.setUserProperty("Laenge myFile", "Datei ist Verzeichnis!");
}

Vor dem Verarbeiten von filename kann man diesen auch prüfen:

if (filename == null || filename.length() == 0) {
    throw new XXXException("filename can't be null or empty!");
} else if (!new File(filename).exists()) {
    throw new XXXException("file for filename " + filename + " can't be found!");
}

Je nachdem, wass man prüfen möchte. Das macht die Fehlersuche bei der Anwendung später um einiges leichter. XXXException muß natürlich gegen eine geeignete Exception ausgetauscht werden (BuildException, IllegalArgumentException etc.).

 

[absoluteBeginner]

Leider wird

 this.project.setUserProperty("Laenge myFile", laenge);

abgelehnt, wenn ich

private Project project;

rausnehme.

(Hinweis: The field ProjectComponent.project is deprecated).

Das Build läuft zwar durch, bringt aber für den javac-task die Meldungen:
Note: /Users/anjawegner/Documents/workspace/EigenerTask/src/DateiLaenge.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

????

Als echo kommt leider nur: Property myFile = ${myFile}


(Imports habe ich übrigens Folgende - fehlt da noch was?:

import org.apache.tools.ant.BuildException;
import org.apache.tools.ant.Task;
import org.apache.tools.ant.Project;
import java.io.*;

 

[fastjack]

        this.project.setUserProperty("myFile", laenge);
    } else
        this.project.setUserProperty("myFile", "Datei ist Verzeichnis!");

der Propertyname "Laenge myFile" ist falsch, es muß nur "myFile" heißen, wenn Du die Property unter "myFile" finden willst. Das mit dem Project funktioniert schon, wirft nur eine "deprecated warning", deshalb wird aber noch laenge nichts abgelehnt. Du mußt mal in der API schauen, welche Variante demnächst benutzt werden soll.

Importe:

import org.apache.tools.ant.BuildException;
import org.apache.tools.ant.Task;
import org.apache.tools.ant.Project;
 
 
import java.io.*;

 

[absoluteBeginner]

Aaah, jetzt ist´s klar! Super, funktioniert - DANKE! :applaus:

 

[fastjack]

No problem, kannst ja den Danke-Knopf drücken