#Verification repetition

12 messages · Page 1 of 1 (latest)

low iron
#

Is there a smarther way to do this verification?

        if (nElemenentosEscritos != 1) { erroEscrita = true; }

        nElemenentosEscritos = fwrite(&nPontosIpAvariados, sizeof(int), 1, ficheiro);
        if (nElemenentosEscritos != 1) { erroEscrita = true; }

        nElemenentosEscritos = fwrite(&nAvariasRegistadas, sizeof(int), 1, ficheiro);
        if (nElemenentosEscritos != 1) { erroEscrita = true; }

        nElemenentosEscritos = fwrite(&nIntervencoesRealizadas, sizeof(int), 1, ficheiro);
        if (nElemenentosEscritos != 1) { erroEscrita = true; }

        nElemenentosEscritos = fwrite(avaria, sizeof(tipoAvaria), nAvariasRegistadas, ficheiro);
        if (nElemenentosEscritos != nAvariasRegistadas) { erroEscrita = true; }

        nElemenentosEscritos = fwrite(ip, sizeof(tipoIp), nPontosIpExistentes, ficheiro);
        if (nElemenentosEscritos != nPontosIpExistentes) { erroEscrita = true; }

        nElemenentosEscritos = fwrite(intervencao, sizeof(tipoIntervencao), nIntervencoesRealizadas, ficheiro);
        if (nElemenentosEscritos != nIntervencoesRealizadas) { erroEscrita = true; }

        
        erro = fclose(ficheiro);

        if (erro != 0)
        {
            mostrarMensagem("Erro no fecho ficheiro dadosPrograma.dat");
        }
        else
        {
            if (erroEscrita == true)
            {
                mostrarMensagem("Erro ao escrever os dados no ficheiro dadosPrograma.dat");
            }
            else
            {
                mostrarMensagem("Dados gravados com sucesso");
            }
        }````
cunning gyroBOT
#

When your question is answered use !solved to mark the question as resolved.

Remember to ask specific questions, provide necessary details, and reduce your question to its simplest form. For tips on how to ask a good question run !howto ask.

low iron
#

Sorry for the non english var names

keen isle
cunning gyroBOT
#

Is there a smarther way to do this verification?

nElemenentosEscritos = fwrite(&nPontosIpExistentes, sizeof(int), 1, ficheiro);
if (nElemenentosEscritos != 1) {
  erroEscrita = true;
}

nElemenentosEscritos = fwrite(&nPontosIpAvariados, sizeof(int), 1, ficheiro);
if (nElemenentosEscritos != 1) {
  erroEscrita = true;
}

nElemenentosEscritos = fwrite(&nAvariasRegistadas, sizeof(int), 1, ficheiro);
if (nElemenentosEscritos != 1) {
  erroEscrita = true;
}

nElemenentosEscritos =
    fwrite(&nIntervencoesRealizadas, sizeof(int), 1, ficheiro);
if (nElemenentosEscritos != 1) {
  erroEscrita = true;
}

nElemenentosEscritos =
    fwrite(avaria, sizeof(tipoAvaria), nAvariasRegistadas, ficheiro);
if (nElemenentosEscritos != nAvariasRegistadas) {
  erroEscrita = true;
}

nElemenentosEscritos =
    fwrite(ip, sizeof(tipoIp), nPontosIpExistentes, ficheiro);
if (nElemenentosEscritos != nPontosIpExistentes) {
  erroEscrita = true;
}

nElemenentosEscritos = fwrite(intervencao,
                              sizeof(tipoIntervencao),
                              nIntervencoesRealizadas,
                              ficheiro);
if (nElemenentosEscritos != nIntervencoesRealizadas) {
  erroEscrita = true;
}

erro = fclose(ficheiro);

if (erro != 0) {
  mostrarMensagem("Erro no fecho ficheiro dadosPrograma.dat");
} else {
  if (erroEscrita == true) {
    mostrarMensagem("Erro ao escrever os dados no ficheiro dadosPrograma.dat");
  } else {
    mostrarMensagem("Dados gravados com sucesso");
  }
}
Loo
Formatted by Zelis 🔱
keen isle
#

This code is not unreasonable

low iron
#

@keen isle Wdym sry

keen isle
#

The code is fine

#

You can save some lines of code, you could use macros, but there's nothing wrong with it as is

low iron
cunning gyroBOT
#

@low iron Has your question been resolved? If so, run !solved :)

low iron
#

!solved