[C] Как считать с клавиатуры целое число ?

yolki

psihodog

int getInt

{

  SUCCESS= FALSE;

  return 0;

}

vall

yolki

vall

yolki

vall

yolki

psihodog

любой ввод, попадающий в тип int.

#define BUFFLEN 1024



int getInt

{

  char buff[BUFFLEN];

  int i, u;

  scanf("%s", buff);

  sscanf(buff, "%i", &i);

  sscanf(buff, "%u", &u);

  SUCCESS= i==u;

  return i;

}

yolki

mira-bella

#define BUFFLEN 1024



int getInt

{

  char buff[BUFFLEN];

  int i, u;

  scanf("%s", buff);

  sscanf(buff, "%i", &i);

  sscanf(buff, "%u", &u);

  SUCCESS= i==u;

  return i;

}

psihodog

"-1" не сработает

psihodog

vall

yolki

yolki

vall

psihodog

vall

yolki

Open Watcom C/C++ CL Clone for 386  Version 1.3

Portions Copyright (c) 1995-2002 Sybase, Inc. All Rights Reserved.

Source code is available under the Sybase Open Watcom Public License.

See http://www.openwatcom.org/ for details.

mira-bella

00: #define BUFFLEN 1024

01: 

02: int getInt

03: {

04:     char buff[BUFFLEN];

05:     int i, u;

06:     scanf("%s", buff);

07:     sscanf(buff, "%i", &i);

08:     sscanf(buff, "%u", &u);

09:     SUCCESS= i==u;

10:     return i;

11: }

э... давай хотя бы две-три назови.

psihodog

mira-bella

и ещё ошибки обрабатывать? понятное дело, что в реальном коде я бы такого не написал..

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

 

if( scanf("%d",&i) == 1 )

  SUCCESS = TRUE;

else

  SUCCESS = FALSE;

> строка 8: scanf по идее символ '-' должен идентифицировать как ошибку формата без заполнения переменной u, хотя фактическое поведение может и зависит от реализации стандартной библиотеки.
ну вот одно дело "по идее", а компилятор мне совсем другое выдал.

по поводу некорректного ввода — ты прав, действительно надо проверять, чтчо вернули сканфы, но т.к. код лишь демонстрировал идею, то и эту придирку можно отбросить.

mira-bella

psihodog

так какую же идею этот код демонстрировал?

Marinavo_0507

строка 4: хотя и не ошибка, но совершенно отвратная практика - стек не резиновый.

yolki

Marinavo_0507

если этот самый getInt будет использован в рекурсивной функции, то на некотором [очень близком] шаге прога схватит переполнение стека.

yolki

Marinavo_0507

mira-bella

если этот самый getInt будет использован в рекурсивной функции, то на некотором [очень близком] шаге прога схватит переполнение стека.

mira-bella

Marinavo_0507

ибо так вот несколько друг друга вызывающих функций скушают по килобайту и получим переполнение

или человек привыкший так писать начнет кушать не по килобайту, а по мегабайту

Ivan8209

sergey_m

Marinavo_0507

Ivan8209

vall

Ivan8209

Olenenok

char buffer[1024];

while (scanf ("%1023s", buffer) < 1);

while (getchar != '\n')  

Ivan8209

bleyman

Marinavo_0507

bleyman

yolki

Marinavo_0507

yolki

Marinavo_0507

garikus

mira-bella

ты хотел сказать "несколько тысяч", да?
самому-то не смешно?

разве что быдлокодер(tm)
а программист с нормальными умственными способностями разницу между килобайтом и мегабайтом осознаёт

mira-bella

...............
PROCEDURE ExpectEOL (c: CHAR): BOOLEAN;
BEGIN
...............
подойдёт

Olyalyau

целое число

#include <limits.h>

#include <stdio.h>



#undef TRUE

#define TRUE 1



#undef FALSE

#define FALSE 0



inline

int

is_numeric (char c)

{

  return c >= '0') && (c <= '9';

}



inline

int

is_terminating (char c)

{

  return c == '\n') || (c == '\r') ||

          (c == ' ') || (c == '\t';

}



int

read_int (int * result)

{

  int   read;

  int   sign = 1;

  int   value = 0;

  int   c_value;

  int   no_number_present = 1;

  char  c;



  read = scanf ("%c", & c);



  if (read != 1)

    return FALSE;



  if (c == '-')

  {

    sign = -1;

    read = scanf ("%c", & c);

  }

  else if (c == '+')

    read = scanf ("%c", & c);



  if (sign == 1)

  {

    while read == 1) && (is_numeric (c

    {

      no_number_present = 0;

      c_value = c - '0';

      if INT_MAX - c_value) / 10) < value)

        return FALSE;

      value = value * 10 + c_value;

      read = scanf ("%c", & c);

    }

  }

  else

  {

    while read == 1) && (is_numeric (c

    {

      no_number_present = 0;

      c_value = c - '0';

      if INT_MIN + c_value) / 10) > value)

        return FALSE;

      value = value * 10 - c_value;

      read = scanf ("%c", & c);

    }

  }



  if no_number_present) ||

      read != EOF) && (! is_terminating (c

    return FALSE;



  (* result) = value;

  return TRUE;

}



int

main 

{

  int   result;

  int   flag = read_int (& result);



  if (flag == TRUE)

    fprintf (stdout, "%d\n", result);

  else

    fprintf (stderr, "Error!\n");



  return (! flag);

}

 

Ivan8209

Russula

Ivan8209

garikus

/*

EBNF:

	number = [sign] digit {digit} eol .

	sign = '+' | '-' .

	digit = '0'..'9' .

	eol = 0x0a | 0x0d .

*/



#include <ctype.h>

#include <stdio.h>

#include <limits.h>



#define expecteol(c) c) == 0x0a) || c) == 0x0d

#define expectdigit(c) isdigit(c)

#define ctoi(c) c) - '0')

#define FALSE 0

#define TRUE 1



void getnumber (int *num, int *success)

{

	int x, i;

	int sign, overflow;

	int c;

	

	c = getchar;

	if (c == '-') {

		sign = TRUE;

		c = getchar;

	} else if (c == '+') {

		sign = FALSE;

		c = getchar;

	} else

		sign = FALSE;

	

	if (expectdigit(c {

		x = 0;

		while (1) {

			i = ctoi(c);

			overflow = (x > (INT_MAX - i) / 10);

			if (overflow)

				break;

			x = x * 10 + i;

			c = getchar;

			if (!expectdigit(c

				break;

		}

		

		if !overflow) && (expecteol(c {

			*num = sign ? -x : x;

			*success = TRUE;

		} else

			*success = FALSE;

	} else

		*success = FALSE;

}



int main (int argc, char *argv[])

{

	int x, success;

	

	getnumber(&x, &success);

	if (success)

		printf("%d\n", x);

	else

		printf("error\n");

	

	return 0;

}

Ivan8209

garikus

> void getnumber (int *num, int *success)
Любишь strto* переизобретать?

> expecteol(c)
Ещё и с жёстко зашитым fgetln/getline.

Ivan8209

garikus

И?
Это повод переизобретать strto*?
Повод переизобретать fgetln/readline?

Ivan8209

garikus

bzero/memset пишется просто, равно как strcpy и т. п.,
но правильно писать --- вызов bzero/memset,
а не соответствующий цикл, каким бы простым он ни был.

Ivan8209

garikus

Притом, что твоё решение --- это написание одного
удовлетворительного качества strto*, и одного очень плохого fgetln.

Ivan8209

garikus

Тем, что оно не задействует (неполностью задействует)
имеющиеся средства операционной системы.

Ivan8209

Olyalyau

Правильный ответ уже дан и не совпадает с твоим.

fgetln

Любишь переизобретать велосипед?

> is_numeric

> scanf ("%c", & c)

Умные люди уже давно придумали err/warn.

Olyalyau

а в стандарте c есть inline?

Olyalyau

qsk78

К примеру, моя реализация std::string обгоняет стандартную из поставки GCC примерно в 1.5 раза, а моя реализация MD5, на little-endian машине работает быстрее OpenSSL'евской в 1.3 раза, на big-endian — вровень с OpenSSL'вской.

poi1981

Olyalyau

bzero/memset пишется просто, равно как strcpy и т. п.

Olyalyau

>К примеру, моя реализация std::string обгоняет стандартную из поставки GCC примерно в 1.5 раза
а sizeof(string) твой во сколько раз больше жэцэцэшного? в 5?

kokoc88

Обожаю. И как показывает практика, делаю это лучше первооткрывателей велосипеда. К примеру, моя реализация std::string обгоняет стандартную из поставки GCC примерно в 1.5 раза, а моя реализация

  if (sign == 1)

  {

    while read == 1) && (is_numeric (c

    {

      no_number_present = 0;

      c_value = c - '0';

      if INT_MAX - c_value) / 10) < value)

        return FALSE;

      value = value * 10 + c_value;

      read = scanf ("%c", & c);

    }

  }

  else

  {

    while read == 1) && (is_numeric (c

    {

      no_number_present = 0;

      c_value = c - '0';

      if INT_MIN + c_value) / 10) > value)

        return FALSE;

      value = value * 10 - c_value;

      read = scanf ("%c", & c);

    }

  }

  

Olyalyau

Ты неимоверно крут.

Только не понятно, почему в коде столько copy-paste

 if INT_MAX - c_value) / 10) < value) 

  if INT_MIN + c_value) / 10) > value) 

int

read_int (int * result)

{

  static int const      max_limit_value [10] = {(INT_MAX - 0) / 10,

                                                (INT_MAX - 1) / 10,

                                                (INT_MAX - 2) / 10,

                                                (INT_MAX - 3) / 10,

                                                (INT_MAX - 4) / 10,

                                                (INT_MAX - 5) / 10,

                                                (INT_MAX - 6) / 10,

                                                (INT_MAX - 7) / 10,

                                                (INT_MAX - 8) / 10,

                                                (INT_MAX - 9) / 10};



  static int const      min_limit_value [10] = {(INT_MIN + 0) / 10,

                                                (INT_MIN + 1) / 10,

                                                (INT_MIN + 2) / 10,

                                                (INT_MIN + 3) / 10,

                                                (INT_MIN + 4) / 10,

                                                (INT_MIN + 5) / 10,

                                                (INT_MIN + 6) / 10,

                                                (INT_MIN + 7) / 10,

                                                (INT_MIN + 8) / 10,

                                                (INT_MIN + 9) / 10};



  int   read;

  int   sign = 1;

  int   value = 0;

  int   c_value;

  int   no_number_present = 1;

  char  c;



  read = scanf ("%c", & c);



  if (read != 1)

    return FALSE;



  if (c == '-')

  {

    sign = -1;

    read = scanf ("%c", & c);

  }

  else if (c == '+')

    read = scanf ("%c", & c);



  if (sign == 1)

  {

    while read == 1) && (is_numeric (c

    {

      c_value = c - '0';

      no_number_present = 0;

      if (max_limit_value [c_value] < value)

        return FALSE;

      value = value * 10 + c_value;

      read = scanf ("%c", & c);

    }

  }

  else

  {

    while read == 1) && (is_numeric (c

    {

      c_value = c - '0';

      no_number_present = 0;

      if (min_limit_value [c_value] > value)

        return FALSE;

      value = value * 10 - c_value;

      read = scanf ("%c", & c);

    }

  }



  if no_number_present) ||

      read != EOF) && (! is_terminating (c

    return FALSE;



  (* result) = value;

  return TRUE;

}

 

      c_value = c - '0';

      no_number_present = 0;

 

Olyalyau

>К примеру, моя реализация std::string обгоняет стандартную из поставки GCC примерно в 1.5 раза
а sizeof(string) твой во сколько раз больше жэцэцэшного? в 5?

error   svg     std

119     410632  883897

error   svg     std

0       249     386

static char     memory [65533];



void

test_size (size_t const size)

{

  static unsigned long int      std_cache (0);

  static unsigned long int      svg_cache (0);



  struct mallinfo const         mi_1 (mallinfo ;

  std::string * const           std_s (new std::string (memory, size;

  struct mallinfo const         mi_2 (mallinfo ;

  svg::string * const           svg_s (new svg::string (memory, size;

  struct mallinfo const         mi_3 (mallinfo ;

  unsigned long int const       std_m (mi_2.uordblks - mi_1.uordblks + std_cache);

  unsigned long int const       svg_m (mi_3.uordblks - mi_2.uordblks + svg_cache);

  unsigned long int const       std_u std_s->capacity  * 1000) / std_m);

//  double const                        std_u double) std_s->capacity  / std_m);

  unsigned long int const       svg_u svg_s->capacity  * 1000) / svg_m);

//  double const                        svg_u double) svg_s->capacity  / svg_m);



  static unsigned long int      old_std_m (0);

  static unsigned long int      old_svg_m (0);

  static std::string::size_type old_std_c (0);

  static svg::string::size_type old_svg_c (0);

  static bool                   print_dots (true);



  if old_std_m == std_m) and

      (old_svg_m == svg_m) and

      (old_std_c == std_s->capacity  and

      (old_svg_c == svg_s->capacity 

  {

    if (print_dots)

    {

      std::cout << "...\n";

      print_dots = false;

    }

  }

  else

  {

    old_std_m = std_m;

    old_svg_m = svg_m;

    old_std_c = std_s->capacity ;

    old_svg_c = svg_s->capacity ;

    print_dots = true;

    std::cout << std::setfill ('0') << size << '\t' << std_m

              << (std_m > svg_m ? "\t>\t" : "\t<=\t")

              << svg_m << '\t'

// double иногда после запятой выводится с большим числом знаков, чем std::setprecision (я так понял , STL пытается вывести хотябы две значащие цифры)

//            << std::setw (7) << std::setprecision (5) << (double) size / std_m << '\t'

              << '.' << std::setw (6) << uint64_t) size * 1000000) / std_m << '\t'

//            << std::setw (7) << std::setprecision (5) << (double) size / svg_m << '\t'

              << '.' << std::setw (6) << uint64_t) size * 1000000) / svg_m << '\t'

              << std_s->capacity  << '\t'

              << svg_s->capacity  << '\t'

//            << std::setw (4) << std::setprecision (2) << std_u << (std_u > svg_u ? "\t>  " : "\t<= ")

              << '.' << std::setw (3) << std_u << (std_u > svg_u ? "\t>  " : "\t<= ")

//            << std::setw (4) << std::setprecision (2) << svg_u << '\t' << std_cache << '\t' << svg_cache << '\n';

              << '.' << std::setw (3) << svg_u << '\t' << std_cache << '\t' << svg_cache << '\n';

  }



  struct mallinfo const         mi_4 (mallinfo ;

  delete svg_s;

  struct mallinfo const         mi_5 (mallinfo ;

  delete std_s;

  struct mallinfo const         mi_6 (mallinfo ;



//  if (mi_4.uordblks != mi_3.uordblks) // Всегда mi_4.uordblks == mi_3.uordblks

//    std::cout << "std::cout memory disturbance: " << mi_4.uordblks - mi_3.uordblks << '\n';

  svg_cache = svg_m - (mi_4.uordblks - mi_5.uordblks); // Всё равно равен нулю, я кеш себе не отвожу, но для симметричности посчитаем

  std_cache = std_m - (mi_5.uordblks - mi_6.uordblks);

}

  std::cout << "sizeof (std::string) = " << sizeof (std::string)

            << "; sizeof (svg::string) = " << sizeof (svg::string) << '\n';



  std::cout << "length\tstd mem\tV\tsvg mem\tstd l/m\tsvg l/m\tstd cap\tsvg cap\tstd c/m\tsvg c/m\tstd cch\tsvg cch\n";



  for (unsigned int i (0); i <= sizeof (memory); ++i)

    test_size (i);

 

Olyalyau

Ну так отправь им свой код, помоги сообществу.

kokoc88

Дополнительный плюс вынесения этого if наружу — снижение icache pressure и увеличение локальности кода

Ivan8209

Ivan8209

Olyalyau

Настоящие программисты документацию не читают.

Olyalyau

Где-то когда-то уже писали про то, сколько времени программисты тратят на ненужную оптимизацию... Забыл ссылку

Olyalyau

> Вот это уже чистой воды arcane knowledge (см. intel optimization guide).
Что если спрашивает про ARM?

Olyalyau

К сведению, изначально вопрос поставлен так:
"[C] Как считать с клавиатуры целое число?"

Olyalyau

> Дополнительный плюс вынесения этого if наружу —
> снижение icache pressure и увеличение локальности кода
Ты думаешь, это имеет значение?

Olyalyau

> Правильный ответ — это который?
man strtod

SYNOPSIS
#include <stdlib.h>
double strtod(const char *nptr, char **endptr);
float strtof(const char *nptr, char **endptr);
long double strtold(const char *nptr, char **endptr);
DESCRIPTION
The strtod, strtof, and strtold functions convert the initial por-
tion of the string pointed to by nptr to double, float, and long
double representation, respectively.

К сведению, изначально вопрос поставлен так:
"[C] Как считать с клавиатуры целое число?"

man strtod

Olyalyau

>К примеру, моя реализация std::string обгоняет стандартную из поставки GCC примерно в 1.5 раза
а sizeof(string) твой во сколько раз больше жэцэцэшного? в 5?

kokoc88

Для кого как, а для меня оптимизация означает возможность ставиться не на $100k-сервак, а на жалкий
$30k.

Olyalyau

Слово "ненужную" ты не прочитал. Такая оптимизация явно не потянет на 100к-30к, она и на доллар-то не вытянет. А время разработчик тратит.

kokoc88

Как показывает опыт, большая оптимизация имеет тенденцию складываться из маленьких

Ivan8209

decimal 100 1000 1000 * * dup . allocate throw free throw 100000000  ok

Ivan8209

Olyalyau

Мне опыт пока что показывает только одно: большая оптимизация - это смена n*n на log n.

Olyalyau

> У меня дома есть большой шарик от подшипника,
> я готов положить его на кнопку "0" и уехать на работу.
И?
Приличный драйвер должен в таком случае заблокировать ввод,
поскольку устройство явно неисправно.
> особенно есть сделать автоповтор быстрым
А если автоповтор отменить?
Это, кстати, более верное решение при вводе с клавиатуры.
Автоповтор нужен для координатного устройства,
типа джойстика или части клавиатуры со стрелками,
на все клавиши автоповтор действовать, по-хорошему, не должен.

> Как там твой "fgetln" на это отреагирует?
Так, как написано в документации.
Кстати, мог бы сам проверить

HISTORY

     The fgetln function first appeared in 4.4BSD.

  

Olyalyau

> Как ты верно заметил,
...
> клавиатура не есть string , так что
Про написание драйвера я уже сказал: данных недостаточно.

mira-bella

Ты это о чём? Исходная задача была про считывание с клавиатуры числа на языке C. Никто не просил тебя писать драйвер клавиатуры

kokoc88

Твоя позиция понятна. Видимо, разработчики std::string такие же небрежные люди, как и ты — и их реализация работает в 1.5 раза медленнее моей svg::string. Видимо оптимизация константы им, так же как и тебе, показалась несущественной.

Codcod

Ivan8209

Olyalyau

И?
В языке С много чего нет, в частности нет средств работы с пространством адресов ввода-вывода,
ибо нет такого у мотороллеров. Про исходные ПДП не помню, но и там, думаю, нет.

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

Olyalyau

Я не знаю, что там у разработчиков std::string, а что там у тебя. Мне вообще плевать, за 0.0001 секунду моя программа ответит или за 0.00015. На реальных проектах большой оптимизации такими мелочами ты не получишь. Зато код они засорят будь здоров.

Впрочем, на мой изначальный вопрос ты вполне ответил.

Ivan8209

kokoc88

> А я всё жду от тебя хоть какого-нибудь решения поставленной задачи.
> Именно решения — программы, а не отсылок к man.
Конкретные ответы я даю только на конкретные вопросы.

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

Ivan8209

kokoc88

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

К сведению, в настоящее время я читаю с клавиатуры с помощью define-key,
global-set-key и read-string. Тебе этого, как показал опыт, не осилить.

Olyalyau

Читать: Кохтпа не умеет программировать. Он может только всех вокруг опускать.