Skip to main content
2 de 3
cuerpo editado
AguaSal
  • 451
  • 6
  • 20

Error en operaciones TAD matriz C++

Estoy implementando unas operaciones para un TAD matriz, las operaciones son: suma, resta, multiplicación y trasponer.

La mayor parte del TAD viene ya programada en el ejercicio, solo tengo que definir e implementar esas operaciones.

El código es el siguiente:

En la cabecera:

#ifndef MATRIZ_HPP
#define MATRIZ_HPP

#include <stdexcept>
#include <iostream>
#include <iomanip>

#define F 3
#define C 3

class Matriz
{
public:

  // Definición de las clases de excepciones
  class ECoordenadasIncorrectas: public std::runtime_error
  {
  public:
    ECoordenadasIncorrectas(const std::string& w = "ECoordenadasIncorrectas"): std::runtime_error(w) {}
  };

  Matriz();
  
  ~Matriz();

  float valor(int i, int j);

  void asignar(int i, int j, float v);

  template <typename ModificaElemento>
  void modificar(ModificaElemento modifica_elemento);

  Matriz& operator=(Matriz& m);

  /** Constructor de copia */
  Matriz(Matriz& m);

  bool operator==(Matriz& m);
  bool operator!=(Matriz& m);    
  
  //ESTA ES LA PARTE QUE YO ESTOY IMPLEMENTANDO
  void identidad();
  void ceros();
  void unos();
  void multiplicar(Matriz m, Matriz& salida);
  void sumar(Matriz m, Matriz& salida);
  void restar(Matriz m, Matriz& salida);
  void trasponer(Matriz& salida);
  int filas();
  int columnas();
  //HASTA AQUÍ
private:
  float elementos_[F][C];
};

// En las templates (TADs genéricos) hacemos la inclusión al revés el .cpp en el .hpp (para que
// al incluir el matrices.hpp vaya todo el código y se pueda hacer la instanciación de la
// template).
#include "matriz.cpp"
#include "matriz_io.cpp"

#endif // MATRIZ_HPP

El cpp de la entrada y salida del que no se puede tocar nada(viene hecho):

std::ostream& operator<<(std::ostream& os, Matriz & m)
{
  os << m.filas() << " " << m.columnas() << std::endl;
  os << std::setprecision(4) << std::fixed;
  for(int i=1; i <= m.filas(); i++)
  {
    for(int j=1; j <= m.columnas(); j++)
    {
      os << m.valor(i,j) << " ";
    }
    os << std::endl;
  }

  return os;
}

std::istream& operator>>(std::istream& is, Matriz& m)
{
  int filas, columnas;
  float v;

  is >> filas >> columnas;
  for (int i=1; i<=filas; i++)
  {
    for (int j=1; j<=columnas; j++)
    {
      is >> v;
      m.asignar(i,j,v);
    }
  }

  return is;
}

El cpp con la implementación de las operaciones(solo pongo las mías):

void Matriz::unos()
{
    for(int i=0; i<F; i++)
    {
        for(int j=0; j<C; j++)
        {
            elementos_[i][j]=1;
        }
    }
}

int Matriz::columnas()
{
    return C;
}

int Matriz::filas()
{
    return F;
}

void Matriz::sumar(Matriz m, Matriz& salida)
{
    for(int i=0; i<F; i++)
    {
        for(int j=0; j<C; j++)
        {
            salida.elementos_[i][j]+=m.elementos_[i][j];
        }
    }

}

void Matriz::restar(Matriz m, Matriz& salida)
{
    for(int i=0; i<F; i++)
    {
        for(int j=0; j<C; j++)
        {
            salida.elementos_[i][j]-=m.elementos_[i][j];
        }
    }
}
void Matriz::multiplicar(Matriz m, Matriz& salida)
{
    for(int i=0; i<F; i++)
    {
        for(int j=0; j<C; j++)
        {
            for(int k=0; k<C;k++)
            {
                salida.elementos_[i][j]+=m.elementos_[i][k]*salida.elementos_[k][j];
            }
        }
    }
}
void Matriz::trasponer(Matriz& salida)
{
    for(int i=0; i<F; i++)
    {
        for(int j=0; j<C; j++)
        {
            salida.elementos_[i][j]=salida.elementos_[j][i];
        }
    }
}

Y el main.cpp para probar todas las operaciones(pongo únicamente las que afectan a mis operaciones, todo viene hecho aquí):

#include "matriz.hpp"

#define Elemento float

#define MatrizPrueba Matriz


void probarCrearMatriz()
{
  MatrizPrueba m;
  std::cout << m;
}
void probarMultiplicar(std::istream& is)
{
  MatrizPrueba m1, m2, m3;

  is >> m1;
  is >> m2;

  m1.multiplicar(m2, m3);

  std::cout << m3;
}

void probarSumar(std::istream& is)
{
  MatrizPrueba m1, m2, m3;

  is >> m1;
  is >> m2;

  m1.sumar(m2, m3);

  std::cout << m3;
}

void probarRestar(std::istream& is)
{
  MatrizPrueba m1, m2, m3;

  is >> m1;
  is >> m2;

  m1.restar(m2, m3);

  std::cout << m3;
}

void probarTrasponer(std::istream& is)
{
  MatrizPrueba m1, m2;

  is >> m1;

  m1.trasponer(m2);

  std::cout << m2;
}
void probarUnos()
{
  MatrizPrueba m;
  m.unos();
  std::cout << m;
}
int main()
{
  char opcion;

  // Lectura de la operaci?n a probar
  std::cin >> opcion;

  try
  {
    switch(opcion)
    {
      case 'c':  probarCrearMatriz(); break;
      case 'i':  probarIdentidad(); break;
      case 'u':  probarUnos(); break;
      case 'z':  probarCeros(); break;
      case 'a':  probarAsignar(std::cin);  break;
      case 'v':  probarValor(std::cin); break;
      case 'm':  probarMultiplicar(std::cin); break;
      case 'r':  probarRestar(std::cin); break;
      case 's':  probarSumar(std::cin); break;
      case 't':  probarTrasponer(std::cin); break;
      case 'o':  probarModificar(std::cin); break;
      case 'C':  probarCopiar(std::cin); break;
      case '=':  probarIgual(std::cin); break;
      case 'D':  probarDestruir(std::cin); break;
    }
  }
  catch (std::exception const& excepcion)
  {
    std::cout << "EXCEPCION GENERADA: "  <<  excepcion.what() << std::endl;
  }
}

El error que no consigo solucionar es el siguiente:

Todas las operaciones que no sean mías excepto unos y ceros funcionan perfectamente, pero al llegar a las operaciones suma resta multiplicacion y trasponer, una de las matrices que tienes que introducir por teclado no se guarda, y en la suma, resta y multiplicación me imprime la 2ª matriz que introduzco por teclado(en vez del resultado de la operación de ambas matrices), en ´trasponer´ me imprime una matriz 3x3 de 0's.

¿Alguien puede ver el error? Yo seguiré buscando.

AguaSal
  • 451
  • 6
  • 20