Професионална програма
Loading...
+ Нов въпрос
tanchev avatar tanchev 2 Точки

Methods Lab - 09. Greater of Two Values

1.Greater of Two Values

Create a method GetMax() that returns the greater of two values (the values can be of type int, char or string)

Examples

Input

Output

int

2

16

16

char

a

z

z

string

aaa

bbb

bbb

 

Това ми дава 77/100 и не знам, защо при положение, че кода работи, ако някой е наясно да ми каже, ще съм супер благодарен:

using System;

namespace ConsoleApp2
{
    class Program
    {
        static void Main(string[] args)
        {
            string command = Console.ReadLine();
            if (command == "int")
            {
                int first = int.Parse(Console.ReadLine());
               int second = int.Parse(Console.ReadLine());
                HigherThenTwo(first, second);
            }
           else if (command == "char")
            {
                char first = char.Parse(Console.ReadLine());
                char second = char.Parse(Console.ReadLine());
                HigherThenTwo(first, second);
            }
            else if (command == "string")
            {
                string first = Console.ReadLine();
                string second = Console.ReadLine();
                HigherThenTwo(first, second);
            }

        }

        static int HigherThenTwo(int first, int second)
        {
       
            int result = 0;
            if (first > second)
            {
                result = first;
            }
            else
            {
                result = second;
            }
            Console.WriteLine(result);
            return result;
        }
        static char HigherThenTwo(char first, char second)
        {
   
            int result;
            if (first > second)
            {
                result = first;
            }
            else
            {
                result = second;
            }
            Console.WriteLine((char)result);
            return (char)result;

        }
        static string HigherThenTwo(string first, string second)
        {
            string result = string.Empty;
            int firstSum = 0;
            int secondSum = 0;
            foreach (char c in first)
            {
                firstSum += c;
            }
            foreach (char c in second)
            {
                secondSum += c;
            }
            if (firstSum > secondSum)
            {
                result = first;
            }
            else
            {
                result = second;
            }
            Console.WriteLine(result);
            return result;
        }





    } 

}

И по интересния факт е, че с готовия мотод за comparison работи и ми дава 100/100:

using System;
namespace ConsoleApp2
{
    class Program
    {
        static void Main(string[] args)
        {
            string command = Console.ReadLine();
            if (command == "int")
            {
                int first = int.Parse(Console.ReadLine());
                int second = int.Parse(Console.ReadLine());
                HigherThenTwo(first, second);
            }
            else if (command == "char")
            {
                char first = char.Parse(Console.ReadLine());
                char second = char.Parse(Console.ReadLine());
                HigherThenTwo(first, second);
            }
            else if (command == "string")
            {
                string first = Console.ReadLine();
                string second = Console.ReadLine();
                HigherThenTwo(first, second);
            }

        }

        static int HigherThenTwo(int first, int second)
        {

            int result = 0;
            if (first > second)
            {
                result = first;
            }
            else
            {
                result = second;
            }
            Console.WriteLine(result);
            return result;
        }
        static char HigherThenTwo(char first, char second)
        {

            int result;
            if (first > second)
            {
                result = first;
            }
            else
            {
                result = second;
            }
            Console.WriteLine((char)result);
            return (char)result;

        }
        static string HigherThenTwo(string first, string second)
        {
            string result = string.Empty;
            int comparison = first.CompareTo(second);
            if (comparison > 0)
            {
                result = first;
            }
            else
            {
                result = second;
            }
            Console.WriteLine(result);
            return result;

     
        }

    }
}

Някакви идеи?

Мерси,

2
Fundamentals Module
MartinBG avatar MartinBG 2790 Точки
Best Answer

Проблемът на твоя метод за сравняване на стрингове е, че той не сравнява стринговете, ами сумата от ASCII стойностите на символите им.

По дефиниция "B" > "A" , както и "B" > "AA". Твоят метод ще върне грешен резултат за втория пример, защото сумата от ASCII кодовете на двата символа 'А' е по-голяма от ASCII кода на единичното 'B'.

Ако искаш да използваш custom логика, трябва да сравняваш двата стринга символ по символ, докато стойността на единия стане по-голяма от стойността на другия. Липсата на символ може да смяташ като стойност 0, така че "аа" да е > "а"

 

0
16/06/2020 16:28:34
willystyle avatar willystyle 1964 Точки

hint: lowercase, uppercase
Когато сравняваш стринг или чар приравнявай кейса, понеже иначе "a" > "B"

1
tanchev avatar tanchev 2 Точки

Здравей,

Ами да, но то това е идеята, иначе как ще вземе точното char value? В смисъл, че в зависимост от големи или малки value-то на чара е различен, идеята е да се намери стринга с по-големият value, това нямам как да стане ако ги приравня всичките към малки или към големи, а те не са били при входните им данни такива.

В смисъл пробвах, не става, дава 66/100.

0
OgnyanPetranov avatar OgnyanPetranov 18 Точки

В последния метод не сравняваш стринговете като такива, а сравняваш чаровете в тях. Не съм сигурен, че това е условието на задачата. За сравнение на стринг се използва метода Compare и няма смисъл да си усложняваш живота. :-)

Между другото методите ти трябва да са void, защото никъде не използваш резултата, който връщаш, а принтираш директно от метода, също така можеш да оптимизираш решението без да овърлоудваш метода, а да го преправиш да приеме три параметъра, тип, първа стойност и втора стойност. 

0
tanchev avatar tanchev 2 Точки

Здравей,

Да искат да се сравни value-то на string-овете. Исках просто да си направя мой си метод затова реших да пробвам да ги обходя с foreach, интересното е, че проблема е решен, но judje гърми някъде, просто не знам какви крайни стойности са му вкарани, за да видя как да отстраня проблема. В смисъл, че каквито и string-ове да му задавам изкарва правилния резултат, както го прави и comparison-а.

А за comparison ясно, просто ми беше любопитно, защо foreach-а гърми някъде, тьй като независимо дали сравнявам целите стрингове или отделните чарове в тях, value-то на чаровете им все ще е един и същ или поне така би трябвало да бъде.

0
OgnyanPetranov avatar OgnyanPetranov 18 Точки

Както вече казах задачата не е да сравниш ASCII стойностите на символите в стринга, а да сравниш стринга като такъв. При сравнение на стрингове се взимат предвид и CultureInfo настройките, които оказват влияние на крайния резултат.

Ако ти е интересно можеш да разгледаш майкрософт документацията по темата, където има и пример как при един и същи вход и различни CultureInfo настройки, резултатът е различен: 

https://docs.microsoft.com/en-us/dotnet/api/system.globalization.compareinfo.compare?view=netcore-3.1

Дано съм бил полезен :-)

1
krum_43 avatar krum_43 352 Точки

А какво се има предвид при сравняване на два стринга?

Може би firstString>secondString ако дължината на първия е > от дължината на втория?

0
tanchev avatar tanchev 2 Точки

Здравей,

И аз така си помислих първоначално, но после обърнах внимание на думата value. Мисля, че искат сравняване на сумата от чаровете на единия стринг със сумата на чаровете от вторият.

Поздрави,

0
Iv_Konov avatar Iv_Konov 383 Точки

Здравей,

 

Обясниха ти проблема с  - "По дефиниция "B" > "A" , както и "B" > "AA"."  Аз ти предлагам един различен варинт за решение. 100/100...

Това е за проба на name1[0]:

===

string name1 = Console.ReadLine();
char n = name1[0];
Console.WriteLine(n);

===

 

и после с:

=

name1[0] > name2[0] ? name1 : name2;

=

 

using System;

namespace _09._Greater_of_Two_Values
{
    class Program
    {
        static void Main(string[] args)
        {
            string inputName = Console.ReadLine();
            string name1 = Console.ReadLine();
            string name2 = Console.ReadLine();

            Console.WriteLine(GameMax(inputName, name1, name2));
        }

        static string GameMax(string inputName, string name1, string name2)
        {
            string result = "";

            switch (inputName)
            {
                case "int":
                    result = (Math.Max(int.Parse(name1), int.Parse(name2))).ToString();
                    break;
                case "char":
                case "string":
                    return name1[0] > name2[0] ? name1 : name2;
                default:
                    break;
            }

            return result;
        }
    }
}

 

Поздрави,

Иван

0
MartinBG avatar MartinBG 2790 Точки

@Iv_Konov

Това няма да работи коректно, ако стринговете започват с еднакъв символ и имат разлика по-натам. wink

Например: "Az" и "Aa" или "А" и "Аааааа"

0
Iv_Konov avatar Iv_Konov 383 Точки

@MartinBG,

 

Да, перфектен си! :)

От друга страна, задачата минава с 100/100..:

https://judge.softuni.bg/Contests/Practice/Index/1208#8

(може би само първата буква се следи)

 

От друга страна, за да има яснота на задачата, коригирам според дълги string-ве (отново работи 100/100). :)

 

using System;

namespace _09._Greater_of_Two_Values
{
    class Program
    {
        static void Main(string[] args)
        {
            string inputName = Console.ReadLine();
            string name1 = Console.ReadLine();
            string name2 = Console.ReadLine();

            Console.WriteLine(GameMax(inputName, name1, name2));
        }

        static string GameMax(string inputName, string name1, string name2)
        {
            string result = "";

            switch (inputName)
            {
                case "int":
                    result = (Math.Max(int.Parse(name1), int.Parse(name2))).ToString();
                    break;
                case "char":
                case "string":
                    result = StringResult(name1, name2);
                    break;
                default:
                    break;
            }

            return result;
        }

        static string StringResult(string name1, string name2)
        {
            int shortLength = Math.Min(name1.Length, name2.Length);

            for (int i = 0; i < shortLength; i++)
            {
                if (name1[i] > name2[i])
                {
                    return name1;
                }
                else if (name2[i] > name1[i])
                {
                    return name2;
                }
            }

            return name1.Length > name2.Length ? name1 : name2;
        }
    }
}

 

 

Поздрави,

Иван

2
MPeychev avatar MPeychev 5 Точки

Ето едно решение на задачата:https://pastebin.com/hUVkMgbS

За първите два метода можеш да използваш Math.Max(); (ще ти спести няколко реда повторение на код)

Използвай си метода CompareTo за да сравниш двата string-а (Все пак хората за това са го измислили). The CompareTo method was designed primarily for use in sorting or alphabetizing operations.https://docs.microsoft.com/en-us/dotnet/api/system.string.compareto?view=netcore-3.1

1
Elena123456 avatar Elena123456 33 Точки

Здравейте,

предлагам и моето решение на задачата с използването на if..else, Math. Max() за сравнение на char и int, и CompareTo() за сравнение на string.

Интересното за тази задача е, че поне в четири лекции за Методи от минали издания на Fundamental modul или я прескачат или предлаганите решения дават 0/100 в Judge. Ето защо тази тема ще бъде изключително полезна за доста хора, които не разполагат с много време.

Поздрави!

 

 

using System;
using System.Linq;
namespace Methods
{
    class MainClass
    {

 

        public static void Main()
        {
            string command = Console.ReadLine();

            if (command == "int")
            {
                int firstNumber = int.Parse(Console.ReadLine());
                int secondNumber = int.Parse(Console.ReadLine());
                int result= GetMax(firstNumber, secondNumber);
                Console.WriteLine(result);
            }

            else if (command == "string")
            {
                string firstText = Console.ReadLine();
                string secondText = Console.ReadLine();
                string result = GetMax( firstText,  secondText);
                Console.WriteLine(result);

            }

            else if (command == "char")
            {
                char firstChar = char.Parse(Console.ReadLine());
                char secondChar = char.Parse(Console.ReadLine());
                char result = GetMax(firstChar, secondChar);
                Console.WriteLine(result);
            }
        }


     static int GetMax(int firstNumber, int secondNumber)
        {
            int result = Math.Max(firstNumber, secondNumber);
            return (result);
        }

        static string GetMax (string firstText, string secondText)
        {
            int comparison = firstText.CompareTo(secondText);
            if (comparison > 0)
            {
              return (firstText);
            }
            else
            {
                return(secondText);
            }
        }

        static char GetMax(char firstChar, char secondChar)
        {
            char result = (char)Math.Max(firstChar, secondChar);
            return (result);
        }
    }
  }


      

 

1
05/07/2020 00:07:25