0

Есть код

public RegClassSave(int sumDay, DateTime date)
    {
        try
        {
            int RegResult = SumResultDay;

            SumDay = sumDay;
            DateN = date;

            RegistryKey KnockOff = currentUserKey.CreateSubKey("KnockOff");

            KnockOff.SetValue("Amout", SumDay);
            KnockOff.SetValue("AmoutResult", SumResultDay);
            KnockOff.SetValue("Date", DateN);
            KnockOff.Close();

            string Amout = currentUserKey.OpenSubKey("KnockOff", true).GetValue("Amout").ToString();
            string AmoutResult = currentUserKey.OpenSubKey("KnockOff", true).GetValue("AmoutResult").ToString();
            string Date = currentUserKey.OpenSubKey("KnockOff", true).GetValue("Date").ToString();

            RegResult += Convert.ToInt32(Amout) + Convert.ToInt32(AmoutResult);       

            Console.WriteLine("Введённая сумма = {0} Результат = {1} Дата внесения = {2}", Amout, RegResult, Date);
        }
        catch (Exception e)
        {
            Console.WriteLine("Ошибка {0}", e);
        }
    }       
}

В реестр всё добовляеться. Я не как не могу сделаь так что бы после запуска второй раз и ввода ещё одного числа, в реестор на место поля (AmoutResult) записывалось уже новое число (Amout + AmoutResult) тоесть сумма чисел. Считывание тоже работает.

Вызов делаю так

class Program
    {
        static void Main(string[] args)
        {
            int EnterSum = Convert.ToInt32(Console.ReadLine());


            RegClassSave regClassSave = new RegClassSave(EnterSum, DateTime.Now);
    }
}

Не могу понять по чему не работает.

4
  • А где Вы считываете старое значение AmoutResult из реестра? 28 апр 2018 в 12:47
  • @default locale Извеняюсь я не тот код скинул. Сейчас правельный код.
    – j. Atisto
    28 апр 2018 в 13:14
  • Зря Вы описание класса убрали, теперь непонятно откуда SumResultDay, лучше верните его. Судя по всему, Вы больше нигде не считываете AmoutResult правда ведь? 28 апр 2018 в 13:43
  • @default locale RegResult += Convert.ToInt32(Amout) + Convert.ToInt32(AmoutResult); вот здесь считываю и добовляю значение в int RegResult = SumResultDay; и таким образом при следующем вызове значение присваеваеться вSumResultDay. Ну по крайне мере задумка такая.
    – j. Atisto
    28 апр 2018 в 13:50

1 ответ 1

1

Здесь считываю и добавляю значение в int RegResult = SumResultDay; и таким образом при следующем вызове значение присваивается вSumResultDay

C# так не работает. RegResult — локальная переменная. Она каждый раз создается заново и изменения ее значения ни на чем не отражаются, если этого отдельно не прописать.

Можно сохранять общую сумму в SumResultDay — это свойство класса и оно будет сохраняться пока есть объект, которому оно принадлежит. Для этого можно ему присваивать значение в конце метода:

SumResultDay = RegResult;

Либо вообще обойтись без RegResult:

SumResultDay = Convert.ToInt32(Amout) + Convert.ToInt32(AmoutResult);    

Но тут возникнет другая проблема: Вы все делаете в конструкторе, а значит при каждом сохранении создается новый объект, с новыми свойствами и значение им присваивается по умолчанию (0).

Логичнее будет каждый раз вычитывать данные из реестра:

RegistryKey knockOff = currentUserKey.CreateSubKey("KnockOff");

//читаем значение из реестра
var value = knockOff.GetValue("AmoutResult");
int amountResult = Convert.ToInt32(value);
//прибавляем текущее значение
amountResult += sumDay;
...
//и лишь потом сохраняем сумму
knockOff.SetValue("AmoutResult", amountResult);

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

Еще, от конструктора ожидается что он будет выполнять только инициализацию класса, а не сложные манипуляции с реестром. Так что лучше было бы код перенести из конструктора в методы и свойства. Например, чтобы с классом можно было работать так:

RegClassSave regClassSave = new RegClassSave;
//свойство для хранения даты
regClassSave.DateN = DateTime.Now;
//метод, который работает с реестром и текущей суммой
regClassSave.Save(EnterSum);
//свойство для чтения общей суммы
Console.WriteLine(regClassSave.SumResultDay);
3
  • 1
    Спасибо за очень ёмкий ответ. Про конструктор вот оно! Огромное спосибо! Два дня возилься не мог понять почему результат не тот! Теперь вроде понял! Буду пробовать!
    – j. Atisto
    28 апр 2018 в 14:57
  • @j.Atisto Пожалуйста! Удачи! 28 апр 2018 в 15:01
  • Спасибо, всё получилось!!!
    – j. Atisto
    28 апр 2018 в 17:20

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.