C# Visual

[Blondy]

Уважаемые форумчане, не могли бы Вы мне подсказать, как правильно оформить условие цикла do ... while

Пользователь вводит размер квадрата до 20. Если пользователь ввел неправильное число или букву, мы вновь запрашиваем ввод параметра. С цифрами я разабралась, букву он тоже различает, но когда он ругается на условие (в скобках) while, связанное с буквой и не запрашивает вновь параметр . А как должно быть это условие правильно написано?

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

namespace Praktika
{
    class Program
    {
        static void Main(string[] args)
        {
            
            char repeate;
            do
            {


                Console.Clear();///Очищаем экран

                Int16 n;
                do
                {
                    Console.Write("Vvedite razmer storoni kvadrata ot 1 do 20: ");
                    ///////////////////////////проверка - пользователь ввел цифру или букву?
                  
                    bool result;

                    result = Int16.TryParse(Console.ReadLine(), out n);
                    if (result == false)
                    {
                        Console.WriteLine("Vy nepravilno vveli razmer storony  kvadrata. Eto dolzno byt chislo.");
                    }
                    else
                    {
                        break;
                    }
                    ////////////////////////////////////////////////////////
                    n = Int16.Parse(Console.ReadLine());

                    /////Проверка правильности ввода
                   
                    if (n < 1 || n > 20)
                    {
                        Console.WriteLine("Vy nepravilno vveli razmer storony  kvadrata!");
                    }

                    else
                    {

                        result = Int16.TryParse(Console.ReadLine(), out n);
                        if (result == false)
                        {
                            Console.WriteLine("Vy nepravilno vveli razmer storony  kvadrata. Eto dolzno byt chislo.");
                        }

                        else
                        {
                            break;
                        }

                       
                    }
                }

                while (n < 1 || n > 20 || result == false);

[netrino]

Можно вот так:

using System;
 
class Program {
 
  public static void Main(string[] args) {
    bool done = false;
    int n;
 
    do {
      Console.WriteLine ("Vvedite chislo ot 1 do 20: ");
      if (int.TryParse(Console.ReadLine(), out n) && n > 1 && n < 20)
        done = true;
      else
        Console.WriteLine ("Nevernoe chislo, poprobujte esche raz");
    } while (!done);
 
    Console.WriteLine ("{0}^2 = {1}", n, n * n);
  }
}

Но вообще говоря, в переменной done особой необходимости нет, можно просто поставить вечный цикл и break в положительной ветви if'а

[rUs_LAN]

 static void Main(string[] args)
        {   
            int inputValue;
            while (true)
            {
                Console.Write("Input size of squre: \t ");
                if (int.TryParse(Console.ReadLine(), out inputValue) && inputValue > 0 && inputValue < 20) break;
                Console.WriteLine("Incorrect value, values mast be in [1..19] interval");
            }
            Console.WriteLine("You did it !!!");
            Console.ReadKey(true);
        }

не успел (((

[Blondy]

Дорогой netrino и Руслан, спасибо!

Буду адаптировать код под Вашу идею. Мне казалось, что у меня просто неправильно прописано условие while в скобках и можно что-то внутри просто довинтить.

А Ваш код, Руслан, очень изящный))))

[rUs_LAN]

Справедливости ради хочу отметить, что мой код точное отображение комментариев netrino к своему коду.

[Blondy]

Не просто ларчик открывался....

Вот, Руслан, практический вопрос - у меня дальше идет кусок, полностью рабочий, где пользователь выбирает цвет и способ заполнения квадрата. А у меня дальше после Вашего примера кода этот блок не идет.... Я и так связывала эти куски, и сяк - не продолжается действо дальше. Что надо написать, после Вашего образца, чтобы он стал дальше код следующего куска видеть?
Вот фрагмент

 Console.Write("Vybirite tsvet:green, blue, red: ");
               
                ConsoleKeyInfo color = Console.ReadKey();
                char colorChar = color.KeyChar;

Прошу прощения, я только осваиваю этот язык, очень сложно пока привыкнуть к его принципам.

[Stilet]

Прошу прощения

Аналогично. Однако осмелюсь превратить твой код в нечто типа Оптимуса Примуса из знаменитого фильма:

            Console.Clear();///Очищаем экран
            Int16 n;
            for (; true; )  {
                for (bool result = Int16.TryParse(Console.ReadLine(), out n); !result; result = Int16.TryParse(Console.ReadLine(), out n))
                    Console.WriteLine("Vy nepravilno vveli razmer storony  kvadrata. Eto dolzno byt chislo.");
                if (n > 1 && n < 20) { break; }
                Console.WriteLine("Vy nepravilno vveli razmer storony  kvadrata!");
            }
            Console.ReadKey();

Если я конечно правильно понял твою задумку, то вот такая реализация мне кажется более красивой.

[Blondy]

Виталик, это все прекрасно, если вводишь цифру все идет, а если букву, он пишет об ошибке, но дальше ничего не запрашивает......А он должен запрашивать снова ввод параметра..Вот я и сижу.....

[rUs_LAN]

Stilet

тогда уж

for(;;)

[Blondy]

В общем, дорогие мальчики, тестовые испытания показали, что в моей итоговой сборке код netrino оказался самый реализуемый. Отрабатывает, как надо...

Ну не гуру я еще пока, чтобы адаптировать нерабочие куски....не хватает пока интеллекта.
Я у преподавателя обязательно спрошу, почему не отработал как надо именно мой вариант и что надо довинтить, чтобы сработал, как надо.

Всем спасибо!!!!

Я всем уже по плюсу поставила))) Кроме Виталика - до сих пор не разрешают ему еще плюсовать(((