Author: tchemit Date: 2008-12-14 22:52:15 +0000 (Sun, 14 Dec 2008) New Revision: 399 Modified: lutingenerator/trunk/src/main/java/org/codelutin/generator/ObjectModelGenerator.java Log: use for-each Modified: lutingenerator/trunk/src/main/java/org/codelutin/generator/ObjectModelGenerator.java =================================================================== --- lutingenerator/trunk/src/main/java/org/codelutin/generator/ObjectModelGenerator.java 2008-12-10 15:44:06 UTC (rev 398) +++ lutingenerator/trunk/src/main/java/org/codelutin/generator/ObjectModelGenerator.java 2008-12-14 22:52:15 UTC (rev 399) @@ -55,10 +55,7 @@ import java.io.Writer; import java.util.ArrayList; import java.util.Enumeration; -import java.util.Iterator; import java.util.List; -import java.util.logging.Level; -import java.util.logging.Logger; import java.util.regex.Matcher; import java.util.regex.Pattern; @@ -86,7 +83,7 @@ * * Le nom de l'argument writer doit absolument etre output et pas autre chose si * vous souhaitez utiliser le processeur - * {@link org.codelutin.processor.filters.GeneratorTemplatesFilter} pour vous + * org.codelutin.processor.filters.GeneratorTemplatesFilter pour vous * s'implifier l'écriture des templates. */ public class ObjectModelGenerator extends ChildGenerator { @@ -226,29 +223,38 @@ // operation avec le nom donné List<ObjectModelElement> elems = new ArrayList<ObjectModelElement>(); if ("class".equals(target)) { - elems.add((ObjectModelElementImpl) omc); + elems.add(omc); } else if ("attribute".equals(target)) { ObjectModelClass classmodel = (ObjectModelClass) omc; - elems.add((ObjectModelElementImpl) classmodel + elems.add(classmodel .getAttribute(targetName)); } else if ("operation".equals(target)) { elems.addAll(omc .getOperations(targetName)); } - for (Iterator it = elems.iterator(); it.hasNext();) { - ObjectModelElementImpl elem = (ObjectModelElementImpl) it - .next(); + for (Object elem1 : elems) { + ObjectModelElementImpl elem = (ObjectModelElementImpl) elem1; if ("stereotype".equals(type)) { // pour les stereotypes ObjectModelImplRef stereotype = new ObjectModelImplRef(); stereotype.setName(value); elem.addStereotype(stereotype); - } else if ("tagvalue".equals(type)) { + continue; + } + + if ("tagvalue".equals(type)) { // pour les tagvalues ObjectModelImplTagValue tagvalue = new ObjectModelImplTagValue(); tagvalue.setName(tag); tagvalue.setValue(value); - elem.addTagValue(tagvalue); + if (log.isDebugEnabled()) { + log.debug("tagValue:"+tagvalue+", tag:" + tag + ", value:" + value+", element:"+elem); + } + if (elem == null) { + log.warn("can not attach tagValue " + key); + } else { + elem.addTagValue(tagvalue); + } } } } else { @@ -291,7 +297,11 @@ String filename = getFilenameForModel(model); File outputFile = getDestinationFile(destDir, filename); - if (getOverwrite() || !isNewerThanSource(outputFile)) { + if (!getOverwrite() && isNewerThanSource(outputFile)) { + if (log.isDebugEnabled()) { + log.debug("file " + outputFile + " is up-to-date"); + } + } else { try { StringWriter out = new StringWriter(); MonitorWriter monitorOut = new MonitorWriter(out); @@ -299,75 +309,72 @@ write(outputFile, monitorOut); } catch (Exception eee) { log.warn("Erreur lors de la génération du fichier " + outputFile); - throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); } - } else { - Logger.getLogger(getClass().getName() + ".generate").log( - Level.FINE, "file " + outputFile + " is up-to-date"); } } - for (Object o : model.getClassifiers()) { - ObjectModelClassifier clazz = (ObjectModelClassifier) o; - + for (ObjectModelClassifier clazz : model.getClassifiers()) { + String filename = getFilenameForClassifier(clazz); File outputFile = getDestinationFile(destDir, filename); - if (getOverwrite() || !isNewerThanSource(outputFile)) { - try { - StringWriter out = new StringWriter(); - MonitorWriter monitorOut = new MonitorWriter(out); - generateFromClassifier(monitorOut, clazz); - write(outputFile, monitorOut); - } catch (Exception eee) { - log.warn("Erreur lors de la génération du fichier " + outputFile); - throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + if (!getOverwrite() && isNewerThanSource(outputFile)) { + if (log.isDebugEnabled()) { + log.debug("file " + outputFile + " is up-to-date"); } - } else { - Logger.getLogger(getClass().getName() + ".generate").log( - Level.FINE, "file " + outputFile + " is up-to-date"); + continue; } + try { + StringWriter out = new StringWriter(); + MonitorWriter monitorOut = new MonitorWriter(out); + generateFromClassifier(monitorOut, clazz); + write(outputFile, monitorOut); + } catch (Exception eee) { + log.warn("Erreur lors de la génération du fichier " + outputFile); + throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + } } - for (Object o : model.getInterfaces()) { - ObjectModelInterface interfacez = (ObjectModelInterface) o; + for (ObjectModelInterface interfacez : model.getInterfaces()) { String filename = getFilenameForInterface(interfacez); File outputFile = getDestinationFile(destDir, filename); - if (getOverwrite() || !isNewerThanSource(outputFile)) { - try { - StringWriter out = new StringWriter(); - MonitorWriter monitorOut = new MonitorWriter(out); - generateFromInterface(monitorOut, interfacez); - write(outputFile, monitorOut); - } catch (Exception eee) { - log.warn("Erreur lors de la génération du fichier " + outputFile); - throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + if (!getOverwrite() && isNewerThanSource(outputFile)) { + if (log.isDebugEnabled()) { + log.debug("file " + outputFile + " is up-to-date"); } - } else { - Logger.getLogger(getClass().getName() + ".generate").log( - Level.FINE, "file " + outputFile + " is up-to-date"); + continue; } + try { + StringWriter out = new StringWriter(); + MonitorWriter monitorOut = new MonitorWriter(out); + generateFromInterface(monitorOut, interfacez); + write(outputFile, monitorOut); + } catch (Exception eee) { + log.warn("Erreur lors de la génération du fichier " + outputFile); + throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + } } - for (Object o : model.getClasses()) { - ObjectModelClass clazz = (ObjectModelClass) o; - + for (ObjectModelClass clazz : model.getClasses()) { + String filename = getFilenameForClass(clazz); File outputFile = getDestinationFile(destDir, filename); - if (getOverwrite() || !isNewerThanSource(outputFile)) { - try { - StringWriter out = new StringWriter(); - MonitorWriter monitorOut = new MonitorWriter(out); - generateFromClass(monitorOut, clazz); - write(outputFile, monitorOut); - } catch (Exception eee) { - log.warn("Erreur lors de la génération du fichier " + outputFile); - throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + if (!getOverwrite() && isNewerThanSource(outputFile)) { + if (log.isDebugEnabled()) { + log.debug("file " + outputFile + " is up-to-date"); } - } else { - Logger.getLogger(getClass().getName() + ".generate").log( - Level.FINE, "file " + outputFile + " is up-to-date"); + continue; } + try { + StringWriter out = new StringWriter(); + MonitorWriter monitorOut = new MonitorWriter(out); + generateFromClass(monitorOut, clazz); + write(outputFile, monitorOut); + } catch (Exception eee) { + log.warn("Erreur lors de la génération du fichier " + outputFile); + throw new RuntimeException("Erreur lors de la génération du fichier " + outputFile, eee); + } } }