Foggy Finder

Дело в том что существует 2 разные подписки на событие Tick - одно "автоматическое" созданное конструктором форм и второе "собственное".

VS показывает все ссылки если нажать на "reference" вверху:



Удалив одну из них получим ожидаемое поведение.

Можно преобразовать DataTable к строго типизированной коллекции с помощью метода AsEnumerable. Тогда вы сможете сделать группировку значений с помощью метода GroupBy.

После этого заполнить TreeView не составит труда.

Пример

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
        var treeView = new TreeView
        {
            Dock = DockStyle.Fill
        };
        this.Controls.Add(treeView);

        var dt = GetDataTable();
        PopulateTreeViewFromDataTable(treeView, dt);
    }

    private DataTable GetDataTable()
    {
        var dt = new DataTable("Цемент");

        dt.Columns.AddRange(new[] {
            new DataColumn("Name"),
            new DataColumn("NameOfCement")
        });

        dt.Rows.Add("Портланд", "Вихрь");
        dt.Rows.Add("Романцемент", "Тайга");
        dt.Rows.Add("Романцемент", "Свой");
        dt.Rows.Add("Алумацемент", "Курон");

        return dt;
    }

    public void PopulateTreeViewFromDataTable(TreeView tv, DataTable dataTable)
    {
        var data =
            dataTable
                .AsEnumerable()
                .Select(row => new
                {
                    Name = row.Field<string>("Name"),
                    Cement = row.Field<string>("NameOfCement")
                })
                .GroupBy(x => x.Name);

        tv.Nodes.Clear();

        var root = tv.Nodes.Add(dataTable.TableName);
        foreach (var gr in data)
        {
            var node = root.Nodes.Add(gr.Key);
            foreach (var subItem in gr)
            {
                node.Nodes.Add(subItem.Cement);
            }
        }
    }
}

Результат

Удобнее всего выполнять рисование в обработчике события Paint. Если в какой-то момент нужно обновить отображение, то следует вызвать перерисовку.

Пример, на основе вашего кода

readonly int[] q;
public Form2(int[] q)
{
    InitializeComponent();
    this.q = q;
    this.WindowState = FormWindowState.Maximized;
    Size size = new Size(Screen.PrimaryScreen.WorkingArea.Width,
        Screen.PrimaryScreen.WorkingArea.Height);
    pictureBox1.Size = size;
    pictureBox1.Paint += PictureBox_Paint;
}

private void PictureBox_Paint(object sender, PaintEventArgs e)
{
    Graphics g = e.Graphics;
    g.Clear(Color.FromArgb(q[2], q[3], q[4]));
    for (int i = 0; i < pictureBox1.Width; i += q[0])
    {
        for (int j = 0; j < pictureBox1.Height; j += q[0])
        {
            g.FillRectangle(new SolidBrush(Color.FromArgb(q[5], q[6], q[7])), i, j, q[1], q[1]);
        }
    }
}

Небольшой комментарий - совет не относящийся непосредственно к вопросу:

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

Тут у вас сразу несколько ошибок:

1. Нужно явно приводить к double иначе при делении двух int получите тоже целое.
2. Вы берете очень большие границы. Рост знаменателя намного превышает скорость роста числителя. Сами посмотрите: возведите 50 в куб, что получите? График вы конечно построите, но наглядности будет немного.

Что более важно: ваш отрезок OA = 2*a, а это значит что ваша правая граница точно должна быть меньше чем 2*a;

3. Посмотрите как задана функция - в виде квадрата. Извлекая корень вы должны не забывать про нижнюю ветку - ту которую дают отрицательные значения y.
Насколько мне известно возможности стандартного контрола довольно ограничены - вам придется заносить значения во вторую линию.

Собирая все вместе, попробуйте что-то вроде такого:

double a = 3;
double step = 0.1;

for (double x = 0; x < 2 * a - 1; x += step)
{
    var n = x * x * x;
    var d = 2 * a - x;
    chart1.Series[0].Points.AddXY(x, Math.Sqrt(n / d));
    chart1.Series[1].Points.AddXY(x, -Math.Sqrt(n / d));
}

Здесь 1 (x < 2 * a - 1) вычитается только для примера, чтобы вы сравнили результат. В реальности вам придется определять границу более сложным путем чтобы для разных a и левой границы (x) получать наглядное представление. Но, по крайней мере, у вас есть от чего отталкиваться.

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

Код в обработчике button1_Click выполняется последовательно. После того как сработает первое условие вы уменьшаете значение на единицу и следующий if тоже становится верным.

Первое что вам нужно сделать это вынести количество попыток в приватную переменную.
kolvopotytok не самое подходящее название (и хотя я, например, могу понять что тут имеется ввиду количество попыток) лучше использовать названия на английском. Давайте возьмем countOfAttempts . Так как вы хотите отслеживать количество, то вам нужно завести еще одну переменную для учета их изначального (максимального) числа:

public Form2(int digitfrom, int digitto)
{
    InitializeComponent();

    Random rnd = new Random();
    r = rnd.Next(digitfrom, digitto);
    maxCountOfAttempts = countOfAttempts = 3;
}
private int r;
private int maxCountOfAttempts, countOfAttempts;

Затем вы можете заметить что каждый раз код внутри проверочного условия одинаковый.
А теперь представьте, что диапазон будет достаточно широкий (скажем около сотни), а количество попыток увеличиться до 10. Копировать один и тот-же участок раз за разом не слишком эффективно. Давайте тогда вынесем весь код в отдельную функцию, которая будет принимать значение и в зависимости от него выводить информационное сообщение. Если пользователь угадал, то функция вернет true, в противном случае - false:

private bool CheckResult(int a)
{
    if (a == r)
    {
        MessageBox.Show($"Поздравляем!!! Вы угадали число с {countOfAttempts - 1} раза.");
        return true;
    }
    else if (a > r)
    {
        MessageBox.Show($"Число которое загадал ИИ меньше вашего!\nУ вас осталось {countOfAttempts - 1} попытки.");
        return false;
    }
    else
    {

        MessageBox.Show($"Число которое загадал ИИ больше вашего!\nУ вас осталось {countOfAttempts - 1} попытки.");
        return false;
    }
}

Теперь немного изменим сам обработчик:

private void button1_Click(object sender, EventArgs e)
{
    if (IsDigitsOnly(answer.Text))
    {
        int a = Convert.ToInt32(answer.Text);
        if (CheckResult(a))
            this.Close();
        else
        {
            countOfAttempts--;
            if (countOfAttempts == 0)
            {
                MessageBox.Show("К сожелению вы проиграли\nЗагаданное число: " + r);
                this.Close();
            }
        }
    }
    else
    {
        MessageBox.Show("Некорректный ввод данных", "Ошибка", MessageBoxButtons.OK, MessageBoxIcon.Error);
    }
    answer.Clear();
}

Должно работать. Как видите код достаточно близкий к вашему, но в то-же время немного понятнее.

И, если позволите, небольшой совет на будущее: старайтесь избегать использования goto.

В WinForms проектах конструктор форм прячет детали пользовательского интерфейса в файл Form1.Designer.cs (вместо Form1 будет название конкретного класса). Поэтому при копировании из других проектов или примеров нужно не забывать перенести также все подписки.

Чтобы проверить их наличие (или отсутствие), в конструкторе форм выделите элемент и перейдите к просмотру его свойств (Properties).

По умолчанию откроется вкладка свойств контрола. Для просмотра информации о событиях нужно перейти на следующую за ней (с нарисованной молнией):



В списке находите требуемый элемент. Если справа ничего не присвоено, значит дело в отсутствующей подписке. Через выпадающий список (ComboBox) выбираете обработчик из доступных:



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

Для подписки не обязательно использовать конструктор форм, можно создавать их самостоятельно, например, в конструкторе форм:

button1.Click += Button1_Click;

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

Обрабатывайте общее событие CellMouseClick. У выделенной на скриншоте ячейки оба индекса равны -1:

private void DataGridView1_CellMouseClick(object sender, DataGridViewCellMouseEventArgs e)
{
    var row = e.RowIndex;
    var column = e.ColumnIndex;

    if(row == -1 && column == -1)
    {
        // Do something
    }
}