Ошибка Windows Forms

 private void topouts_TextChanged(object sender, EventArgs e)
    {
        int x;
        int.TryParse(topouts.Text, out x);
        if (x>0 & x<=16)
        {
            core.TopOuts = x;
            logbox.Text = $" core.TopOuts-> {topouts.Text.Length}";//не важно
        }
        else
        {  topouts.Text = topouts.Text.Remove(topouts.Text.Length-1);
            topouts.SelectionStart = topouts.Text.Length;
        }

ваша проблема тут:

topouts.Text.Remove(topouts.Text.Length-1)

для пустой строки "".Length-1 == -1, Remove принимает строго положительные индексы и 0.

Вторая проблема - событие TextChanged срабатывает после любого присваивания значения свойству TextBox.Text, а так как вы при предыдущем срабатывании удалили некорректный символ - на входе опять будет пустая строка. Т.е. фактически у вас получается косвенная рекурсия (присваивание - обработчик - присваивание - обработчик - ...), в результате символы будут удаляться до тех пор, пока значение не станет корректным, либо строка не кончится и произойдет исключение.

По поводу исправления, я бы советовал не затирать символы за пользователя, а только проверять корректность ввода. Если ввод некорректный - перекрасить цвет фона в красный например, чтобы пользователь видел введенное значение. Ну и подсказка всплывающая или просто Label под полем мелким шрифтом, о допустимых значениях. Почему именно так - ну хотя бы потому, что в этом случае не возникнет проблем при вставке значения из буфера обмена и не придется дополнительно обрабатывать и этот случай.

Ну ок, пусть будет тут:

private void topouts_TextChanged(object sender, EventArgs e)
{
    if(string.IsNullOrEmpty(topouts.Text))
    {
        this.topouts.BackColor = System.Drawing.SystemColors.Window;
        return;
    }
    int x;
    if(int.TryParse(topouts.Text, out x) && x >= 0 && x <= 16)
    {
        core.TopOuts = x;
        this.topouts.BackColor = System.Drawing.SystemColors.MenuHighlight;
        logbox.Text = $" core.TopOuts-> {core.TopOuts}";
    }
    else
    {
        this.topouts.BackColor = System.Drawing.Color.Red;
    }
}

private void topouts_TextChanged(object sender, EventArgs e)
    {
        int x=17;
        if(topouts.Text.All(char.IsDigit)) { int.TryParse(topouts.Text, out x); }  
        if (topouts.Text.Length == 0) { this.topouts.BackColor = 
System.Drawing.SystemColors.Window; }
        if (x>=0 && x<=16 )
        {
            core.TopOuts = x;
            this.topouts.BackColor = System.Drawing.SystemColors.MenuHighlight;
            logbox.Text = $" core.TopOuts-> {core.TopOuts}";
        }
        else
        {
            this.topouts.BackColor = System.Drawing.Color.Red;
        }
    }

В общем сделал чтоб подсвечивалось красным если ввод не корректный и синим если корректный,но при вводе числа больше 10000000000 фон опять становится синим...почему?

Вроде так все нормально:

private void topouts_TextChanged(object sender, EventArgs e)
    { 
        int x=17;
        if(topouts.Text.All(char.IsDigit)&& topouts.Text.Length<10) { 
int.TryParse(topouts.Text, out x); }  
        if (topouts.Text.Length == 0) { this.topouts.BackColor = 
System.Drawing.SystemColors.Window; }
        if (x>=0 && x<=16 )
        {
            core.TopOuts = x;
            this.topouts.BackColor = System.Drawing.SystemColors.MenuHighlight;
            logbox.Text = $" core.TopOuts-> {core.TopOuts}";
        }
        else
        {
            this.topouts.BackColor = System.Drawing.Color.Red;
        }
    }

и очень хочется заменить topouts.Text.Length <10 на x.GetType().MaxValue.ToString().Lengh<10 я знаю,извращенство но можно как нибудь так?суть думаю поняли)