Вернитесь к циклу функции, когда пользователь не вводил ничего в scanf () в C [duplicate]

Это означает, что указанная переменная не указана ни на что. Я мог бы сгенерировать это так:

SqlConnection connection = null;
connection.Open();

Это вызовет ошибку, потому что, пока я объявил переменную «connection», она не указала ни на что. Когда я пытаюсь вызвать член «Open», для его устранения нет ссылки, и он будет вызывать ошибку.

Чтобы избежать этой ошибки:

  1. Всегда инициализируйте свои объекты, прежде чем пытаться что-либо с ними делать.
  2. Если вы не уверены, что объект имеет значение null, проверьте его с помощью object == null.

Инструмент Resharper JetBrains определит каждое место в вашем коде, которое имеет возможность ошибки нулевой ссылки, позволяя вам ввести нулевую проверку. Эта ошибка является источником ошибок номер один, IMHO.

7
задан Matt 16 September 2010 в 02:16
поделиться

4 ответа

Используйте fgets для чтения ввода в консоль:

   int res = 2;
   while (res == 2) {
       char buf[100];
       fgets(buf, sizeof(buf), stdin);
       res = sscanf(buf, "%f %f", &real, &img);
       if (res == 2)
           c[i++] = real + img * I;
   }
   c[i++] = 1 + 0*I; // most significant coefficient is assumed to be 1
   return i;
8
ответ дан MattSmith 27 August 2018 в 09:41
поделиться

Есть способ сделать то, что вы хотите, используя только scanf:

int readCoefficents(double complex *c) {
    int i = 0;
    double real;
    double img;
    char buf[2];
    while (scanf("%1[\n]", buf) == 0) {         // loop until a blank line or EOF
        if (scanf("%lf %lf", &real, &img) == 2) // read two floats
            c[i++] = real + img * I;
        scanf("%*[^\n]");                       // skip the rest of the line
        scanf("%*1[\n]");                       // and the newline
    }
    c[i++] = 1 + 0*I; // most significant coefficient is assumed to be 1
    return i;
}

Если пользователь вводит только 1 поплавок в строке, он будет читать следующую строку для второго значения. Если какой-либо случайный мусор введен, он перейдет к новой строке и повторит попытку со следующей строкой. В противном случае он просто начнет считывать пары значений float до тех пор, пока пользователь не войдет в пустую строку или не достигнет EOF.

2
ответ дан Chris Dodd 27 August 2018 в 09:41
поделиться

re PAXDIABLO: он не работает должным образом с EMPTY-линией, введенной пользователем, поэтому эта строка добавляется в вашу функцию getLine ()

if (strlen(buff) <= 1) return NO_INPUT;

после строки:

if (fgets (buff, sz, stdin) == NULL)
    return NO_INPUT;

Таким образом, он станет:

...
   if (strlen(buff) <= 1) return NO_INPUT;
   if (fgets (buff, sz, stdin) == NULL) return NO_INPUT;
....
0
ответ дан Derlo 27 August 2018 в 09:41
поделиться

Конкретная проблема, с которой вы сталкиваетесь, состоит в том, что строка формата scanf в %f пропускает пробел (, включая новые строки), пока не найдет фактический символ для сканирования. Из стандарта c99:

Спецификация преобразования выполняется в следующих шагах: & nbsp; & nbsp; - & nbsp; & nbsp; & nbsp; & nbsp; & nbsp; & nbsp; & nbsp; & nbsp; & nbsp; & nbsp; Входные символы пробела (как указано в isspace), если спецификация не включает спецификатор '[', 'c' или 'n'.

и в другом месте описывает isspace():

< blockquote>

Стандартными символами пробела являются: пробел ' ', подача формы '\f', новая строка '\n', возврат каретки '\r', горизонтальная вкладка '\t' и вертикальная вкладка '\v'.

Лучше всего использовать fgets для получения строки (и это можно легко защитить от переполнения буфера), а затем использовать sscanf в результирующей строке.

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

#include <stdio.h>
#include <string.h>

#define OK       0
#define NO_INPUT 1
#define TOO_LONG 2
static int getLine (char *prmpt, char *buff, size_t sz) {
    int ch, extra;

    // Get line with buffer overrun protection.
    if (prmpt != NULL) {
        printf ("%s", prmpt);
        fflush (stdout);
    }
    if (fgets (buff, sz, stdin) == NULL)
        return NO_INPUT;

    // If it was too long, there'll be no newline. In that case, we flush
    // to end of line so that excess doesn't affect the next call.
    if (buff[strlen(buff)-1] != '\n') {
        extra = 0;
        while (((ch = getchar()) != '\n') && (ch != EOF))
            extra = 1;
        return (extra == 1) ? TOO_LONG : OK;
    }

    // Otherwise remove newline and give string back to caller.
    buff[strlen(buff)-1] = '\0';
    return OK;
}

& nbsp;

// Test program for getLine().

int main (void) {
    int rc;
    char buff[10];

    rc = getLine ("Enter string> ", buff, sizeof(buff));
    if (rc == NO_INPUT) {
        // Extra NL since my system doesn't output that on EOF.
        printf ("\nNo input\n");
        return 1;
    }

    if (rc == TOO_LONG) {
        printf ("Input too long [%s]\n", buff);
        return 1;
    }

    printf ("OK [%s]\n", buff);

    return 0;
}

Тестирование его различными комбинациями:

pax> ./prog
Enter string>[CTRL-D]
No input

pax> ./prog
Enter string> a
OK [a]

pax> ./prog
Enter string> hello
OK [hello]

pax> ./prog
Enter string> hello there
Input too long [hello the]

pax> ./prog
Enter string> i am pax
OK [i am pax]

Я бы использовал эту функцию, чтобы безопасно получить строку, а затем просто используйте:

sscanf (buffer, "%f %f", &real, &img)

, чтобы получить фактические значения (и проверить количество).


На самом деле, вот полная программа, которая ближе к тому, что вы хотите:

#include <stdio.h>
#include <string.h>

#define OK       0
#define NO_INPUT 1
#define TOO_LONG 2
static int getLine (char *prmpt, char *buff, size_t sz) {
    int ch, extra;

    // Get line with buffer overrun protection.
    if (prmpt != NULL) {
        printf ("%s", prmpt);
        fflush (stdout);
    }
    if (fgets (buff, sz, stdin) == NULL)
        return NO_INPUT;

    // If it was too long, there'll be no newline. In that case, we flush
    // to end of line so that excess doesn't affect the next call.
    if (buff[strlen(buff)-1] != '\n') {
        extra = 0;
        while (((ch = getchar()) != '\n') && (ch != EOF))
            extra = 1;
        return (extra == 1) ? TOO_LONG : OK;
    }

    // Otherwise remove newline and give string back to caller.
    buff[strlen(buff)-1] = '\0';
    return OK;
}

& nbsp;

int main (void) {
    int i = 1, rc;
    char prompt[50], buff[50];
    float real, imag;

    while (1) {
        sprintf (prompt, "\nEnter real and imaginary for #%3d: ", i);
        rc = getLine (prompt, buff, sizeof(buff));
        if (rc == NO_INPUT) break;
        if (*buff == '\0') break;

        if (rc == TOO_LONG) {
            printf ("** Input too long [%s]...\n", buff);
        }

        if (sscanf (buff, "%f %f", &real, &imag) == 2) {
            printf ("Values were %f and %f\n", real, imag);
            i++;
        } else {
            printf ("** Invalid input [%s]\n", buff);
        }
    }

    return 0;
}

вместе с тестовым прогоном:

pax> ./testprog

Enter real and imaginary for #  1: hello
** Invalid input [hello]

Enter real and imaginary for #  1: hello there
** Invalid input [hello there]

Enter real and imaginary for #  1: 1
** Invalid input [1]

Enter real and imaginary for #  1: 1.23 4.56
Values were 1.230000 and 4.560000

Enter real and imaginary for #  2:

pax> _
9
ответ дан paxdiablo 27 August 2018 в 09:41
поделиться
Другие вопросы по тегам:

Похожие вопросы: