Необработанное исключение по адресу при вводе даты

@FokycHuk · Регистрация: 04.04.2016

имеется код

C++

#include <stdio.h>
#include <iostream>
#include <conio.h> 
using namespace std;
 
 
int main() 
{ 
int day, year; 
char month[10]; 
printf_s("Vvedite day month year\n"); 
scanf_s("%d %s %d", &day, &month, &year); 
printf_s("Segodny %d %s %d", day, month, year);  
return 0; 
}

по заданию:
Пусть формат ввода даты с экрана имеет вод: 5 Septemba 2005
Как следует модифицировать приведенную выше программу для ввода даты?

у моего соседа по комнате данный код работает, у меня же приходится добавлять "_s" чтобы он хотябы запускался
все идет до тех пор пока я не ввожу "Septemba" то есть пока не забиваю масив.
далее происходит следующее
ошибок в обычном коде не выдает, но ссылается на ассемблер

@pav1uxa · 04.04.2016, 21:48

Сообщение от FokycHuk

у моего соседа по комнате данный код работает, у меня же приходится добавлять "_s"

Возьмите у соседа по комнате нормальную среду для программирования с его компиляторами)

@nikolay1982 · 04.04.2016, 22:02

Сообщение от FokycHuk

scanf_s("%d %s %d", &day, &month, &year);

Дело в том, что у вам массив символов, имя(month) которого на языке C можно рассматривать как указатель на первый элемент массива.
При передачи в функцию, именно это и работает, передается как указатель. Поэтому не стоит дополнительно брать указатель на указатель на первый элемент массива.
Поэтому правильно так:

C++

1	scanf("%d %s %d", &day, month, &year);

Или же так:

C++

1	scanf("%d %s %d", &day, &month[0], &year);

Второй вариант &month[0] более знаком тем, кто привык использовать массивы в Pascal в качестве указателей(там было бы @month[0]).

@djair · 04.04.2016, 23:26

C++

#include <stdio.h>
#include <iostream>
#include <conio.h> 
using namespace std;
 
 
int main()
{
    int day, year;
    char month[10];
    cout << "Vvedite day month year" << endl;
    cin >> day;
    cin >> month;
    cin >> year;
    cout << day << " " << month << " " << year << endl;
    return 0;
}

а так не проще сделать?

@FokycHuk · 05.04.2016, 00:28 **[ТС]**

первый вариант уже перепробовал нет никаких изменений, второй проверил, так же без изменений

Добавлено через 48 секунд
целевую функцию надо оставить

Добавлено через 5 минут

Сообщение от nikolay1982

Дело в том, что у вам массив символов, имя(month) которого на языке C можно рассматривать как указатель на первый элемент массива.
При передачи в функцию, именно это и работает, передается как указатель. Поэтому не стоит дополнительно брать указатель на указатель на первый элемент массива.
Поэтому правильно так:

C++

1	scanf("%d %s %d", &day, month, &year);

Или же так:

C++

1	scanf("%d %s %d", &day, &month[0], &year);

Второй вариант &month[0] более знаком тем, кто привык использовать массивы в Pascal в качестве указателей(там было бы @month[0]).

так вот оба способа испробованны а результат тотже, все таже ошибка и отправление к языку на ассемблере

Добавлено через 1 минуту

Сообщение от pav1uxa

Возьмите у соседа по комнате нормальную среду для программирования с его компиляторами)

дело в том что мне охота разобраться, а иначе в чем смысл)

Добавлено через 8 минут

Сообщение от djair

C++

#include <stdio.h>
#include <iostream>
#include <conio.h> 
using namespace std;
 
 
int main()
{
    int day, year;
    char month[10];
    cout << "Vvedite day month year" << endl;
    cin >> day;
    cin >> month;
    cin >> year;
    cout << day << " " << month << " " << year << endl;
    return 0;
}

а так не проще сделать?

надо решить именно через этот способ

@DrOffset · 05.04.2016, 08:41

FokycHuk, читаем здесь: https://msdn.microsoft.com/en-... 768et.aspx

Unlike scanf and wscanf, scanf_s and wscanf_s require the buffer size to be specified for all input parameters of type c, C, s, S, or string control sets that are enclosed in []. The buffer size in characters is passed as an additional parameter immediately following the pointer to the buffer or variable.

C++

#include <stdio.h>
#include <iostream>
 
int main() 
{ 
    int day, year; 
    char month[10]; 
    printf_s("Vvedite day month year\n"); 
    scanf_s("%d %9s %d", &day, month, sizeof(month), &year); 
    printf_s("Segodny %d %s %d", day, month, year);  
    return 0; 
}

Естественно, замечание из 3го поста это никак не отменяет. Оно абсолютно правомерено.

@Izual · 05.04.2016, 09:41

Сообщение от FokycHuk

Septemba

=) Это новый язык, как английский только ещё хуже..

+ к предыдущим постам, и разширь char массив, т.к. скорее всего ещё и мало места, т.к. если ты вводиш реальное слово "September", то это 9 символов +1 символ для \0, а если ты ошибаешся с вводом на 1 элемент, то вылетает, брать надо с запасом, а у тебя запаса нет.

@FokycHuk · 05.04.2016, 15:44 **[ТС]**

Сообщение от DrOffset

FokycHuk, читаем здесь: https://msdn.microsoft.com/en-... 768et.aspx

C++

#include <stdio.h>
#include <iostream>
 
int main() 
{ 
    int day, year; 
    char month[10]; 
    printf_s("Vvedite day month year\n"); 
    scanf_s("%d %9s %d", &day, month, sizeof(month), &year); 
    printf_s("Segodny %d %s %d", day, month, year);  
    return 0; 
}

Естественно, замечание из 3го поста это никак не отменяет. Оно абсолютно правомерено.

спасибо, возврат размера помог, но всеже остался один вопрос в чем же тогда отличия почему мне пришлось писать именно так и соседу, именно с тем же кодом это не понадобилось? (у него VS 2010 у меня VS 2012 )

@DrOffset · 05.04.2016, 16:36

Сообщение от FokycHuk

соседу, именно с тем же кодом это не понадобилось?

Ему "повезло".
Если у него был именно такой код (со scanf_s и без указания размера), то он у него точно так же некорректен, как был у тебя. Ошибка совсем необязатально должна выражаться в падении программы. Ты поймешь это на своей шкуре, если будешь потом работать программистом.
В общем, ссылку я выше дал, там все написано. И про отличия scanf от scanf_s, и зачем они нужны, и как правильно ими пользоваться. Если правда хочешь разобраться - читай, вникай.

Необработанное исключение по адресу при вводе даты

Решение

Решение

@pav1uxa 1932 / 1761 / 822 Регистрация: 23.01.2014 Сообщений: 6,205
	04.04.2016, 21:48	2
	Сообщение от FokycHuk у моего соседа по комнате данный код работает, у меня же приходится добавлять "_s" Возьмите у соседа по комнате нормальную среду для программирования с его компиляторами) 0

@Izual 143 / 122 / 21 Регистрация: 13.11.2012 Сообщений: 1,564
	05.04.2016, 09:41	7
	Сообщение от FokycHuk Septemba =) Это новый язык, как английский только ещё хуже.. + к предыдущим постам, и разширь char массив, т.к. скорее всего ещё и мало места, т.к. если ты вводиш реальное слово "September", то это 9 символов +1 символ для \0, а если ты ошибаешся с вводом на 1 элемент, то вылетает, брать надо с запасом, а у тебя запаса нет. 0

@DrOffset 13712 / 7335 / 1751 Регистрация: 30.01.2014 Сообщений: 12,306
	05.04.2016, 16:36	9
	Сообщение от FokycHuk соседу, именно с тем же кодом это не понадобилось? Ему "повезло". Если у него был именно такой код (со scanf_s и без указания размера), то он у него точно так же некорректен, как был у тебя. Ошибка совсем необязатально должна выражаться в падении программы. Ты поймешь это на своей шкуре, если будешь потом работать программистом. В общем, ссылку я выше дал, там все написано. И про отличия scanf от scanf_s, и зачем они нужны, и как правильно ими пользоваться. Если правда хочешь разобраться - читай, вникай. 0