C# Intro and Basic Syntax - Exercises - Софтуерен университет

Plamena7777 0 Точки

C# Intro and Basic Syntax - Exercises

Здравейте,

Извинявам се за глупавия спам, но ... би ли могъл някой дам и подскаже.. не мога да си открия грешката, а с няколко примера в средата за разработка, кодът работи. :) Благодаря!

https://dotnetfiddle.net/

using System;

public class Program
{
   public static void Main()
   {
       string name = Console.ReadLine();

       int CurrentHealth = int.Parse(Console.ReadLine());

       int MaximumHealth = int.Parse(Console.ReadLine());

       int CurrentEnergy = int.Parse(Console.ReadLine());

       int MaximumEnergy = int.Parse(Console.ReadLine());


       Console.WriteLine("Name: " + name);

       Console.Write("Health: |");

       for(int i = 0; i < 10; i ++)

       {
           if(CurrentHealth - i > 0)

           {
               Console.Write("|");

           }
           else

           {
               Console.Write(".");
               }
            }
      Console.WriteLine("|");
            {
                 Console.Write("Energy: |");

       for(int i = 0; i < 10; i ++)

       {
           if(CurrentEnergy - i > 0)

           {
             Console.Write("|");
           }
           elsе
           {
               Console.Write(".");
           }

       }
         Console.WriteLine("|");
       }
   }


   }

Тагове:

16/05/2018 11:59:12 Programming Fundamentals

Vesso1980 486 Точки

Може ли да дадеш линк към условието на задачата ?

16/05/2018 12:02:51

JuganD 15 Точки

Здравей, първото което забелязах е, че и в двата for loop-a си предположила, че максимум кръвта е 10. Което е така само в първия пример, но в следващите примери е различно. Другото е,че тези loop-ове са ненужни в случая, тъй като е доста по-лесно с new string("|",count) (има го като hint под задачката).

16/05/2018 12:07:49

Vesso1980 486 Точки

Аз всъщност я намерих задачата. Проблемът е, че ти въртиш циклите винаги до 10, а максималните стойности могат да са по- големи както са във втория пример. Ето ти едно решение с new string, което е много лесно.

string name = Console.ReadLine();
int health = int.Parse(Console.ReadLine());
int maxHealth = int.Parse(Console.ReadLine());
int energy = int.Parse(Console.ReadLine());
int maxEnergy = int.Parse(Console.ReadLine());

Console.WriteLine($"Name: {name}");
Console.WriteLine($"Health: |{new string('|', health)}{new string('.', maxHealth - health)}|");
Console.WriteLine($"Energy: |{new string('|', energy)}{new string('.', maxEnergy - energy)}|");

16/05/2018 12:11:04

Plamena7777 0 Точки

Благодаря! :)

16/05/2018 22:08:10

Milchev11 1 Точки

Просто върти циклите до MaximumHealth, MaximumEnergy!
Не знам на какъв език си бил в бейсикс, но има много по-лесен начин - с new string.
new string('символ', колко пъти да се повтаря). :)

16/05/2018 12:17:24

VasilKotsev 830 Точки

Поствам едно решение с материал от следваща лекция (масиви). Определено задачата може и доста по-просто да се реши, но според мен това е по-удачният подход ако се желае запзаване на данните и тяхното манипулиране вместо да се печатат само new string-ове.

namespace BlankProject
{
using System;
using System.Collections.Generic;
using System.Linq;
using System.Numerics;
using System.Globalization;
    class Program
    {
        static void Main()
        {
            string name = Console.ReadLine();
            int currentHealth = int.Parse(Console.ReadLine());
            int maxHealth = int.Parse(Console.ReadLine());
            int currentEnergy = int.Parse(Console.ReadLine());
            int maxEnergy = int.Parse(Console.ReadLine());

            char[] healthBar = new char[maxHealth];
            for (int i = 0; i < healthBar.Length; i++)
            {
                bool checkForHp = currentHealth > i;
                if(checkForHp)
                {
                    healthBar[i] = '|';
                }

                else
                {
                    healthBar[i] = '.';
                }
            }

            char[] energyBar = new char[maxEnergy];
            for (int i = 0; i < energyBar.Length; i++)
            {
                bool checkForEnergy = currentEnergy > i;
                if (checkForEnergy)
                {
                    energyBar[i] = '|';
                }

                else
                {
                    energyBar[i] = '.';
                }
            }

            string energyBarStr = $"Energy: |{string.Join("", energyBar)}|";
            string healthBarStr = $"Health: |{string.Join("", healthBar)}|";

            Console.WriteLine($"Name: {name}");
            Console.WriteLine(healthBarStr);
            Console.WriteLine(energyBarStr);

        }
    }
}

16/05/2018 12:43:39 16/05/2018 12:45:20

Waikiki 0 Точки

using System;

namespace CharacterSet
{
    class CharacterSet
    {
        static void Main()
        {
            var name = Console.ReadLine();
            var currentHealth = int.Parse(Console.ReadLine());
            var maxHealth = int.Parse(Console.ReadLine());
            var currentEnergy = int.Parse(Console.ReadLine());
            var maxEnergy = int.Parse(Console.ReadLine());

            Console.WriteLine($"Name: {name}");
            Console.WriteLine("Health: |{0}{1}|", new string('|', currentHealth), new string('.', maxHealth - currentHealth));
            Console.WriteLine("Energy: |{0}{1}|", new string('|', currentEnergy), new string('.', maxEnergy - currentEnergy));
        }
    }
}

Ето едно друго решение, доста по-кратичко.

16/05/2018 13:08:38