Проблема с написанием программы. Си.

[Chelovekpredel]

#include "stdafx.h"
#include <stdio.h>

int any(char s1[],char s2[]);

int any(char s1[],char s2[])
{
	int k=1,i=0,j=0,p;
	while ((k>0) && (s1[i]!='\0'))
	{
		while ((s2[j]!='\0') && (k>0))
		{
			if (s1[i]==s2[j])
			{
				k=0;
			}
		}
		++i;
	}
	if (k==0)
		return (i-1);
	else
		return -1;
}

int _tmain(int argc, _TCHAR* argv[])
{
	char s1[1000],s2[1000],c;
	int i=0,j=0;
	while ((c=getchar())!='\n')
		s1[i++]=c;
	s1[i]='\0';
	while ((c=getchar())!='\n')
		s2[j++]=c;
	s2[j]='\0';
	printf("%d\n",any(s1,s2));
	getchar();
	return 0;
}

Только начал учить си и столкнулся с небольшой проблемой.
Задача заключается в следующем:
Нужно вывести позицию в строке s1, элемент которой впервые совпадает с любым элементом строки s2. Если такой позиции нет, то вывести -1.

Например: abcdefgh
bsufew
Ответ: 2
Мой код впринципе компилируется нормально, но ничего не выводит.
То есть, данные вводятся, а результата нет.
Прошу помочь с избавлением от этой неприятности и если нужно переписать код на правильный лад. Заранее спасибо.

[Stilet]

Упростим?:

	char *s="abcdefgh";
	char *d="sufew";
	int i=0;bool b=false;
	for(;!b && s[i];i++)for(int k=0;!b && d[k];k++)b=s[i]==d[k];
	i=(b)?i:-1;
	printf("%d",i);getchar();

Ась?

[MooNDeaR]

for(;!b && s[i];i++)for(int k=0;!b && d[k];k++)b=s[i]==d[k];

Какая жуткая конструкция...

По-моему, вот так будет чуть яснее:

#include <stdio.h>

int main()
{
	char* str1 = "Hello";
	char* str2 = "Wild";
	int   pos = -1;
	int   i = 0;

	while(*str1 && *str2)  //Проверка на конец строки
	{
		if(*str1++ == *str2++) //Проверка на идентичность + автоматическое итерирование
		{
			pos = i + 1;
			break;
		}
		i++;
	}

	printf("%d",pos);
	return 0;
}

[Stilet]

Какая жуткая конструкция...

Зато работает

По-моему, вот так будет чуть яснее:

Автору не это нужно. Ему подавай аналог регулярки - найти первое вхождение любого символа из списка (списком является вторая строка)

[MooNDeaR]

Ему подавай аналог регулярки - найти первое вхождение любого символа из списка (списком является вторая строка)

Да действительно, что-то криво задание читаю.

Вообще ваша конструкция правильная и даже хорошо работающая. Только вот, по-моему, уж очень вульгарно двойной цикл одной строкой писать.

[8Observer8]

to Chelovekpredel

1) При создании проекта ставьте галочку "Пустой проект"

2) Реализация функции после main. Перед main прототип.

3) getchar() не нужен. При запуске нажимайте Ctrl+F5 (или тоже самое - "Отладка" -> "Запуск без отладки"). Либо
ставьте точку останова.

4) В Меню: "Проект" -> "Свойства" -> слева "Общие" -> "Набор знаков" -> выбрать "Использовать многобайтовую кодировку"

5) Главный файл называем (для с программ) main.c (так будет запускаться сишный компилятор)

6) Константы через define (или const если компилятор поддерживает)

7) Ваш код ошибочный. Программа попадает в бесконечный цикл (вложенный while). Вы всего-то
забыли инкрементировать j.

    while ((k>0) && (s1[i]!='\0'))
    {
        while ((s2[j]!='\0') && (k>0))
        {
            if (s1[i]==s2[j])
            {
                k=0;
            }
            j++;
        }
        ++i;
    }

[Chelovekpredel]

Спасибо огромное. Всего-то забыл инкрементировать)

[Chelovekpredel]

Хотя нет. Теперь всегда -1 выводит.

// номер позиции.строки.cpp: определяет точку входа для консольного приложения.
//

#include "stdafx.h"
#include <stdio.h>

int any(char s1[],char s2[]);

int any(char s1[],char s2[])
{
	int k=1,i=0,j=0,p;
	while ((k>0) && (s1[i]!='\0'))
	{
		while ((s2[j]!='\0') && (k>0))
		{
			if (s1[i]==s2[j])
			{
				k=0;
			}
			++j;
		}
		++i;
	}
	if (k==0)
		return (i-1);
	else
		return -1;
}

int _tmain(int argc, _TCHAR* argv[])
{
	char s1[1000],s2[1000],c;
	int i=0,j=0;
	while ((c=getchar())!='\n')
		s1[i++]=c;
	s1[i]='\0';
	while ((c=getchar())!='\n')
		s2[j++]=c;
	s2[j]='\0';
	printf("%d\n",any(s1,s2));
	getchar();
	return 0;
}

[8Observer8]

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

Хотя нет. Теперь всегда -1 выводит.

Я вчера торопился и не заметил. А вообще говоря и вам минус, так как вы не умеете пользоваться отладчиком (ставить точки останова и выполнять пошагам). Пока не научитесь им пользоваться - будете постоянно спотыкаться.

По вашему коду: видите, в внутреннем while предусмотрен только один проход по строке (j только увеличивается).Нужно j сбрасывать, когда мы прошли по второй строке:

        while ((s2[j]!='\0') && (k>0))
        {
            if (s1[i]==s2[j])
            {
                k=0;
            }
            j++;
        }
        j = 0;
        ++i;

[Stilet]

уж очень вульгарно двойной цикл одной строкой писать.

Ай лайк ит
Что поделать?