Комплексное число в консольном приложении: повторный запрос данных при недопустимом вводе

@Orein_Chis · Регистрация: 24.12.2012

Знаете, задача на самом деле простая. Но замутнааяяя

Понимаете, программу написала, всё работает отлично. Подправила где надо. Загвоздка вся в защите данных от ошибки в классе: т.е. требуется, в случае ввода с клавиатуры чего-либо, помимо числа, и первое число меньше или равное нулю, заново в том месте запустить запрос на ввод. Я так полагаю цикл нужен. Но с каким условием? Использовала try...catch, но кроме отлова ошибок и запуска новой консоли, да и закрытия старой, я так впихнуть и не смогла... Мозги закипели. Ведь вводить нужно действительную и мнимую часть два раза (для двух комплексных чисел). И если первое число задано нормально, а во втором что-то "накосячили", то не заново запускать программу, а то же место повторить и продолжить.
Ни с кем посоветоваться не могу, очень нуждаюсь в вашей помощи...

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Diagnostics;
using System.Runtime.InteropServices;
 
namespace ComplexNumber
{
    class ComplexNumber
    {
        int p; double f; double a, b;
        Process process = new Process();
        
        public int P
        {
            get{return p;}
            set
            {
                if (value > 0) value = p;
                else Console.WriteLine("Модуль числа должен быть положительным!");
            }
        }
        public double F
        {
            get { return f; }
            set { value = f; }
        }
      
       public ComplexNumber(int mod, double arg)
        {
            try
            {
                P = mod;
                F = arg;
            }
            catch (Exception)
            {
                Console.WriteLine("Число введено некорректно!");
                Process.Start("ComplexNumber.exe", "ComplexNumber\\bin\\Release");
                //Environment.Exit(0); - закрывает консоль
            }
        }
         
        public ComplexNumber()
        {
            try
            {
                Console.WriteLine("Введите модуль комплексного числа");
                p = Convert.ToInt32(Console.ReadLine());
                Console.WriteLine("Введите коэфициент аргумента Pi комплексного числа");
                f = Convert.ToDouble(Console.ReadLine());
            }
            catch (Exception)
            {
                Console.WriteLine("Число введено некорректно!");
                Process.Start("ComplexNumber.exe", "ComplexNumber\\bin\\Release");
                //Environment.Exit(0); - закрывает консоль
            }
        }
        
             
        public void PokazForm ()
        {
            Console.WriteLine("Показательная форма:");
            Console.WriteLine("     "+f+"*Pi");
            Console.WriteLine("z="+p+"+e");
        }
 
        public void TrigForm ()
        {
            Console.WriteLine("Тригонометрическая форма:");
            Console.WriteLine("z="+p+"*(cos("+f+")+i*sin("+f+"))");
        }
 
        public void AlgForm()
        {
            Console.WriteLine("Алгебраическая форма:");
            a = p*Math.Cos(f);
            b = p*Math.Sin(f);
            Console.WriteLine("z=" + a + "+" + "i*(" + b + ")");
        }
 
        //перегрузки
        public static ComplexNumber operator + (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
            ComplexNumber tmp = new ComplexNumber(0, 0);
            tmp.p = ComNum1.p + ComNum2.p;
            tmp.f = ComNum1.f + ComNum2.f;
            return tmp;
        }
 
        public static ComplexNumber operator - (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
            ComplexNumber tmp = new ComplexNumber(0, 0);
            tmp.p = ComNum1.p - ComNum2.p;
            tmp.f = ComNum1.f - ComNum2.f;
            return tmp;
        }
 
        public static ComplexNumber operator / (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
            ComplexNumber tmp = new ComplexNumber(0, 0);
            tmp.p = ComNum1.p / ComNum2.p;
            tmp.f = ComNum1.f / ComNum2.f;
            return tmp;
        }
 
        public static ComplexNumber operator * (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
            ComplexNumber tmp = new ComplexNumber(0, 0);
            tmp.p = ComNum1.p * ComNum2.p;
            tmp.f = ComNum1.f * ComNum2.f;
            return tmp;
        }
 
    }
}

Ладно, допустим вместо try...catch вставим бесконечный while, но условие - наличие ошибки? Ошибку выловить надо, так? Знаний у меня на это не хватает

На всякий пожарный вот код программы:

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Diagnostics;
 
namespace ComplexNumber
{
    class Program
    {
        static void Main(string[] args)
        {
            
            
                Console.WriteLine('\t' + "ПЕРВОЕ ЧИСЛО");
                ComplexNumber ComNum1 = new ComplexNumber();
                ComNum1.PokazForm();
                ComNum1.TrigForm();
                ComNum1.AlgForm();
 
                Console.WriteLine('\t' + "ВТОРОЕ ЧИСЛО");
                ComplexNumber ComNum2 = new ComplexNumber();
                ComNum2.PokazForm();
                ComNum2.TrigForm();
                ComNum2.AlgForm();
 
                ComplexNumber ComNum3 = new ComplexNumber(0, 0);
                ComNum3 = ComNum1 + ComNum2;
                Console.WriteLine('\t' + "Сумма:");
                ComNum3.PokazForm();
 
                ComplexNumber ComNum4 = new ComplexNumber(0, 0);
                ComNum4 = ComNum1 - ComNum2;
                Console.WriteLine('\t' + "Разность:");
                ComNum4.PokazForm();
 
                ComplexNumber ComNum5 = new ComplexNumber(0, 0);
                ComNum5 = ComNum1 / ComNum2;
                Console.WriteLine('\t' + "Деление:");
                ComNum5.PokazForm();
 
                ComplexNumber ComNum6 = new ComplexNumber(0, 0);
                ComNum6 = ComNum1 * ComNum2;
                Console.WriteLine('\t' + "Произведение:");
                ComNum6.PokazForm();
 
                Console.ReadLine();
            
        }
    }
}

@dondublon · 25.12.2012, 08:44

Ну как не помочь симпатичной девушке?
Посмотрел ваш код. Пришел в легкий ужас.
1. Ввод данных в конструкторе не делается. Сделайте создание - чтобы конструктор принимал параметры, но ввод отдельно. Может быть - отдельный метод для считывания из консоли, чтобы вызывался не в конструкторе.
2. В случае ошибки вы запускаете этот процесс опять??? Добропорядочные программы так не пишутся, так делаются форк-бомбы

( ru.wikipedia.org/wiki/Fork-бомба )
3.

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

Ладно, допустим вместо try...catch вставим бесконечный while, но условие - наличие ошибки?

Да, для ввода можно поставить бесконечный while, условие - успешный ввод данных.

@Orein_Chis · 25.12.2012, 14:35 **[ТС]**

Спасибо за отклик)) Комп сломался не вовремя, инет не работал, но я успела найти кое-какую инфу и закончила программу сама. По крайней мере исполняется как надо (я действительно билась головой об чужой монитор). Не успела правда сдать задание))) Сама понимаю, что код грубый и неотёсанный.
1. На счёт ввода в конструкторе мне ничего не сказал преподаватель, даже наоборот, посоветовал. Хочет он видеть там ввод. А если я начну опять что-то переделывать, то точно запутаюсь

2. Форк-бомба - милая весчь, мой кругозор расширился)))) Действительно, получается вводя данные заново я заполняю память, так? Опять ткнут в нерациональность

Вывод - чистить переменную после отлова ошибки... Хммм... Хорошее замечание.
3. Использовала do...while и try...catch внутри для p и f, условия - несовпадение типов данных и равенство нулю)))

Ну, собсно, нынешний результат...

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace ComplexNumber
{
    class ComplexNumber
    {
        int p; double f; double a, b;
        
        public int P
        {
            get{return p;}
            set{value = p;}
        }
        public double F
        {
            get { return f; }
            set { value = f; }
        }
      
       public ComplexNumber(int mod, double arg)
        {
            P = mod;
            F = arg;
        }
         
        public ComplexNumber()
        {
            do
            {
                try
                {
                    Console.WriteLine("Введите модуль комплексного числа");
                    p = Convert.ToInt32(Console.ReadLine());
                    if (p < 0 || p == 0) Console.WriteLine("Число введено некорректно!");
                }
                catch (Exception)
                {
                    Console.WriteLine("Число введено некорректно!");
                }
            }
            while (p.GetType() != typeof(int) || p < 0 || p == 0);
            
            do
            {
                try
                {
                    Console.WriteLine("Введите коэфициент аргумента Pi комплексного числа");
                    f = Convert.ToDouble(Console.ReadLine());
                    if (f == 0) Console.WriteLine("Число введено некорректно!");
                }
                catch (Exception)
                {
                    Console.WriteLine("Число введено некорректно!");
                }
            }
            while (f.GetType() == typeof(Exception) || f == 0);
        }
 
 
        public void PokazForm()
        {
            if (p.GetType() == typeof(int) && f.GetType() != typeof(Exception))
            {
                Console.WriteLine("Показательная форма:");
                Console.WriteLine("     " + f + "*Pi");
                Console.WriteLine("z=" + p + "+e");
 
            }
        }
 
        public void TrigForm ()
        {
            if (p.GetType() == typeof(int)&& f.GetType() != typeof(Exception))
            {
                Console.WriteLine("Тригонометрическая форма:");
                Console.WriteLine("z=" + p + "*(cos(" + f + ")+i*sin(" + f + "))");
            }
        }
 
        public void AlgForm()
        {
            if (p.GetType() == typeof(int) && f.GetType() != typeof(Exception))
            {
                Console.WriteLine("Алгебраическая форма:");
                a = p * Math.Cos(f);
                b = p * Math.Sin(f);
                Console.WriteLine("z=" + a + "+" + "i*(" + b + ")");
            }
        }
 
        //перегрузки
        public static ComplexNumber operator + (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
                ComplexNumber tmp = new ComplexNumber(0, 0);
                tmp.p = ComNum1.p + ComNum2.p;
                tmp.f = ComNum1.f + ComNum2.f;
                return tmp;
        }
 
        public static ComplexNumber operator - (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
                ComplexNumber tmp = new ComplexNumber(0, 0);
                tmp.p = ComNum1.p - ComNum2.p;
                tmp.f = ComNum1.f - ComNum2.f;
                return tmp;
        }
 
        public static ComplexNumber operator / (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
                ComplexNumber tmp = new ComplexNumber(0, 0);
                tmp.p = ComNum1.p / ComNum2.p;
                tmp.f = ComNum1.f / ComNum2.f;
                return tmp;
        }
 
        public static ComplexNumber operator * (ComplexNumber ComNum1, ComplexNumber ComNum2)
        {
                ComplexNumber tmp = new ComplexNumber(0, 0);
                tmp.p = ComNum1.p * ComNum2.p;
                tmp.f = ComNum1.f * ComNum2.f;
                return tmp;
        }
 
    }
}

@dondublon · 25.12.2012, 15:25

Не знаю, что там у вас не совпало. Делайте так:

C#

    class Program
    {
        static void Main(string[] args)
        {
            var c = Complex.ReadFromConsole();
        }
    }
 
    class Complex
    {
        float p; double f;
        public float P
        {
            get { return p; }
            set { value = p; }
        }
        public double F
        {
            get { return f; }
            set { value = f; }
        }
        public Complex(float ap, double af)
        {
            p = ap;
            f = af;
        }
        public static Complex ReadFromConsole()
        {
            String str;
            float p;
            do
            {
                str = Console.ReadLine();
            } 
            while (!float.TryParse(str, out p));
            return new Complex(p, 18);
        }
    }

И никакого ввода в конструкторе, и, тем более, других процессов.

@Orein_Chis · 29.01.2013, 00:42 **[ТС]**

Ой, руки не доходили ответить Х)))
Всё нормально, спасибо за советы, всё работает как надо. Спасибо ещё раз, но за отклик)))))

@dondublon 4606 / 2027 / 359 Регистрация: 17.03.2012 Сообщений: 10,081 Записей в блоге: 6
	25.12.2012, 08:44	2
	Ну как не помочь симпатичной девушке? Посмотрел ваш код. Пришел в легкий ужас. 1. Ввод данных в конструкторе не делается. Сделайте создание - чтобы конструктор принимал параметры, но ввод отдельно. Может быть - отдельный метод для считывания из консоли, чтобы вызывался не в конструкторе. 2. В случае ошибки вы запускаете этот процесс опять??? Добропорядочные программы так не пишутся, так делаются форк-бомбы ( ru.wikipedia.org/wiki/Fork-бомба ) 3. Сообщение от Orein_Chis Ладно, допустим вместо try...catch вставим бесконечный while, но условие - наличие ошибки? Да, для ввода можно поставить бесконечный while, условие - успешный ввод данных. 0

@Orein_Chis 0 / 0 / 0 Регистрация: 24.12.2012 Сообщений: 18
	29.01.2013, 00:42 [ТС]	5
	Ой, руки не доходили ответить Х))) Всё нормально, спасибо за советы, всё работает как надо. Спасибо ещё раз, но за отклик))))) 0

Опции темы