Процедура идеологически правильна.. Лишь с тем уточнением что все-таки на Select в качестве параметра стоит передавать таки @Original_CityID а не @CityID, хотя в данном контексте это не столь важно. А процедура твоя выполнится в один бач-проход так что одновременный Update с последующим Selectom других данных невозможен по определению.

А параметры по умолчанию ты можешь выставить и в коллекции Parameters у самого SqlCommand, инициализировав свойство Value, если тебе не хочется исправлять процедуру.

Вот, это я нашёл, но пдскажи плиз, что я там могу написать или где об этом можно почитать? Тоесть там в value я могу подставлять только константу или можно вписать туда переменную? типа this.label1.text
Спасибо

Да пожалуста:

this.SqlAdapterYou.SelectCommand.Parameters["@CityID"].Value =
				this.label1.text;

this.SqlAdapterYou.UpdateCommand.Parameters["@CityID"].Value =
				this.label1.text;

ну или

this.sqlCommandYou.Parameters["@CityID"].Value =
				this.label1.text;

А потом (после инициализации параметра ) делай Update или Fill, что по смыслу тебе нужно...

this.SqlAdapterYou.Update(this.YourDataSet);