Autor Tema: Error al declarar una clase... (Leído 2556 veces)

LenguajeC · « **en:** Sábado 21 de Mayo de 2011, 21:49 »

Hola a todos.
Espero que me tengan mucha paciencia y comprencíon ya que estoy estudiando con un manual y no tengo otra consulta que los foros y tutoriales de la web( y tengo 59 años).-
El programa que expongo a continuación me genera un el siguiente error.-

Citar

'Clientes.datos.vec' no es accesible debido a su nivel de protección

Seguramente estoy declarando mal la clase pero si le pongo public empeora.-

Código: C#

using System;
using System.Collections.Generic;
 
class Clientes
{
    class datos
    {
        private string [] vec;
        public datos()
        {   
            vec = new String[7]; 
        }
        public string this[int indice]
        {
            set
            {
                vec[indice] = value;
            }
            get
            {
                return vec[indice];
            }
        }   
    }   
 
    public static void detalle()
    {
        for(byte a = 1; a<50; a++)
        {
            Console.SetCursorPosition(a,00); Console.Write("="); 
            Console.SetCursorPosition(a,02); Console.Write("="); 
        }
        Console.SetCursorPosition(17,01);
        Console.Write(" Alta de Clientes ");
        Console.SetCursorPosition(01,04);
        Console.Write(" Nombre completo....:");
        Console.SetCursorPosition(01,05);
        Console.Write(" Dirección..........:");
        Console.SetCursorPosition(01,06);
        Console.Write(" Cód. postal........:");
        Console.SetCursorPosition(01,07);
        Console.Write(" Ciudad.............:");
        Console.SetCursorPosition(01,08);
        Console.Write(" Celular............:");
        Console.SetCursorPosition(01,09);
        Console.Write(" Crédito............:");
        Console.SetCursorPosition(01,10);
        Console.Write(" Plazo en días......:");
    }
    
    public static void ingreso()
    {
        datos _datos = new datos();
        byte ver = 04; 
        string linea = "";
        for(int cont = 0; cont<7; cont++)
        {
            do
            {
                Console.SetCursorPosition(22,ver);
                linea = Console.ReadLine(); 
            }while((linea == null) || (linea.Length==0));
           // _datos.vec[cont] = linea; 
            ver++;
        }   
    }   
    
    public static void Main(string[] args)
    {
        Console.Title = " Alta de Clientes"; 
        detalle();
        ingreso();
        
        
        Console.SetCursorPosition(01,12);
        Console.Write(" Pulse una tecla para finalizar..."); Console.ReadKey();  
    }
}
 

Espero me puedan ayudar.-

Saludos.
Daniel

Valinor73

Creo que el problema esta en que dentro de la definicion de una clase no es posible la definicion de otra. Estas definiendo la clase datos dentro de la clase Clientes, cuando lo correcto a mi entender es que la crees separada. Una vez definidas las dos clases si que te sera posible crear en cualquier clase, objectos de la otra.
Pruebalo. Si no es asi que alguien me corrija.
Un saludo

LenguajeC

Hola Valinor73.
Esta línea estaba mal: // _datos.vec[cont] = linea; esta es la manera correcta para que funcione _datos[cont] = linea;.-
Ahora tengo otro inconveniente, no logro leer el array desde el método principal, si desearan ayudarme yo agradecido.-

Saludos.
Daniel

ProfesorX

Bueno, escribi mi respuesta hace 3 horas, pero no pude publicarla antes porque tuve que salir, veo que te contestaron en parte, aun asi, publico mi respuesta, con comentarios a ver si asi te queda mas claro.

Código: C#

 
using System;
using System.Collections.Generic;
 
// Las clases no se anidan una dentro de otra, deben ir separadas.
// Y de preferencia, una clase en cada archivo de codigo, esto
// ultimo no es obligatorio, pero si recomendado.
class datos
{
    // Declara vec de tipo public, no private, para evitar el error que te marca,
    // no se cual era el problema que dices que
    // te da haciendolo public, pero a mi no me da problema. Si te da
    // problema con public, menciona especificamemnte cual es.
    public string[] vec;
    public datos()
    {
        vec = new String[7];
    }
    // Ya que estas declarando vec como public, no es necesario usar
    // set/get, este solo se utilizaria si vec fuera private, pero
    // lo estas haciendo mal de todas formas, ya que no puedes usar
    // this como nombre de propiedad, pues this es palabra reservada
    // de C#. La convencion normal (digo convencion, por que no es
    // obligatorio, pero es algo que casi todo programador de C# hace)
    // es usar nombres de variables con minuscula para variables privadas
    // y nombres de variables con mayuscula para variables publicas.
    // En otro bloque te pondre la manera correcta de hacerlo
    //public string this[int indice]
    //{
    //    set
    //    {
    //        vec[indice] = value;
    //    }
    //    get
    //    {
    //        return vec[indice];
    //    }
    //}
}
 
 
class Clientes
{
 
    public static void detalle()
    {
        for (byte a = 1; a < 50; a++)
        {
            Console.SetCursorPosition(a, 00); Console.Write("=");
            Console.SetCursorPosition(a, 02); Console.Write("=");
        }
        Console.SetCursorPosition(17, 01);
        Console.Write(" Alta de Clientes ");
        Console.SetCursorPosition(01, 04);
        Console.Write(" Nombre completo....:");
        Console.SetCursorPosition(01, 05);
        Console.Write(" Dirección..........:");
        Console.SetCursorPosition(01, 06);
        Console.Write(" Cód. postal........:");
        Console.SetCursorPosition(01, 07);
        Console.Write(" Ciudad.............:");
        Console.SetCursorPosition(01, 08);
        Console.Write(" Celular............:");
        Console.SetCursorPosition(01, 09);
        Console.Write(" Crédito............:");
        Console.SetCursorPosition(01, 10);
        Console.Write(" Plazo en días......:");
    }
 
    public static void ingreso()
    {
        datos _datos = new datos();
        byte ver = 04;
        string linea = "";
        for (int cont = 0; cont < 7; cont++)
        {
            do
            {
                Console.SetCursorPosition(22, ver);
                linea = Console.ReadLine();
            } while ((linea == null) || (linea.Length == 0));
            _datos.vec[cont] = linea;
            ver++;
        }
    }
 
    public static void Main(string[] args)
    {
        Console.Title = " Alta de Clientes";
        detalle();
        ingreso();
 
 
        Console.SetCursorPosition(01, 12);
        Console.Write(" Pulse una tecla para finalizar..."); Console.ReadKey();
    }
}
 
 

Clase datos usando set/get, y variables privadas.

Código: C#

 
class datos
{
    // Declara vec con minuscula para la variable private
    private string[] vec;
    // Declara la propiedad vec con mayuscula para la parte public
    // Recuerda que ahora utilizaras _datos.Vec[cont] en lugar de
    // _datos.vec[cont], ya que vec ahora es private
    public string[] Vec
    {
        set
        {
            vec = value;
        }
        get
        {
            return vec;
        }
    }
    public datos()
    {
        vec = new String[7];
    }
}
 
 

Citar

Ahora tengo otro inconveniente, no logro leer el array desde el método principal, si desearan ayudarme yo agradecido.-

No entiendo aqui especificamente que es lo que deseas hacer, si pudieras explicarnos mejor y de manera mas clara, seria excelente.

Si hiciste modificaciones al codigo que publicaste originalmente, publicalo de nuevo, especificando el error que te marca, y en que linea obtienes el error.

Saludos

LenguajeC

Hola Profe
Que interesante para un estudiante solitario como yo encontrar a alguien con tanta predisposición para ayudar(enseñar) y tan didáctico.-
Te dejo mi código para que me digas porque desde el método Main no puedo leer el array.-

Código: C#

using System;
using System.Collections.Generic;
 
class Clientes
{
    class datos
    {
        private string [] vec;
        public datos()
        {   
            vec = new String[7]; 
        }
        public string this[int indice]
        {
            set
            {
                vec[indice] = value;
            }
            get
            {
                return vec[indice];
            }
        }   
    }   
 
    public static void detalle()
    {
        for(byte a = 1; a<50; a++)
        {
            Console.SetCursorPosition(a,00); Console.Write("="); 
            Console.SetCursorPosition(a,02); Console.Write("="); 
        }
        Console.SetCursorPosition(17,01);
        Console.Write(" Alta de Clientes ");
        Console.SetCursorPosition(01,04);
        Console.Write(" Nombre completo....:");
        Console.SetCursorPosition(01,05);
        Console.Write(" Dirección..........:");
        Console.SetCursorPosition(01,06);
        Console.Write(" Cód. postal........:");
        Console.SetCursorPosition(01,07);
        Console.Write(" Ciudad.............:");
        Console.SetCursorPosition(01,08);
        Console.Write(" Celular............:");
        Console.SetCursorPosition(01,09);
        Console.Write(" Crédito............:");
        Console.SetCursorPosition(01,10);
        Console.Write(" Plazo en días......:");
    }
    
    public static void ingreso()
    {
        datos _datos = new datos();
        string linea = "";
        for(int cont = 0; cont<7; cont++)
        {
            do
            {
                Console.SetCursorPosition(22,cont+4);
                linea = Console.ReadLine(); 
            }while((linea == null) || (linea.Length==0));
            _datos[cont] = linea; 
        }   
    }   
    
    public static void Main(string[] args)
    {
        Console.Title = " Alta de Clientes"; 
 
        detalle();
        ingreso();
        datos d_atos = new datos();
        
        Console.SetCursorPosition(01,12);  
        [i]Console.WriteLine(" Dirección...:" + d_atos[0]);[/i] Console.ReadKey(); 
    }
}
 

Bueno eso es todo, luego pondré en práctica lo del post anterior.-

Saludos Profe.-
Daniel

ProfesorX · « **Respuesta #5 en:** Lunes 23 de Mayo de 2011, 03:42 »

Bueno, en el nuevo codigo que publicaste no has hecho las correcciones que te dije. Debiste hacerlo, pero bueno

Te dejo el codigo corregido de nuevo, con las correcciones que te mencione anteriormente, ademas de nuevas correcciones con comentarios para que sepas el porque de tu error, espero que lo comprendas mejor.

Código: C#

 
using System;
using System.Collections.Generic;
 
class datos
{
    public string[] vec;
    public datos()
    {
        vec = new String[7];
    }
}
 
class Clientes
{
    // Debes declarar la variable como static, para poder
    // acceder a ella en metodos static
    static datos _datos;
    public static void detalle()
    {
        for (byte a = 1; a < 50; a++)
        {
            Console.SetCursorPosition(a, 00); Console.Write("=");
            Console.SetCursorPosition(a, 02); Console.Write("=");
        }
        Console.SetCursorPosition(17, 01);
        Console.Write(" Alta de Clientes ");
        Console.SetCursorPosition(01, 04);
        Console.Write(" Nombre completo....:");
        Console.SetCursorPosition(01, 05);
        Console.Write(" Dirección..........:");
        Console.SetCursorPosition(01, 06);
        Console.Write(" Cód. postal........:");
        Console.SetCursorPosition(01, 07);
        Console.Write(" Ciudad.............:");
        Console.SetCursorPosition(01, 08);
        Console.Write(" Celular............:");
        Console.SetCursorPosition(01, 09);
        Console.Write(" Crédito............:");
        Console.SetCursorPosition(01, 10);
        Console.Write(" Plazo en días......:");
    }
 
    public static void ingreso()
    {
        string linea = "";
        for (int cont = 0; cont < 7; cont++)
        {
            do
            {
                Console.SetCursorPosition(22, cont + 4);
                linea = Console.ReadLine();
            } while ((linea == null) || (linea.Length == 0));
            _datos.vec[cont] = linea;
        }
    }
 
    public static void Main(string[] args)
    {
        Console.Title = " Alta de Clientes";
 
        // Creas un nuevo objeto de tipo "datos"
        _datos = new datos();
        detalle();
        ingreso();
 
        // Esto es incorrecto para lo que creo que quieres hacer.
        // Cada vez que usas new creas un nuevo objeto.
        // por lo tanto, el nuevo objeto d_atos, no es el mismo
        // que _datos, son del mismo tipo, pero NO son el mismo
        // objeto, cada objeto es independiente uno del otro,
        // por lo tanto no puedes esperar que d_atos contenga lo
        // que capturaste en _datos, para obtener lo que capturaste
        // en _datos, debes declarar _datos como static al
        // principio de la clase para poder acceder a el desde los
        // metodos static (Lo que ya hice en esta ejemplo corregido).
        // Si los metodos no fueran static, no seria
        // necesario declarar _datos como static, pero entonces 
        // necesitarias crear una instancia (objeto) de
        // la clase clientes para acceder a los metodos no estaticos
        // detalle() e ingreso(). No se si me entiendas.
 
        //datos d_atos = new datos();
       
        Console.SetCursorPosition(01,12);  
        Console.WriteLine(" Nombre...:" + _datos.vec[0]); Console.ReadKey();
    }
}
 
 

Saludos

LenguajeC · « **Respuesta #6 en:** Martes 24 de Mayo de 2011, 02:32 »

Hola Profe.
Me pudrías decir que criterio debo utilizar para saber cuando utilizar las propiedades en c#,
me parecía que era una manera avanzada de programar y me encuentro que en el código anterior la terminaste descartando, te pido x favor que me lo expliques para saber a que atenerme.-

Un gran abrazo.-
Daniel

ProfesorX · « **Respuesta #7 en:** Martes 24 de Mayo de 2011, 04:55 »

Cita de: "LenguajeC"

Me pudrías decir que criterio debo utilizar para saber cuando utilizar las propiedades en c#,

En realidad es un poco a gusto del programador, y segun lo necesite el programa. cada programa es diferente.

Segun el paradigma de la programacion orientada a objetos, no es bueno tener acceso directo a las variables internas de una clase, para evitar efectos secundarios inesperados, todo acceso debe ser por medio de metodos set/get, parametros y mensajes.

Yo en la practica solo utilizo set y get cuando necesito que la variable sea solo de lectura o solo de escritura. O sea que si necesito leer la variable, pero no escribirla, utilizo get, y omito el get, y si necesito escribir la variable, pero no leerla, utilizo set y omito el get. Si necesito leer y escribir la variable, la hago public directamente. Aun asi, no debes abusar de ello, en lo posible debes tratar de que cada objeto sea independiente del exterior, y evitar las variables public en lo posible. Solo cuando sea de veras necesario

De hecho, tu programa no esta correctamente encapsulado, ya que llamas por fuera a variables de tu clase, y manipulas la clase con metodos externos, lo mas correcto hubiera sido definir los metodos dentro de la misma clase datos, para manipular la variable vec dentro de la clase y evitar manipulacion externa de la clase.

Te dejo un ejemplo de como lo hubiera hecho, eso te lo aclarara un poco mas. Nota como manejo el vector internamente dentro de la clase datos, y no necesito acceder al vector dentro de main(), si esta correctamente encapsulado

Código: C#

 
using System;
using System.Collections.Generic;
 
class datos
{
    private string[] vec;
    
    public datos()
    {
        vec = new String[7];
    }
    
    public void detalle()
    {
        for (byte a = 1; a < 50; a++)
        {
            Console.SetCursorPosition(a, 00); Console.Write("=");
            Console.SetCursorPosition(a, 02); Console.Write("=");
        }
        Console.SetCursorPosition(17, 01);
        Console.Write(" Alta de Clientes ");
        Console.SetCursorPosition(01, 04);
        Console.Write(" Nombre completo....:");
        Console.SetCursorPosition(01, 05);
        Console.Write(" Dirección..........:");
        Console.SetCursorPosition(01, 06);
        Console.Write(" Cód. postal........:");
        Console.SetCursorPosition(01, 07);
        Console.Write(" Ciudad.............:");
        Console.SetCursorPosition(01, 08);
        Console.Write(" Celular............:");
        Console.SetCursorPosition(01, 09);
        Console.Write(" Crédito............:");
        Console.SetCursorPosition(01, 10);
        Console.Write(" Plazo en días......:");
    }
 
    public void ingreso()
    {
        string linea = "";
        for (int cont = 0; cont < vec.Length ; cont++)
        {
            do
            {
                Console.SetCursorPosition(22, cont + 4);
                linea = Console.ReadLine();
            } while ((linea == null) || (linea.Length == 0));
            vec[cont] = linea;
        }
    }
 
    public void imprime()
    {
        Console.SetCursorPosition(01, 12);
        Console.WriteLine("Nombre...:" + vec[0]); 
        Console.ReadKey();
    }
}
 
class Clientes
{
    static datos _datos;
 
    public static void Main(string[] args)
    {
        Console.Title = " Alta de Clientes";
 
        _datos = new datos();
        _datos.detalle();
        _datos.ingreso();
        _datos.imprime();
 
    }
}
 
 
 

Saludos

LenguajeC

Hola profe.
Muchas gracias y tendré que combinar más la práctica con la teoría.-

Saludos.
Daniel

SoloCodigo

Autor Tema: Error al declarar una clase... (Leído 2556 veces)

LenguajeC

Error al declarar una clase...

Valinor73

Re: Error al declarar una clase...

LenguajeC

Re: Error al declarar una clase...

ProfesorX

Re: Error al declarar una clase...

LenguajeC

Re: Error al declarar una clase...

ProfesorX

Re: Error al declarar una clase...

LenguajeC

Re: Error al declarar una clase...

ProfesorX

Re: Error al declarar una clase...

LenguajeC

Re: Error al declarar una clase...