0
#include<iostream>
#include<time.h>
using namespace std;
int main()
{
    int filas,columnas,matriz[100][100];
    cout<<"por favor introdusca la cantidad de filas : ";cin>>filas;
    cout<<"por favor introdusca la cantidad de columnas : ";cin>>columnas;

    for(int i=0;i<filas;i++){
        for(int j=0;j<columnas;j++){
              srand(time(NULL));
              matriz[i][j]=1+rand()%(100+1-1);
        }
    }
    cout<<"la matriz quedo de esta forma"<<endl;
    for(int i=0;i<filas;i++){
        for(int j=0;j<columnas;j++){
            cout<< matriz[i][j];                   
        }
        cout<<"\n";
    }
    system("pause");
    return 0;
 }
1
  • Primero, deberías editar tu pregunta, para que el código quede ordenado y entendible. Segundo, ¿a que te refieres con que suma el doble de filas que debería tener?
    – JYass
    Commented el 21 jun. 2016 a las 2:39

1 respuesta 1

5

Tienes una serie de problemas de estilo en tu código que hacen difícil seguir el problema que planteas.

Problemas


int filas,columnas,matriz[100][100];

No has dado valor inicial a ninguna variable; esto no es estrictamente necesario pero además de ser considerado una buena práctica puede ayudarte a detectar errores.


matriz[100][100];

Estás usando una matriz de dimensiones determinadas en tiempo de compilación pero después solicitas las dimensiones en tiempo de ejecución esto es un problema porque...

  • Malgastas recursos al solicitar más memoria de la realmente requerida cuando el usuario introduce dimensiones menores a 100.
  • Causas desbordamientos de búfer cuando el usuario introduce alguna dimensión mayor a 100, o ambas.

for(int i=0;i<filas;i++){
    for(int j=0;j<columnas;j++){
          srand(time(NULL));
          matriz[i][j]=1+rand()%(100+1-1);
    }
}

Estableces la semilla aleatoria a cada vuelta del bucle, dado que la estableces a el valor del tiempo actual (time(NULL)) si el bucle es rápido estarás asignando la misma semilla a cada vuelta; reasignar la misma semilla provoca que la secuencia de números pseudo-aleatorios se reinicie con lo que obtendrás el mismo valor contínuamente.


(100+1-1)

Debería ser evidente que cien más uno menos uno es cien.


Soluciones


Da un valor inicial a tus variables:

int filas = 0, columnas = 0;

Utiliza un contenedor para la matriz, std::vector ofrece memoria anexa:

std::vector<int> matriz(filas * columnas, 0);

Establece la semilla fuera del bucle:

srand(time(NULL));

for(int i=0;i<filas;i++){
    for(int j=0;j<columnas;j++){
          matriz[(i * filas) + j]=1+(rand()%100);
    }
}

Tras las correcciones, [no veo qué problema tiene tu código] ¿Podrías ser más específico?

Por cierto, en los bucles for usa pre-increment siempre que te sea posible.

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.