¿Alguien me ayuda a programar?

[Nanaky]

Hola buenas noches!
Soy nuevo, y la información que hay en la pagina me resulta muy útil porque soy bastante novato en Delphi (bueno, y en cualquier tipo de programacion... aunque una vez programé/copié el Arkanoid en Basic, pero eso no cuenta mucho...).

A ver si alguien es tan amable de ayudarme con un trabajo que tengo que hacer, en el cual me he quedado atascado. De este trabajo depende mi nota de informática, y que me compensen matematicas. Lo veo muy negro.

Con la siguiente premisa:

Realizar una aplicación informática que gestiones los vuelos aereos de una companyia X. El programa debe guardar de cada vuelo la siguiente información. Codigo de vuelo(3 letras y 3 numeros), aeropuerto de salida, aeropuerto de llegada, fecha y hora real de salida, fecha y hora real de llegada, si el vuelo ha sido puntual o no y el número de pasajeros transportados. Se considera que un vuelo es puntual si la diferencia entre la hora estimada y la hora real de salida y/o llegada es menor a 5 minutos.

Tengo que hacer un programa que muestre en pantalla las siguientes opciones (con case of):

1.Entrar datos
2.Modificar datos.
3. Determinar puntualidad de los vuelos y guardar la información en el campo correspondiente.
4. Dadas dos fechas, dar el porcentaje de vuelos puntuales sobre el total de vuelos entre estas fechas.

Lo que llevo hecho es lo siguiente , pero me he quedado estancado y no sé como seguir:

uses
SysUtils;

Const MAXvuelos=100;

type
t_fecha_salida= record
minuto:0..59;
hora:0..23;
dia:1..31;
mes:1..12;
anyo: 2000..2100;
end;
t_fecha_llegada= record
minuto:0..59;
hora:0..23;
dia:1..31;
mes:1..12;
anyo: 2000..2100;
end;
t_vuelo= record
cod_vuelo: string[6];
aero_salida: string[3];
aero_llegada: string[3];
pasajeros: integer;
fecha_salida: t_fecha_salida;
fecha_llegada: t_fecha_llegada;
end;
t_index=1..MAXvuelos;
t_lista_vuelos= array[1..MAXvuelos]of t_vuelo;

var
l:t_lista_vuelos;
op:integer;
i:t_index;


procedure Leer_datos;
begin
writeln('Introduzca el codigo de vuelo: ');
readln(l[i].cod_vuelo);
writeln('Introduzca el codigo del aeropuerto de salida: ');
readln(l[i].aero_salida);
writeln('Introduzca el codigo del aeropuerto de llegada: ');
readln(l[i].aero_llegada);
writeln('Introduzca la fecha de salida: ');
write('Dia: ');
readln(l[i].fecha_salida.dia);
write('Mes: ');
readln(l[i].fecha_salida.mes);
write('Anyo: ');
readln(l[i].fecha_salida.anyo);
writeln('Introduzca la fehca de llegada: ');
write('Dia: ');
readln(l[i].fecha_llegada.dia);
write('Mes: ');
readln(l[i].fecha_llegada.mes);
write('Anyo: ');
readln(l[i].fecha_llegada.anyo);
writeln;

end;
begin
repeat

writeln ('Escoja una de las siguentes opciones: ');
writeln ('--------------------------------------');
writeln ('1) Leer datos');
writeln ('2) Modificar los datos de vuelo por referencia');
writeln ('3) Conocer la puntualidad de los vuelos');
writeln ('4) Porcentajes de puntualidad segun fechas');
writeln ('0) Salir');
readln (op);
if (op>=0) and (op<=4) then
else
writeln('La opción no es correcta, escoja otra opcion.');
(*else lo he puesto como: si no es ni 0,1,2,3 o 4 , opción incorrecta. No se si esto está bién así*)
case op of

'1': Leer_datos;
'2': Modificar_datos;
'3': Puntualidad;
'4': Porcentaje;

end;
until op='0'
end.


-Aquí ya no compila. Me dice que en 'op' incompatible types:integer y char son incompatibles, pero es que yo a la variable op le he dado valor de integer.

- También quiero cambiar el procedure leer_datos, porque de esta manera, luego no se como voy a modificarlos por referencia en el siguiente subprograma.
Quiero hacerlo de la siguiente manera (adjunto pseudocódigo):

Procedimiento leer_datos (ent/sal, VAR v1 : t_lista_vuelos, nvuelos:entero);
VAR j:t_index;
Inicio_secuencia
leer (nvuelos1);
desde j:=1 hasta nvuelos1 hacer
leer v1[j].codigo_vuelo;
f_desde;
f_secuencia;

donde luego en el case of irían así v y , n vuelos.
1:leer_datos (v,nvuelos)

Lo ha explicado el profesor así, pero no lo acabo de entender.
¿Cual sería la entrada y la salida del procedure? y en el caso de nvuelos, ¿puedo luego poner un nvuelos1 sin definirlo como variable, solo poniendole un 1? lo mismo pasa con el v1 y v.

De verdad, está bién copiado.

Estoy bastante perdido.

Agradecería muchíiiiisimo que alguien me ayudase a seguir con el trabajo.
Muchas gracias por anticipado, estaré muy agradecido al que conteste, y también al que le tome interés.

Espero vuestras respuestas!

P.D. Se ha alineado solo a la izquierda, lo siento si no se diferencian mucho los saltos de linea.

[droguerman]

hola nanaki vamos por partes, los integer no van entre cadenas solo se ponen los números por lo tanto debes modificar tu código:

Código Delphi [-]
     case op of
    1: Leer_datos;
    2: Modificar_datos;
    3: Puntualidad;
    4: Porcentaje;
end;
  until op=0;
end.

ahora en el otro caso estás pasando la variable nro de vuelos entonces no necesitas volver a leer cuantas veces vas a leer solo bastar con hacer algo parecido a:

Código Delphi [-]
procedure leer_datos (VAR v1 : t_lista_vuelos, nvuelos:entero);
VAR 
j: integer;
begin
   for j:=1 to nvuelos do
       readln(v1[j].codigo_vuelo);
end;

[Nanaky]

Muchas gracias droguerman. A ver si me voy enterando.
Quieres decir que en el case en lugar de poner la opcion entre comillas:

'1': Leer_datos;
'2': Modificar_datos;
'3': Puntualidad;
'4': Porcentaje;

la, he de poner sin? Es la unica diferencia que veo.

Por otro lado, justo antes del case of:

if (op>=0) and (op<=4) then
else
writeln('La opción no es correcta, escoja otra opcion.');

(*else lo he puesto como: si no es ni 0,1,2,3 o 4 , opción incorrecta. No se si esto está bién así*)

esto que he puesto, no funciona correctamente. Cuando escojo una opcion que no sea 0, me vuelve a repetir el case, y cuando es 0 me dice que la opcion no es correcta, y el programa tampoco se cierra.

Muchas gracias por la explicación. Agradeceria todo lo que me podáis explicar. Esta tarde cuando vuelva de trabajar continuaré intentando programar. Alguna idea para el tercer y cuarto apartado?

[droguerman]

estas poniendo un if sin acción mejor quedaria haciendo esto

Código Delphi [-]
if (op<0) or (op>4) then
begin
  writeln('La opción no es correcta, escoja otra opcion.');
  continue; //repetir el bucle
end;

case op of
  1: Leer_datos;
  2: Modificar_datos;
  3: Puntualidad;
  4: Porcentaje;
end;

[Nanaky]

Vale ya me he puesto manos a la obra y he conseguido que funcione el Case of. Solo me funciona con la opcion 0 (salir) o si le doy a una opcion que no está entre 0 y 4 ( esto es lo siguiente que preguntaba, ahora ya funciona).

Lo que pasa ahora es que cuando le doy a la opcion 1 ( la unica que tengo de momento) no hace nada , ni puedo volver al menu ni cerrar el programa. A ver si alguien me ayuda .

Pongo el codigo tal cual lo tengo ahora:

Código Delphi [-]
program Algoprueba3;

{$APPTYPE CONSOLE}

uses
  SysUtils;
Const MAXvuelos=100;

type
  t_fecha_salida= record
    minuto:0..59;
    hora:0..23;
    dia:1..31;
    mes:1..12;
    anyo: 2000..2100;
  end;
  t_fecha_llegada= record
    minuto:0..59;
    hora:0..23;
    dia:1..31;
    mes:1..12;
    anyo: 2000..2100;
  end;
  t_vuelo= record
    cod_vuelo: string[6];
    aero_salida: string[3];
    aero_llegada: string[3];
    pasajeros: integer;
    fecha_salida: t_fecha_salida;
    fecha_llegada: t_fecha_llegada;
  end;
  t_index=1..MAXvuelos;
  t_lista_vuelos= array[1..MAXvuelos]of t_vuelo;

  var
  v1:t_lista_vuelos;
  nvuelos:integer;
  op:integer;
  procedure Leer_datos (VAR v1 : t_lista_vuelos; nvuelos:integer );
    VAR
      j: t_index;
      begin
        readln (nvuelos);
        for j:=1 to nvuelos do
        writeln ('Escriba el codigo de vuelo(AAA111):  ');
        readln(v1[j].cod_vuelo); (*fallo*)
        writeln ('¿Cual es el codigo del aeropuerto de salida?(AAA):  ');
        readln(v1[j].aero_salida);
        writeln ('¿Cual es el codigo del aeropuerto de llegada?(AAA):  ');
        readln(v1[j].aero_llegada);
        writeln ('¿Cuantos pasajeros hay en este vuelo?:  ');
        readln(v1[j].pasajeros);
      end;
   begin
      repeat

      writeln ('Escoja una de las siguentes opciones: ');
      writeln ('--------------------------------------');
      writeln ('1) Leer datos');
      writeln ('2) Modificar los datos de vuelo por referencia');
      writeln ('3) Conocer la puntualidad de los vuelos');
      writeln ('4) Porcentajes de puntualidad segun fechas');
      writeln ('0) Salir');
      writeln;
      readln (op);
      writeln;
        if (op<0) and (op>4) then
          begin
            writeln('La opcion no es correcta, escoja otra opcion.');
            continue;
          end;
      case op of

        1: Leer_datos (v1,nvuelos);(*fallo*)

      end;
      until op=0;
end.

y estos son los mensajes de error en las lineas que he marcado como fallo.

[Warning] Algoprueba3.dpr(46): FOR-Loop variable 'j' may be undefined after loop

[Warning] Algoprueba3.dpr(73): Variable 'nvuelos' might not have been initialized


Muchas a todos por vuestra colaboración.


No se si será que le faltan variables de entrada y/o salida al procedure.

[Nanaky]

Si has visto, he cambiado lo que has dicho, para no poner un if sin ninguna orden, y ahora funciona, porque vuelve al menu de opciones pero no da el mensaje:La opcion no es correcta, escoja otra opcion.

lo he probado con y sin begin/end; y con y sin continue.


edito:

Ya se que pasaba, he dejado el AND que habia antes y por eso no iba. Lo he cambiado por un OR y va correctamente.
Así queda más optimizado, gracias.

A ver si me ayudáis a continuar.

[[seoane]]

Vamos por partes y despacito.

El contrario de esto:

Código Delphi [-]
if (op>=0) and (op<=4) then

no es esto:

Código Delphi [-]
if (op<0) and (op>4) then

es esto:

Código Delphi [-]
if (op<0) or (op>4) then

Sigamos, aqui lo unico que haces es mandar escribir la misma frase varias veces:

Código Delphi [-]
for j:=1 to nvuelos do
        writeln ('Escriba el codigo de vuelo(AAA111):  ');

Si quieres meter mas de una instruccion dentro de un bucle vas a tener que usar begin/end

Código Delphi [-]
for j:=1 to nvuelos do
begin
        writeln ('Escriba el codigo de vuelo(AAA111):  ');
      // Las demas instrucciones
end;

Y todavía sigo sin entender porque le pasas el parámetro nvuelos al procedure, si lo primero que haces en el procedure es leer su valor por teclado