Line breaking problems and correctly display real value conversion on C#

Question

I made a simple C# code that asks you to enter name, age, weight, city and state, performs normally only that it appears some problems as instead of the weight being 60.5KG gets 605KG and wanted to know how to break the line to print the part filling the form and printing separately.

// Meu Código
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace TerceiroCd
{
    class Program
    {
        static void Main(string[] args)
        {
            String nomeNome, cidadeC, estadoE;
            double peso;
            int idade;

            Console.Write("Digite o seu nome: ");
            nomeNome = Console.ReadLine();
            Console.Write("Digite a sua idade: ");
            idade = int.Parse(Console.ReadLine());
            Console.Write("Digite o seu peso: ");
            peso = double.Parse(Console.ReadLine());

            Console.Write("Digite sua cidade: ");
            cidadeC = Console.ReadLine();
            Console.Write("Digite o seu estado: ");
            estadoE = Console.ReadLine();

            Console.WriteLine($"Seu nome é {nomeNome}");
            Console.WriteLine($"Possui {idade} de idade e pesa {peso}KG");
            Console.WriteLine($"Mora na cidade de {cidadeC} no estado de {estadoE}");

            Console.ReadKey();
        }
    }
}

// A impressão fica assim
Digite seu nome: Alex F.
Digite sua idade: 20
Digite o seu peso: 60.5
Digite sua cidade: Guanambi
Digite o seu estado: Bahia
Seu nome é Alex F.
Possui 20 anos de idade e pesa 605KG // Valor errado
Mora na cidade de Guanambi no estado da Bahia

// Queria que fosse assim:
Digite seu nome: Alex F.
Digite sua idade: 20
Digite o seu peso: 60.5
Digite sua cidade: Guanambi
Digite o seu estado: Bahia

Seu nome é Alex F.
Possui 20 anos de idade e pesa 60.5KG
Mora na cidade de Guanambi no estado da Bahia

Answer 1

Line breaker

Put a WriteLine() Wherever you jump a line, so you have printed nothing but a line break, after all the method calls WriteLine.

It is also possible \n in the text that is the character that jumps line.

Data conversion

As to the Parse() will break your application when you type something wrong. It’s good to read too What is the main difference between int. Parse() and Convert.Toint32()? because the other answer makes the same mistake.

Note that the person needs to type within the format of the current culture that the thread has at the moment, if nothing is specified then it is the one that is configured in the operating system. As it should be in the format pt-BR then you must enter the comma as the decimal separator.

To ensure certain format it is necessary to configure the method of parse in the specific format that accepts or not, then you should use the signature of the most complete method that asks for the style and culture. In the case maybe the style Float should solve the issue and not accept the decimal point (I did a quick test and it worked but would need a more extensive test and better evaluation), it would be something like this:

double.TryParse("60.5", NumberStyles.Float, CultureInfo.CurrentCulture, out var peso)

I put in the Github for future reference.

It was commented to give a Replace(). This is a huge gambit. This should not be done, this is asking for future problems.

First, rare cases that change the data the person typed is a good idea. I’m going to say it again because I know everybody does it: it’s a rare good idea to change something that a person has typed, and yes almost everyone gets it wrong. It is even possible in some cases to change, show the user in one way or another the modified value and let them confirm this data or not, the exact way to do this depends on the context. Want to make have all this work? If that’s the case you can do so using this pattern.

You can do an extra validation and not allow the input of the data in this format, but that’s exactly what the above command does. Then you give the most appropriate treatment for the case, usually the TryParse() is used as a condition of a if, and inside it you can do whatever you want, for example you can give a hint to the user about the error. Unfortunately `Tryparse() does not give more details of which specific error occurred, may be a fault of it, but the way to solve this is to make your own parser, simple or complex to find the error and give a more specific hint. in this example you can verify that you are using a comma culture as decimal separator and show the user that it is using dot. It takes work for a simple case like this I wouldn’t do.

In this case if you change the point by the comma if the person type the point as a thousand separator or for any other crazy reason, if he put random things will give a nice problem.

Do you want a robust system or one that accepts anything and breaks? If you want the second, ok, just tell the user that he has to type right and accept when he type wrong. In a real system in production this is not acceptable. If you want something robust then you have to think of all cases, not just some.

There are other things that can be simplified in the code.

Answer 2

// Meu Código
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace TerceiroCd
{
    class Program
    {
        static void Main(string[] args)
        {
            String nomeNome, cidadeC, estadoE;
            double peso;
            int idade;

            Console.Write("Digite o seu nome: ");
            nomeNome = Console.ReadLine();
            Console.Write("Digite a sua idade: ");
            idade = Convert.ToInt32(Console.ReadLine());
            Console.Write("Digite o seu peso: ");
            peso = Convert.ToDouble(Console.ReadLine());

            Console.Write("Digite sua cidade: ");
            cidadeC = Console.ReadLine();
            Console.Write("Digite o seu estado: ");
            estadoE = Console.ReadLine();

            Console.WriteLine($"\n Seu nome é {nomeNome}. \n Possui {idade} anos de idade e pesa {Convert.ToString(peso).Replace(",", ".")} Kg. \n Mora na cidade de {cidadeC}, estado de(a) {estadoE}.");

            // Use o \n para quebrar a linha dentro de uma string
            // Ao invés de Parse tente o Convert
            // Ao invés de um ponto tente usar uma virgula ao inserir o peso
            // Replace() => Substitui um caracter por outro

            Console.ReadKey();
        }
    }
}