seguro sea un error muy bobo..
2 respuestas
Te voy a explicar por qué te está ocurriendo este error, que como bien dices es una cosa bien sencilla.
En la línea 32 intentas hacer uso del valor SALTA
del enum
Provincia
. Al menos eso es lo que interpreto que has intentado hacer. El fallo es muy sencillo, no puedes acceder a un elemento de un enum
sin referenciarlo. Sí indicas directamente SALTA
, no se puede encontrar como una variable. Si te fijas te lo pone claramente:
Cannot find symbol
Symbol: variable SALTA
Para solucionar este tema no es necesario que añadas el modificador de acceso Public
al enum Provincias, ya que está usando el modificador de acceso "Package-Private". Este modificador le da acceso a todos los elementos que están en su paquete, pero no a los que estan en niveles superiores. Te lo digo porque te han recomendado cambiarlo a Public
cuando eso no va a cambiar nada, ya que el enum
Provincias
sí está accesible desde la clase CasaEmpanadas
. Como iba diciendo, lo único que tienes que hacer es añadir la referencia al enum
Provincias
, te lo pongo en código para que lo veas.
Antes:
System.out.println(PromocionLocal(SALTA, 20, 20));
Despues:
System.out.println(PromocionLocal(Provincia.SALTA, 20, 20));
Con este cambio el error que nos has mostrado en pantalla se debería arreglar, pero no se puede asegurar que sea el único error, ya que no has puesto más que una captura de pantalla del código, y una descripción muy deficiente.
-
Acerca del public del enum: ¿qué pasaría si se quisiera llamar al método "PromocionLocal" (que es public, de una clase public) desde fuera del package "casaempanadas"?– Racsoel 19 dic. 2017 a las 17:52
-
Simple! no tendría visibilidad para el tipo de dato que de requiere para el primer parámetro, por lo tanto no se podría usar dicho método bajo esas circunstancias, sería necesario cambiarle el modificador de acceso al enum Provincias. Es mas, sería recomendable sacarlo del fichero en el que se encuentra, y ponerlo por separado en el pqquete que tenga más sentido. He hecho hincapie en que el modificador public no era necesario porque con lq definicion y el código que tenemos es así. el 19 dic. 2017 a las 18:04
-
Tenemos entonces una discrepancia entre la visibilidad del método y la visibilidad del tipo de uno de sus parámetros, que tiene consecuencias reales. Tienes razón en que es un detalle irrelevante para arreglar el error que se plantea, pero no es cierto que sea un cambio que no vaya a cambiar nada, como dices. Es un problema que se debe corregir. Si te parece que las respuestas deben limitarse exclusivamente a lo que se está preguntando, vale, es respetable, pero creo que el otro extremo (contradecir la sugerencia y decir explícitamente que el cambio no hace nada) es igual de dañino.– Racsoel 19 dic. 2017 a las 18:24
-
Gracias por la respuesta, es justo lo qeu me pasaba, y añadiendo que el metodo esta como no static asique me obligaba a instanciarlo para que funcione, el caso es que lo cambie a static, puse Provincia. y funciono, :) el 19 dic. 2017 a las 19:23
-
@Racso, Estoy completamente de acuerdo en una cosa que has indicado! En que tenemos discrepancia de opiniones :). Estas basando tu último comentario en una suposición. Y si la pregunta de Cosoforo Decimo así lo indicase sabrías que lo que he dicho en mi respuesta pierde mucho sentido. Tu última frase... Si te soy sincero, no la entiendo. Me da a entender que limitar la respuesta exclusivamente a lo que se pregunta es tan dañino como el otro extremo. ¿Te refieres a dar una respuesta que no soluciona lo que se pregunta? La verdad es que no entiendo lo que quieres decir, lo siento. el 19 dic. 2017 a las 20:05
Prueba a cambiar SALTA
por Provincia.SALTA
en la línea 32, y a establecer Provincia
(el enum) como public
.
EDITO: Establecer Provincia
como public
no es necesario para corregir el error concreto que se muestra en la imagen (como bien indica Mario Castillo en su respuesta), pero actualmente hay una discrepancia entre la accesibilidad del método PromocionLocal
(que es public
) y la de uno de sus parámetros: prov
, que es de tipo Provincia
, que es menos accesible que el método. El problema causado por esa discrepancia se visibilizará el día en que intentes invocar el método desde fuera del package casaempanadas
. Dependiendo de lo que estés haciendo, arreglar el detalle de una vez puede parecerte más o menos necesario, pero sin duda es mala idea dejar smells o bugs por ahí.
public enum Provincia ...
desaparezca el problema... no se aprecia bien, pero parece que quieres usarla en un método en cuyo ámbito la variable no existe... Para futuros preguntas, no olvides este principio: en Stackoverflow, un código vale más que mil imágenes. Si compartes el texto del código comentando dónde está el error y cuál es el error, será de más utilidad que la imagen.