função não esta retornando

Mauro Del Gaudio · 27 de outubro de 2015

Boa noite galera!

estou com problemas no seguinte código

//matriz.h

#ifndef matriz_h
#define matriz_h

typedef struct
{
int linhas,colunas,*v;
}Mat;

Mat CriarMatriz();
void AdicionarElemento(Mat **);
void AcessarElemento(Mat **);
void getNumeroLinhas(Mat **);
void getNumeroColunas(Mat **);
void LiberarMemoria(Mat **);
#endif

//matriz.cpp

#include<iostream>
#include "matriz.h"
using namespace std;

Mat CriarMatriz()
{
   int lin,col,i;
   cout<<"Entre com o numero de linhas"<<endl;
   cin>>lin;
   cout<<"Entre com o numero de colunas"<<endl;
   cin>>col;
   Mat *M= new Mat [lin];
   M->linhas=lin;
   M->colunas=col;
   M->v=new int [lin*col];
   return *M;
}

//main.cpp

#include<iostream>
#include "matriz.h"
using namespace std;
int main()
{
   Mat *M;
   *M=CriarMatriz();
   AdicionarElemento(&M);
   AcessarElemento(&M);
   getNumeroLinhas(&M);
   getNumeroColunas(&M);
   LiberarMemoria(&M);
   return 0;
}

ele não esta retornando M na função CriarMatriz(), pelo menos eu acho q é isso alguem pode me ajudar

eu consigo colocar a linha e a coluna, mas depois disso da falha de segmentação

desde já obrigado!

CiroboyBR · 28 de outubro de 2015

é porque você está criando a função com "void", esse tipo de função não retorna nada. Se você quer retornar um inteiro, você tem que criar uma função do tipo inteiro:

int CriaMatriz() {  return numero_inteiro;}

Se quer que a função retorne um numero real, tem que ser do tipo float ou double:

double CriaMatriz() {  return numero_real;}

E assim por diante...

Lucas Souza_774044 · 28 de outubro de 2015

Na verdade acredito que o problema é que, na função CriaMatriz(), você está alocando a memória para um array de "lin" objetos do tipo Mat e a armazenando em um ponteiro temporário e ainda retornando o conteúdo daquele ponteiro. Já na main, por sua vez, você está declarando um ponteiro sem inicializá-lo e está tentando atribuir o retorno da CriaMatriz nele, o que está gerando a falha de segmentação.

O mais apropriado seria você retornar o ponteiro que alocou na CriaMatriz(), lembrando de, na função LiberarMemoria, liberar tanto a memória do vetor de elementos "v" quanto do objeto Mat.

Tente este código:

//matriz.h#ifndef matriz_h#define matriz_h typedef struct{    int linhas,colunas,*v;} Mat; Mat* CriarMatriz();void AdicionarElemento(Mat **);void AcessarElemento(Mat **);void getNumeroLinhas(Mat **);void getNumeroColunas(Mat **);void LiberarMemoria(Mat **);#endif//matriz.cpp#include <iostream>#include "matriz.h"using namespace std;Mat* CriarMatriz(){    int lin,col;    cout<<"Entre com o numero de linhas"<<endl;    cin>>lin;    cout<<"Entre com o numero de colunas"<<endl;    cin>>col;    Mat *M= new Mat();    M->linhas=lin;    M->colunas=col;    M->v=new int [lin*col];    return M;}//main.cpp#include <iostream>#include "matriz.h"using namespace std;int main(){    Mat *M;    M = CriarMatriz();    AdicionarElemento(&M);    AcessarElemento(&M);    getNumeroLinhas(&M);    getNumeroColunas(&M);    LiberarMemoria(&M);    return 0;}

Vou testar ele aqui agora e se der algum problema edito o post.

CiroboyBR · 28 de outubro de 2015

Na verdade acredito que o problema é que, na função CriaMatriz(), você está alocando a memória para um array de "lin" objetos do tipo Mat e a armazenando em um ponteiro temporário e ainda retornando o conteúdo daquele ponteiro. Já na main, por sua vez, você está declarando um ponteiro sem inicializá-lo e está tentando atribuir o retorno da CriaMatriz nele, o que está gerando a falha de segmentação.

O mais apropriado seria você retornar o ponteiro que alocou na CriaMatriz(), lembrando de, na função LiberarMemoria, liberar tanto a memória do vetor de elementos "v" quanto do objeto Mat.

Tente este código:
//matriz.h#ifndef matriz_h#define matriz_h typedef struct{    int linhas,colunas,*v;} Mat; Mat* CriarMatriz();void AdicionarElemento(Mat **);void AcessarElemento(Mat **);void getNumeroLinhas(Mat **);void getNumeroColunas(Mat **);void LiberarMemoria(Mat **);#endif//matriz.cpp#include <iostream>#include "matriz.h"using namespace std;Mat* CriarMatriz(){    int lin,col,i;    cout<<"Entre com o numero de linhas"<<endl;    cin>>lin;    cout<<"Entre com o numero de colunas"<<endl;    cin>>col;    Mat *M= new Mat();    M->linhas=lin;    M->colunas=col;    M->v=new int [lin*col];    return M;}//main.cpp#include <iostream>#include "matriz.h"using namespace std;int main(){    Mat *M;    M = CriarMatriz();    AdicionarElemento(&M);    AcessarElemento(&M);    getNumeroLinhas(&M);    getNumeroColunas(&M);    LiberarMemoria(&M);    return 0;}
Vou testar ele aqui agora e se der algum problema edito o post.

VDD, bem observado, não tinha visto que a função era do tipo Mat.

Lucas Souza_774044 · 28 de outubro de 2015

Testei aqui e ao que parece não deu nenhum problema, então o código deve resolver

Para a função LiberarMemoria, usei esse código:

void LiberarMemoria(Mat **M){	delete[](*M)->v;	delete *M;}

A função também pode usar um parâmetro do tipo Mat * só, se quiser:

void LiberarMemoria(Mat *M){	delete[] M->v;	delete M;}

Nesse último caso, para chamá-la, é só usar "LiberarMemoria(M);".

Mauro Del Gaudio · 29 de outubro de 2015

Deu certo aqui faltava o Mat* hehehe valeu ai galera!

agora na função liberar memoria ta dando falha de segmentação

void LiberarMemoria(Mat **M)
{
delete[](*M)->v;
delete *M;
}

Lucas Souza_774044 · 29 de outubro de 2015

Como está o código completo?

Mauro Del Gaudio · 30 de outubro de 2015

==================matriz.h=============================

#ifndef matriz_h
#define matriz_h

typedef struct
{
int linhas,colunas,*v;
}Mat;

Mat* CriarMatriz();
void AdicionarElemento(Mat **);
void AcessarElemento(Mat **);
void getNumeroLinhas(Mat **);
void getNumeroColunas(Mat **);
void LiberarMemoria(Mat **);
#endif

================matriz.cpp=============================

#include<iostream>
#include "matriz.h"
using namespace std;

Mat* CriarMatriz()
{
   int lin,col,i;
   cout<<"Entre com o numero de linhas"<<endl;
   cin>>lin;
   cout<<"Entre com o numero de colunas"<<endl;
   cin>>col;
   Mat *M= new Mat [lin];
   M->linhas=lin;
   M->colunas=col;
   M->v=new int [lin*col];
   return M;
}
void AdicionarElemento(Mat **M)
{

   int elemento,i,j,k,n,l;
   n=((*M)->linhas)*((*M)->colunas);
   for(l=0;l<n;l++)
   {
       cout<<"Digite qual o numero voce deseja adicionar"<<endl;
       cin>>elemento;
       cout<<"Digite a coordenada em que deseja adicionar o numero"<<endl;
       cin>>i>>j;
       if(i<0 || i>=(*M)->linhas || j<0 || j>=(*M)->colunas)
       {
           cout<<"Acesso invalido"<<endl;
       }
       k=(i-1)*(*M)->colunas+j-1;
       (*M)->v[k]=elemento;
   }
}
void AcessarElemento(Mat **M)
{
   int i,j,k;
   cout<<"Digite a coordenada"<<endl;
   cin>>i>>j;
   if(i<0 || i>=(*M)->linhas || j<0 || j>=(*M)->colunas)
   {
       cout<<"Acesso invalido"<<endl;
   }
   k=(i-1)*(*M)->colunas+j-1;
   cout<<"O elemento é:"<<(*M)->v[k]<<endl;
}
void getNumeroLinhas(Mat **M)
{
   cout<<"Numero de linhas: "<<(*M)->linhas<<endl;
}
void getNumeroColunas(Mat **M)
{
   cout<<"Numero de linhas: "<<(*M)->colunas<<endl;
}
void LiberarMemoria(Mat** M)
{
   delete[] (*M)->v;
   cout<<"!!!!!!!!!!"<<endl;
   delete M;
}

=========================main.cpp====================================

#include<iostream>
#include "matriz.h"
using namespace std;
int main()
{
   Mat *M;
   M=CriarMatriz();
   AdicionarElemento(&M);
   AcessarElemento(&M);
   getNumeroLinhas(&M);
   getNumeroColunas(&M);
   LiberarMemoria(&M);
   return 0;
}

Lucas Souza_774044 · 30 de outubro de 2015

Tem 2 erros nesse código. Na função CriaMatriz, você está alocando um array de Mat, e não um objeto:

Mat *M= new Mat [lin]; //Troque isso por Mat *M = new Mat();

Na função LiberarMemoria, você está passando um ponteiro de ponteiro como parâmetro, então você não está liberando a memória da matriz, mas sim do endereço dela:

 void LiberarMemoria(Mat** M){    delete[] (*M)->v;    cout<<"!!!!!!!!!!"<<endl;    delete M; //Troque isso por delete *M;}

Só um outro aviso, nos códigos AdicionarElemento e AcessarElemento, se o usuário digita um valor inválido, o código está avisando que deu um acesso inválido, mas ainda está prosseguindo, o que pode causar outra falha de segmentação dependendo dos valores.

Mauro Del Gaudio · 31 de outubro de 2015

ainda ta dando o erro, coloquei um break la no adicionar elemento caso a operação seja invalida, mas o de acessar nao ta em um loop então n tem como colocar break,e tambem mudei o if, tirei o || e coloquei &&.

ainda ta dando falha de segmentação

acho q esta alocando errado por que quando procuro v na posição 2 da 0 e era pra conter um numero que foi digitado.

mas n estou conseguindo ver o que pode ser hehehehe

#include<iostream>
#include "matriz.h"
using namespace std;

Mat* CriarMatriz()
{
   int lin,col,i;
   cout<<"Entre com o numero de linhas"<<endl;
   cin>>lin;
   cout<<"Entre com o numero de colunas"<<endl;
   cin>>col;
   Mat *M= new Mat();
   M->linhas=lin;
   M->colunas=col;
   M->v=new int [lin*col];
   return M;
}
void AdicionarElemento(Mat **M)
{

   int elemento,i,j,k,n,l;
   n=((*M)->linhas)*((*M)->colunas);
   for(l=0;l<n;l++)
   {
       cout<<"Digite qual o numero voce deseja adicionar"<<endl;
       cin>>elemento;
       cout<<"Digite a coordenada em que deseja adicionar o numero"<<endl;
       cin>>i>>j;
       if(i<0 && i>(*M)->linhas && j<0 && j>(*M)->colunas)
       {
           cout<<"Acesso invalido"<<endl;
           break;
       }
       k=(i-1)*(*M)->colunas+j-1;
       (*M)->v[k]=elemento;
   cout<<(*M)->v[k]<<endl;
   }
}
void AcessarElemento(Mat **M)
{
   int i,j,k;
   cout<<"Digite a coordenada"<<endl;
   cin>>i>>j;
   if(i<0 && i>(*M)->linhas && j<0 && j>(*M)->colunas)
   {
       cout<<"Acesso invalido"<<endl;
   }
   k=(i-1)*(*M)->colunas+j-1;
   cout<<"O elemento é:"<<(*M)->v[k]<<endl;
}
void getNumeroLinhas(Mat **M)
{
   cout<<"Numero de linhas: "<<(*M)->linhas<<endl;
}
void getNumeroColunas(Mat **M)
{
   cout<<"Numero de linhas: "<<(*M)->colunas<<endl;
}
void LiberarMemoria(Mat** M)
{
   cout<<(*M)->v[2]<<"!!!!!!!!!"<<endl;
   delete[] (*M)->v;
   cout<<"!!!!!!!!!!"<<endl;
   delete *M;
}

Lucas Souza_774044 · 2 de novembro de 2015

O || dentro dos ifs estava certo, porque é para dar esse aviso se qualquer uma das condições for verdadeira; com o &&, só aconteceria por exemplo se o i fosse menor que 0 e maior que o número de linhas ao mesmo tempo, o que obviamente não pode acontecer. Onde está dando erro aí eu não sei dizer de cara... tente fazer algum teste, tipo dando cout em algumas linhas para ver onde exatamente está dando erro, testando vários valores e tal.