我该如何优化asp.net的代码

Question

我想优化下面写的代码，如果你有答案请快速回复我，在这段代码中我想提出不同的条件并在每个条件下返回不同的url如果有可能与优化的方式，然后回答我：

if (Offer1_Rb_Yes.Checked == true || DropDownList1.SelectedIndex > 0) 
    { 
     int offerid = MyOffers[0].OfferId; 
     DAL.offers Offer = new DAL.offers(); 
     Offer = obj.GetOffer(offerid); 
     if (Offer.CampId == "WINE-MAKERS-CHOICE") 
     { 
      url1 = url1 + "WINE-MAKERS-CHOICE&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&c1=" + Gender; 
     } 
     else if (Offer.CampId == "LETS-INSURE-CA") 
     { 
      url1 = url1 + "LETS-INSURE-CA&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&c1=" + Age; 
     } 
     else if (Offer.CampId == "SOLAR-BROKER") 
     { 
      url1 = url1 + "SOLAR-BROKER&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&c1=" + Age; 
     } 
     else if (Offer.CampId == "WENATEX-AU") 
     { 
      url1 = url1 + "WENATEX-AU&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&street1=" + street1 + "&towncity=" + suburb + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&source=" + Source; 
     } 
     else if (Offer.CampId == "MODERN-SOLAR-COMP") 
     { 
      url1 = url1 + "MODERN-SOLAR-COMP&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&street1=" + street1 + "&towncity=" + suburb + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&phone2=" + Phone2 + "&gender=" + Gender + "&solar_pv=" + Solar_pv; 
     } 
     else if (Offer.CampId == "CAREERS-AUSTRALIA-3") 
     { 
      url1 = url1 + "CAREERS-AUSTRALIA-3&sid=TPF34" + "&firstname=" + FirstName + "&lastname=" + LastName + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1; 
     } 
     else if (Offer.CampId == "GOOD-LIFE") 
     { 
      url1 = url1 + "GOOD-LIFE&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&postcode=" + PostCode + "&phone1=" + Phone1; 
     } 
     else if (Offer.CampId == "GRYPHON-SOLAR") 
     { 
      url1 = url1 + "GRYPHON-SOLAR&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&county=VIC&postcode=" + PostCode + "&phone1=" + Phone1 + "&source=" + Source + "&age=" + Age; 
     } 
     else if (Offer.CampId == "GRYPHON-SOLAR-WA") 
     { 
      url1 = url1 + "GRYPHON-SOLAR-WA&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&county=WA&postcode=" + PostCode + "&phone1=" + Phone1 + "&source=" + Source + "&age=" + Age; 
     } 
     else if (Offer.CampId == "ACQUIRE-LEARNING") 
     { 
      url1 = url1 + "ACQUIRE-LEARNING&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&industry=" + Industry; 
     } 
     else if (Offer.CampId == "KOGAN") 
     { 
      url1 = url1 + "KOGAN&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&towncity=" + suburb + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&ipaddress=" + ipaddress; 
     } 
    } 

Answer 1

我会把它下面写：

if (offer5_Rb_Yes.Checked == true) 
{ 
    int offerid = MyOffers[4].OfferId; 
    DAL.offers Offer = new DAL.offers(); 
    Offer = obj.GetOffer(offerid); 
    url5=url5+Offer.CampId+"&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&c1=" + Age; 
    return url5; 
} 

我看到唯一获取您的网址更改为Offer.CampId和其他部分保持不变！所以你可以像上面那样做！

UPDATE

我只注意到的是，在第一个条件你url有在年底小的变化！不知道是有意的还是错误的！如果是故意的，那么你可以尝试使用ternary operators为您url只为一个条件的变化，并保持相同的其他2

url5= Offer.CampId=="WINE-MAKERS-CHOICE"? 
     url5+Offer.CampId+"&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&c1=" + Gender 
     :url5+Offer.CampId+"&sid=TPF34" + "&email=" + Email + "&firstname=" + FirstName + "&lastname=" + LastName + "&dob=" + DOB + "&county=" + State + "&postcode=" + PostCode + "&phone1=" + Phone1 + "&c1=" + Age;