Как я могу опровергнуть этот код 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; 
} 

Я вижу, что единственное, что получает изменилось в вашем URL является 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;