Enviado em 18/05/2019 - 19:06h
A descrição do formato PPM está em
http://netpbm.sourceforge.net/doc/ppm.html . Existem dois formatos, na verdade, que você pode reconhecer pelas assinaturas "P6" ou "P3", que indicam respectivamente o formato completo ou o formato “simples” (“
plain ”). Em ambos os casos, o cabeçalho do PPM — isto é: tudo o que vem desde a assinatura até imediatamente antes dos componentes RGB de cada pixel — pode conter comentários, que são blocos de texto que não afetam o formato da imagem, mas podem dar informações adicionais sobre ela, tal como o nome do arquivo original, data de criação ou edição, assinatura do programa usado para manipulá-la etc.
O próprio exemplo mostrado no site de arquivo PPM de formato simples (com assinatura “P3”, e que é o mesmo formato que você está usando) inclui um comentário, como você pode ver na transcrição abaixo.
P3
No exemplo acima a segunda linha é um comentário, ocorrendo logo após a assinatura e antes do tamanho da imagem e da escala usada para os componentes RGB. O conteúdo do comentário parece ser um nome de arquivo.
Pelo que eu vi no seu programa, ele não conseguiria tratar o arquivo de exemplo mostrado acima, pois ele não dá margem para o aparecimento de comentários no cabeçalho.
Na verdade, todas as operações de leitura do seu programa estão sujeitas a problemas. Um bom programa deve procurar se proteger contra situações indesejáveis durante a leitura de dados, mas seu programa não faz isso.
Mesmo que você tivesse considerado que pode haver variações no cabeçalho devidas à possível presença de comentários em qualquer posição entre a assinatura e o fim do cabeçalho, ainda podem ocorrer situações de arquivos com formato diferente (por exemplo, um PPM com assinatura “P6” em lugar de “P3”), arquivos corrompidos, arquivos truncados etc. Você não deve assumir que as operações de leitura serão bem sucedidas, porque uma eventual falha pode deixar com valor indevido uma variável que será usada num ponto mais a diante no programa, e esse valor indevido pode provocar efeitos deletérios.
Aqui, eu vou analisar partes do seu código, mostrando como robustecê-lo contra erros. Mas antes, quero lhe dar uma orientação geral: sempre que você usar uma função da família de
scanf () (que inclui a própria, mas também
fscanf (),
sscanf (),
vscanf () etc.), verifique o valor retornado pela função, comparando-o com a quantidade de conversões contidas na
string de formatação. Se o valor for retornado for menor, eis um sinal de alerta de que chegaram dados que não atendiam o formato esperado, e isso quase sempre indica que você não deve usar as variáveis que deveriam ter sido modificadas em decorrência das atribuições, mas provavelmente não o foram ou o foram de modo incompleto.
// Leitura da imagem
Na invocação acima, além do problema geral de não confirmar se a conversão foi bem-sucedida ou não, você tem três outros problemas, a saber:
• Você não diz qual a quantidade máxima de caracteres que podem ser salvos em
nomeImagem , de modo que se o usuário digitar mais do 49 caracteres (visto que o
array foi declarado com 50 caracteres, e o último caráter deveria ser um
byte nulo), o programa pode acabar invadindo áreas de memória alocadas para outra variáveis, estruturas de controle usadas durante a execução do programa, ou mesmo memória que não foi de modo nenhum reservada para o programa, e os efeitos disso não-raramente são desastrosos.
• A conversão
"%s" descarta espaços no começo da digitação e para quando encontra o primeiro caráter de espaço após o início da conversão, o que impede o uso de nomes de arquivos que porventura contenham espaços.
• Admitindo que o usuário encerrou a digitação do nome do arquivo apertando a tecla
Enter , o caráter correspondente a essa tecla é interpretado na categoria de espaço em branco, interrompendo a conversão que já tenha sido iniciada, mas
permanecendo no buffer de leitura até a próxima operação de leitura. Dependendo de qual operação seja essa, a presença desse (e de outros) espaços em branco pode produzir efeitos indesejados, se você não tomar os devidos cuidados.
Uma forma melhor de fazer essa mesma leitura, tratando o problema geral e os três específicos da
string de formatação contendo apenas a conversão
"%s" seria a seguinte.
int a=0, b=0; // Contadores de caracteres consumidos durante a leitura da linha (ver abaixo).
Eventualmente, em lugar de apenas indicar erro e sair, você poderia querer um tratamento mais sofisticado, como distinguir entre erro de leitura e erro de digitação ou, neste último caso, permitir que o usuário tente novamente. Tais diferentes casos podem ser percebidos por variações no valor de retorno de
scanf () (
EOF indicando erro de leitura, valores não-negativos mas menores do que o número de conversões indicando sucesso parcial, e valor positivo igual ao número de conversões indicando sucesso total), e mais os valores de ‘a’ e ‘b’ (ou outras, dependendo da complexidade da
string de formatação) como forma de garantir que mesmo eventuais separadores serão devidamente consumidos, sem espirrar para operações de leitura futuras.
Se você achar a
string de formatação acima cabalística demais, saiba, em primeiro lugar, que eu concordo, mas isso decorre do fato de que
scanf () é possivelmente a função mais complexa da biblioteca padrão do C, ao mesmo tempo muito poderosa e difícil de aprender a usar devidamente. Nunca é demais indicar que você leia bem sua documentação.
Mas, além da baixa legibilidade, a
string de formatação acima tem outro problema, que é o de conter um número fixo (49), que depende do tamanho de um
array declarado em outra parte do programa (que eu nem transcrevi nesta resposta). E se eu quisesse mudar o tamanho usado na declaração do
array ? Teria de varrer todo o programa encontrando associações indiretas como essa. E se eu quisesse ler um arquivo com nome maior do que 49 caracteres? Teria de localizar não apenas a declaração, para alterar ali o tamanho para alguma coisa maior (lembrando do elemento a mais para o
byte nulo), mas também todas as outras operações envolvendo o
array em que o tamanho possa fazer diferença.
Por essas e outras complicações, muita gente (inclusive eu) prefere, na hora de ler
strings , usar outras funções em lugar de
scanf (), tais como
fgets () ou
getline (). Mas, como não existe almoço grátis, essas funções também têm sua própria dose de idiossincrasias, especialmente se você quiser fazer uma detecção de erros detalhada.
imagemAtual = fopen (nomeImagem, "r");
Aqui há alguns problemas. Os mais óbvios são que você não testou o valor de retorno de
fscanf () e não limitou o tamanho da
string na conversão
"%s" , de modo que você está vulnerável a corrupção de memória e falha de segmentação se simplesmente mandar abrir um arquivo cuja primeira palavra tenha 3 ou mais caracteres que não sejam considerados espaços.
Ainda há outras falhas associadas ao uso de
"%s" . O fato de tal conversão ignorar espaços que apareçam antes de a conversão começar permite que um arquivo que não comece realmente com a assinatura “P3” nas duas primeiras posições do arquivo ainda fosse interpretado como válido (por exemplo, se esse arquivo começar com uma quantidade arbitrária de caracteres considerados como espaços, e o primeiro texto diferente de espaços só ocorra bem lá para frente, como na décima, milésima ou mesmo bilionésima posição do arquivo).
Para além disso, há dois outros problemas relacionados à sinalização de erro, não graves, mas que vale a pena apontar e aprender. Um deles é jogar a mensagem de erros na saída padrão, através de
printf (). Como você deve saber, o C possui um canal separado para indicações de erro, chamado de “saída padrão de erros”, ou
stderr , e essa separação se dá para que mensagens decorrentes da execução normal e esperada, quando tudo vai bem, não se confundam com mensagens decorrentes de operações que falharam, inclusive permitindo redirecionamentos para locais diferentes. Por isso, convém que você façam como o resto dos programadores do mundo, e jogue suas mensagens de erro no canal destinado para tais mensagens, como eu fiz no código que postei acima. O outro problema é você encerrar uma execução que deu errado com o código de retorno 0, pois esse valor é convencionalmente usado para indica execução bem-sucedida (note que eu, no código que postei acima, chamo “
exit(1) ” no caso de erro, para que o programa use esse
1 como código indicador de falha).
Considerando tudo isso, eis uma forma mais ou menos óbvia de corrigir os problemas acima.
int a=0, b=0, c=0;
Mas há um jeito mais simples de fazer, que dispensa o
array codigo e o uso de
strcmp (): é usar
fscanf () para procurar por texto arbitrário, e não por conversões ou coisa parecida.
int a=0;
Essas leituras (e as que vem depois, que eu não mostrei) também têm de ser tratadas com cuidado, testando-se o valor de retorno de cada uma, para se ter certeza de que o arquivo não está corrompido e que os valores lidos fazem sentido. Mas, mais do que isso, antes e depois de cada uma das leituras acima pode haver um ou mais cabeçalhos, e você tem de incluir código que saiba se desvencilhar deles, já que não agregam informação visual ao PPM (ou que os retenha, se você preferir).
Eu fico muito tentado a lhe mostrar o código para descartar os cabeçalhos, mas penso que será um bom exercício para você se conseguir fazê-lo por conta própria.
Uma dica, porém, que lhe pode ser útil é saber usar a função
ungetc ().
... “Principium sapientiae timor Domini, et scientia sanctorum prudentia.” (Proverbia 9:10)
