while reading xml file line by line need to update some tag based on class name in java

Question

i am reading one xml file to another file it is fine but while reading i am checking condition for updation of one statement but it is not updating, please suggest me on this

public class PutMesageDebugger {


    public static void main(String[] args) throws IOException {

    BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream("C:/PutMessageDebugger/Sub_Workflow_HSS.xml"), "UTF-8"));
    BufferedWriter writer = new BufferedWriter(new OutputStreamWriter(new FileOutputStream("C:/PutMessageDebugger/tempSub_Workflow_HSS.xml"), "UTF-8"));

     String line = null;

     while ((line = reader.readLine()) != null){                   
     //System.out.println("line"+ line);

      //System.out.println(line.contains("<Process-Node"));
        /*System.out.println(line.contains("<Class-Name>com.hp.ov.activator.mwfm.component.builtin.PutMessage</Class-Name>"));
           System.out.println(!line.contains("inactive=true"));
                         */
         if ( line.contains("<Class-Name==com.hp.ov.activator.mwfm.component.builtin.PutMessage")){
             line = line.replaceAll("<Process-Node","<Process-Node inactive=\"true\" ");

                  }

             writer.write(line + "\n");



                  }

              reader.close();
              writer.close();

           }

I would strongly avoid reading XML line by line to start with. Why not read it *as an XML document*? At the moment your code is *very* sensitive to the exact format, which appears to contain a rather bizarre element name. If it's actually valid XML, treat it as such. You may be able to just use XSLT to perform the transformation you want, in fact. — Jon Skeet, Jan 13 '16 at 06:48
Hi Jon, xml transaformation is fine but in that if condition if(line.contains(" — shekar vanamala, Jan 13 '16 at 06:56

score 0 · Answer 1 · edited May 23 '17 at 10:28

0

It is not a good idea to parse XML like that. You should use a XML library, which takes care of white-space handling and many other things for you. Take a look at the javax.xml package.

If all you want to do is to add the inactive="true" attribute to the ProcessNode element, you could go for XSLT. There are many resources on how to do that, e.g. add attribute example

edited May 23 '17 at 10:28

Community

1
1

answered Jan 13 '16 at 06:56

stevops

429
3
9

can you share any example of XSLT – shekar vanamala Jan 13 '16 at 07:00
not for all processNode tags only class name in some put message – shekar vanamala Jan 13 '16 at 07:03
inactive="true " is not for all proccesNodes only class name is equal to putmessage – shekar vanamala Jan 13 '16 at 07:06
The link provided shows an example. If you want me to write the transformation for you, you will at least have to post your XML document instance. – stevops Jan 13 '16 at 07:06
how to share my xml its too long – shekar vanamala Jan 13 '16 at 07:16
share the relevant fragment! – stevops Jan 13 '16 at 07:22
PutMessage42 com.hp.ov.activator.mwfm.component.builtin.PutMessage Update the sub order status "rollback failure" – shekar vanamala Jan 13 '16 at 07:25
like this to add – shekar vanamala Jan 13 '16 at 07:27

while reading xml file line by line need to update some tag based on class name in java

1 Answers1