error en tipo de datos matriz c++

[drd222]

Estoy aprendiendo esto de matrices en c++ y bueno me toco hacer una matriz transpuesta... pero al compilar el programa me sale errores en esta funcion x favor alguien si me pudiera ayudar con esto de "tipos de datos" os agradeceria...
Gracias de antemano... Aca esta el codigo de la funcion que mencione.

Código:

double trans(double **a, unsigned n, unsigned m) {
	double nfa,nca; double **at;
	*at=new double(nca);
	for(double i=0;i<n;i++)
		(*at)[i]=new double(nfa);
	for(double j=0;j<n;j++){
		for(double k=0;j<m;k++)
			(*at)[j][k]=a[k][j];
	}
	return **at;
}

[cesar_casla]

Hola! nose mucho de C++, pero tenes algunos errores
1º tenes problemas de punteros. Esto *at significa indireccion, osea "lo apuntado de at", es lo mismo hacer at[i].
2º usas nfa y nca, pero a esas variables nunca le diste un valor. Por lo que entiendo del codigo tendria que ser n y m, osea nfa=m y nca=n // si es q n son filas y m columnas de double **a

Si queres hacer matrices dinamicas debe ser:

Código C:

double **trans(double **a, unsigned n, unsigned m) {
    double nfa,nca; double **at;
at=new double *[nfa];
    for(double i=0;i<m;i++)
        at[i]=new double [nca];
    for(double j=0;j<n;j++){
        for(double k=0;k<m;k++)
            at[k][j]=a[j][k];
    }
        return at;
}
 

fijate lo que retorno y lo que devuelve la funcion. Retorno at que es un puntero a puntero a double, por lo tanto la funcion debe devolver lo mismo.. Sino qdo muy claro el tema de la matriz dinamica, mira aca:
http://c.conclase.net/curso/?cap=013b


Saludos

[Fw190]

Primero, estás utilizando nca y nfa (que, supongo, se refieren a número de columnas y de filas) sin haberles asignado valor alguno, por lo que tomarán valores aleatorios.

Segundo, para lo que estás empleando estas variables es para reservar memoria de forma dinámica. Aunque el compilador haga la conversión correspondiente, debido a que los valores que hayan tomado son aleatorios, puede que sean valores no válidos para una reserva de memoria.

Tercero, estás utilizando variables de tipo double como variables de control para bucles e índices para los elementos de la matriz. No es ilegal, pero tampoco conveniente. Para esto, utiliza valores int o unsigned int.

Cuarto, estás devolviendo un doble puntero a double en una función que has declarado como que retorna un double.

Para transponer una matriz, sería más sencillo así:

Código C++:

void Trans1(double **A, unsigned filas, unsigned cols)
{
double x;
for (int i=0; i<filas; i++)
for (int j=0; j<columnas; j++)
{
x = A[i][j];
A[i][j] = A[j][i];
A[j][i] = x;
}
 

Esta te sobreescribe la matriz de entrada. Si necesitas conservar la matriz de entrada, tendrás que pasar también la matriz de salida:

Código C++:

void Trans2(double **A, double **B, unsigned filas, unsigned cols)
{
for (int i=0; i<filas; i++)
for (int j=0; j<columnas; j++)
B[i][j] = A[j][i];
}
 

Claro está, suponiendo que las matrices A y B han sido convenientemente creadas antes de llamar a la función.

[drd222]

Muchas gracias por las respuestas ... ya entendi mejor el uso de estas variables... ahora no se por que este programa no me funciona aca el codigo... Creo que es algun error de logica en la funcion "**trans" si me pudieran ayudar les agradeceria ... aca el codigo completo... El programa debe devolver la matriz transversa... El programa funciona hasta que pongo numeros muy grandes para las filas y columnas de la matriz por ej 15 y 12

Código C++:

#include <iostream>
#include <iomanip>
#include <vector>
#include <cmath>
#include <cstdlib>
using namespace std;
 
void leerDimensiones(unsigned &n, unsigned &m){
    try{
        cout<<"Numero de filas y columnas (0 y 0) para salir: "<<endl;
        cin>>n>>m;
        if(n==0 && m==0) throw string("¡Error! Matriz vacía.");
        if(n<=0) throw 1;
        if(m<=0) throw 2;
    }catch(int e){
        switch(e){
        case 1:
            cout<<"El numero de filas debe ser mayor a 0!"<<endl;break;
        case 2:
            cout<<"El numero de columnas debe ser mayor a 0!"<<endl; break;
        }
        leerDimensiones(n,m);
    }
}
 
void leerlim(double &li, double &ls){
    try{
        cout<<"Limites inferior y superior: "<<endl;
        cin>>li>>ls;
        if(li>=ls) throw 1;
    }catch (int e){
        cout<<"El limite inferior debe ser menor al superior !"<<endl;
        leerlim(li,ls);
    }
}
 
int **randMatE(unsigned n, unsigned m, int li, int ls) {
    unsigned i,j,d=ls-li; int **a;
    a=new int*[n];
    for (i=0;i<n;i++){
        a[i]=new int[m];
        for (j=0;j<m;j++)
            a[i][j]= (int)((double)rand()/RAND_MAX*d+li);}
    return a;
}
 
void showMat(int **a, unsigned n, unsigned m){
    unsigned i,j;
    for (i=0;i<n;i++){
        for (j=0;j<m;j++){
            cout<<" "<<setw(8)<<setprecision(5)<<a[i][j];
            if ((j+1)%10==0) cout<<endl;
        }
        cout<<endl;
    }
}
 
 
int **trans(int **a, unsigned n, unsigned m) {
    unsigned i,j;int **at;
    at=new int *[n];
    for(i=0;i<m;i++)
        at[i]=new int(m);
    for(i=0;i<n;i++){
        for(j=0;j<m;j++)
            at[j][i]=a[i][j];
    }
    return at;
}
 
int main(){
    double li,ls;int **a;
    unsigned n,m;
    cout<<"***** Matriz traspuesta *****"<<endl;
    try{
        do {
            leerDimensiones(n,m);
            leerlim(li,ls);
            a=randMatE(n,m,li,ls);
            showMat(a,n,m);
            a=trans(a,n,m);
            cout<<"Matriz transpuesta"<<endl;
            showMat(a,m,n);
        } while(n!=0 && m!=0);
    } catch(string &e) {
        cout<<e;
    }
    return 0;
}