Foros aprenderaprogramar.com
Didáctica y divulgación de la programación

Hola Luis Fdo Carvajal

Tu clase Policia no cumple con lo que pedía el enunciado relativo a "La clase debe implementar la interface Cloneable y disponer de un método que permita ver el resultado de concatenar sus atributos."

Por otro lado, la idea del ejercicio era modificar únicamente un atributo de los objetos clonados, mientras que tú has modificado o establecido los cuatro atributos.

Este ejercicio deberías replantearlo

Puedes consultar una solución posible para el ejercicio en https://www.aprenderaprogramar.com/foros/index.php?topic=5703.0

Saludos

Hola Luis Fdo Carvajal lo veo todo correcto

Saludos

Hola Luis Fdo Carvajal el ejercicio no es correcto puesto que no cumple con lo que indicaba el enunciado "un programa que pida por pantalla..."

En tu código no se piden los datos por pantalla

Saludos

Hola Luis Fdo Carvajal

En el primer código al ejecutarlo me ha saltado un error java.lang.ArrayIndexOutOfBoundsException, deberías revisarlo

En el segundo código tienes unas declaraciones de LinkedList y ArrayList que no son útiles, deberías eliminar todo lo que no sea útil o no sea pedido en el ejercicio

Saludos

Hola Luis Fdo Carvajal tu código cumple con lo que se pedía y además añade otras cosas que no se indicaban en el enunciado del ejercicio

Una cosa a cambiar sería el hecho de que al ejecutar el programa todos los legisladores indica que son diputados, cuando lo correcto sería que en algunos casos indicara que son diputados y en otros casos indicara que son senadores.

Esto se debe a esta línea:

if (tmp instanceof Diputado) { tmpStr1 = "Diputado";tmpStr2 = "Camara de Representantes";tmpStr3 = "ANTIOQUIA";} else { tmpStr1 = "Diputado"; tmpStr2 = "Senado de la Republica";tmpStr3 = "VALLE"; }

en el else deberíamos tener tmpStr1 = "Senador"

Saludos

Hola SidritaParaFestejar, para plantear consultas es conveniente leer las normas sobre cómo publicar en los foros, disponibles en https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0

Hay diferentes maneras para encontrar el número más frecuente en un array

Usando Hashtable

class MainClass
    {
 
        static void MaxOccurrence(int[] array, Hashtable hs)
        {
            int mostCommom = array[0];
            int occurences = 0;
            foreach (int num in array)
            {
                if (!hs.ContainsKey(num))
                {
                    hs.Add(num, 1);
                }
                else
                {
                    int tempOccurences = (int)hs[num];
                    tempOccurences++;
                    hs.Remove(num);
                    hs.Add(num, tempOccurences);
 
                    if (occurences < tempOccurences)
                    {
                        occurences = tempOccurences;
                        mostCommom = num;
                    }
                }
            }
            foreach (DictionaryEntry entry in hs)
            {
                Console.WriteLine("{0}, {1}", entry.Key, entry.Value);
            }
            Console.WriteLine("The commmon numer is " + mostCommom + " And it appears " + occurences + " times");
        }
 
        public static void Main(string[] args)
        {
 
            int[] array = new int[20] { 3, 6, 8, 5, 3, 5, 7, 6, 4, 3, 2, 3, 5, 7, 6, 4, 3, 4, 5, 7 };
            Hashtable hs = new Hashtable();
            MaxOccurrence(array, hs);
 
        }
    }

Da como resultado:

8,1
7,3
6,3
5,4
4,3
3,5
2,1
The common number is 3 and it appears 5 times

Usando Dictionary

class MainClass
    {
 
        public static void MaxOccurrence(int[] numbers)
        {
            var counts = new Dictionary<int, int>();
            foreach (int number in numbers)
            {
                int count;
                counts.TryGetValue(number, out count);
                count++;
                //Automatically replaces the entry if it exists;
                //no need to use 'Contains'
                counts[number] = count;
            }
            int mostCommonNumber = 0, occurrences = 0;
            foreach (var pair in counts)
            {
                if (pair.Value > occurrences)
                {
                    occurrences = pair.Value;
                    mostCommonNumber = pair.Key;
                }
            }
            Console.WriteLine("The most common number is {0} and it appears {1} times",
                mostCommonNumber, occurrences);
        }
 
        public static void Main(string[] args)
        {
 
            int[] array = new int[20] { 3, 6, 8, 5, 3, 5, 7, 6, 4, 3, 2, 3, 5, 7, 6, 4, 3, 4, 5, 7 };
            MaxOccurrence(array);
 
        }

Resultado: The common number is 3 and it appears 5 times

Usando LINQ

class MainClass
    {
 
        static void MaxOccurrence(int[] numbers)
        {
            var groups = numbers.GroupBy(x => x);
            var largest = groups.OrderByDescending(x => x.Count()).First();
            Console.WriteLine("The most common number is {0} and it appears {1} times", largest.Key, largest.Count());
        }
 
        public static void Main(string[] args)
        {
 
            int[] array = new int[20] { 3, 6, 8, 5, 3, 5, 7, 6, 4, 3, 2, 3, 5, 7, 6, 4, 3, 4, 5, 7 };
            MaxOccurrence(array);
 
        }     
 
    }

Resultado: The common number is 3 and it appears 5 times

Saludos

Hola Luis Fdo Carvajal. En la respuesta a) no has indicado si es posible o no que un objeto declarado Taxi puede contener a un objeto Vehiculo en tiempo de ejecución

En la clase Taxi que has escrito no cumples con lo que pide el enunciado: "que además de los atributos de Vehiculo lleve un atributo adicional nombreDelConductor (String) y numeroDePlazas (int)..."

Este ejercicio debería rehacerse pues no cumple con lo que se pedía

Saludos

Hola Jesu

Usar while true se considera una mala práctica de programación. Lee este hilo donde se explica esto: https://www.aprenderaprogramar.com/foros/index.php?topic=3074.0

Otra cosa es que has llamado a la clase UsoVectores. En Java Vector es una clase del api de java, y no es lo mismo un vector que un array o arreglo, por eso no estaría bien nombrada la clase.

Saludos

Hola paramonso, y bienvenido. Puedes empezar por leer este hilo: https://www.aprenderaprogramar.com/foros/index.php?topic=1460.0

Navega por el foro y comprueba cómo se plantean las preguntas y respuestas

Si tienes dudas sobre el funcionamiento del foro, plantéalas aquí mismo

Saludos

Hola dferna el ejercicio está bien resuelto

Para hacer uso del polimorfismo, podríamos hacer que el tipo estático de todos los objetos fuera el tipo abstracto Legislador. Es decir, en lugar de

Diputado diputado1 = new Diputado("Pablo", "Iglesias", 38);
Diputado diputado2 = new Diputado("Iñigo", "Errejon", 34, "Madrid");
Senador senador1 = new Senador("Pedro", "Sanchez", 40);
Senador senador2 = new Senador("Antonio", "Morales", 45, "Sevilla");
[code]

escribiríamos esto otro

[code]
ArrayList<Legislador> legisladores = new ArrayList<Legislador>();
Legislador diputado1 = new Diputado("Pablo", "Iglesias", 38);
Legislador diputado2 = new Diputado("Iñigo", "Errejon", 34, "Madrid");
Legislador senador1 = new Senador("Pedro", "Sanchez", 40);
Legislador senador2 = new Senador("Antonio", "Morales", 45, "Sevilla");

Otra mejora que podría hacerse es evitar que se muestre por pantalla [provinciaQueRepresenta=null] cuando se trate de un senador

Saludos