Feda /dev/ - No Javascript allowed

#59522 creo que el problema que te está dando es porque no hay return si el bucle nunca empieza por no cumplirse la condición inicial. Y luego tienes otro posible fallo relacionado.

Edit: no había visto lo de la conversación con el bot. Cuidado con tomarte todo lo que dice en serio. Está bien para cosas de detalles técnicos, pero para la lógica de lo que haces puede desvariar muchísimo

#59522 tienes el return false mal. debes devolver despues del bucle for, lo que ahora estas haciendo es comprovar solo el primero, si lo he han votado true, sino false. y tu quieres buscar si le han votado o no.

non-void function does not return a value in all control paths

obvio, si la lista es vacia, candidate_count = 0, nunca entra en el bucle for y como ademas has puesto el return false mal, no devuelve nada tu funcion

bool vote(string name) {
    for (int i = 0; i < candidate_count; i++) {
        if (strcmp(name, candidates[i].name) == 0) {
            candidates[i].votes++;
            printf("Candidate %s has now %i votes\n", candidates[i].name, candidates[i].votes);
            return true;
        }
    }
    return false;
}

#59528 da igual que el flujo de tu código obligue a que la lista de candidatos no esté vacía, eso la función vote no lo sabe ni tiene por qué saberlo, lo sabes tú que estás haciendo el código para un caso de uso concreto. La función no sabe que tú no vas a hacer otra llamada en cualquier otra parte o momento. Podrías pasar como parámetro la lista de candidatos y evaluar que no fuese nula o este vacía dentro de la función.

#59530 No tiene nada que ver con que el compilador lo tenga claro, la función esta mal escrita.

El compilador no sabe si quieres devolver false o es que el programador es imbecil y ha escrito mal el codigo, como es tu caso que has puesto el return false dentro del for loop.

Porque debería saber el compilador que esa funcion tiene que devolver false y no devolver true?

el usage no se entiende, hay un standard y una guia de estilo para las CLI, usalos o dile a chatgpt que te lo genere.

Usage: plurality [candidate ...]

deberia ser:

Usage: plurality candidate [candidate...]

xq minimo 1 obligatrio

#59532 #59531 Anotado, gracias.

#include <cs50.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <stdint.h>

#define MAX_CANDIDATES 9

struct candidate {
    string name;
    uint8_t votes;
};

bool vote(const size_t candidates_len, struct candidate candidates[], const string name);
void print_winner(const size_t candidates_len, const struct candidate candidates[]);

int main(const int argc, const string argv[]) {
    if (argc < 2) {
        printf("Usage: plurality candidate [candidate ...]\n");
        return EXIT_FAILURE;
    }

if (argc > MAX_CANDIDATES + 1) {
    printf("Maximum number of candidates is %i\n", MAX_CANDIDATES);
    return EXIT_FAILURE;
}

const size_t number_of_candidates = argc - 1;
struct candidate candidates[number_of_candidates];
for (size_t i = 0; i < number_of_candidates; i++) {
    candidates[i].name = argv[i + 1];
    candidates[i].votes = 0;


    for (size_t j = 0; j < i; j++) {
        if (strcmp(candidates[i].name, candidates[j].name) == 0) {
            printf("Error: Duplicate candidate name '%s'\n", candidates[i].name);
            return EXIT_FAILURE;
        }
    }
}

const int number_of_votes = get_int("Number of voters: ");
if (number_of_votes < 1) {
    printf("Invalid number of voters\n");
    return EXIT_FAILURE;
}

for (size_t i = 0; i < number_of_votes; i++) {
    const string name = get_string("Vote to candidate: ");

    if (!vote(number_of_candidates, candidates, name)) {
        printf("Invalid vote.\n");
        return EXIT_FAILURE;
    }
}

print_winner(number_of_candidates, candidates);
return EXIT_SUCCESS;
}

bool vote(const size_t candidates_len, struct candidate candidates[], const string name) {
    for (size_t i = 0; i < candidates_len; i++) {
        if (strcmp(name, candidates[i].name) == 0) {
            candidates[i].votes++;
            return true;
        }
    }
    return false;
}

void print_winner(const size_t candidates_len, const struct candidate candidates[]) {
    uint8_t max_votes = 0;

for (size_t i = 0; i < candidates_len; i++) {
    if (candidates[i].votes > max_votes) {
        max_votes = candidates[i].votes;
    }
}

printf("Winner(s):\n");
for (size_t i = 0; i < candidates_len; i++) {
    if (candidates[i].votes == max_votes) {
        printf("%s with %i votes\n", candidates[i].name, candidates[i].votes);
    }
}
}

• candidates inicializado a MAX en lugar de number_of_candidates, mal

// Array of candidates
candidate candidates[MAX];

// Number of candidates
int candidate_count;

• ambas variables globales en lugar de locales
• el return 1 y return 2? xq no siempre return 1??? mejor usar la variable de alias
• candidatos repetidos
• el caso de empate estoy seguro q te lo habrias dejado, mal también
• los loops no se hacen sobre int, se hacen sobre size_t porque asi puedes ejecutarse en cualqueir arquitectura 32bit o 64bit o XXXbit
• me faltan consts

mejor:

struct {
  string name;
  int votes;
} candidate;

VS

typedef struct
{
    string name;
    int votes;
} candidate;

• los votos siempre son positivos, no hace falta usar int que acepta negativos, mejor usar unsigned int, uint8, uint16 etc

Nota final: 2/10 suspendido

Edit: Nota final: 1/10, que seguro que te dejarias pasar la len del array por parametros.

#59536

Loguean el error todos. No necesariamente van a loguear el mismo error. C sacará el stacktrace del error mientras que A igual simplemente loguean que C le ha tirado un 500.

Piensa que es posible que A no conecte solo con C, si no con más servicios

Metes algún sistema de tracing (Zipkin, tempo, etc) y te hace el trabajo solito con su graph de servicios incluidos

#59537 el tracing distribuido es una mala practica
#59538 ahi esta el meme, q nada de lo q te digo te sirve HAHAHAHAHAAHAHAHAHHAHAHAH es como si Einstein da una clase de fiisca en pre-escolar

#59539desu:

ahi esta el meme, q nada de lo q te digo te sirve HAHAHAHAHAAHAHAHAHHAHAHAH es como si Einstein da una clase de fiisca en pre-escolar

El tema es que un buen profesor sabe adaptarse y explicar en función de la pregunta que le hagan y del nivel que tenga el que necesita la respuesta, no se pone a dejar el código perfecto con conceptos de 3er año. Que después te extrañas de que no te aceptaran de profesor en Carnegie, y quizás es que es eso lo que te falló, que vas a demostrar tus máximas capacidades como programador y no como educador. Probablemente necesitasen una persona que mostrase el equilibrio óptimo entre ambas aptitudes. Como ya te dije, no se puede ser bueno en todo.

#59542 Los primeros días llegué a pensar "algún día me dirán que es un bot al que le han puesto de prompt que demuestre máximos conocimientos y soberbia con mínima explicación", y la verdad es que me quedaría con cara de "tiene todo el sentido". Cuesta creer que una persona sea así 24/7 sin que le dé un poquito de vergüenza.

Mi nivel de profesor es Feynman.

Dudar de ello solo es negar la realidad.

Ahora, te quiero enseñar?

Obvio que no.