Why is the last letter of my code replaced by a question mark "?"?

3

The teacher asked for semester work a code in C # that encrypts a text from a file.

The code is still not polished or anything, I'm only doing the brute then I'll add more things, but my problem is that every time the last letter of my text comes out ? .

For example:

  

Good evening > encrypt > decrypt > Good night?

There are two codes one to encrypt and the other to decrypt:

Encrypt:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.IO;

namespace Cript
{
class Program
{
    static void Main(string[] args)
    {
        //declaração das variáveis
        string sDrive, sArquivo = "", sPalavra, sTexto = "",sOpcao;

        //método write escreve na tela do usuario
        Console.WriteLine("|---------------------------------------------------|");
        Console.WriteLine("| Gostaria de Criptografar Uma Mensagem ? (Sim/Não) |");
        Console.WriteLine("|---------------------------------------------------|");
        Console.Write("--->>>>>>>>");

        //Aqui é feito uma conversão, pois a opcao é inicialmente uma string
        sOpcao = Console.ReadLine();

        //PEDE VALORES DO ARQUIVO A SER CRIADO
        Console.Write("Digite em Qual driver o arquivo sera salvo: ");
        sDrive = Console.ReadLine();
        Console.Write("Digite o Nome do Arquivo(Sem a Extensão .txt): ");
        sArquivo = Console.ReadLine();

        //depois da conversão o switch verifica a opcao digitada
        if ((sOpcao == "s") || (sOpcao == "sim") || (sOpcao == "S" ) || (sOpcao == "SIM") || (sOpcao == "Sim"))
        {
            Console.Write("Entre com a mensagem para ser criptografada: ");

            //sPalavra é a variavel que o usuario vai digitar.                    
            sPalavra = Console.ReadLine();

            int[] iAscii = new int[sPalavra.Length];
            char[] cChar = new char[sPalavra.Length];

            //enquanto a palavra for menor que i
            for (int i = 0; i < sPalavra.Length; i++)
            {
                //Transforma o texto no codigo ASCII dele e retira 5 posições na tabela ASCII (para criptografar) 
                iAscii[i] = ((int)sPalavra[i]);
                iAscii[i] = iAscii[i] + 5;
                cChar[i] = ((char)iAscii[i]);
                sTexto = sTexto + cChar[i];
            }

            // Cria e Grava o Arquivo
            sArquivo = sDrive + ":" + sArquivo + ".txt";
            using (StreamWriter arquivo = File.CreateText(sArquivo))
            {
                arquivo.WriteLine(sTexto);
                arquivo.Close();
            }

            // Mostra na Tela o Arquivo Criado
            Console.WriteLine("Arquivo Criado Como: " + sArquivo);
            Console.ReadKey();
        }
    }
}
}

Decrypt:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.IO;

namespace DeCrypt
{
    class Program
    {
        static void Main(string[] args)
        {
            //declaração das variáveis
            string sDrive, sArquivo, sPalavra = "", sTexto = "",sOpcao;


            //método write escreve na tela do prompt do usuario
            Console.WriteLine("|---------------------------------------------------|");
            Console.WriteLine("| Gostaria de Descriptografar Uma Mensagem ? (S/N)  |");
            Console.WriteLine("|---------------------------------------------------|");
            Console.WriteLine("--->>>>>> ");

            //Aqui é feito uma conversão, pois o opcao é inicialmente uma string
            sOpcao = Console.ReadLine();

            //Aqui, A pessoa digitara o nome e o drive do arquivo
            Console.WriteLine("Letra do Drive do Arquivo Instalado: ");
            sDrive = Console.ReadLine();
            Console.WriteLine("Digite o Nome do Arquivo(Sem Extensão): ");
            sArquivo = Console.ReadLine();
            sArquivo = sDrive + ":" + sArquivo + ".txt";

            if (!File.Exists(sArquivo))
            {
                Console.WriteLine("Arquivo " + sArquivo + " não Existe." );
            }
            else
            {
                Console.WriteLine("O ARQUIVO DESCRIPTOGRAFADO FICOU ASSIM: ");

                // ABRE ARQUIVO TEXTO
                sPalavra = File.ReadAllText(sArquivo);
                //depois da conversão o switch verifica a opcao digitada
                switch (sOpcao)
                {
                    //caso a opcao escolhida for 2
                    case "s":
                        int[] iAscii = new int[sPalavra.Length];
                        char[] cChar = new char[sPalavra.Length];

                        //enquanto a palavra for menor que i
                        for (int i = 0; i <= sPalavra.Length; i++)
                        {
                            //Transforma o texto no codigo ASCII dele e Acrescenta 5 posições na tabela ASCII (para criptografar) 
                            iAscii[i] = ((int)sPalavra[i]);
                            iAscii[i] = iAscii[i] - 5;
                            cChar[i] = ((char)iAscii[i]);
                            sTexto = sTexto + cChar[i];
                        }
                        Console.WriteLine(sTexto);
                        Console.ReadKey();
                        break;
                }
            }
        }
    }
}
    
asked by anonymous 02.11.2016 / 15:17

2 answers

5

Your cryptography routine is working, as this example in dotNETFiddle can demonstrate:

link

using System;

public class Program
{
    public static void Main()
    {
        string sPalavra, sTexto = "";

        Console.WriteLine("Criptografando...");

        sPalavra = "Boa noite";

        int[] iAscii = new int[sPalavra.Length];
        char[] cChar = new char[sPalavra.Length];

        for (int i = 0; i < sPalavra.Length; i++)
        {
            iAscii[i] = ((int)sPalavra[i]);
            iAscii[i] = iAscii[i] + 5;
            cChar[i] = ((char)iAscii[i]);
            sTexto = sTexto + cChar[i];
        }

        Console.WriteLine(sTexto);

        Console.WriteLine("Descriptografando...");

        sPalavra = "Gtf%stnyj";
        sTexto = "";

        iAscii = new int[sPalavra.Length];
        cChar = new char[sPalavra.Length];

        for (int i = 0; i < sPalavra.Length; i++)
        {
            iAscii[i] = ((int)sPalavra[i]);
            iAscii[i] = iAscii[i] - 5;
            cChar[i] = ((char)iAscii[i]);
            sTexto = sTexto + cChar[i];
        }

        Console.WriteLine(sTexto);
    }
}
  

Encrypting ...
  Gtf% stnyj
  Decrypting ...
  Good night.

What happens is that you are using the TextWriter.WriteLine method, which automatically adds the end-of-line marker ( Environment.NewLine ) composed of Carriage Return (ASCII 13) and Line Feed (ASCII 10) characters, thus increasing your string from 9 to 11 characters (% with% + "Boa noite" +% with%).

I can not test now, but I think your file reading routine is including this marker in the 'decryption' cycle - what would be a possible cause of the return with the '?' character.

    
02.11.2016 / 16:56
3

Complementing OnoSendai's analysis:

It is yes including an undue line when writing to text file, after adjusting it worked.

Also remove the "=" from your loop in the decryp method, otherwise it will display the error: System.IndexOutOfRangeException.

for (int i = 0; i <= sPalavra.Length; i++)

Whenever you translate "good night" will it be the same text? Read about asymmetric cryptography to make your code less vulnerable

    
02.11.2016 / 17:22