--- xmlTools/trunk/posUpdate/XmlTools.cs 2013/05/08 21:52:38 861 +++ xmlTools/trunk/posUpdate/XmlTools.cs 2013/05/09 10:37:17 867 @@ -40,7 +40,7 @@ namespace xmlTools List myElements = new List(); Util.getAllSpecificElements(xdoc.DocumentElement, ref myElements, this.posElement, this.posParentElement); //Returns all after "Oni" element - if (valuePositions != "") + if (!String.IsNullOrEmpty(valuePositions)) { checkValidSpecificPositions(valuePositions, value, myElements[0].InnerText); } @@ -49,11 +49,11 @@ namespace xmlTools { if (element.Name == posElement) { - if (posParentElement != "" && element.ParentNode.Name != posParentElement) + if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement) { continue; } - if (valuePositions == "") //replace every value for the new + if (String.IsNullOrEmpty(valuePositions)) //replace every value for the new { element.InnerText = value; } @@ -90,7 +90,7 @@ namespace xmlTools { if (element.Name == posElement) { - if (posParentElement != "" && element.ParentNode.Name != posParentElement) + if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement) { continue; } @@ -140,7 +140,7 @@ namespace xmlTools { if (element.Name == posElement) { - if (posParentElement != "" && element.ParentNode.Name != posParentElement) + if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement) { continue; } @@ -184,7 +184,7 @@ namespace xmlTools { if (element.Name == posElement) { - if (posParentElement != "" && element.ParentNode.Name != posParentElement) + if (!String.IsNullOrEmpty(posParentElement) && element.ParentNode.Name != posParentElement) { continue; } @@ -229,7 +229,7 @@ namespace xmlTools List myElements = new List(); Util.getAllSpecificElements(xdoc.DocumentElement, ref myElements, this.posElement, this.posParentElement); //Returns all after "Oni" element - if (valuePositions == null) + if (String.IsNullOrEmpty(valuePositions)) { newXmlLastPos = new XmlNumberValue(newValue); } @@ -238,18 +238,18 @@ namespace xmlTools newXmlLastPos = new XmlNumberValue(replaceSpecificPositions(myElements[0].InnerText, newValue, valuePositions)); } - if (valuePositions != null) + if (!String.IsNullOrEmpty(valuePositions)) { checkValidSpecificPositions(valuePositions, newValue, myElements[0].InnerText); } foreach (XmlNode element in myElements) { - if (element.Name == this.posElement && (this.posParentElement == "" || this.posParentElement == element.ParentNode.Name)) + if (element.Name == this.posElement && (String.IsNullOrEmpty(this.posParentElement) || this.posParentElement == element.ParentNode.Name)) { XmlNumberValue xmlCurrValue = new XmlNumberValue(element.InnerText); - if (xmlLastPos != null) + if (xmlLastPos!=null) { newXmlLastPos = XmlNumberValue.sum(newXmlLastPos, XmlNumberValue.difference(xmlCurrValue, xmlLastPos)); element.InnerText = newXmlLastPos.ToString(); @@ -257,7 +257,7 @@ namespace xmlTools } else { // first time just assign to last value - if (valueRelation != null) + if (!String.IsNullOrEmpty(valueRelation)) { newXmlLastPos = XmlNumberValue.difference(newXmlLastPos, XmlNumberValue.difference(new XmlNumberValue(valueRelation), xmlCurrValue)); }