Проблемы с функцией gets (С)

[spirit_trapper]

Есть такое задание:
Дана строка. Преобразовать ее следующим образом: если символ «*» в ней отсутствует, то оставить ее без изменения, иначе – заменить на символ «–» число символов после первого вхождения «*».

Я попытался сделать вот так:

#include<stdio.h>
#include<ncurses.h>

int Star(char str[]);
void Change(char s[],char z[]);

int main ()
{
char str[1000],z[1000];

clear;
printf ("Введите строку с символом '*' или без такового (на ваш выбор):\n");
gets(str);
Change (str,z);
printf(z);
return 0;
getch();
}

int Star (char str[])
{
int n;
for (;str[n]=='*');
return n;
}

void Change (char s[],char z[])
{
int i,k;
k=Star(z);

for (;i>k;i++)
{
if (s[i]==k) z[i]='-';
else z[i]=s[i];
}
}

Однако g++ ругается вот так:
laba_3.cpp: In function ‘int main()’:
laba_3.cpp:15: warning: format not a string literal and no format arguments
/tmp/ccZqFiL6.o: In function `main':
laba_3.cpp (.text+0x32): warning: the `gets' function is dangerous and should not be used.


Не понятно следующее:
1. Почему printf (z); некорректно?
2. Почему компилятор пишет что gets опасна, и если причины есть, то какая может быть альтернатива gets?
3. Как вам вообще такой код? Что следует поправить? Потому что я чувствую что тут далеко не 2 ошибки.

Помогите, пожалуиста.

Заранее спасибо.

[netrino]

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

3. Как вам вообще такой код? Что следует поправить? Потому что я чувствую что тут далеко не 2 ошибки.

У Вас на удивление хорошая интуиция
Ошибок действительно достаточно.
Насчёт 1-го и 2-го вопроса:
1) Он(компилятор) говорит, что в качестве параметра в printf передаётся не строковый литерал, содержащий строку форматирования, а переменная, в которой неизвестно что, более того, за этой переменной не следуют другие параметры. В общем не стоит особо обращать на это внимание, это лишь предупреждение о некорректно использованной функции.

2) gets действительно опасна, потому как не предлагает никаких средств для ограничения ввода, в силу чего может быть "переполнение буфера". Можете попробовать fgets вместо gets.

Немного изменил Вашу программу, будут вопросы - спрашивайте

#include<stdio.h>
#include<ncurses.h>

int  Star(const char* s);
char* Change(char* s);

int main ()
{
	char str[1000];

	clear();
	printf ("Введите строку с символом '*' или без такового (на ваш выбор):\n");
	fgets(str, 1000, stdin);
	printf( Change (str) );
	getch();

	return 0;
}

int Star(const char* s)
{
	int i;

	for(i = 0; s[i] != 0; i++)
		if(s[i] == '*')
			break;

	if(s[i] == '\0')
		i = -1;

	return i;
}

char* Change (char* s)
{
	int i;
	int pos = Star(s);
	
	if(pos != -1)
		for( i = pos; s[i] != 0; i++ )
			s[i] = '-';

	return s;
}

[spirit_trapper]

netrino

Спасибо огромное. Буду разбирать код по кусочкам. Вопросы будут.