Скрыть область видимости переменной?

Question

[Alex XYZ]

Всем привет.

Из области "хочу". Хочу в методе/функции скрыть переменную из объявленной ранее в коде.

int i=0;
.
.
.
for(int j=0; j<=N; j++){
    // Вот тут хочу, чтобы i была не видна и я получал ошибку компиляции.
    // Очевидно, что я хочу использовать в качестве индекса массива переменную "j",
    // но из-за того, что применил copy-paste мог не заметить.
    if(elem[i]==null){
    }
}

// Тут нужно вернуть "видимость" i.

Понимаю, что немного перебор, но вдруг есть решение или даже костыль?

P.S.
Вынос кода в другую функцию не предлагать )))

Comments

[Alex XYZ]

SmInc, Это-то я знаю. Внутри блоков можно скрывать переменные от внешнего доступа, но в данном случае переменная i объявлена в корне области видимости. Понимаю, что даже слова "область видимости" намекают, но ведь вроде нигде не написано, что её нельзя отменить?

[Даша Циклаури]

Захаров Александр, нельзя, для проверки таких кейсов существуют статические анализаторы, которые проверяют код на подозрительные места (например переменная j не используется)

[Alex XYZ]

Даша Циклаури, Никто не сказал, что она не используется в коде совсем. Я просто привёл пример только по отношению к переменной i.

[Даша Циклаури]

Захаров Александр, нууу тогда пиши тесты, так можно сказать про все что угодно. Там вообще может быть тьма кода, именовать переменные правильно я тебе советовать не буду - поможет частично. Так для этого есть юнит-тесты. Старайся писать чистые функции, их тестировать легче. Это что касается билд-тайм. А дальше код ревью другим разрабом.

Answer 1

[#]

в заданном наборе требований решения нет

Comments

[Alex XYZ]

Предположим, что решения в таком наборе условий нет, но если задача бороться с совпадением перемен при copy-paste, то может быть что-то может предложить рефакторинг? например, сообщить, что при вставке обнаружены совпадения имён переменных. И показать их список. Тоже было бы подспорье. Не знаете такого?

[#]

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

Answer 2

[Алексей Павлов]

Переменную можно поместить в блок, тогда переменная будет ограничена только этим блоком:

int Test()
{
    int result = 0;

    {
        int i = 10;
        for (int j = 0; j < 10; j++)
        {
            result += i;
        }
    }
    {
        string i = "xyz";
        Console.WriteLine(i);
    }
    return result;
}

Comments

[Alex XYZ]

Спасибо, но это фокус я знаю. Я там в примере кода косвенно указал что переменная i находится в корне области видимости.
Речь идёт о корневой области видимости.

Answer 3

[sttrox]

Одним из вариантов может быть удаление переменной i.
После удаления будут подсвечены все использования переменной i

Comments

[#]

а можно пример, как удалить int в C#

[sttrox]

Гражданин, видимо Вы владеет прекраснейшем даром -- виденье между строк.

Пояснение

тип int != переменной i

Удалив строку
int i=0;
Получим ошибку среды разработки о не объявленной переменной, что визуально выделит использование переменно i. Чего автору будет достаточно для обнаружения ошибки

[GavriKos]

sttrox, ну как бы давно есть стандартные средства типа "показать все места где используется переменная" - find all references. Чтобы не делать вот такую вот фигню с удалением.

[Alex XYZ]

Шарпер, >>а можно пример, как удалить int в C#
Но как узнать, что эта переменная вообще была в коде copy-paste, чтобы хотеть выполнить это действие?

[sttrox]

Захаров Александр, если не желаете пользоваться поисковиком, написав
int i;
в начале всех скопированных функций и классов Вы получите этот поиск от среды разработки

[Alex XYZ]

sttrox, Т.е. просмотреть код на предмет объявленных переменных перед copy-paste?

[#]

sttrox, топикстартер уже во многих местах говорил что ранее объявленная декларация неприкасаема

так что если вы про удаление строки в исходнике - это ему объясняйте

Answer 4

[GavriKos]

Вынести цикл в отдельный метод.

Comments

[Александр]

Или в локальную функцию.

[Alex XYZ]

Александр, Но вроде как локальная функция обладает той же областью видимости где и находится?

[Alex XYZ]

GavriKos, в условии написано отдельного метода/функции не предлагать )))

[GavriKos]

Захаров Александр, по моему добавили после моего ответа. Ну и в любом случае - это чуть ли не единственный вариант ))

[Alex XYZ]

GavriKos, Да, менял. Не сразу в голову пришло. Извините.

[Александр]

Захаров Александр, таки да, прошу прощенья.

Answer 5

[lam0x86]

Можно написать Code Analyzer, который будет выдавать ошибку компиляции в случае использовании переменной внутри блока, помеченного каким-либо способом, например специальным комментарием с именами запрещенных переменных:

int i = 0;
// HIDE: i
for(int j=0; j<=N; j++) {
  if(elem[i]==null) { // Ошибка компиляции: использование запрещённой переменной.
  }
}
// SHOW: i
i++; // Нет ошибки.

или блоком using:

int i = 0;
using (CompilerUtils.HideVar(i))
{
  for(int j=0; j<=N; j++){
    if(elem[i]==null) { // Ошибка компиляции: использование запрещённой переменной
    }
  }
}
i++; // OK.

Сам код HideVar может не делать ничего:

public static class CompilerUtils
{
  public static IDisposable HideVar(params object[] vars)
  {
    return <пустая реализация IDisposable>;
  }
}