Проверить восьмеричное число

Совершенно не ясно, чего пытается достичь ваш текущий подход, поскольку он преобразуется как в десятичный, так и в восьмеричный. Если вы просто хотите проверить, содержит ли строка только цифры 0-7, вероятно, проще всего использовать регулярное выражение:

using System;
using System.Text.RegularExpressions;

class Test
{
    public static void Main(string[] args)
    {
        // Do you want this to be valid or not?
        Console.WriteLine(IsOctal(""));
        Console.WriteLine(IsOctal("012"));
        Console.WriteLine(IsOctal("890"));
        Console.WriteLine(IsOctal("abc"));
        // Do you want this to be valid or not?
        Console.WriteLine(IsOctal("-0123"));
    }

    private static bool IsOctal(string text)
    {
        return Regex.IsMatch(text, "^[0-7]+$");
    }
}

Это предполагает, что:

• Вас интересуют только неотрицательные значения (поэтому нам не нужно принимать начальный -)
• Пустая строка не должна считаться действительной
• После этого вам не нужно фактическое значение — оно может быть сколь угодно длинным.

Если важна производительность, вы можете сравнить решение регулярного выражения со сканированием (адаптировано из этой записи):

private bool IsOctal(IEnumerable<char> chars)
{
    bool isOctal;
    foreach (var c in chars)
    {
        isOctal = (c >= '0' && c <= '7');

        if (!isOctal)
            return false;
    }
    return true;
}

И, как указывалось в предыдущих ответах, это делает некоторые предположения о ваших числовых значениях.

Это сильно зависит от ваших правил проверки. Не существует единственно правильного определения набора символов восьмеричного числа. Иногда начинать с 0 обязательно. Я считаю, что в Python вы можете использовать _ в качестве разделителя цифр. Отрицательные? Научная/инженерная нотация? Десятичные?

Да, некоторые из них странные, но дело в том, что это зависит от того, где вы их используете.

Вам нужно, чтобы его можно было анализировать на С#? Тогда int.TryParse() - это путь. Вы только что проанализировали его и получили успех или неудачу. Нужно ли передавать его в виде строки другому сервису/API? Посмотрите правила проверки этой службы/API. Также не перепроверяйте, как вы делаете выше, это просто сбивает с толку, что такое реальный допустимый набор символов, и если вы не доверяете своей среде выполнения, вы можете также начать создавать ту, которой вы доверяете (не делайте это, это была шутка).

Узнайте, каковы настоящие правила и требования валидации, и ваш ответ будет ясен. Не переусердствуйте.

Вот решение linq:

public static void Main(string[] args)
{
    var a = IsOctal("023"); //true
    var b = IsOctal("678"); //false
}

public static bool IsOctal(string text)
{
    //get a collection of chars that each text element can be
    var possibleChars = Enumerable.Range(0, 8)
        .Select(x => x.ToString()[0]);

    //"are there any chars in 'text' that are not in 'possibleChar'?"
    var anyAreInvalid = text
        .Where(x => !possibleChars.Contains(x))
        .Any();

    //return the inverse of your question
    return !anyAreInvalid;
}

Основная работа заключается в операторе Where, который проверяет каждый символ в вашем вводе по списку «допустимых символов».

Это также использует те же предположения из ответа Джона.