Посмотрел код, стало страшно. Вот мой вариант (написан в лоб, по факту можно было бы вынести в отдельные функции)
#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include <ctype.h>
int dN(int N)
{
srand(time(NULL));
return rand()%N + 1;
}
void MdN(int M, int N)
{
printf("==== MdN(%d,%d)===\n", M, N); // это для отладки
int s = 0, tmp;
if (N == 0) // защита, Ваш код падал при нуле
return;
while (M>0)
{
s += tmp = dN(N);
printf("+%d", tmp);
M--;
}
printf("=%d;\r \n", s);
}
int main()
{
int m = 0;
int n = 0;
while (1)
{
printf("Enter two numbers (MdN) or zero to exit\n");
int r = scanf("%d", &m); // ввод первого
if (r != 1) {
puts("first number is not numer\n");
break;
}
if (m == 0) {
// пользователь ничего не ввел либо ввел один ноль
puts("exit\n");
break;
}
// удалось прочитать только одно число. Попробуем прочитать ещё одно
r = scanf("%d", &n);
if (r != 1) {
puts("ups, second number is not number\n");
break;
}
MdN(m,n);
}
printf("end\n");
return 0;
}
scanf возвращает результат и корректно будет проверять, что он возвратил. По факту - возвращается количество переменных, которые удалось вычитать. 0 - если ничего не прочитали. Те переменные, которые удалось прочитать, будут иметь значения. Если случится что то странное, scanf
возвратит -1 (по честному - константу EOF
).
Можно делать ввод в виде sscanf("%d %d", &m, &n)
. Но если пользователь введет только один ноль, то код будет ждать, пока будет введет ещё один символ. Поэтому я разбил ввод на две части.
Как ни странно, в обеих средах код работал неверно.
логично, так как с 99% вероятностью компилятор один и тот же - gcc.
Для Вашего случая больша подошла бы getchar() с проверкой введенного значения
очень вредный совет. С int
еще проверку сделаете, а с float
?
Почему же оно работает в Visual studio? есть подозрение, что там sscanf("%d %d", &m, &n)
при вводе одно числа и перевода строки возвращает 1 и заполненное m
.
upd
Учитывая последние изменения в примере в вопросе. Если Вы только начинаете изучать С и делаете много ошибок (зачастую элементарных), то нужно компилятору намекнуть, что нужно быть строже. Да, он будет придираться по пустякам, но на первое время самое оно.
Я рекомендую такой наборчик
-std=c99 -pedantic -Wall -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
Как использовать:
- если это в консоли, то просто `gcc имяфайла.с -std=c99 -pedantic -Wall -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
- если это monodevelop. В дереве "решений" слева выделяете свой проект, кликаете правой, "свойства", в окне "свойства проекта" находите "Генерация кода", выбираете вкладку с таким же именем и внизу видите окно "Extra Compiler Options", куда и вставляете вышеприведенный список.
- В QtCreator нужно открыть pro файл и добавить туда строку вида
QMAKE_CFLAGS = -std=c99 -pedantic -Wall -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
Что значат эти флаги?
std=c99
следовать стандарту с99. Сейчас есть и другие, но этот более промышленный (но это мое мнение)
pedantic
быть попридирчивей
Wall
включает большой набор проверок. К примеру придерется к коду вида if (a = 0)
(некоторые любители любят это лечить переворачиванием условия, а некоторые используют для сознательного запутывания). Также эта опция включает -Wformat
, который заставляет компилятор пересчитывать параметры у функций вида printf/scanf
и ругаться, если их не хватает или типы не совпадают.
Wshadow
придираться, когда в функции используется перекрывание переменных. К примеру, внутри функции объявляется переменная, имя которой совпадает с параметром. В некоторых случаях может приводить к веселым ошибкам.
Wpointer-arith
в новых версиях это уже включается в -pedantic
. Придирается, когда с химичат с арифметикой указателей.
Wcast-qual
придирается, когда делается приведение типов, которое потенциально опасное. это часто вылазит у новичков, когда они осваивают работу со строками в стиле с.
Wstrict-prototypes
придерется к int main()
, так как желательно писать int main(int argc, char ** argv)
.
Wmissing-prototypes
потребует определять прототипы используемых функций. То есть, добавлять строки вида int dN(int N);
.
Конечно, некоторые найдут эти вещи абсурдными и мешающими (особенно для "продвинутых программистов"). Но я в своих проектах время от времени включаю подобные наборы и смотрю - не вылезло ли глупостей - опечатку то сделать легко.
На первый порах рекомендую все эти замечания компилятора устранять.
%d
в соответствующийprintf()
. Также советую завершать вывод символом '\\n'.