C# Problem ze zmianą stanu pola

Question

Witam

Mam prosty program TODOList, z jedną klasą o nazwie TodoListElement.

Mam w nim metodę ChangeToDoListElementStatus() służącą do zmiany wartości pola _todoListElementStatus z true na false (i analogicznie do odwróconej sytuacji.) Jednak mimo wywołania metody wartość się nie zmienia. Na chwilę obecną nie mam pojęcia co jest nie tak.

klasa Program

using System;

namespace TODOList
{
    internal class Program
    {
        private static void Main()
        {
            var todoListElement = new TodoListElement();

            todoListElement.SetTodoListElement("Write this program!");
            Console.Write("After: ");
            todoListElement.ShowTodoListElementStatus();
            todoListElement.ChangeToDoListElementStatus();
            Console.Write("Before: ");
            todoListElement.ShowTodoListElementStatus();
            todoListElement.ShowTodoListElement();
        }
    }
}

klasa TodoListElement

using System;

namespace TODOList
{
    public class TodoListElement
    {
        private string _todoListElementValue;
        private bool _todoListElementStatus;

        public TodoListElement()
        {
            _todoListElementValue = "Move this fat ass (づ>‿‿<)づ";
            _todoListElementStatus = true;
        }

        public void ShowTodoListElementStatus()
        {
            Console.WriteLine("TodoListElementStatus: " + _todoListElementStatus);
        }

        public void ShowTodoListElementValue()
        {
            Console.WriteLine("TodoListElementValue: " + _todoListElementValue);
        }

        public void ShowTodoListElement()
        {
            var status = _todoListElementStatus ? "(during)" : "(finished)";
            Console.WriteLine(_todoListElementValue + " " + status);
        }

        public string SetTodoListElement(string todoListElementValue)
        {
            _todoListElementValue = todoListElementValue;
            return _todoListElementValue;
        }

        public bool ChangeToDoListElementStatus()
        {
            return !_todoListElementStatus;
        }
    }
}

Z góry dziękuję za pomoc.

Comments

Rozwiązałem problem zamieniając tą metodę na voida o treści _todoListElementStatus = !_todoListElementStatus;

Wszytko działa, jednak dlaczego wcześniej był z tym problem? W czym rzecz?

Answer 1

Tak jak już zdążyłeś zauważyć problem:

public bool ChangeToDoListElementStatus()
{
     return !_todoListElementStatus;
}

Ta metoda jedynie zwraca odwrotność zmiennej _todoListElementStatus, ale nie zmienia jej wartości i to właśnie było problemem.

 

Edit: A tak przy okazji, to jaki cel ma ta funkcja, oprócz przypisania wartość?

public string SetTodoListElement(string todoListElementValue)
{
      _todoListElementValue = todoListElementValue;
      return _todoListElementValue;
}

Po co zwracasz tę wartość, która została podana w argumencie?

Comments

Masz rację, nie ma to sensu :D Zamieniłem tą metodę na voida i zostawiłem tylko _todoListElementValue = todoListElementValue;

I oczywiście dziękuję za odpowiedź, jak teraz o tym pomyślę to trochę wolno myślałem pisząc te metody i przkminiając ich działanie :>