Skip to main content
conteúdo editado
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) p->qnt = 0;
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++) printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone. E no repl.itno repl.it. Também coloquei no GitHub para referência futura.

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) p->qnt = 0;
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++) printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) p->qnt = 0;
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++) printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

adicionou 118 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) {
        p->qnt = 0;
    }
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++)
        printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone efuncionando no ideone. E no Coding Groundno repl.it. Também coloquei no GitHub para referência futura.

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) {
        p->qnt = 0;
    }
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++)
        printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone e no Coding Ground.

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) p->qnt = 0;
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++) printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.

Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

Eu dei uma melhorada em várias coisas, entre elas simplifiquei a passagem de parâmetro, não precisa passar ponteiro de ponteiro. Mas o erro principal é que o operador >= não podia ser usado no while. Quando chega a 0 tem que parar, por causa do = ele fazia uma operação a mais. Mas também há um problema do outro lado. Começa pegando o índice equivalente à quantidade de elementos na pilha. Como índice começa em 0, o último é a quantidade menos 1. Com essas duas alterações funciona. Mas ainda não está liberando a memória (eu sei que é algo desnecessário nesse exemplo, mas é bom se acostumar fazer certo sempre):

#include <stdio.h>
#include <stdlib.h>

#define MAX 10

typedef struct pilha {
    int qnt;
    int dados[MAX];
} Pilha;
Pilha* cria() {
    Pilha *p = malloc(sizeof(Pilha));
    if (p != NULL) {
        p->qnt = 0;
    }
    return p;
}
void insere(Pilha *p, int valor) {
    p->dados[p->qnt] = valor;
    p->qnt++;
}
void exibe(Pilha *p) {
    for (int i = 0; i < p->qnt; i++)
        printf("%3d", p->dados[i]);
}
Pilha* inverte(Pilha *p) {
    Pilha *outra = cria();
    while (p->qnt > 0) {
        outra->dados[outra->qnt] = p->dados[p->qnt - 1];
        p->qnt--;
        outra->qnt++;
    }
    return outra;
}
int main() {
    Pilha *p = cria();
    insere(p, 32);
    insere(p, 45);
    insere(p, 78);
    exibe(p);
    p = inverte(p);
    printf("\n");
    exibe(p);
}

Veja funcionando no ideone e no Coding Ground.