Ir ao conteúdo
  • Cadastre-se
Mauro Del Gaudio

RESOLVIDO função não esta retornando

Recommended Posts

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!

Compartilhar este post


Link para o post
Compartilhar em outros sites

é 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...

Compartilhar este post


Link para o post
Compartilhar em outros sites

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.

  • Curtir 1

Compartilhar este post


Link para o post
Compartilhar em outros sites

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.

Compartilhar este post


Link para o post
Compartilhar em outros sites

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);".

Compartilhar este post


Link para o post
Compartilhar em outros sites

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;
}

Compartilhar este post


Link para o post
Compartilhar em outros sites

==================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;
}
 

Compartilhar este post


Link para o post
Compartilhar em outros sites

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.

Compartilhar este post


Link para o post
Compartilhar em outros sites

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;
}

Compartilhar este post


Link para o post
Compartilhar em outros sites

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.

Compartilhar este post


Link para o post
Compartilhar em outros sites

Crie uma conta ou entre para comentar

Você precisar ser um membro para fazer um comentário

Criar uma conta

Crie uma nova conta em nossa comunidade. É fácil!

Crie uma nova conta

Entrar

Já tem uma conta? Faça o login.

Entrar agora





Sobre o Clube do Hardware

No ar desde 1996, o Clube do Hardware é uma das maiores, mais antigas e mais respeitadas publicações sobre tecnologia do Brasil. Leia mais

Direitos autorais

Não permitimos a cópia ou reprodução do conteúdo do nosso site, fórum, newsletters e redes sociais, mesmo citando-se a fonte. Leia mais

×