Numericupdown событие mousewheel увеличивает десятичное число более чем на один шаг

Question

Я пытаюсь переопределить элемент управления колесом мыши, чтобы при перемещении колесика мыши вверх или вниз оно только увеличивало значение в числовом поле вверх на 1. Я считаю, что в настоящее время оно использует то, что хранится в панели управления, и увеличивает/ уменьшая значение на 3 каждый раз.

Я использую следующий код.Даже когда numberOfTextLinesToMove равно только 1 и я вижу, что txtPrice.Value заполняется, как и ожидалось, что-то еще перезаписывает его, потому что значение, которое я установил, не соответствует отображаемому в числовом поле внизу

void txtPrice_MouseWheel(object sender, MouseEventArgs e)
        {
            int numberOfTextLinesToMove = e.Delta  / 120;
            if (numberOfTextLinesToMove > 0)
            {
                txtPrice.Value = txtPrice.Value + (txtPrice.Increment * numberOfTextLinesToMove);
            }
            else 
            {

                txtPrice.Value = txtPrice.Value - (txtPrice.Increment * numberOfTextLinesToMove);
            }

        }

Answer 1

У меня недавно была эта проблема, и я обошел ее, изменив приращение.

numericUpDown1.Increment = 1m / SystemInformation.MouseWheelScrollLines;

Редактировать: Это только хорошее решение, если вы собираетесь использовать колесо мыши только для изменения значения.Чтобы исправить это для всех ситуаций, вы должны переопределить класс.Вот простая версия.

public class NumericUpDownFix : System.Windows.Forms.NumericUpDown
{
    protected override void OnMouseWheel(MouseEventArgs e)
    {
        HandledMouseEventArgs hme = e as HandledMouseEventArgs;
        if (hme != null)
            hme.Handled = true;

        if (e.Delta > 0)
            this.Value += this.Increment;
        else if (e.Delta < 0)
            this.Value -= this.Increment;
    }
}

Answer 2

Пользователь user3610013 работает очень хорошо.Это небольшая модификация, которая может кому-то пригодиться.

Answer 3

Я воспользовался советом Ханса Пассанта и подклассом NumericUpDown, чтобы добавить эту функцию.Он использует модифицированную версию кода Reflector для метода OnMouseWheel.Вот то, что я закончил для тех, кто заинтересован.

using System;
using System.ComponentModel;
using System.Windows.Forms;

/// <summary>
/// Extends the NumericUpDown control by providing a property to 
/// set the number of steps that are added to or subtracted from
/// the value when the mouse wheel is scrolled.
/// </summary>
public class NumericUpDownExt : NumericUpDown
{
    private int wheelDelta = 0;
    private int mouseWheelScrollLines = 1;

    /// <summary>
    /// Gets or sets the number of step sizes to increment or 
    /// decrement from the value when the mouse wheel is scrolled.
    /// Set this value to -1 to use the system default.
    /// </summary>
    [DefaultValue(1)]
    [Description("Gets or sets the number of step sizes to increment or decrement from the value when the mouse wheel is scrolled. Set this value to -1 to use the system default.")]
    public Int32 MouseWheelScrollLines {
        get { return mouseWheelScrollLines; }
        set {
            if (value < -1)
                throw new ArgumentOutOfRangeException("value must be greater than or equal to -1.");
            if (value == -1)
                mouseWheelScrollLines = SystemInformation.MouseWheelScrollLines;
            else
                mouseWheelScrollLines = value;
        }
    }

    protected override void OnMouseWheel(MouseEventArgs e) {
        HandledMouseEventArgs args = e as HandledMouseEventArgs;
        if (args != null) {
            if (args.Handled) {
                base.OnMouseWheel(e);
                return;
            }
            args.Handled = true;
        }

        base.OnMouseWheel(e);

        if ((Control.ModifierKeys & (Keys.Alt | Keys.Shift)) == Keys.None && Control.MouseButtons == MouseButtons.None) {
            if (mouseWheelScrollLines != 0) {
                this.wheelDelta += e.Delta;
                float num2 = (float)this.wheelDelta / 120f;
                if (mouseWheelScrollLines == -1) 
                    mouseWheelScrollLines = 1;
                int num3 = (int)(mouseWheelScrollLines * num2);
                if (num3 != 0) {
                    int num4;
                    if (num3 > 0) {
                        for (num4 = num3; num4 > 0; num4--) 
                            this.UpButton();
                        this.wheelDelta -= (int)(num3 * (120f / (float)mouseWheelScrollLines));
                    } else {
                        for (num4 = -num3; num4 > 0; num4--) 
                            this.DownButton();
                        this.wheelDelta -= (int)(num3 * (120f / (float)mouseWheelScrollLines));
                    }
                }
            }
        }
    }
}

Answer 4

Посмотрев вокруг похожих вопросов, я обнаружил, что вы также можете обойти это без создания подкласса - например, этот код даст вам numericUpDown, который будет прокручиваться только на один (и только когда элемент управления имеет фокус).

Настройка делегата в конструкторе контейнера:

numericUpDown1.MouseWheel += new MouseEventHandler(this.ScrollHandlerFunction);

... затем определите функцию в другом месте класса:

private void ScrollHandlerFunction(object sender, MouseEventArgs e)
{
    HandledMouseEventArgs handledArgs = e as HandledMouseEventArgs;
    handledArgs.Handled = true;
    numericUpDown1.Value += (handledArgs.Delta > 0) ? 1 : -1;
}

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

Answer 5

Следующий код будет проходить по всей вашей Форме и исправлять каждый дочерний элемент управления NumericUpDown.

Переписано на основе этого и это

добавьте следующий код в ваш класс (или некоторый служебный класс):

static void FixNumericUpDownMouseWheel(Control c) {
    foreach (var num in c.Controls.OfType<NumericUpDown>())
        num.MouseWheel += FixNumericUpDownMouseWheelHandler;

    foreach (var child in c.Controls.OfType<Control>())
        FixNumericUpDownMouseWheel(child);
}

static private void FixNumericUpDownMouseWheelHandler(object sender, MouseEventArgs e) {
    ((HandledMouseEventArgs)e).Handled = true;
    var self = ((NumericUpDown)sender);
    self.Value = Math.Max(Math.Min(
        self.Value + ((e.Delta > 0) ? self.Increment : -self.Increment), self.Maximum), self.Minimum);

}

затем в конструкторе формы, после InitializeComponent(), вставить строку:

FixNumericUpDownMouseWheel(this);

Answer 6

На основании ответа Джея Риггса, вот более чистая версия (на мой взгляд):

namespace MyControls
{
    public partial class NumericUpDown : System.Windows.Forms.NumericUpDown
    {
        public Decimal MouseWheelIncrement { get; set; }

        public NumericUpDown()
        {
            MouseWheelIncrement = 1;
            InitializeComponent();
        }

        protected override void OnMouseWheel(MouseEventArgs e)
        {
            decimal newValue = Value;
            if (e.Delta > 0)
                newValue += MouseWheelIncrement;
            else
                newValue -= MouseWheelIncrement;
            if (newValue > Maximum)
                newValue = Maximum;
            else
                if (newValue < Minimum)
                    newValue = Minimum;
            Value = newValue;
        }
    }
}

Answer 7

Об этой ошибке сообщается здесь: NumericUpDown - использование колеса мыши может привести к различному приращению

В ответе Microsoft в феврале 2007 года говорится, что они не могут обратиться к Visual Studio 2008.

Существует два опубликованных обходных пути, оба из которых подкласс NumericUpDown.Проверьте вкладку Временное решение по ссылке.

Тот, который я попробовал, сработал для меня (опубликовано 'NanoWizard'):

using System;
using System.Windows.Forms;

internal class NumericUpDownControl : NumericUpDown
{
#region Constants
protected const String UpKey = "{UP}";
protected const String DownKey = "{DOWN}";
#endregion Constants

#region Base Class Overrides
protected override void OnMouseWheel(MouseEventArgs e_)
{
    String key = GetKey(e_.Delta);
    SendKeys.Send(key);
}
#endregion Base Class Overrides

#region Protected Methods
protected static String GetKey(int delta_)
{
    String key = (delta_ < 0) ? DownKey : UpKey;
    return key;
}
#endregion Protected Methods
}