Skip to main content
se añadieron 2502 caracteres en el cuerpo
Origen Enlace
Antonio S.F.
  • 1.2k
  • 5
  • 12

Bueno; entendiendo un poco más lo que pretendes creo que es cuestión de organización. Yo lo haría con List< T > y lo plantearía así:

class CApartamento : IEquatable<CApartamento>
{
    public string Clave;
    public string Descripcion;
    public ...

    // Constructor
    public CApartamento(string str_Key)
    {
         Clave = str_Key;
    }

    // Permite la comparación de dos objetos CApartamento
    public bool Equals(CApartamento other)
    {
       return (this.Clave == other.Clave);
    }
}

// Esta quedaría obsoleta si usamos List<T>.Contains()
private bool ExistKey(List<CApartamento> inmobiliaria, string temporary_key)
{
    foreach(CApartamento CA in inmobiliaria)
    {
        if (CA.Clave == temporary_key) return true;      
    }
    return false;
}

Luego en tu cuerpo de programa o método encargado ponemos:

List<CApartamento> inmobiliaria = new List<CApartamento>();

bool SeguirMetiendoDatos = true;

while( SeguirMetiendoDatos == true )
{
    // Introducimos una clave
    Console.Write("CLAVE [A-###]: ");
    temporary_key = Console.ReadLine();
    // Creamos nuestro objeto provisional con la clave introducida
    CApartamento CA = new CApartamento(temporary_key);

    // *** if (ExistKey(inmobiliaria, temporary_key)  **
    // Este IF es innecesario ya que dotamos a nuestra clase
    // con la capacidad de compararse y por lo tanto podemos usar
    // el método Contains de la lista inmobiliaria

    if (inmobiliaria.Contains(CA))
    {
        Console.WriteLine("El elemento ya existe, introduzca otra clave.");
    }
    else // Si no existe lo añadimos a la lista
    {
         // Rellenamos nuesto objeto CApartamento con
         // el resto de datos antes de añadirlo a la lista.
         CA.Descripcion = ...;

         // Lo incluimos en la lista  
         inmobiliaria.Add(CA);

         Console.Write("Introducir otra clave (S/N): ");
         string respuesta = Console.ReadLine();

         SeguirMetiendoDatos = (respuesta.ToUpper() == "S");
    }
}

El hecho de usar List nos permite operaciones como Añadir o Borrar dinámicamente elementos de la lista, ordenarlos... a la vez que nos permite simplificar el código y añadirle legibilidad.

Bueno; entendiendo un poco más lo que pretendes creo que es cuestión de organización. Yo lo haría con List< T > y lo plantearía así:

class CApartamento : IEquatable<CApartamento>
{
    public string Clave;
    public string Descripcion;
    public ...

    // Constructor
    public CApartamento(string str_Key)
    {
         Clave = str_Key;
    }

    // Permite la comparación de dos objetos CApartamento
    public bool Equals(CApartamento other)
    {
       return (this.Clave == other.Clave);
    }
}

// Esta quedaría obsoleta si usamos List<T>.Contains()
private bool ExistKey(List<CApartamento> inmobiliaria, string temporary_key)
{
    foreach(CApartamento CA in inmobiliaria)
    {
        if (CA.Clave == temporary_key) return true;      
    }
    return false;
}

Luego en tu cuerpo de programa o método encargado ponemos:

List<CApartamento> inmobiliaria = new List<CApartamento>();

bool SeguirMetiendoDatos = true;

while( SeguirMetiendoDatos == true )
{
    // Introducimos una clave
    Console.Write("CLAVE [A-###]: ");
    temporary_key = Console.ReadLine();
    // Creamos nuestro objeto provisional con la clave introducida
    CApartamento CA = new CApartamento(temporary_key);

    // *** if (ExistKey(inmobiliaria, temporary_key)  **
    // Este IF es innecesario ya que dotamos a nuestra clase
    // con la capacidad de compararse y por lo tanto podemos usar
    // el método Contains de la lista inmobiliaria

    if (inmobiliaria.Contains(CA))
    {
        Console.WriteLine("El elemento ya existe, introduzca otra clave.");
    }
    else // Si no existe lo añadimos a la lista
    {
         // Rellenamos nuesto objeto CApartamento con
         // el resto de datos antes de añadirlo a la lista.
         CA.Descripcion = ...;

         // Lo incluimos en la lista  
         inmobiliaria.Add(CA);

         Console.Write("Introducir otra clave (S/N): ");
         string respuesta = Console.ReadLine();

         SeguirMetiendoDatos = (respuesta.ToUpper() == "S");
    }
}

El hecho de usar List nos permite operaciones como Añadir o Borrar dinámicamente elementos de la lista, ordenarlos... a la vez que nos permite simplificar el código y añadirle legibilidad.

Origen Enlace
Antonio S.F.
  • 1.2k
  • 5
  • 12

El fallo pueda estar en las tres primeras líneas de código:

Console.Write("CLAVE [A-###]: ");
inmobiliaria[i].clave = Console.ReadLine();
temporary_key = inmobiliaria[i].clave;

Lo que lees de consola ya lo tratas de guardar en inmobiliaria[i].clave y creo que debería ser así:

Console.Write("CLAVE [A-###]: ");
temporary_key = = Console.ReadLine();

Después de tener el dato introducido en temporary_key es cuando compruebas si ya existe dicho elemento en tu array inmobiliaria[]. Ten en cuenta las correcciones de Eduardo Reyes en la función Check.

Espero haber sido de ayuda aunque sólo has mostrado parte del código, no sabemos de dónde viene esa i, ni su valor ni tampoco qué función tiene la condicional j==i.