Как обработать изображение циклом?

Question

[absoluteST]

Добрый день, есть лаба по созданию простого графического редактора, в которой есть задание с которым возникли трудности, нужно создать функцию которая добавляет на изображение 1000 точек с координатами, заданными случайным образом, цвет тоже задается случайно. Я нашёл готовую функцию, но возникли проблемы с её встраиванием в кнопку(button3), постоянные ошибки "не содержит определения". Прошу помощи.
Вот код функции:

Код функции

public Color RandColor(Random rand)
        {
            return Color.FromArgb(rand.Next(0, 256), rand.Next(0, 256), rand.Next(0, 256));
        }
        public float NextF(Random rand, float min, float max)
        {
            return min + (float)rand.NextDouble() * (max - min);
        }
        public void Draw1000(Bitmap bmp, float r = 1F)
        {
            Random rand = new Random();
            using (var g = Graphics.FromImage(bmp))
            {
                for (int i = 0; i < 1000; i++)
                {
                    using (var br = new SolidBrush(rand.RandColor()))
                    {
                        g.FillEllipse(br, rand.NextF(0, bmp.Width) - r, rand.NextF(0, bmp.Height) - r, 2 * r, 2 * r);
                    }
                }
            }
        }

Ошибки

Код приложения

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WindowsFormsApp21
{
    public partial class Form1 : Form
    {
        private Point PreviousPoint, point;
        private Bitmap bmp;
        private Pen blackPen;
        private Graphics g;
        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            blackPen = new Pen(Color.Black, 4);
        }
        private void pictureBox1_MouseDown(object sender, MouseEventArgs e)
        {
            PreviousPoint.X = e.X;
            PreviousPoint.Y = e.Y;
        }
        private void pictureBox1_MouseMove(object sender, MouseEventArgs e)
        {
            if (e.Button == MouseButtons.Left)
            {   point.X = e.X;
                point.Y = e.Y;
                g.DrawLine(blackPen, PreviousPoint, point);
                PreviousPoint.X = point.X;
                PreviousPoint.Y = point.Y;
                pictureBox1.Invalidate();
            }
        }

        private void button1_Click_1(object sender, EventArgs e)
        {
            OpenFileDialog dialog = new OpenFileDialog();
            dialog.Filter = "Image files (*.BMP, *.JPG, " + "*.GIF, *.PNG)|*.bmp;*.jpg;*.gif;*.png";
            if (dialog.ShowDialog() == DialogResult.OK)
            {
                Image image = Image.FromFile(dialog.FileName);
                int width = image.Width;
                int height = image.Height;
                pictureBox1.Width = width;
                pictureBox1.Height = height;
                bmp = new Bitmap(image, width, height);
                pictureBox1.Image = bmp;
                g = Graphics.FromImage(pictureBox1.Image);
            }
        }

        public Color RandColor(Random rand)
        {
            return Color.FromArgb(rand.Next(0, 256), rand.Next(0, 256), rand.Next(0, 256));
        }
        public float NextF(Random rand, float min, float max)
        {
            return min + (float)rand.NextDouble() * (max - min);
        }
        public void Draw1000(Bitmap bmp, float r = 1F)
        {
            Random rand = new Random();
            using (var g = Graphics.FromImage(bmp))
            {
                for (int i = 0; i < 1000; i++)
                {
                    using (var br = new SolidBrush(rand.RandColor()))
                    {
                        g.FillEllipse(br, rand.NextF(0, bmp.Width) - r, rand.NextF(0, bmp.Height) - r, 2 * r, 2 * r);
                    }
                }
            }
        }
        private void button3_Click(object sender, EventArgs e)
        {

        }

        private void button2_Click_1(object sender, EventArgs e)
        {
            SaveFileDialog savedialog = new SaveFileDialog();
            savedialog.Title = "Сохранить картинку как ...";
            savedialog.OverwritePrompt = true;
            savedialog.CheckPathExists = true;
            savedialog.Filter =
             "Bitmap File(*.bmp)|*.bmp|" + "GIF File(*.gif)|*.gif|" + "JPEG File(*.jpg)|*.jpg|" + "PNG File(*.png)|*.png"; 
            if (savedialog.ShowDialog() == DialogResult.OK)
            {
                string fileName = savedialog.FileName;
                string strFilExtn = fileName.Remove(0,
                fileName.Length - 3);
                switch (strFilExtn)
                {
                    case "bmp":
                        bmp.Save(fileName, System.Drawing.Imaging.ImageFormat.Bmp);
                        break;
                    case "jpg":
                        bmp.Save(fileName, System.Drawing.Imaging.ImageFormat.Jpeg);
                        break;
                    case "gif":
                        bmp.Save(fileName, System.Drawing.Imaging.ImageFormat.Gif);
                        break;
                    case "tif":
                        bmp.Save(fileName, System.Drawing.Imaging.ImageFormat.Tiff);
                        break;
                    case "png":
                        bmp.Save(fileName, System.Drawing.Imaging.ImageFormat.Png);
                        break;
                    default:
                        break;
                }
            }
        }
    }
}

Comments

[edward_freedom]

rand.RandColor()

а что тут происходит? У класса рандома действительно нет такой функции

[Foggy Finder]

Просто передавайте созданный экземпляр в функцию. Вместо

using (var br = new SolidBrush(rand.RandColor()))
{
    g.FillEllipse(br, rand.NextF(0, bmp.Width) - r, rand.NextF(0, bmp.Height) - r, 2 * r, 2 * r);
}

напишите

using (var br = new SolidBrush(RandColor(rand)))
{
    g.FillEllipse(br, NextF(rand, 0, bmp.Width) - r, NextF(rand, 0, bmp.Height) - r, 2 * r, 2 * r);
}

а еще лучше почитайте хотя-бы документацию

Extension Methods (C# Programming Guide)

[absoluteST]

Foggy Finder, так и сделал, осталось решить как инициализировать цикл этой функции на уже открытой картинке.

[Foggy Finder]

absoluteST, не совсем понятно о какой картинке идет речь

[absoluteST]

Foggy Finder,

Форма

вот у меня такая форма. В чём сейчас у меня проблема. Я открываю в форме картинку, нажимаю на кнопку Точки и у меня должна срабатывать функция Draw1000, но проблема в том что я не знаю как правильно прописать её в коде кнопки, потому что она должна принимать картинку которую я открыл, а не заранее прописанную параметром.

[Foggy Finder]

absoluteST, для этого нужно сохранять где-то картинку при открытии

bmp = new Bitmap(dialog.FileName);

Answer 1

[freeExec]

Класс Random не имеет метода NextF, что вам в этой фразе не понятно?
Если же там кто-то хотел сделать надстройку над ним, то должно быть

public float NextF(this Random rand, float min, float max)

Comments

[absoluteST]

Если прописать аргумент this, ошибки всё равно остаются и добавляются новые, требующие определения этой функции в неуниверсальном статическом классе.

Ошибки

Как тогда мне использовать эту функцию в кнопке?

[edward_freedom]

absoluteST, просто вызови как обычную функцию NextF и укажи аргументы

[absoluteST]

edward_freedom, а с ошибками то что делать?

[edward_freedom]

absoluteST, ты зачем то вызываешь функции, которые обьявлены у тебя локально, в обьекте рандом. Вызывай их как обычные функции и ошибок не будет

[freeExec]

требующие определения этой функции в неуниверсальном статическом классе

absoluteST, ну так и сделай, что требудют.

[absoluteST]

freeExec,

В новом статическом классе

Сделал так, теперь ошибок нет, как мне вызвать Draw1000 в button3?

[edward_freedom]

absoluteST, как обычный метод расширения
bitmap.Draw1000(...)

[absoluteST]

edward_freedom, не так поставил вопрос, у меня есть форма, в которой ты открываешь рисунок, на которой уже по нажатию button3 на форме должна сработать эта функция, как мне передать bitmap уже открытую картинку?

[edward_freedom]

absoluteST, так у тебя в твоей кнопке уже есть битмап
bmp = new Bitmap(image, width, height);

[absoluteST]

edward_freedom, я это понимаю, но как мне прописать в Draw1000(), картинку которую я открыл уже в форме,?

[edward_freedom]

absoluteST, твояКартинка.Draw1000(...)