protected void Page_Load(object sender, EventArgs e)
{
if (!Page.IsPostBack)
{
loadCountries();
loadRegions();
loadCities();
}
}
private void loadCountries()
{
Country country = new Country();
ddlCountry.DataSource = country.GetDataTable();
ddlCountry.DataTextField = "countryName";
ddlCountry.DataValueField = "countryID";
ddlCountry.DataBind();
}
private void loadRegions()
{
Region region = new Region();
ddlRegion.DataSource = region.GetRegionID(ddlCountry.SelectedValue);
ddlRegion.DataTextField = "regionName";
ddlRegion.DataValueField = "regionID";
ddlRegion.DataBind();
}
private void loadCities()
{
City city = new City();
ddlCity.DataSource = city.GetCityID(ddlRegion.SelectedValue);
ddlCity.DataTextField = "cityName";
ddlCity.DataValueField = "cityID";
ddlCity.DataBind();
}
protected void ddlCountry_SelectedIndexChanged(object sender, EventArgs e)
{
loadRegions();
if (ddlRegion.SelectedItem.Text == "No Province")
{
ddlRegion.Enabled = false;
loadCities();
}
else
{
ddlRegion.Enabled = true;
loadCities();
}
}
コードがは私のコードデザインを評価する - シンプルなスニペット
国に関連するビジネスロジックは、国のクラスに入れているのDefault.aspx(プレゼンテーション層)のバックエンドで、同じ規則は、地域や都市に適用されます。
このスニペットのデザインは問題ありませんか?言い換えれば、プレゼンテーションレイヤーの標準デザインを満たしていますか?このスニペットのデザインを改善するにはどうすればいいですか?
私はこれに新しいです、私はそれをゆっくりと確実に取るようにしています。
これはOKです... – Xaqron