По принцип е супер. :) Ето малко предложения от мен:

1. Когато имаш подобен статичен множител/делител/константа в случая делтата между гравитацията на земята и на луната, не го ползвай директно като число в сметките си. Представи си, че по-натам имаш да сметнеш още 8 неща по това изменение (17%), и на всякъде ще трябва да пишеш *17.0/100. Пък и може да е нещо, което е според програмата ти - например си декларирал, че всички потребители получават 10% бонус към нещо си. Но след 5 дни решаваш, че ще са 20% - ще трябва да обходиш целия код и да напишеш 20% вместо 10%. В този случай бих ти предложил да изнесеш в класа константа (private static ако е нужно) const double moonGravityDelta = 0.17; Или в мейн метода си double moonGravityDelta = 0.17; Пък мисля и, че 0,17 като множите/делител е по-четимо от *17.0/100.

2. Другото, което е, то е по-скоро мое лично мнение, ама смятам, че текстовете за пред юзъра трябва да са на английски. Почти не съм виждал програма в съвремието, която да е таргетирана само към локалната езикова група. Т.е. програма само на български, програма само на румънски и т.н. Напиши текстовете на английски, а пък можеш чисто за интереса, от сега да се разровиш как би могъл рънтайм да преведеш текст според локейла на юзъра или да му изведеш правилния превод според въведеш от него език. Тук може би ще ти излязат варианти с конфизи, бази данни, етс. Честно да си призная не знам в .NET world-a как е най-правилно.

3. Ако не ти е нужно някъде другаде теглото на точно този потребител на луната, можеш сметката да я извършиш направо в принта в конзолата: Console.WriteLine("Your weight on the moon will be: " + weight * moonGravityDelta);

Единствената ми препоръка е да кръщаваш променливите camelCase (да започват с малка буква). Иначе решението можеш да го сбиеш  и в по-малко редове, за да бъде кода по четим, например:

double weight = double.Parse(Console.ReadLine());

double weightOnMoon = weight * 17.0 / 100;

Console.WriteLine("Your weight on moon is: " + weightOnMoon);

Успех :)

Можеш да го направиш от един floating point тип след което направо да го уможиш по 0.17f.

Този string на мен ми се струва излишен.

using System;

class GravitationMoon
{
       static void Main()
       {
                Console.WriteLine("Разбери колко е телесното ти тегло на Луната");
                Console.WriteLine("Въведи Теглото си :");

                float weight = float.Parse(Console.ReadLine());
                float moonWeight = weight * 0.17f;
                Console.WriteLine("Теглото на луната ви ще е {0} кг",moonWeight);

       }

}

Най - важното, което първо трябва да ти кажа е да не започваш променлива с главна буква. WorldWeight - може да е име на клас, на метод така наречения Pascal Case, ако го пиша правилно - за променлива Camel Case.Разликата е, че първия започва с главна буква и всяка съдържаща се следваща дума е отново с главна - както в твоя случай, втория започва с малка буква и всяка следваща дума е с главна.В случая worldWeight. Другото, което мога да ти препоръчам е да не отделяш (в случая не е нужно) входа.Какво имам предвид string weight и double worldWeight.

double worldWeight = double.Parse(Console.Readline()); ще свърши същата работа, не че не е вярно в твоя случай, но пестиш код и ако сега имаш 1 променлива за вход, какво ще стане ако имаш 100.По - удачно е. Същото е и при последния ред, за да спестиш от писане:

Console.WriteLine("Теглото на луната ви ще бъде:" + worldWeight * 17 / 100); и махаш предния ред.

Не съм убеден, че последното е по-добрата практика, просто ти го давам за пример. И една последна препоръка, разделяй кода и го прави четлив:

WorldWeigth = WorldWeigth*17.0/100;

WorldWeigth = WorldWeigth * 17.0 / 100; - по - добре

WorldWeigth = WorldWeigth * (17.0 / 100); - още по ясно и четливо.

worldWeigth = worldWeigth * (17 / 100); - с правилното изписване ;)

Дано съм бил полезен. Успех!

Пичове, до колкото съм наясно, PascalCase е вариация на CamelCase-а. Т.е. да кажеш кръщавай си променливите CamelCase не винаги означава, че WorldWeigth трябва да е worldWeigth.

http://en.wikipedia.org/wiki/CamelCase

CamelCase (camel case) or medial capitals is the practice of writing compound words or phrases such that each word or abbreviation begins with a capital letter. Camel case may start with a capital or, especially in programming languages, with a lowercase letter

За CamelCase е еднакво валидно да се пише с главна или малка първа буква.

P.S.: Странно ми е, че в C# конвенцията за писане на методи е да започват с главна буква. :)

Благодаря ви за мненията.

A за четливостта, много вече ми направиха забележка явно ще трябва да се фокусирам върху това .

Ползвам кирилица защото ми изглежда по прилежно някак си на мен знам че не е правилно . 

За напред ще пробвам да не ползвам.

Ще използвам темата на колегата. Това е моето решение на задачата, но имам и един въпрос. Ще се радвам ако ми отговорите. 

В условието е дадено също да провериме килограми като " 74.5кг" , "53.7кг", а в случая когато въведа такова число ми излиза грешка. Програма работи само когато въведа цяло число като първото 86кг.  Та какво би трябвало да променя за да може да ми изчислява и другите килограми. Мерси. 

using System;

class GravitationOnTheMoon
{
static void Main()
{
Console.Write("Write the weight of the man here : ");
double weight = double.Parse(Console.ReadLine());

double moonWeight = (weight * 0.17);

Console.WriteLine("The weight on the moon it will be : {0} kg", moonWeight);
}
}

Ето и моето решение :)

using System;

class GravitationontheMoon
{
static void Main()

{
Console.Write("Въведете вашето тегло: ");
double weight = double.Parse(Console.ReadLine());
double moonW = weight * 17;
double mw = moonW / 100;
Console.WriteLine("Вашето тегло на луната е: {0}", mw);
}
}

Елате в групата ни за домашни в FB: "Software University - Homeworks"

Привет колега!

Тъй като един нормален човек не може да тежи повече от 200кг.  да речем, както и след десетичната запетая може да има най-много 3 числа (ако използваме обикновен кантар), можем спокойно да използваме float като тип на променливата ни

Друго, което мисля, че може да се избегне TryParse, тъй като входа по условие винаги ще е верен. А ако тръгнем да правим проверки, можем да изпишем сумати редове код.

Също така както се вижда входа е с десетична точка, а не запетая за това трябва да добавим следния ред.

System.Threading.Thread.CurrentThread.CurrentCulture = new System.Globalization.CultureInfo("en-US");

Аз бих използвал следния начин за да решим задачата:

System.Threading.Thread.CurrentThread.CurrentCulture = new System.Globalization.CultureInfo("en-US");

float weight = float.Parse(Console.ReadLine());

float moonWeight = weight * 0.17;

Console.WriteLine("Your weight on the Moon will be {0}.", moonWeight);

Дано съм помогнал