Foro de elhacker.net

Hola. No sé cual es el error en el siguiente código, ¿alguien podría ayudarme?

//Problema 1: Programa que cree una matriz nxn, ingrese datos y luego imprima la suma de los elementos de la diagonal principal

#include<iostream>
#include<cstdlib>

using namespace std;

void llenarymostrarM(int *matriz, int f, int c);
int sumadiagp(int *matriz, int f, int c);

int main()
{
    int *mat, fil, col;
    cout<<"El numero de filas es: ";
    cin>>fil;
    cout<<"El numero de columnas es: ";
    cin>>col;
    mat=new int[fil, col];
    llenarymostrarM(mat, fil, col);
    cout<<endl;
    cout<<"La suma de los elementos de la diagonal principal es: "      <<sumadiagp(mat, fil, col)<<endl;
    delete []mat;
    system("pause");
}

void llenarymostrarM(int *matriz, int f, int c){
     int i,j;
     cout<<"Ingrese los elementos de la matriz: ";
     for(i=0; i<f; i++){
              for(j=0; j<c; j++){                                         
                       cin>>matriz[i,j];
                       }
     }
     cout<<endl;
     cout<<"La matriz es: ";
     for(i=0; i<f; i++){
              for(j=0; j<c; j++){
                       cout<<matriz[i,j]<<endl;
                       }
     }
}                           

int sumadiagp(int *matriz, int f, int c){
      int i,j, sdp=0;
      for(i=0; i<f; i++){
              for(j=0; j<c; j++){
                       sdp=sdp+matriz[i,i];
                               }
              }
      return sdp;
}

Por ejemplo cuando pongo una matriz 2x2: 1, 2, 3, 4, me sale que la matriz es: 3 4 3 4 y la suma de la diagonal principal es 14  :huh:

* Tienes errores en los punteros del array bidimensional, que han de ser **, tanto en su declaración como en los argumentos de las funciones.

* Se te ha ido la olla al hacer mat[ i , j ] en lugar de mat [ i ][ j ]

* Tienes mal declarado el array bidimensional con el operador new.

* Te falta el return de main.

Con esas correcciones o puntualizaciones, ya que el resto del código es el tuyo, y tabulando o indentando el código de forma más razonable -no escatimes con los espacios en blanco, si no queda todo apelotonado- una posible solución, según tienes planteado, sería:

---

EI: codigo suprimido.

---

Por cierto, si quieres la suma de la diagonal no olvides que fil = col. Y si han de ser iguales te puedes ahorrar un cin, tomando col = fil si la matriz ha de ser del tipo n x n.

Muchas gracias por la ayuda. Creo que este el código final además le agregue otro problema jeje.

#include<iostream>
#include<cstdlib>

using namespace std;

void llenarymostrarM(int **matriz, int o);
int mayor(int **matriz, int o);
int sumadiagp(int **matriz, int o);

int main()
{
    int **mat, orden;
    cout<<"El orden de la matriz es: ";
    cin>>orden;
    mat=new int*[orden];
    llenarymostrarM(mat, orden);
    cout<<endl;
    cout<<"El mayor elemento es: "<<mayor(mat, orden)<<endl;
    cout<<"La suma de los elementos de la diagonal principal es: "<<sumadiagp(mat, orden)<<endl;
    delete []mat;
    system("pause");
}

void llenarymostrarM(int **matriz, int o){
     int i,j;
     cout<<"Ingrese los elementos de la matriz: ";
     for(i=0; i<o; i++){
              for(j=0; j<o; j++){                                        
                       cin>>matriz[j];
                       }
     }
     cout<<endl;
     cout<<"La matriz es: ";
     for(i=0; i<o; i++){
              for(j=0; j<o; j++){
                       cout<<matriz[j]<<endl;
                       }
     }
}                            

int mayor(int **matriz, int o){
    int i,j, mayor=matriz[0][0];
    for(i=1; i<o; i++){
              for(j=0; j<o; j++){
                       if(matriz[j]>mayor){
                                      mayor=matriz[j];
                                      }
              }
    }
    return mayor;
}

int sumadiagp(int **matriz, int o){
      int i,j, sdp=0;
      for(i=0; i<o; i++){
              for(j=0; j<o; j++){
                       if(i==j)
                       sdp=sdp+matriz[j];
                               }
              }
      return sdp;
}

Con respecto al return 0, ¿es siempre obligatorio?, nunca lo he usado en otros códigos.  Donde dice matriz[j] en realidad debe ser matriz i[j], no sé porque se ve asi. Por último, cuando termino de ejecutar el programa sale que debe cerrarse  :huh:

Respecto a lo último se debe a que [ i ] , sin los espacios en blanco que he puesto, es la secuencia de escape para las letras en cursivas. Por ello a partir del primer [ i ] que escribes la letra cambia a cursiva. Fíjate en el código de tu mensaje y lo veras. Para evitarlo usa las llamadas etiquetas GeSHi, en la parte superior de los iconos a la derecha, y en el desplegable que aparece elige las de C++ y en medio de las etiquetas "code" que aparecen pegas tu código.

Sí, el return es obligatorio porque por cuestiones del estándar del C/C++ main se debe declarar como una función de tipo int y estas han de retornar algo. Lo normal es que si no lo pones el programa te lance un warning de advertencia, al menos en el modo pedantic.
 
Y respecto al código sigues declarando mal el array. Como el mio lo han borrado te lo vuelvo a poner:

int **mat = new int*[orden];
    for( int i = 0; i < orden; ++i )
      mat[i] = new int[orden];

Y ya está.

También te aconsejo el uso de mensajes para introducir los valores, como:

cout <<"Ingrese los elementos de la matriz: "<< endl;
  for( i = 0; i < f; i++ )
    for( j = 0; j < c; j++ ){
      cout << "matriz[" << i << "][" << j << "] = " ;
      cin >> matriz[i][j];
    }
  cout << endl;

Y en la impresión que lo hagas como matriz:

cout << "La matriz es: " << endl;
  for( i = 0; i <f ; i++){
    for( j = 0; j < c; j++ )
      cout << matriz[i][j] << "  ";
    cout <<  endl;
  }

Así te quedará todo mucho más claro.

Muchas gracias por la ayuda, recién por cuestiones de tiempo he podido entrar.