Compare two strings in C

Question

Hello, people, I want to compare two strings but the result is not returning what is expected, in this case the index in the function search_name, the error is in the fourth line of the function search, but I am providing all the code. the fourth line test is not returning true even when the two strings are equal, can anyone tell me why? Thanks from now on.

/* 
 * File:   main.c
 * Author: pmargreff
 *
 * Created on 6 de Novembro de 2014, 00:25
 */
#include <stdio.h>
#include <stdlib.h>
#define MAX 100

struct end {
    char nome[100];
    char rua[100];
    char cidade[100];
    char estado[100];
    unsigned long int cep;
} info[MAX];

/*
 * struct para conseguir comparar 
 * dois nomes em diversas funções
 */
struct compara {
    char nome[100];
} busca;

void cria_lista(void);
void insere(void);
void apaga(void);
void imprime(void);
int menu(void);
int livre(void);
void ler_string(char palavra[100], int tamanho);

int main(void) {
    int escolha;
    cria_lista();
    for (;;) {
        escolha = menu();
        switch (escolha) {
            case 1: insere();
                break;
            case 2: apaga();
                break;
            case 3: imprime();
                break;
            case 4: exit(0);
                break;
        }
    }
}

void cria_lista(void) {
    int i;
    for (i = 0; i < MAX; i++)
        info[i].nome[0] = '\0';
}

int menu(void) {
    int c = 0;
    do {
        printf("-- MENU:\n");
        printf("\t 1. Inserir um nome\n");
        printf("\t 2. Excluir um nome\n");
        printf("\t 3. Listar o arquivo\n");
        printf("\t 4. Sair\n");
        printf("-- Digite sua escolha: ");
        scanf("%d", &c);
    } while (c <= 0 || c > 4);
    getchar();
    return c;
}

void ler_string(char palavra[100], int tamanho) {
    int i = 0;
    char c;
    c = getchar();
    while ((c != '\n') && (i < tamanho)) {
        palavra[i++] = c;
        c = getchar();
    }
    palavra[i] = '\0';
    if (c != '\n') {
        c = getchar();
        while ((c != '\n') && (c != EOF)) {
            c = getchar();
        }
    }
}

void insere(void) {
    int posicao;
    posicao = livre();
    if (posicao == -1) {
        printf("\nEstrutura Cheia!!");
        return;
    }
    printf("-- Registro %d:\n", posicao);
    printf("\t Nome: ");
    ler_string(info[posicao].nome, 30);
    printf("\t Rua: ");
    ler_string(info[posicao].rua, 40);
    printf("\t Cidade: ");
    ler_string(info[posicao].cidade, 20);
    printf("\t Estado: ");
    ler_string(info[posicao].estado, 2);
    printf("\t CEP: ");
    scanf("%lu", &info[posicao].cep);
}

int livre(void) {
    int i;
    for (i = 0; info[i].nome[0] && i < MAX; i++);
    if (i == MAX) return -1;
    return i;
}

/*
 * chama a funçao busca_nome que retorna o indice
 * para a variavel posicão que devera ser apagado
 * se nenhum nome igual for encontrado retorna -1
 */
void apaga(void) {
    int posicao;
    posicao = busca_nome();
    if (posicao >= 0 && posicao < MAX)
        info[posicao].nome[0] = '\0';
}

void imprime(void) {
    int i;
    for (i = 0; i < MAX; i++)
        if (info[i].nome[0] != '\0') {
            printf("-- Registro %d:\n", i);
            printf("\t Nome: %s", info[i].nome);
            printf("\t Rua: %s", info[i].rua);
            printf("\t Cidade: %s", info[i].cidade);
            printf("\t Estado: %s\n", info[i].estado);
            printf("\t CEP: %lu\n", info[i].cep);
        }
}

/*
 * função que faz a busca de um nome no programa
 * le o nome que está localizado na struct busca.nome
 * e retorna o indice se encontrar nome igual ou
 * -1 se não encontrar nome como aquele
 */
int busca_nome(void) {
    int i;
    ler_string(busca.nome, 30);
    for (i = 0; i < MAX; i++) {
        if (info[i].nome == busca.nome){
            return i;
        }
    }
    return -1;
}

Answer 1

Your problem is in the comparison:

    if (info[i].nome == busca.nome){
        return i;
    }

In reality it is comparing the pointers and not the contents of the vectors, hence the fact that even strings were false in comparison.

Use before strcmp to make the comparison:

#include <string.h> 

...

if (strcmp(info[i].nome, busca.nome) == 0){
    return i;
}

---

Extra

(Note: prefer to use strcmp but from the point of view of learning):

You can implement your own comparison using the following code:

int index = 0;
// Enquanto os caracteres forem iguais, corre os vectores.
while( a[index] == b[index] )
{
    // Verifica se alguma das strings terminou.
    // Se sim, sai do loop. Caso contrario, analisa o próximo carácter.
    if( a[index] == '\0' || b[index] == '\0' )
        break;
    ++index;
}

// Se ambos terminaram, as strings contidas são iguais.
if( a[index] == '\0' && b[index] == '\0' )
    return 0;
else
    return -1;