is this ok?

I need to delete files. Those files are listed in a .log file and I also need to delete the logfile itself. I've written this like that... is this ok or would you do something different? Maybe adding anoter @ construct or whatever? (for example, writing the Condition and File parameter differently??) ... 

<Target ...><ItemGroup><IDLDeleteFiles /></ItemGroup><ReadLinesFromFile Condition="Exists('%(IDLDeletes.out)\%(IDLDeletes.Filename).log')" File="%(IDLDeletes.out)\%(IDLDeletes.Filename).log"><Output TaskParameter="Lines" ItemName="IDLDeleteFiles"/></ReadLinesFromFile><Delete Files="@(IDLDeleteFiles)" /><Delete Files="@(IDLDeletes->'%(out)\%(Filename).log')" /></Target>

Rudolf