Ir ao conteúdo
  • Comunicados

    • Gabriel Torres

      Seja um moderador do Clube do Hardware!   12-02-2016

      Prezados membros do Clube do Hardware, Está aberto o processo de seleção de novos moderadores para diversos setores ou áreas do Clube do Hardware. Os requisitos são:   Pelo menos 500 posts e um ano de cadastro; Boa frequência de participação; Ser respeitoso, cordial e educado com os demais membros; Ter bom nível de português; Ter razoável conhecimento da área em que pretende atuar; Saber trabalhar em equipe (com os moderadores, coordenadores e administradores).   Os interessados deverão enviar uma mensagem privada para o usuário @Equipe Clube do Hardware com o título "Candidato a moderador". A mensagem deverá conter respostas às perguntas abaixo:   Qual o seu nome completo? Qual sua data de nascimento? Qual sua formação/profissão? Já atuou como moderador em algo outro fórum, se sim, qual? De forma sucinta, explique o porquê de querer ser moderador do fórum e conte-nos um pouco sobre você.   OBS: Não se trata de função remunerada. Todos que fazem parte do staff são voluntários.
    • DiF

      Poste seus códigos corretamente!   21-05-2016

      Prezados membros do Fórum do Clube do Hardware, O Fórum oferece um recurso chamado CODE, onde o ícone no painel do editor é  <>     O uso deste recurso é  imprescindível para uma melhor leitura, manter a organização, diferenciar de texto comum e principalmente evitar que os compiladores e IDEs acusem erro ao colar um código copiado daqui. Portanto convido-lhes para ler as instruções de como usar este recurso CODE neste tópico:  
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.

Editado por Lucas Souza_774044
  • 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.

Editado por Lucas Souza_774044

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






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

×