Ir ao conteúdo

função não esta retornando


Ir à solução Resolvido por Mauro Del Gaudio,

Posts recomendados

Postado

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!

Postado

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

Postado

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
Postado

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.

Postado

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

Postado

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

Postado

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.

Postado

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

Postado

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.

Visitante
Este tópico está impedido de receber novas respostas.

Sobre o Clube do Hardware

No ar desde 1996, o Clube do Hardware é uma das maiores, mais antigas e mais respeitadas comunidades 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

×
×
  • Criar novo...

Ebook grátis: Aprenda a ler resistores e capacitores!

EBOOK GRÁTIS!

CLIQUE AQUI E BAIXE AGORA MESMO!