C# Не получается выводить корректные данные

@Guiho · Регистрация: 09.09.2022

Author24 — интернет-сервис помощи студентам

Доброго времени суток. Делал задание и столкнулся с проблемой.
Я ввожу три значения веса и время/температуру. По заданию я должен вводить составляющие трёх рецептов, и, после нажатия кнопки, они должны выводиться списком.
Подсчитываю общий вес и добавляю в список. При выведении списка общий вес отображается некорректно: выдаётся лишь последнее введённое значение, в остальных же просто 0. В чём ошибка, как исправить?

Прикрепил архив с проектом

@big1991 · 26.01.2023, 19:40

Потому что Вы добавляете в список reciper несколько ссылок на один и тот же объект recipe.

Ваши методы по добавлению рецептов должны создавать новый объект и добавлять в список его, а не изменять один и тот же объект и добавлять в список ещё одну ссылку на тот же объект.

Например вместо

C#

        private void NewRecipe1_Click(object sender, RoutedEventArgs e)
        {
            recipe.Weight1 = Convert.ToDouble(TBw1.Text);
            recipe.Weight2 = Convert.ToDouble(TBw2.Text);
            recipe.Weight3 = Convert.ToDouble(TBw3.Text);
            recipe.C_time = Convert.ToDouble(Cook.Text);
            recipe.Sum = Convert.ToDouble(Total1.Text);
            reciper.Add(recipe);
        }

Хотя бы так:

C#

        private void NewRecipe1_Click(object sender, RoutedEventArgs e)
        {
            var defaultRecipe = new Recipe();
            defaultRecipe.Weight1 = Convert.ToDouble(TBw1.Text);
            defaultRecipe.Weight2 = Convert.ToDouble(TBw2.Text);
            defaultRecipe.Weight3 = Convert.ToDouble(TBw3.Text);
            defaultRecipe.C_time = Convert.ToDouble(Cook.Text);
            defaultRecipe.Sum = Convert.ToDouble(Total1.Text);
            reciper.Add(defaultRecipe);
        }

Не по теме:

Я сначала попытался присваивать новый объект также в поле класса, как и было, и если честно не совсем понял почему это не сработало, ну да ладно, к делу не относится.

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

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

	26.01.2023, 19:40