如何开发一种替代重复代码块的方法

Question

我是ant的新手，所以我没能找到一种方法让我的buld文件更优雅。我相信有一种方法可以在我的构建中替换重复的代码块。所以这里是构建文件：

<project basedir="../../../" name="do-report" default="zip-all">
    <xmlproperty keeproot="false" file="implementation/xml/ant/properties.xml"/>
    <!--    -->
    <taskdef resource="net/sf/antcontrib/antcontrib.properties">
        <classpath>
            <pathelement location="${infrastructure-base-dir}/apache-ant-1.9.6/lib/ant-contrib-0.3.jar"/>
        </classpath>
    </taskdef>
    <!--    -->
    <target name="clean">
        <delete dir="${dita-odt.path.odt-unzipped-base-dir}" includeemptydirs="true" failonerror="no"/>
        <delete dir="examples/intermediate/odt-files" includeemptydirs="true" failonerror="no"/>
    </target>
    <!--    -->
    <target name="unzip-writing-odt-file" depends="clean">
        <unzip src="${dita-odt.path.writing-odt}" dest="${dita-odt.path.writing-odt-unzipped}"/>
    </target>
    <!--    -->
    <target name="extract-common-paths" depends="unzip-writing-odt-file">
        <foreach target="copy-text-path" param="file">
            <path>
                <fileset dir="${dita-odt.path.text-xml-base-dir}">
                    <include name="**/content.xml"/>
                </fileset>
            </path>
        </foreach>
    </target>
    <!--    -->
    <target name="copy-text-path" description="copy text-xml path relative to text-xml-base-dir">
        <dirname property="text-absolute-dir-path" file="${file}"/>
        <property name="absolute-path-text-base-dir" location="${dita-odt.path.text-xml-base-dir}"/>
        <pathconvert property="common-path" dirsep="/">
            <path location="${text-absolute-dir-path}"/>
            <map from="${absolute-path-text-base-dir}/" to=""/>
        </pathconvert>
        <antcall target="copy-writing-unzipped">
            <param name="common-path" value="${common-path}"/>
        </antcall>
    </target>
    <!--    -->
    <target name="copy-writing-unzipped">
        <echo>${common-path}</echo>
        <copy todir="${dita-odt.path.odt-unzipped-base-dir}/${common-path}">
            <fileset dir="${dita-odt.path.writing-odt-unzipped}">
                <include name="**/*"/>
            </fileset>
        </copy>
    </target>
    <!--   -->
    <target name="transform-all" depends="extract-common-paths">
        <foreach target="transform" param="file">
            <path>
                <fileset dir="${dita-odt.path.text-xml-base-dir}">
                    <include name="**/content.xml"/>
                </fileset>
            </path>
        </foreach>
    </target>
    <!--   -->
    <target name="transform">
        <basename property="file-base-name" file="${file}"/>
        <dirname property="file-dir-absolute-path" file="${file}"/>
        <property name="text-xml-base-dir-absolute-path" location="${dita-odt.path.text-xml-base-dir}"/>
        <pathconvert property="common-path" dirsep="/">
            <path location="${file-dir-absolute-path}"/>
            <map from="${text-xml-base-dir-absolute-path}/" to=""/>
        </pathconvert>
        <!--Substitutes backslashes with forword slashes. Basedir is a reserved property that returns absolute path with separator symbols of the current OS.-->
        <pathconvert dirsep="/" property="base-dir-unix">
            <path location="${basedir}"/>
        </pathconvert>
        <echo>TRANSFORM TO: ${dita-odt.path.odt-unzipped-base-dir}/${common-path}/${file-base-name}</echo>
        <xslt in="${file}" out="${dita-odt.path.odt-unzipped-base-dir}/${common-path}/${file-base-name}" style="${dita-odt.path.text-odt-xsl}" extension=".xml" force="true">
            <param name="dir-path-styles-xml" expression="${dita-odt.path.odt-unzipped-base-dir}/${common-path}"/>
            <param name="project-base-dir-absolute-path" expression="${base-dir-unix}"/>
            <classpath location="${infrastructure-base-dir}/${dita-odt.text-odt-xsl.processor}"/>
        </xslt>
    </target>
    <!--   -->
    <target name="zip-all" depends="transform-all" description="Turns all unzipped text folders into ODT files">
        <foreach target="zip-odt" param="file">
            <path>
                <fileset dir="${dita-odt.path.odt-unzipped-base-dir}" includes="**/content.xml" excludes="writing/**"/>
            </path>
        </foreach>
    </target>
    <!--    -->
    <target name="zip-odt">
        <basename property="file-base-name" file="${file}"/>
        <dirname property="file-dir-absolute-path" file="${file}"/>
        <!--This property will be used to provided name for the produced ODT file. The document will have the same name as the folder that contains it.-->
        <basename property="odt-doc-name" file="${file-dir-absolute-path}.odt"/>
        <property name="odt-unzipped-base-dir-absolute-path" location="${dita-odt.path.odt-unzipped-base-dir}"/>
        <pathconvert property="common-path" dirsep="/">
            <path location="${file-dir-absolute-path}"/>
            <map from="${odt-unzipped-base-dir-absolute-path}/" to=""/>
        </pathconvert>
        <echo>COMMON PATH: ${common-path}</echo>
        <zip destfile="examples/intermediate/odt-files/${common-path}/${odt-doc-name}" basedir="${dita-odt.path.odt-unzipped-base-dir}/${common-path}" update="true"/>
    </target>
    <!--    -->
</project>

因此，脚本的这一部分做得几乎相同，但在项目中几乎所有的目标之间共享：

        <dirname property="file-dir-absolute-path" file="${file}"/>
        <property name="text-xml-base-dir-absolute-path" location="${dita-odt.path.text-xml-base-dir}"/>
        <pathconvert property="common-path" dirsep="/">
            <path location="${file-dir-absolute-path}"/>
            <map from="${text-xml-base-dir-absolute-path}/" to=""/>
        </pathconvert>

这部分只不过是为了获得一条路径的一部分。例如，如果${file}代表/folder/subfolder1/subfolder2，则取/folder之后的路径，即subfolder1/subfolder2，并将其分配给属性。在这种情况下，该属性名为common-path，为所有目标保留相同的路径。我检查了MacroDef任务，但据我所知，它不会返回，只接受一些属性形式的参数。不管怎样，任何帮助都将不胜感激。

Answer 1

您考虑<macrodef>以减少重复代码的做法是正确的。

虽然<macrodef>确实不返回任何内容，但可以为<macrodef>指定要设置的属性的名称。例如

<macrodef name="my-hello">
    <attribute name="person"/>
    <attribute name="output-property"/>
    <sequential>
        <property name="@{output-property}" value="Hello, @{person}!"/>
    </sequential>
</macrodef>
<my-hello person="Riko" output-property="say-hi-to-riko"/>
<echo>my-hello said: ${say-hi-to-riko}</echo>

输出。。。

[echo] my-hello said: Hello, Riko!

在本例中，<my-hello>的调用者告诉macrodef在say-hi-to-riko属性中"返回"其结果。

知道了这一点，脚本中的几个<target>可以转换为设置属性的<macrodef>。。。

<project name="ant-macrodef-pathconvert" default="extract-common-paths">
    <taskdef resource="net/sf/antcontrib/antlib.xml" />
    <property name="dita-odt.path.text-xml-base-dir" value="C:tempdita-odt"/>
    <macrodef name="my-pathconvert">
        <attribute name="file"/>
        <attribute name="common-path-property"/>
        <sequential>
            <!-- <local> allows multiple calls to a macrodef. -->
            <local name="file-dir-absolute-path"/>
            <echo>In my-pathconvert for @{file}</echo>
            <dirname property="file-dir-absolute-path" file="@{file}"/>
            <property name="text-xml-base-dir-absolute-path"
                location="${dita-odt.path.text-xml-base-dir}"/>
            <pathconvert property="@{common-path-property}" dirsep="/">
                <path location="${file-dir-absolute-path}"/>
                <map from="${file-dir-absolute-path}/" to=""/>
            </pathconvert>
        </sequential>
    </macrodef>
    <macrodef name="copy-text-path"
        description="copy text-xml path relative to text-xml-base-dir">
        <attribute name="file"/>
        <sequential>
            <local name="common-path"/>
            <echo>In copy-text-path for @{file}</echo>
            <my-pathconvert file="@{file}" common-path-property="common-path"/>
            <copy-writing-unzipped common-path="${common-path}"/>
        </sequential>
    </macrodef>
    <macrodef name="copy-writing-unzipped">
        <attribute name="common-path"/>
        <sequential>
            <echo>In copy-writing-unzipped for @{common-path}</echo>
            <echo>copy task goes here.</echo>
        </sequential>
    </macrodef>
    <target name="extract-common-paths">
        <for param="file">
            <path>
                <fileset dir="${dita-odt.path.text-xml-base-dir}">
                    <include name="**/content.xml"/>
                </fileset>
            </path>
            <sequential>
                <copy-text-path file="@{file}"/>
            </sequential>
        </for>
    </target>
</project>

通常，与其直接调用<target> s，不如调用<macrodef> s。在上面的例子中，<foreach>被替换为<for>，因为<for>允许我们调用<macrodef> s。

输出

 [echo] In copy-text-path for C:tempdita-odtdir1content.xml
 [echo] In my-pathconvert for C:tempdita-odtdir1content.xml
 [echo] In copy-writing-unzipped for C:/temp/dita-odt/dir1
 [echo] copy task goes here.
 [echo] In copy-text-path for C:tempdita-odtdir2content.xml
 [echo] In my-pathconvert for C:tempdita-odtdir2content.xml
 [echo] In copy-writing-unzipped for C:/temp/dita-odt/dir2
 [echo] copy task goes here.