Click to See Complete Forum and Search --> : please check the format


vinit005
June 12th, 2008, 12:55 AM
if (!(GetField(header, fields, "Declared Value for Carriage").Equals("") || GetField(header, fields, "Declared Value for Carriage").Equals(null)) && !(GetField(header, fields, "Number of Packages").Equals("") || GetField(header, fields, "Number of Packages").Equals(null)))
{
writer.WriteElementString("InsuranceAmount", (int.Parse(GetField(header, fields, "Declared Value for Carriage")) / int.Parse(GetField(header, fields, "Number of Packages"))).ToString());
//writer.WriteElementString("InsuranceAmount", "");
}
else
{
writer.WriteElementString("InsuranceAmount", "");
//writer.WriteElementString("InsuranceAmount", (Convert.ToInt16(GetField(header, fields, "Declared Value for Carriage")) / Convert.ToInt16(GetField(header, fields, "Number of Packages"))).ToString());
}









i m getting error in the third line .the error states that the input string format is not correct.please help me............

cilu
June 12th, 2008, 01:17 AM
I cannot follow your code. What about reorganizing a little it. Introducing some variables to simplify it?

yraen
June 12th, 2008, 08:52 AM
int declaredValue = int.Parse(GetField(header, fields, "Declared Value for Carriage"));
int packageNumber = int.Parse(GetField(header, fields, "Number of Packages"));
writer.WriteElementString("InsuranceAmount", Convert.ToString(declaredValue / packageNumber));

Make it cleaner... :sick: