C#

if (string.isNullOrEmpty(apiVariableName)) { return ; }

internal class Shipment {
        [Required]
        [StringLength(12)]
        public string ShippingNumber { get; set; }

        [Required(ErrorMessage = "City cannot be empty.")]
        public string City { get; set; }

        [Required(ErrorMessage = "Zip is Required")]
        [RegularExpression(@"^\d{5}(-\d{4})?$", ErrorMessage = "Invalid Zip")]
        public string PostalCode { get; set; }
    }

var prc = new Data.General.Shipment();
                    var node = mainNode.SelectSingleNode("TrackingNumber");

                    if (node == null)
                    {
                        return; // If your method is not void return the appropriate data type, null, false, etc...
                    }

                    decimal trackingNumber;

                    if (!decimal.TryParse(node.InnertText, out trackingNumber))
                    {
                        return; // Same as above with return type
                    }

                    prc.GetByProNumber(trackingNumber);

                    try
                    {
                        rspxml.Root.Add(new XElement("API", "4.0"));
                        rspxml.Root.Add(new XElement("PackageTrackingInfo"));
                        rspxml.Root.Element("PackageTrackingInfo").Add(new XElement("TrackingNumber", prc.ProNumber.ToString())); // These ToString calls are dangerous without first validating the data
                        rspxml.Root.Add(new XElement("PackageDestinationLocation"));
                        rspxml.Root.Element("PackageDestinationLocation").Add(new XElement("City", prc.Consignee.ToString()));
                        rspxml.Root.Element("PackageDestinationLocation").Add(new XElement("StateProvince", prc.Consignee.ToString()));
                        rspxml.Root.Element("PackageDestinationLocation").Add(new XElement("PostalCode", prc.Consignee.ToString()));
                        rspxml.Root.Element("PackageDestinationLocation").Add(new XElement("CountryCode", prc.Consignee.ToString()));
                        // remainder omitted for brevity
                    }
                    catch (NullReferenceException n)
                    {
                        // NullReferenceException is not super helpful but it will tell you there is an error and provides a general catch all.
                        // Log or alert user there is a problem
                        // A more defensive strategy is to validate that prc.Consignee is not null before trying to covnert to a string
                    }

SqlConnection conn = new SqlConnection(); //properly initiliased connection object with connection string
          SqlCommand command1 = new SqlCommand(); //properly initiliased command with sql text or procedure name
          SqlCommand command2 = new SqlCommand(); //properly initiliased command with sql text or procedure name

          SqlTransaction trx = conn.BeginTransaction();
          command1.Transaction = trx;
          command2.Transaction = trx;

          try
          {
              command1.ExecuteNonQuery();
              command2.ExecuteNonQuery();
              trx.Commit();
          }
          catch (SqlException ex)
          {
              trx.Rollback();
          }
          finally
          {
              //clean up resources
          }

Quote:

I thought I assigned it properly.

int cookies = 2; // Or any

// Now the framework allocates a memory for "cookies" and places "2" there. 

Member 13413476 wrote:

use of unassigned local variable