Что лучше, если "заявление" использовать в C #

Question

Оба эти метода работают, однако, какой метод лучше использовать из следующих:

PlaceHolder PH = ctl.PlaceHoldNu == 1 ? (PlaceHolder)Page.Master.FindControl("PlaceHolder1") : (PlaceHolder)FindControl("PlaceHolder" + ctl.PlaceHoldNu);

Или

           PlaceHolder PH;
                    if (ctl.PlaceHoldNu == 1)
                        PH = (PlaceHolder)Page.Master.FindControl("PlaceHolder1");
                    else
                        PH = (PlaceHolder)FindControl("PlaceHolder" + ctl.PlaceHoldNu);

Answer 1

Другое решение будет:

PlaceHolder PH = ctl.PlaceHoldNu == 1
    ? (PlaceHolder)Page.Master.FindControl("PlaceHolder1") 
    : (PlaceHolder)FindControl("PlaceHolder" + ctl.PlaceHoldNu);

В этом решении используется минимум строк, но поддерживается удобочитаемость.

Answer 2

Это вопрос вкуса.Однако, если они получатся слишком длинными, я бы предпочел второе.Но до того, как это произойдет, я думаю, что это выглядит хорошо:

PlaceHolder PH =
    ctl.PlaceHoldNu == 1
    ? (PlaceHolder)Page.Master.FindControl("PlaceHolder1")
    : (PlaceHolder)FindControl("PlaceHolder" + ctl.PlaceHoldNu);

Answer 3

Обычно я использую условный оператор только в назначениях (как вы сделали), но только тогда, когда они просты и удобны для чтения.

Длина ваших заданий заставит меня отдать предпочтение второму по удобочитаемости.

По существу, любой из них легче всего анализировать, если вы предполагаете, что никогда раньше не просматривали код.

Answer 4

В этом случае мне нравится первый, так как я чувствую, что он более читабелен и лаконичен.

Немного сложнее, и ваш второй подход будет более предпочтительным.

Но это такой же собственный вкус, как правильный

Answer 5

Лучшим является тот, который вы предпочитаете использовать.

Они будут компилироваться до одного и того же, так что это зависит от контекста.Какой из них делает цель того, что вы пытаетесь сделать, более ясной?