set in model class is not functionnal!

Question

set in model class is not functionnal!

sblb 1,231

Hi, I have the class model below and when the condition DTb is verified Statut take the "Open".
When the Status of the DTb changes my app is blocked. Is not possible to use dev tool because the application is blocked.

public string CloseOn { get; private set; }  
        public string Statut  
        {  
              
         get  
           {  
              if (DTb != "Validated")  
                  {  
                      string var1;  
                       var1 = "Open";  
                      return var1;  
                 }  
               return Statut;  
            }  
         set  
            {                  
                   if (DTb == "Validated")  
                   {  
                     CloseOn = "Closed";  
                   }  
                  
            }  
                 
        }

Is the model class I wrote correct?
Have you an idea why my application is blocked ?
Thanks in advance

Accepted answer

1 additional answer

Your answer

Answer 1

Viorel 122.5K

I think that the 'return Statut' calls Statut again and again (infinitely or until an overflow error). This blocks your application.

You should return something else. (Maybe 'return CloseOn'?).

sblb 1,231

You're right. I've changed return Statut by return CloseOn.

In this case my application is no more blocked.

But when the the DTb take the value Validated that return anything! Have you any suggestions?

public string CloseOn { get; private set; }
public string Statut
{

      get  
        {  
           if (DTb != "Validated")  
               {  
                   string var1;  
                    var1 = "Open";  
                   return var1;  
              }  
            return CloseOn;  
         }  
      set  
         {                  
                if (DTb == "Validated")  
                {  
                  CloseOn = "Closed";  
                }  
                  
         }  
                 
     }

Viorel 122.5K Reputation points

2022-08-18T19:30:52.403+00:00

Maybe you should return "Closed" instead of CloseOn.

It is not clear what is the purpose of these values and of the setter of Statut (which does not use the passed value).

sblb 1,231

Thanks to your support.
it works the way I want it to.

public string CloseOn { get; private set; }  
        public string Statut  
        {  
  
            get  
            {  
                if (DTb != "Validated")  
                {  
                    string var1;  
                    var1 = "Open";  
                    return var1;  
                }  
                else if (DTb == "Validated")  
                {  
                    string var2;  
                    var2 = "Close";  
                    return var2;  
                }  
                return CloseOn;  
            }  
  
            private set {}  
          
                 
        }

Answer 2

AgaveJoe 30,126

You have to understand the code you've shared is nonsensical. Therefore, the community is unable to infer your intent from the code. Please explain the use case and perhaps the community can provide a logical approach.

The C# programming guide has a section on properties which might prove helpful.

Properties (C# Programming Guide)

Share via

set in model class is not functionnal!

1 additional answer

Your answer