Excepción de desbordamiento de pila: ¿por qué?

Question

Sigo recibiendo una excepción de desbordamiento de pila con esto. Lo he reducido a esta clase para tratar de averiguar lo que estaba mal pero simplemente no tengo idea de por qué sigo recibiendo este mensaje de error. Originalmente tengo la interfaz de usuario en otra clase, pero para eliminar todo lo demás, como problemas en mis métodos de llamada, moví los elementos esenciales a esta clase para tratar de descubrir qué estaba mal. Pensé que podrían ser mis propiedades? Tal vez sea obvio para todos los demás, pero simplemente no entiendo.

Como soy muy nuevo en la programación, agradecería algo de ayuda sobre lo que he hecho mal. Por lo que entiendo, ¿este problema ocurre cuando tienes algo así como un ciclo infinito?

namespace MyNameSpace 
{ 
    public class Customers 
    { 
     private List<Customers> customers; 

     public Customers() 
     { 
      customers = new List<Customers>(); 

      AddCustomer(new Customers() 
      { 
      Name = "A", Telephone="1" 
      }); 
     } 

     public string Name 
     { 
      get; 
      set; 
     } 
     public string Telephone 
     { 
      get; 
      set; 
     } 

     public void RunTest() 
     { 

      Console.WriteLine(); 
      Console.WriteLine("****** VIDEOSTORE ******"); 
      Console.WriteLine(); 
      Console.WriteLine("1. Show Customers"); 
      Console.WriteLine("6. Quit"); 

      string userChoice = Console.ReadLine(); 

      switch (userChoice) 
      { 
       case "1": 
        View(); 
        break;   

        break; 
       case "2": 
        break; 
      } 
     } 

     public void View() 
     { 
      foreach (Customers c in customers) 
      { 
       Console.WriteLine(); 
       Console.WriteLine(c.Name); 
       Console.WriteLine(c.Telephone); 
       Console.WriteLine(); 
      } 
     } 

     public void AddCustomer(Customers custom)       
     { 
      customers.Add(custom);   
     } 
    } 
} 

Answer 1

Está llamando a su creación de un nuevo objeto Clientes en el constructor Clientes.

Answer 2

Está creando una lista dentro del constructor de su clase Customers. Esto causará un bucle infinito y dará como resultado un desbordamiento de la pila.

Creo que deberías tratar de separar tu código en varias clases.

public class Customer 
{ 
    public string Name { get; set; } 
    public string Telephone { get; set; } 
} 

public class Program 
{ 
    private List<Customer> _customers = new List<Customer(); 

    public Program() 
    { 
     _customers.Add(new Customer() 
     { 
      Name = "A", Telephone="1" 
     }); 
    } 

    // your other methods here - like View() 
} 

Answer 3

En el constructor Clientes llama al constructor Clientes nuevamente, creando una recursión infinita.

usted debe tener una clase separada para obtener una lista de los clientes y para un solo cliente:

namespace MyNameSpace 
{ 
    public class Customer 
    { 
     public string Name 
     { 
      get; 
      set; 
     } 
     public string Telephone 
     { 
      get; 
      set; 
     } 
    } 

    public class Customers 
    { 
     private List<Customer> customers; 

     public Customers() 
     { 
      customers = new List<Customer>(); 

      AddCustomer(new Customer() 
      { 
      Name = "A", Telephone="1" 
      }); 
     } 


     public void RunTest() 
     { 

      Console.WriteLine(); 
      Console.WriteLine("****** VIDEOSTORE ******"); 
      Console.WriteLine(); 
      Console.WriteLine("1. Show Customers"); 
      Console.WriteLine("6. Quit"); 

      string userChoice = Console.ReadLine(); 

      switch (userChoice) 
      { 
       case "1": 
        View(); 
        break;   

        break; 
       case "2": 
        break; 
      } 
     } 

     public void View() 
     { 
      foreach (Customer c in customers) 
      { 
       Console.WriteLine(); 
       Console.WriteLine(c.Name); 
       Console.WriteLine(c.Telephone); 
       Console.WriteLine(); 
      } 
     } 

     public void AddCustomer(Customer customer)       
     { 
      customers.Add(customer);   
     } 
    } 
} 

Answer 4

Su constructor está llamando a sí mismo (nuevos clientes()), lo que hace que no volver nunca.

Una buena regla general, si obtiene un desbordamiento de pila en C#, busque recursividad que nunca termina.

Answer 5

Su constructor para clientes se llama a sí mismo, causando un bucle infinito.

public Customers() 
{ 
    customers = new List<Customers>(); 

    AddCustomer(new Customers() // <- Here 
    { 
    Name = "A", Telephone="1" 
    }); 
} 

Las llamadas recursivas interminables a una función causarán un StackOverFlow.

Answer 6

Los errores de desbordamiento de pila generalmente no provienen de bucles infinitos, sino de recurrencia interminable (bueno, no realmente interminable en la práctica, continúa llamándose hasta que la pila está llena y luego se lanza la excepción).

Si tiene un método que usa recursividad (es decir, se llama a sí mismo), debe asegurarse de que esto solo suceda un número limitado de veces. Si no haces eso, obtienes un método que se llama a sí mismo con una llamada a método que se llama a sí misma con una llamada a método que se llama a sí misma con una llamada a método que se llama a sí misma (continúa muchas veces hasta que la pila esté llena). ... Al igual que Clientes() Llamar Clientes() Llamar Clientes() Llamar Clientes() Llamar Clientes() Llamar Clientes() Llamar Clientes() Llamar Clientes() .....