структура C#...почему не работает сортировка по "НАЗВАНИЮ АВТО"??

[Serg94]

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

namespace laba_programmirovanie
{
    public struct salon
    {
        public int data;
        public double ob;
        public string nazv;
    }
    public class A : IComparer<salon>
    {
        public int Compare(salon x, salon y)
        {
            if (x.data < y.data)
                return 1;
            else return 0;
        }
    }
    public class B : IComparer<salon>
    {
        public int Compare(salon x, salon y)
        {
            if (x.ob < y.ob)
                return 1;
            else return 0;
        }
    }

    class Program
    {


        static void Main(string[] args)
        {
            List<salon> myspisok = new List<salon>();
            Console.WriteLine("введите количество учетных записей:");
            int n = Convert.ToInt32(Console.ReadLine());
            for (int f = 0; f < n; f++)
            {

                Console.WriteLine("\tМеню:");
                Console.WriteLine("1- ввод параметров авто");
                Console.WriteLine("2- сортировка по дате выпуска");
                Console.WriteLine("3- сортировка по объему");
                Console.WriteLine("4- сортировка по названию");
                Console.WriteLine("5- вывод параметров");
                Console.WriteLine("6- выход из программы");
                int otvet = Convert.ToInt32(Console.ReadLine());

                if (otvet == 1)
                {

                    salon my;
                    Console.WriteLine("введите марку:");
                    my.nazv = Console.ReadLine();
                    Console.WriteLine("дата выпуска: ");
                    my.data = Convert.ToInt32(Console.ReadLine());
                    Console.WriteLine("объем двигателя:");
                    my.ob = Convert.ToDouble(Console.ReadLine());
                    myspisok.Add(my);
                    Console.ReadLine();

                    if (otvet == 4)
                    {
                        myspisok.Sort();

                        foreach (salon a in myspisok)
                        {
                            Console.WriteLine(a);
                        }
                    }
                }
                if (otvet == 2)
                {
                    A d = new A();
                    myspisok.Sort(d);
                    Console.WriteLine("Сортировка по дате выпуска:");
                    foreach (salon pp in myspisok)
                        Console.WriteLine("{0}\t{1}", pp.nazv, pp.data);
                    Console.ReadKey();
                }
                if (otvet == 3)
                {

                    B dnew = new B();
                    myspisok.Sort(dnew);
                    Console.WriteLine("Сортировка по объему:");
                    foreach (salon pp in myspisok)
                        Console.WriteLine("{0}\t{1}", pp.nazv, pp.ob);
                    Console.ReadKey();
                }
                if (otvet == 5)
                {
                    foreach (salon pp in myspisok)
                    {
                        Console.WriteLine("название авто: {0}", pp.nazv);
                        Console.WriteLine("объем двигателя :{0}", pp.ob);
                        Console.WriteLine("год выпуска {0}", pp.data);
                    }
                }

                if (otvet == 6)
                    return;

            }
            Console.ReadLine();

        }

    }

}

[Lasur]

Не смущает, что блок if(otvet == 4) внутри блока if(otvet == 1)?
Кроме того, почему для сортировки по названию используется дефолтный Comparer?

[Serg94]

как по-другому сделать?!

[Guy]

switch(otvet)
{
case 1:
case 2:
...
}

[Serg94]

Сообщение от Guy

switch(otvet)
{
case 1:
case 2:
...
}

я как бы функции не писал, и вот это использовать не могу

[Lasur]

Какие функции?
Что мешает сделать так?

switch (otvet)
{
	case 1:
		salon my;
		Console.WriteLine("введите марку:");
		my.nazv = Console.ReadLine();
		Console.WriteLine("дата выпуска: ");
		my.data = Convert.ToInt32(Console.ReadLine());
		Console.WriteLine("объем двигателя:");
		my.ob = Convert.ToDouble(Console.ReadLine());
		myspisok.Add(my);
		Console.ReadLine();
		break;
	case 2:
		//... и т.д.
		break;
	//... и т.д.
}

[Guy]

По сути дела тебе во втором сообщении сказали почему

 if (otvet == 1)
                {

                    salon my;
                    Console.WriteLine("введите марку:");
                    my.nazv = Console.ReadLine();
                    Console.WriteLine("дата выпуска: ");
                    my.data = Convert.ToInt32(Console.ReadLine());
                    Console.WriteLine("объем двигателя:");
                    my.ob = Convert.ToDouble(Console.ReadLine());
                    myspisok.Add(my);
                    Console.ReadLine();

                    if (otvet == 4)
                    {
                        myspisok.Sort();

                        foreach (salon a in myspisok)
                        {
                            Console.WriteLine(a);
                        }
                    }
                }

Надо поменять на

 if (otvet == 1)
                {

                    salon my;
                    Console.WriteLine("введите марку:");
                    my.nazv = Console.ReadLine();
                    Console.WriteLine("дата выпуска: ");
                    my.data = Convert.ToInt32(Console.ReadLine());
                    Console.WriteLine("объем двигателя:");
                    my.ob = Convert.ToDouble(Console.ReadLine());
                    myspisok.Add(my);
                    Console.ReadLine();

                    
                }
if (otvet == 4)
                    {
                        myspisok.Sort();

                        foreach (salon a in myspisok)
                        {
                            Console.WriteLine(a);
                        }
                    }

А switch про который я написал просто читаемость кода улучшит

[Serg94]

 }
                    if (otvet == 4)
                    {
                        myspisok.Sort();

                        foreach (salon a in myspisok)
                        {
                            Console.WriteLine(a);
                        }
                    }

почему эта часть не срабатывает???

[Serg94]

Сообщение от Guy

По сути дела тебе во втором сообщении сказали почему

 if (otvet == 1)
                {

                    salon my;
                    Console.WriteLine("введите марку:");
                    my.nazv = Console.ReadLine();
                    Console.WriteLine("дата выпуска: ");
                    my.data = Convert.ToInt32(Console.ReadLine());
                    Console.WriteLine("объем двигателя:");
                    my.ob = Convert.ToDouble(Console.ReadLine());
                    myspisok.Add(my);
                    Console.ReadLine();

                    if (otvet == 4)
                    {
                        myspisok.Sort();

                        foreach (salon a in myspisok)
                        {
                            Console.WriteLine(a);
                        }
                    }
                }

Надо поменять на

 if (otvet == 1)
                {

                    salon my;
                    Console.WriteLine("введите марку:");
                    my.nazv = Console.ReadLine();
                    Console.WriteLine("дата выпуска: ");
                    my.data = Convert.ToInt32(Console.ReadLine());
                    Console.WriteLine("объем двигателя:");
                    my.ob = Convert.ToDouble(Console.ReadLine());
                    myspisok.Add(my);
                    Console.ReadLine();

                    
                }
if (otvet == 4)
                    {
                        myspisok.Sort();

                        foreach (salon a in myspisok)
                        {
                            Console.WriteLine(a);
                        }
                    }

А switch про который я написал просто читаемость кода улучшит

я понял о чем ты..я просто забыл скобку поставить..просто оно не сортирует..по названию

[Lasur]

Как один из вариантов (я писал об этом выше) - для сортировки по salon.nazv не написан Comparer<salon>, как это сделано для salon.data и salon.ob. При сортировке по названию вылетает Exception или просто выдает неверный порядок?