Java - LinkedList

tengo dos clases, especialidad y centro



en el centro lo que tengo como atributo es una linkedList
LinkedList<Especialidad> especialidades = new LinkedList<>();

y un metodo


lo que busco es meter en la linked list el nombre de una especialidad y su valor,
pero no sabria como hacerlo para q cumpla con los getter y setter que tiene,
la finalidad que busco es
en el main que esta en centro


y que se agregen, alguna ayuda con respecto a esto ? se me hace complicado y debe ser uan tonteria.

Creo que no requieres usar getters o setters, solo el constructor.

claro pero pasa que el nombre de la especialidad son solo 3 tipos pediatria cardiologia y general, con lo que decis podria ingresar el nombre pepino y ponerle el valor -1
por eso habia pensado en los getter y setter para ponerle una restriccion digamos a lo q puedo ingresar

Podrias hacer 2 modificaciones:
- Que el constructor invoque los setters y entonces el constructor haga el chequeo.
- Que los setters modifiquen los campos que se supone hay que setear. Ahora solo chequean, pero no setean (los setters tienen que comprobar lo que se intenta setear, no lo que tiene ahora en los campos del objeto)

- Que el constructor invoque los setters y entonces el constructor haga el chequeo.

no sabia que se podia hacer eso

Pudiera poner la restriccion a nivel del constructor o puedes crera un enum con loas valores que va a atenr la especializadad. Por ejemplo

enum Especialidad{
GENERAL
PEDRIATRIA,
CARDIOLOGIA,
}

y

Por cierto, otra cosa, para comparar Strings no es recomendable usar los operadores == o !=.
En ocasiones pueden dar resultados inesperados.

Lo adecuado es usar el método equals().
Y al usar this, no estás comprobando el dato que se recibe, si no que estás comprobando el valor del atributo ya existente previamente.

Supongo que la intención es comprobar si el "nombre" recibido se corresponde con una de las tres especialidades admitidas.
Si corresponde, lo seteamos (que por cierto tu método no setea nada)
Si no corresponde, lanzamos excepción.
Y la comparación debería ser con AND (&&) y no con OR (||)

Si usamos OR, basta con que se cumpla una sola de las tres comparaciones para que se considere TRUE, y por lo tanto, se lanzaría la excepción.
Si pasamos como nombre "General", es un nombre válido. Pero como es distinto de "Pediatria" y al usar OR con eso basta para que toda la expresión del IF se considere TRUE, se lanzará la excepción rechazando este nombre, que en realidad es válido.

En cambio si usamos AND, las tres comparaciones han de ser ciertas para que toda la expresión se considere TRUE. De este modo, solo se rechazan los nombres que sean distintos de los tres admitidos.

Quizás sea mejor así:

Aún se puede mejorar un poco.
En este caso solo hay tres especialidades, así que bueno, no queda mal hacerlo así.
Pero imagina que quieres añadir más especialidades admisibles.., el if podría ser muy largo.
Usando un switch, se puede hacer más cómodo de escribir, y fácil de entender al leerlo:



El método setValor() también requiere ser completado.
Si el valor recibido es negativo, se lanza excepción. Bien,
Pero, ¿qué pasa si es un valor que se puede admitir? Nada, no pasa nada, no se está pasando ese valor al atributo.