Полезно ли инициализировать переменные с нулевыми / пустыми значениями перед назначением «реальных» значений?

Question

Я пытаюсь обновить старый модуль словаря данных на моем новом рабочем месте.У меня есть какой-то код, который кажется излишним, но я не уверен, существует ли он из-за «наилучшей практики» или, возможно, по какой-то другой причине, которую я не знаю.

SqlCommand objSQLCommand = null;
string strquery = string.Empty;
strquery = string.Format("SELECT Name FROM sys.databases ORDER BY name");
objSQLCommand = new SqlCommand(strquery);

Мне кажется,как это можно сделать в две строки (максимум), но я не совсем уверен.

Answer 1

Для этого запроса string.Format() кажется бесполезным. Это, вероятно, было скопировано из другого запроса с параметрами на мой взгляд.
Однако string.Format() все еще очень плохо для использования параметров в запросе. Это может привести к проблемам безопасности .
Что вы можете сделать, чтобы очистить этот код:

SqlCommand cmd = new SqlCommand("SELECT Name FROM sys.databases ORDER BY name");

Тем не менее, я настоятельно рекомендую вам прочитать некоторые документы и получить необходимые знания, чтобы понять, почему используются или не используются части этого кода.