1

Программа написана на языке Си. ОС: Linux Ubuntu . Редактор : VSC 2017. Вот часть кода :

while (true)
{
    printf("Введите рейтинг: ");
    char S11[100];
    scanf("%s", S11);
    double v0 = atof(S11);
    if (v0 == 0)
    {
        if (!(strlen(S11) == 1 && S11[0] == '0'))
        {
            Console_clear();
            printf("Вы ввели не число !");
            getchar();
            Console_clear();
            continue;
        }
    }
    mas[j]->Rang = v0;
    break;
}

При компилировании все проходит нормально. Однако, при запуске программы, и последовательном выполнении ее пунктов (введении имени, фамилии и так далее), когда доходит до этого места, возникает ошибка Segmentation fault (core dumped). Если же, преобразовать эту часть кода в более простую, например :

printf("Введите рейтинг: ");
scanf("%f", &newUser.Rang);

Все будет работать нормально, однако необходима проверка на наличие символов отличных от цифр в этой строчке.

В чем причина ошибки?

12
  • "...однако необходима проверка на наличие символов отличных от цифр в этой строчке". Но у вас в коде нет такой проверки. Так "необходима" такая проверка или нет? 14 дек 2017 в 2:44
  • "...когда доходит до этого места..." До какого "этого" места? Что такое mas? Что такое j? Что такое newUser? Почему вы пользуетесь ущербной функцией atof, а не нормальной strtod? 14 дек 2017 в 2:46
  • Вот эта часть кода и является данной проверкой. mas - массив значений структуры данных. j - номер массива, так как таких массивов может быть много. newUser - структура данных, у нас есть структура user, в которой мы меняем значения, я назвал ее newUser
    – Griffin
    14 дек 2017 в 3:04
  • Нет. Например, если ввести 123vasya, то ваша проверка не заметит ничего странного. А если ввести 0.0, то она скажет, что это не число. Почему? И почему вообще вы работаете с плавающим типом, если вы хотите разрешать только цифры. "Только цифрами" можно набрать только целые значения. Откуда тогда вдруг взялся double? 14 дек 2017 в 3:05
  • Да, согласен, проверку реализовал неверно. Не подскажете тогда, каким образом ее лучше всего сделать ? через strtod упомянутый выше ?
    – Griffin
    14 дек 2017 в 3:08

1 ответ 1

-1

Два замечания по коду.

Первое. Условие

strlen(S11) == 1 && S11[0] == '0' 

не имеет смысла, так как если в S11[0] находится 0, то длина строки == 0.

Второе. А где присовоение значения переменной j?

mas[j]->Rang = v0;
7
  • 1
    Ваше первое замечание не имеет смысла, ибо в коде сделана проверка на '0', а не на 0. Не путайте 0 и '0'. 14 дек 2017 в 2:45
  • Присвоение починил, однако проблему это не решило
    – Griffin
    14 дек 2017 в 3:02
  • ибо в коде сделана проверка на '0' - виноват, проглядел.
    – Sergey
    14 дек 2017 в 4:34
  • 1
    Ошибка Segmentation fault возникает тогда, когда программа пытается обращение к памяти за пределом сегмента, ей выделенного ОС. Как правило, такие ошибки возникают либо при использовании индексов, вышедших за переделы размерности массива, либо при использовании "битых" указателей.
    – Sergey
    14 дек 2017 в 4:38
  • 1
    В приведённом Вами участке кода есть только одно "подозрительное" место: mas[j]->Rang = v0; Окружите этот оператор операторами типа prinf("1111\n"); ... prinf("2222\n"); что бы проверить эту гипотезу. Если единички напечатались, а двойки - нет, значит Ваше присвоение j неверно.
    – Sergey
    14 дек 2017 в 4:42

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.