1

Estoy intentando crear una aplicacion que permita crear matriculas de carros de forma aleatoria, como tambien tomar de forma aleatoria una cantidad de vehiculos ya definidos, para luego almacenarlos en un array y poder dar los datos del cliente, nombre + vehiculo aleatorio + matricula aleatoria. Todo lo estoy separando mi Principal Main y luego dentro de una clase hago los llamados, de esta forma:

Esta es mi clase de Matriculas:

namespace ProvaCuadro
{
    internal class Matricula
    {
        public static string getMatricula()
        {
            for (int i = 0; i < 10; i++)
            {
                Random randomChar = new Random();
                char[] letras = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z' };
                string matriculaRandom = randomChar.Next(10, 99) + "-" + letras[randomChar.Next(26)].ToString() + letras[randomChar.Next(26)].ToString() + "-" + randomChar.Next(10, 99);

                return matriculaRandom;
            }

            return getMatricula().ToString();
        }
    }
}  

Y esta este es mi MAIN:

public static void Main(string[] args)
{
    for (int i = 1; i < 10 + 1; i++)
    {
        Console.WriteLine($"{Viaturas.getViatura()}  {Matricula.getMatricula()}");
    }
}

Pero solamente me hace un llamado, es decir, solamente me trae un vehiculo y una matriculas repetida 10 veces, y no quiero eso, quiero que cambie por cada llamado, y me pueda dar carros y matriculas diferentes por cada iteracion.

Resolver este dilema


Problema

Este es el gran problema que estoy teniendo, al momento de llamarlo desde mi clase principal "Program.cs" y hacer o no el ciclo, siempre me traera las mismas, y necesito 10 pero distintas cada una entre si

PROBLEMA RESUELTO: Parece realmente que se tratara de un problema con la version de Visual Studio, ya que ahora, que coloque que abriera un proyecto nuevo con Aplicativo de Consola, parece funcionar. Sin embargo, ahora tengo un nuevo problema.

PROBLEMA #2:

He conseguido realizar las correcciones, me queda de esta forma, pero ahora quiero almacenar en un array los vehiculos y las matriculas, para luego, asignarle dichos datos a un cliente en particular, para que asi, el vehiculo y la matricula corresponda realmente al cliente. Esto es, porque debo hacer un historico de todos los clientes (los cuales se muestran en la imagen), y despues, crear una ficha de usuario para un solo cliente, el cual, como en este caso, Gustavo Serrano tiene un Seat(Arosa) con su matricula, y yo quiero presentarlos aqui:introducir la descripción de la imagen aquí, como tambien, quiero saber que es eso que ahora me aparece debajo de la consola : C:\Users\gusta\Desktop\Mi repositorio de C Sharp\Prueba2C_Projeto\Prueba2C_Projeto\bin\Debug\net6.0\Prueba2C_Projeto.exe (proceso 9264) se cerró con el código 0. introducir la descripción de la imagen aquí

5
  • Te faltan detalles, cual es la longitud de la matricula, cuantas letras y numeros puede tener, donde van, etc, aclara mejor tu problema, recuerda que los paises tienen diferentes estandares de matriculas
    – Japv
    el 11 ene. 2023 a las 12:55
  • Mira Cómo preguntar para que tu pregunta sea mejor recibida. También, aprovecha y haz el recorrido de bienvenida para entender mejor cómo funcionamos y de paso obtener tu primera medalla!
    – gbianchi
    el 11 ene. 2023 a las 13:35
  • Creo que hay un error en el bucle for del método estático getMatricula(). No estoy muy familiarizado con la sintaxis de C#, pero creo que el return matriculaRandom; podría estar causándote problemas el 11 ene. 2023 a las 13:36
  • @AndresGardiol el codigo en si, no tiene sentido y seguro esta tirando warnings... el for no tiene sentido, no itera sobre nada.
    – gbianchi
    el 11 ene. 2023 a las 13:41
  • Si, realmente pensé que era un tema de sintaxis de C#, pero ahora que veo con más detenimiento, la función no tiene ningún tipo de sentido el 11 ene. 2023 a las 13:42

2 respuestas 2

2

Tu función getMatricula(), dentro del ciclo haces un return, luego sales del ciclo y llamas a tu misma funcion recursivamente y creas un bucle erróneo, si te das cuenta en el propio Visual Studio te da el hint que tienes código inaccesible.

Debes de simplemente en tu función getMatricula() sin necesidad de hacer bucles, generar el código y luego en tu main, allí hacer tu ciclo las veces que sean necesarias.

public static string getMatricula()
{
     string matriculaRandom = String.Empty;

     Random randomChar = new Random();
     char[] letras = { 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z' };
     matriculaRandom = randomChar.Next(10, 99) + "-" + letras[randomChar.Next(26)].ToString() + letras[randomChar.Next(26)].ToString() + "-" + randomChar.Next(10, 99);

     return matriculaRandom;
}

introducir la descripción de la imagen aquí

6
  • Cuando hace el return, ya no sale del ciclo ni sigue ejecutando ese metodo...
    – gbianchi
    el 11 ene. 2023 a las 13:54
  • No hay ningún ciclo dentro de getMatricula() en esta respuesta
    – javdromero
    el 11 ene. 2023 a las 13:55
  • Lo digo por tu explicacion del problema...
    – gbianchi
    el 11 ene. 2023 a las 14:01
  • Es que sinceramente han varios problemas con el método original, creí que eso era la mejor forma que se entienda. Como tú indicas, en el primer return el ciclo se acaba y nunca accede al otro return, pero por eso lo intenté explicar para que el autor entienda que lo que estaba haciendo en sí tenía 2 problemas y 1 ya se lo habían explicado en los comentarios
    – javdromero
    el 11 ene. 2023 a las 14:06
  • la respuesta es buena.. solo ese punto puede confundir a alguien que no entienda bien (porque la llamada recursiva no se hace nunca, ese codigo es inaccesible) ;)
    – gbianchi
    el 11 ene. 2023 a las 14:09
0

Si la función getMatricula devuelve una matricula aleatoria, no necesitas ningún bucle:

public static string getMatricula()
{
    var r = new Random();
    String l = "ABCDEFGHIJKLMN0PQRSTUVWXYZ";

    int n1 = r.Next(10, 99);
    int l1 = r.Next(0, l.Length);
    int l2 = r.Next(0, l.Length);
    int n2 = r.Next(10, 99);

    return $"{n1}-{l[l1]}{l[l2]}-{n2}";
}

Puedes usarlo así:

static void Main(string[] args)
{
    for (int i = 0; i != 10; ++i)
        Console.WriteLine($"Matricula {i + 1}: {getMatricula()}");
}

Ten en cuenta que hay letras que no se ponen en las matrículas por ser fácilmente confundibles entre si (por ejemplo Q y O) o que se pueden confundir con números (por ejemplo 1 I o O 0).

Puedes ver el código funcionando en Try it online!.

1
  • Estoy mismo usando tu codigo, pero me siguen apareciendo las mismas matriculas, el mismo resultado multiplicado 10 veces, no se que esta sucediendo, el codigo de mi profesor tambien era similar y funcionaba, daba 10 matriculas distintas, pero el mio, a pesar de copiar y pegar, solo da una matricula 10 veces el 11 ene. 2023 a las 23:02

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.