1

Buenas, mi problema es que en la parte del código que aparece el primer if del Case 1: dice if (pos < vectorsize) esto lo hago para que el usuario no pueda agregar más datos y no se exceda del limite,

El problema es que al volver al menú principal, la variable pos vuelve a 0 por lo que nunca va a llagar al else donde muestra el error de limite execido, mi pregunta es ¿Cómo hago para que no se vuelva a 0 cada vez que entra al menú?

static void Main(string[] args)
{
    int oba = 0, obb = 0, obc = 0, pra = 0, prb = 0, prc = 0, proy = 0;
    int opcion = 0;
    int confirmacion = 0;
    int ci = 0;
    int vectorsize = 0;
    int pos = 0;
    if (vectorsize == 0)
    {
        Console.WriteLine("Ingrese la cantidad de alumnos.");
        vectorsize = Convert.ToInt32(Console.ReadLine());
    }

    double[] alumnos = new double[vectorsize];
    while (opcion != 4)
    {
        string opcionsino = "SI";
        Console.Clear();
        //Console.WriteLine(vectorsize);
        Console.WriteLine("Bienvenido");
        Console.WriteLine("");
        Console.WriteLine("1-> Agregar alumno.");
        Console.WriteLine("2-> Asignar notas.");
        Console.WriteLine("3-> Informe.");
        Console.WriteLine("4-> Salir.");
        Console.WriteLine("");
        Console.WriteLine("Elija una opcion.");
        Console.WriteLine(pos);
        opcion = Convert.ToInt32(Console.ReadLine());
        switch (opcion)
        {
            case 1:
            {
                Console.Clear();
                for (int i = 0; opcionsino == "SI" || opcionsino == null; i++)
                {
                    if (pos < vectorsize)
                    {
                        for (int i2 = 0; i2 < alumnos.Length && alumnos[i2] != 0; i2++)
                            pos = i2 + 1;
                        Console.Clear();
                        Console.WriteLine("Ingrese la cedula del alumno sin puntos ni guiones Ej: 12345678");
                        ci = Convert.ToInt32(Console.ReadLine());
                        for (int i3 = 0; i3 < alumnos.Length; i3++)
                        {
                            confirmacion = 0;
                            //int control = 0;
                            if (ci == alumnos[i3])
                            {
                                Console.Clear();
                                Console.WriteLine("Alumno ya ingresado.");
                                Console.ReadLine();
                                confirmacion = 7;
                                break;
                            }
                        //control++;
                        }
                    }
                    else
                    {
                        Console.WriteLine("Limite exidido.");
                        break;
                    }

                    if (confirmacion != 7)
                    {
                        alumnos[pos] = ci;
                        pos = pos + 1;
                    }

                    Console.WriteLine("¿Deseas agregar otro alumno? Si/No.");
                    opcionsino = Console.ReadLine();
                    opcionsino = opcionsino.ToUpper();
                }

                Console.Clear();
                Console.WriteLine("Precione una tecla para volver al menú principal.");
                Console.ReadLine();
                pos = 0;
                break;
            }
        }
    }
}

Muchas gracias de antemano.

0

2 respuestas 2

2

El problema está en que pones a 0 el valor de pos después de leer del teclado para volver al menú principal. Comenta esa línea y ya no se reinicializará a 0:

            Console.Clear();
            Console.WriteLine("Precione una tecla para volver al menú principal.");
            Console.ReadLine();
            // pos = 0;  <--- Aquí lo pones a 0, comenta esta línea
            break;
0
0

- Propongo estas 5 correcciones (añado comentarios en cada punto) ... -


static void Main(string[] args
    int oba = 0, obb = 0, obc = 0, pra = 0, prb = 0, prc = 0, proy = 0;
    int opcion = 0;
    int confirmacion = 0;
    int ci = 0;
    int vectorsize = 0;
    int pos = -1;
    if (vectorsize == 0)
    {
        Console.WriteLine("Ingrese la cantidad de alumnos.");
        vectorsize = Convert.ToInt32(Console.ReadLine());
    }

double[] alumnos = new double[vectorsize];

Se ha de inicializar el array de double ( como luego se compara con 0 para considerar vacío un elemento, se inicializarán en 0 ) ...

for (int cont = 0; cont < vectorsize; cont++)  
   alumnos[cont] = 0;

while (opcion != 4)
{
    string opcionsino = "SI";
    Console.Clear();
    //Console.WriteLine(vectorsize);
    Console.WriteLine("Bienvenido");
    Console.WriteLine("");
    Console.WriteLine("1-> Agregar alumno.");
    Console.WriteLine("2-> Asignar notas.");
    Console.WriteLine("3-> Informe.");
    Console.WriteLine("4-> Salir.");
    Console.WriteLine("");
    Console.WriteLine("Elija una opcion.");
    Console.WriteLine(pos);
    opcion = Convert.ToInt32(Console.ReadLine());
    switch (opcion)
    {
        case 1:
        {
            Console.Clear();
            for (int i = 0; opcionsino == "SI" || opcionsino == null; i++)
            {

Si manejamos pos como posición (indice+1), no podemos usarlo luego como si fuera el índice. (Error : if (pos < vectorsize) ...

                if (pos <= vectorsize)
                {
                    for (int i2 = 0; i2 < alumnos.Length && alumnos[i2] != 0; i2++);

Solo se asignará a pos la posición del primer elemento que contenga 0, únicamente si no existe una posición previa de modo que se puedan ingresar valores se alumnos sencuencialmente ...

                    if (pos == -1)                            
                       pos = i2 + 1;  

                    Console.Clear();
                    Console.WriteLine("Ingrese la cedula del alumno sin puntos ni guiones Ej: 12345678");
                    ci = Convert.ToInt32(Console.ReadLine());
                    for (int i3 = 0; i3 < alumnos.Length; i3++)
                    {
                        confirmacion = 0;
                        //int control = 0;
                        if (ci == alumnos[i3])
                        {
                            Console.Clear();
                            Console.WriteLine("Alumno ya ingresado.");
                            Console.ReadLine();
                            confirmacion = 7;
                            break;
                        }
                    //control++;
                    }
                }
                else
                {
                    Console.WriteLine("Limite exidido.");
                    break;
                }

                if (confirmacion != 7)
                {

Se debe poner el índice para la asignación, que es la posición-1, de modo que no se asigne por error el valor al próximo alumno (Error : alumnos[pos] = ci; ) ...

                    alumnos[pos-1] = ci;   
                    pos++; // << Más directo que : pos = pos + 1;      

                }

                Console.WriteLine("¿Deseas agregar otro alumno? Si/No.");
                opcionsino = Console.ReadLine();
                opcionsino = opcionsino.ToUpper();
            }

            Console.Clear();
            Console.WriteLine("Precione una tecla para volver al menú principal.");
            Console.ReadLine();

Se eliminará la siguiente linea para poder ingresar valores de alumnos secuencialmente ...

            // pos = 0;
            break;
            }
        }
    }
}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.