Ініціалізація рядків за замовчуванням: NULL або Empty? [зачинено]

Я завжди ініціалізував свої рядки до NULL, маючи на увазі, що NULL означає відсутність значення та "" або String.Empty - це дійсне значення. Останнім часом я бачив більше прикладів коду, де String.Empty вважається значенням за замовчуванням або не представляє значення. Це здається мені дивним, коли щойно додані нульові типи в c # здається, ми робимо кроки назад за допомогою рядків, не використовуючи NULL для позначення "Без значення".

Що ви використовуєте як ініціалізатор за замовчуванням і чому?

Редагувати: Грунтуючись на відповідях, я розвиваю подальші думки

1. Уникнення поводження з помилками Якщо значення не повинно бути нульовим, чому воно встановилося NULLв першу чергу? Можливо, було б краще визначити помилку в місці, де вона виникає, а не приховати її через решту вашої бази коду?

2. Уникнення нульових перевірок Якщо ви втомилися робити нульові перевірки в коді, чи не було б краще абстрагувати нульові чеки? Можливо, оберніть (або розтягніть!) Рядкові методи, щоб зробити їх NULLбезпечними? Що трапиться, якщо ви постійно користуєтесь, String.Emptyа нуль працює, це входить у вашу систему, ви все одно починаєте додавати NULLчеки?

Не можу не повернутись до думки, що це лінь. Будь-яка DBA ударить вас дев'ятьма способами безглуздо, якби ви використовували '' замість nullйого бази даних. Я думаю, що ті ж принципи застосовуються і в програмуванні, і повинен бути хтось, хто б'є по голові тих, хто використовує, String.Emptyа не NULLпредставляє ніякої цінності.

Пов’язані запитання

• Чи слід використовувати C #: string.Empty або String.Empty або “”?
• Яка різниця між String.Empty та ""
• Пустий або порожній рядок не відображає даних у стовпці таблиці?

+1 для розрізнення "порожній" та NULL. Я погоджуюся, що "порожній" повинен означати "дійсний, але порожній", а "NULL" повинен означати "недійсний".

Тож я відповів би на ваше запитання так:

порожній, коли я хочу дійсне значення за замовчуванням, яке може бути або не може бути змінено, наприклад, прізвище користувача.

NULL, коли це помилка, якщо наступний код не встановлює значення явно.

За даними MSDN :

Ініціалізуючи рядки зі Emptyзначенням замість null, ви можете зменшити шанси на NullReferenceExceptionвиникнення.

Завжди використовувати IsNullOrEmpty()хорошу практику.

Чому ви хочете, щоб ваша рядок взагалі була ініціалізована? Не потрібно ініціалізувати змінну, коли ви оголошуєте її, і IMO, ви повинні робити це лише тоді, коли значення, яке ви призначаєте, дійсне в контексті блоку коду.

Я бачу це багато:

string name = null; // or String.Empty
if (condition)
{
  name = "foo";
}
else
{
  name = "bar";
}

return name;

Неініціалізація до нуля була б настільки ж ефективною. Крім того, найчастіше потрібно призначити значення. Ініціалізуючи null, ви потенційно можете пропустити кодові шляхи, які не присвоюють значення. Так:

string name = null; // or String.Empty
if (condition)
{
  name = "foo";
}
else if (othercondition)
{
  name = "bar";
}

return name; //returns null when condition and othercondition are false

Коли ви не ініціалізуєтеся на null, компілятор генерує помилку, кажучи, що не всі кодові шляхи призначають значення. Звичайно, це дуже простий приклад ...

Маттійс

Для більшості програмного забезпечення, яке насправді не є рядковим програмним забезпеченням, логіка програми не повинна залежати від вмісту струнних змінних. Щоразу, коли я бачу щось подібне в програмі:

if (s == "value")

У мене погані відчуття. Чому в цьому методі є рядковий літерал? Що налаштування s? Чи відомо, що логіка залежить від значення рядка? Чи знає він, що це має бути нижчим регістром для роботи? Чи слід це виправляти, змінюючи його на використання String.Compare? Чи повинен я створювати Enumі аналізувати його?

З цього погляду можна потрапити до філософії коду, яка є досить простою: ви уникаєте вивчення вмісту рядка, де це можливо. Порівняння рядка зString.Empty справді - це лише особливий випадок порівняння його з буквальним: цього потрібно уникати, якщо цього не потрібно.

Знаючи це, я не моргну, коли бачу щось подібне в нашій кодовій базі:

string msg = Validate(item);
if (msg != null)
{
   DisplayErrorMessage(msg);
   return;
}

я це знаю Validate ніколи не повернетьсяString.Empty , тому що ми пишемо кращий код, ніж цей.

Звичайно, решта світу не працює так. Коли ваша програма має справу з введенням користувача, базами даних, файлами тощо, вам доведеться враховувати інші філософії. Там завдання вашого коду - накладати порядок на хаос. Частина цього порядку - це знати, коли має означати порожній рядокString.Empty і коли він повинен означатиnull .

(Просто для того, щоб переконатися, що я не розмовляв із своєю дупою, я просто шукав нашу кодову базу на предмет "String.IsNullOrEmpty". Усі 54 події в ній знаходяться в методах, які обробляють введення користувача, повертають значення з сценаріїв Python, вивчають значення, отримані з зовнішні API та ін.)

Це насправді зяюча дірка у мові C #. Немає можливості визначити рядок, який не може бути нульовим. Це спричиняє проблеми настільки ж прості, як та, яку ви описуєте, що змушує програмістів приймати рішення, якого вони не повинні приймати, оскільки в багатьох випадках NULL та String.Empty означають те саме. Це, в свою чергу, може згодом змусити інших програмістів працювати з NULL і String.Empty, що дратує.

Більшою проблемою є те, що бази даних дозволяють визначати поля, які відображають у рядок C #, але поля бази даних можуть бути визначені як NOT NULL. Отже, немає способу точно представити, скажімо, поле varchar (100) NOT NULL у SQL Server, використовуючи тип C #.

Інші мови, наприклад Spec #, дозволяють це зробити.

На мій погляд, нездатність C # визначити рядок, який не дозволяє null, так само погано, як і його попередня нездатність визначити int, яка дозволяє null.

Щоб повністю відповісти на ваше запитання: я завжди використовую порожню рядок для ініціалізації за замовчуванням, оскільки це більше схоже на те, як працюють типи даних бази даних. (Редагувати: Цей вислів був дуже незрозумілим. У ньому слід писати "Я використовую порожню рядок для ініціалізації за замовчуванням, коли NULL є зайвим станом, так само, як я встановив стовпчик бази даних як НЕ НУЛЬНИЙ, якщо NULL був би зайвим станом". , багато моїх стовпців БД налаштовані як НЕ NULL, тому коли я переношу їх у рядок C #, рядок буде порожнім або матиме значення, але ніколи не буде NULL. Іншими словами, я ініціалізую лише рядок у NULL якщо null має значення, яке відрізняється від значення String.Empty, і я вважаю, що цей випадок рідше (але тут люди наводили законні приклади цього випадку). ")

Це залежить.

Чи потрібно мати змогу сказати, чи не вистачає значення (чи не можна його визначити)?

Чи порожній рядок є дійсним значенням для використання цього рядка?

Якщо ви відповіли "так" обом, тоді ви хочете використовувати null. Інакше ви не можете вказати різницю між "без значення" та "порожнім рядком".

Якщо вам не потрібно знати, чи немає значення, порожній рядок, ймовірно, безпечніший, оскільки він дозволяє пропускати нульові чеки там, де ви його використовуєте.

Схоже, це особливий випадок http://en.wikipedia.org/wiki/Null_Object_pattern

Я або встановив його на "" або null - я завжди перевіряю за допомогою String.IsNullOrEmpty, тому або добре.

Але внутрішній вуха в мені говорить, що я повинен встановити його на нуль, перш ніж мати для нього належне значення ...

Я завжди оголошую рядок з string.empty;

Чи можливо це метод уникнення помилок (бажано чи ні ..)? Оскільки "" все ще є рядок, ви могли б викликати на ньому рядкові функції, що призвело б до винятку, якби це було NULL?

Я завжди ініціалізую їх як NULL.

Я завжди використовую string.IsNullOrEmpty(someString)для перевірки його цінності.

Простий.

Це залежить від ситуації. У більшості випадків я використовую String.Empty, тому що я не хочу робити нульові перевірки кожен раз, коли я намагаюся використовувати рядок. Це робить код набагато простішим, і ви рідше вводите небажані збої NullReferenceException.

Я встановлюю нульовий рядок лише тоді, коли мені потрібно знати, встановлений він чи ні, і де порожня рядок є чимось дійсним для її встановлення. На практиці я вважаю ці ситуації рідкими.

Порожній рядок - це значення (фрагмент тексту, який, до речі, буває, не містить літер). Null означає не-значення.

Я ініціалізую змінні до нуля, коли хочу вказати, що вони не вказують на або містять фактичні значення - коли у наміру немає значення.

Повторюючи відповідь Tomalak, майте на увазі, що при призначенні змінної рядка початковому значенню null, ваша змінна більше не є об'єктом рядка; те саме з будь-яким об'єктом в C #. Отже, якщо ви спробуєте отримати доступ до будь-яких методів або властивостей для вашої змінної, і ви вважаєте, що це об'єкт рядка, ви отримаєте виняток NullReferenceException.

Null слід використовувати лише у випадках, коли значення необов’язкове. Якщо значення не є необов’язковим (наприклад, "Ім'я" або "Адреса"), то значення ніколи не повинно бути нульовим. Це стосується баз даних, а також POCO та інтерфейсу користувача. Null означає "це значення необов’язкове і наразі відсутнє".

Якщо ваше поле є необов’язковим, тоді слід ініціалізувати його як порожній рядок. Якщо ініціалізувати його як null, то об'єкт переведе в недійсний стан (недійсний за вашою власною моделлю даних).

Особисто я вважаю за краще, щоб рядки не були нульовими за замовчуванням, а замість цього лише нульовими, якщо ми оголосили "рядок?". Хоча, можливо, це не здійснене чи логічне на більш глибокому рівні; не впевнений.

Рядки не є типовими значеннями і ніколи їх не буде ;-)

Я думаю, що немає причин не використовувати null для непризначеного (або в цьому місці в потоці програми не зустрічається) значення. Якщо ви хочете розрізнити, є == null. Якщо ви просто хочете перевірити певне значення і не байдуже, чи це нульове чи щось інше, String.Equals ("XXX", MyStringVar) справляється чудово.