Skip to main content
se eliminó 1 carácter en el cuerpo
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86
  1. Estás programando en pero haces uso de cabeceras de .

    Las cabeceras <stdlib.h> y <time.h> no deben ser usadas en programas de C++, si realmente necesitas algo de ellas (que no es el caso) deberías usar las versiones adaptadaadaptadas a C++ que son <cstdlib> y <ctime>. Lee este hilo para saber más del tema.

  2. Evita usar variables globales.

    Las variables globales son muy problemáticas y propensas a errores, evita usarlas. Lee este hilo para saber más del tema.

  3. La función rand pertenece a las rutinas de C.

    Se considera una función lenta, ineficiente e insegura, tanto es así que se valora deprecar su uso en C++, no la uses. Lee este hilo para saber más del tema.

  1. Estás programando en pero haces uso de cabeceras de .

    Las cabeceras <stdlib.h> y <time.h> no deben ser usadas en programas de C++, si realmente necesitas algo de ellas (que no es el caso) deberías usar las versiones adaptada a C++ que son <cstdlib> y <ctime>. Lee este hilo para saber más del tema.

  2. Evita usar variables globales.

    Las variables globales son muy problemáticas y propensas a errores, evita usarlas. Lee este hilo para saber más del tema.

  3. La función rand pertenece a las rutinas de C.

    Se considera una función lenta, ineficiente e insegura, tanto es así que se valora deprecar su uso en C++, no la uses. Lee este hilo para saber más del tema.

  1. Estás programando en pero haces uso de cabeceras de .

    Las cabeceras <stdlib.h> y <time.h> no deben ser usadas en programas de C++, si realmente necesitas algo de ellas (que no es el caso) deberías usar las versiones adaptadas a C++ que son <cstdlib> y <ctime>. Lee este hilo para saber más del tema.

  2. Evita usar variables globales.

    Las variables globales son muy problemáticas y propensas a errores, evita usarlas. Lee este hilo para saber más del tema.

  3. La función rand pertenece a las rutinas de C.

    Se considera una función lenta, ineficiente e insegura, tanto es así que se valora deprecar su uso en C++, no la uses. Lee este hilo para saber más del tema.

Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

Tu código, a parte del error mencionado por Trauma, tiene errores de algoritmo y hace uso de malas prácticas.

  1. Estás programando en pero haces uso de cabeceras de .

    Las cabeceras <stdlib.h> y <time.h> no deben ser usadas en programas de C++, si realmente necesitas algo de ellas (que no es el caso) deberías usar las versiones adaptada a C++ que son <cstdlib> y <ctime>. Lee este hilo para saber más del tema.

  2. Evita usar variables globales.

    Las variables globales son muy problemáticas y propensas a errores, evita usarlas. Lee este hilo para saber más del tema.

  3. La función rand pertenece a las rutinas de C.

    Se considera una función lenta, ineficiente e insegura, tanto es así que se valora deprecar su uso en C++, no la uses. Lee este hilo para saber más del tema.


Mi propuesta es que uses el motor de números pseudo-aleatorios de C++ y que por supuesto, sigas las recomendaciones anteriores, si lo haces tu código podría parecerse a:

#include <iostream>
#include <random>
#include <vector>
#include <utility>
#include <algorithm>
#include <limits>

using Doblete = std::pair<int, int>;
using Fila = std::vector<int>;
using Matriz = std::vector<Fila>;

Usaremos <vector> para crear la matriz dinámicamente, haciendo que el tamaño sea verdaderamente arbitrario en lugar de limitarlo a un máximo de cien. La cabecera <utility> nos da acceso al tipo std::pair, la <algorithm> a las funciones std::max y std::max_element y <limits> a la plantilla std::numeric_limits. Los alias Doblete, Fila y Matriz hacen que el código sea más fácil de comprender.

Doblete pedir()
{
    Doblete resultado;

    std::cout << "introducir el numero de filas\n";
    std::cin >> resultado.first;

    std::cout << "introducir el numero de columnas\n";
    std::cin >> resultado.second;

    return resultado;
}

Pedimos los datos dentro de la función pedir y los devolvemos como Doblete, nos ahorramos así una variable global.

Matriz generar(const Doblete &tamanyo)
{
    Matriz resultado(tamanyo.first, Fila(tamanyo.second, 0));

    std::random_device dispositivo;
    std::mt19937 generador(dispositivo());
    std::uniform_int_distribution<> distribucion(1, 100);

    for (auto &fila : resultado)
    {
        for (auto &valor : fila)
        {
            valor = distribucion(generador);
        }
    }

    return resultado;
}

Generamos la matriz dentro de la función generar; no nos debe preocupar la copia del resultado desde dentro de la función a fuera de la función pues el estándar de C++ garantiza que ese copia será obviada (copy elysion optimization). Se usa el motor de números pseudo-aleatorios de C++.

El resto del código quedaría así:

void imprimir(const Matriz &matriz)
{
    for (const auto &fila : matriz)
    {
        for (const auto &valor : fila)
        {
            std::cout << " [" << valor << "] ";
        }
        std::cout << '\n';
    }
}

int comprobar(const Matriz &matriz)
{
    int resultado = std::numeric_limits<int>::min();

    for (const auto &fila : matriz)
    {
        resultado = std::max(resultado, *std::max_element(fila.begin(), fila.end()));
    }

    return resultado;
}

int main()
{
    const auto &matriz = generar(pedir());
    imprimir(matriz);

    std::cout << "El mayor numero de la matriz es: " << comprobar(matriz) << '\n';

    return 0;
}

Puedes verlo funcionando en Try it online!.