Tu código, a parte del error mencionado por Trauma, tiene errores de algoritmo y hace uso de malas prácticas.
Estás programando en c++ pero haces uso de cabeceras de c++.
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.
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.
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!.