Problema con cursor en pl/sql

[k0m3t4]

Buenas noches colegas. este es mi primer post por aca y quisiera dar las gracias en general a todos ya que esta pagina me ha servido mucho y espero también ser de ayuda en un futuro a esta web.

Quisiera un poco de su conocimiento a que me ayuden a revisar este pequeño problema que aun no he podido resolver el cual tengo que crear un cursor que sume los empleados que hay en un determinado pais. esto es con la base de datos HR que viene por defecto en oracle.
Yo estuve metiendole mano y cree algo así aunque se que esta malo pero lo dejo para ver si me pueden dar ideas de como mejorarlo o podria hacerlo. no soy como las demas personas que piden que le hagan las cosas, con tal que me ayuden con la logica yo tratare de implementar sus ideas y de buscar mi propia solucion. aca les dejo lo que he hecho hasta el momento:

Código SQL:

cursor suma_cursor IS
SELECT COUNT(employee_id)
FROM employees
WHERE department_id = (SELECT department_id
                       FROM departments
                       WHERE location_id = (SELECT location_id
                                            FROM locations
                                            WHERE country_id = (SELECT country_id
                                                                FROM countries
                                                                WHERE country_name = '&nombre_pais');
 

lo que se es que en esta consulta:

Código SQL:

SELECT department_id
FROM departments
WHERE location_id = (SELECT location_id
                                 FROM locations
                                 WHERE country_id =
 

tengo problemas ya que en esta fila obtengo mas de 1 departamento el cual aun no he podido resolver de como solo obtener 1 solo eso eso uno y lo otro no se si estara correcto.

Espero de vuestra ayuda y ojala puedan ayudarme en algo.
Agradeciendo por darse la paja de leer este testamento xD se depside

K0m3t4
Cristian Saavedra

[gnzsoloyo]

Es que estás complicando innecesariamente la consulta, pero además no estás considerando que la condición puesta no ´devolverá un único registro.
Piensa: ¿Cuántos empleados puede haber, de cuántos departamentos, que estén en qué lugares de un mismo país?
Con eso sólo, ya tienes una cantidad enorme de posibilidades.

Ahora bien, si lo único que necesitas es la cantidad de empleados, ni siquiera es un CURSOR, porque devuelve un único valor. Se usan cursores si vas a realizar una iteración de valores, y no es el caso.
La solución sería definir una variable y obtener el resultado.

Código SQL:

totalempleados NUMBER;
totalempleados := 0;
 
SELECT COUNT (DISTINCT employee_id) total
INTO totalempleados
FROM employees E, departments D, locations L, countries C
WHERE E.department_id = Ddepartment_id
    AND  D.location_id = L.location_id
    AND L.country_id = C.country_id
    AND C.country_name = '&nombre_pais'
 

[k0m3t4]

Primero que nada gnzsoloyo quisiera agradecerte por tu ayuda brindada y si, tienes bastante razon, me estaba complicando un poco mas de lo necesario, pero en el ejercicio me obligan por asi decir a utilizar un cursor y eso es lo que me complica. mira te dejo a continuación el enunciado que corresponde al cursor:

1.-Crear un cursor que contabilice el número de empleados de cada país y el nombre del país en que existen sucursales.

obiamente este cursor debo anexarlo a otras cosas que me piden que haga pero lo demas ya lo tengo hecho solo he estado topado en lo del cursor. tratare de ver como implemento parte de tu codigo en el mio haber que resulta y ver si es que puedo hacer que el cursor de alguna manera funcione :)

Nuevamente te agradesco por tu ayuda =)

[k0m3t4]

creo que ya implementando lo que tengo que hacer el codigo seria algo así :

Código SQL:

SET serveroutput ON
DECLARE
 
pais countries.country_name%TYPE:='&pais';
t_empleados NUMBER :=0;
 
cursor empleados_cursor IS
SELECT COUNT (DISTINCT employee_id)
FROM employees E, departments D, locations L, countries F
WHERE E.department_id = D.department_id 
      AND D.location_id = L.location_id
      AND L.country_id = F.country_id
      AND UPPER(F.Country_name) = UPPER(pais);
BEGIN
 
dbms_output.put_line('                    EMPLEADO POR PAISES                     ');
dbms_output.put_line('============================================================');
 
OPEN empleados_cursor;
  loop
    fetch empleados_cursor
      INTO t_empleados;
      exit WHEN empleados_cursor%notfound;
      IF (t_empleados < 10 ) THEN
        dbms_output.put_line(RPAD('CANTIDAD EMPLEADOS',25)||RPAD('PAIS',15)||RPAD('CATEGORIA',20) );
        dbms_output.put_line('============================================================');
        dbms_output.put_line(RPAD(t_empleados,25)||RPAD(UPPER(pais),15)||RPAD('PEQUEÑA SUCURSAL',20) );
      ELSE
        dbms_output.put_line(RPAD('CANTIDAD EMPLEADOS',25)||RPAD('PAIS',15)||RPAD('CATEGORIA',20) );
        dbms_output.put_line('============================================================');
        dbms_output.put_line(RPAD(t_empleados,25)||RPAD(UPPER(pais),15)||RPAD('GRAN SUCURSAL',20) );
      END IF;
  END loop;
close empleados_cursor;
END;
 

Ahora creo que solo me queda que me muestre todas las sucursales con empleados de una y no solo buscar una en especifico ;)

[gnzsoloyo]

Bueno, en realidad lo que has hecho no está mal.... pero tampoco está bien. AL menos no lo está si se lee con cuidado la consigna que te dieron.
Fijate lo que te dicen:

1.-Crear un cursor que contabilice el número de empleados de cada país y el nombre del país en que existen sucursales.

Es decir que se buscan estas cosas
1) Cantidad de empleados por país.
2) Países en que hay sucursales

En realidad el cursor no es para contar los empleados, sino para listar los datos resultantes, y para ello la consulta no es exactamente igual. Incluso, el proceso entero es mucho menos complicado de lo que hiciste.
Un detalle que se infiere de la consigna es que todo empleado pertenece a una sucursal, y que no hay sucursales sin empleados.

Sería mas o menos así:

Código SQL:

CURSOR cursor_suma AS
SELECT C.country_name COUNT (DISTINCT employee_id) totalempleados
FROM employees E, departments D, locations L, countries C
WHERE E.department_id = Ddepartment_id
    AND  D.location_id = L.location_id
    AND L.country_id = C.country_id
GROUP BY C.country_name;
 

Luego de eso, simplemente lo recorres y muestras los resultados. Incluso puedes sumar el total de empleado para mostrar al final, e incluso poner la cantidad de países.

[k0m3t4]

de hecho es exactamente lo que tengo que hacer. debo ver cuantas personas trabajan en cada pais y mostrar los paises donde trabajen personas y si es menor a 10 que salga por panta que la empresa es pequeña y si es mas de 10 que la empresa es grande. intentare el codigo y lo coloco aca si tengo alguna duda al respecto y gracias por la ayuda prestada de todas maneras :)

[k0m3t4]

ya ahora si coloco el codigo casi final ya que solo me falta afinar un detalle el cual es mostrar la cantidad de empleados en la siguiente forma:

35 United Kingd GRAN SUCURSAL
68 United State GRAN SUCURSAL
1 Germany PEQUEÑA SUCURSAL
2 Canada PEQUEÑA SUCURSAL

pero es un pelo de la cola lo que me queda =)

Código:

set serveroutput on
declare
c_empleados numeric;
c_paises varchar2(40);

cursor empleados_cursor is
select f.country_name, count(distinct employee_id)
from employees E, departments D, locations L, countries F
where E.department_id = D.department_id
    and  D.location_id = L.location_id
    and L.country_id = F.country_id
group by F.country_name;


begin
 
dbms_output.put_line('                          EMPLEADO POR PAISES                           ');
dbms_output.put_line('========================================================================');
dbms_output.put_line(RPAD('CANTIDAD EMPLEADOS',25)||RPAD('PAIS',30)||RPAD('CATEGORIA',20) );
dbms_output.put_line('========================================================================'); 
open empleados_cursor;
  loop
    fetch empleados_cursor
    into c_paises,c_empleados;
      exit when empleados_cursor%notfound;
      IF (c_empleados < 10 ) then
        dbms_output.put_line(RPAD(c_empleados,25)||RPAD(upper(c_paises),30)||RPAD('PEQUEÑA SUCURSAL',20) );
      else
        dbms_output.put_line(RPAD(c_empleados,25)||RPAD(upper(c_paises),30)||RPAD('GRAN SUCURSAL',20) );
      end if;
  end loop;
close empleados_cursor;
end;

P.D: gracias por darte el tiempo de editar mis post y colocar los codigos como se debe ya que aun no encuentro como colocarlo como tu lo haces para que se vea mas ordenado cuando los posteo.