Написание программы

[KOzh1r]

Доброго времени суток, гуру программирования!
Недавно стал открывать для себя увлекательный мир программирования и стал читать "всякие штуки-дрюки" (начал изучать основы программирования на языке С#). Очень интересное и увлекательное это дело! Но произошла со мной не очень классная ситуация:
Влез Я проходить один тест.. Очень такой интересный тест про программирование. Несколько вариантов прохождения каждого этапа - от простого, до сложного. И на одном из таких этапов в моё обучение решил вмешаться младший сын, трёхлетний мальчуган Алёша, и с удовольствием ткнул в красивую кнопочку с надписью "Сложный вариант"... 
Я читаю задание:
Поздравляем, Вы решили опробовать себя в сложном задании!(тут Я иронически усмехнулся) Вам необходимо написать программу на одном из популярных языков (Java, Python, C/C++, Javascript), которая будет выводить марки автомобилей в зависимости от запроса. Например, на вход программе подаётся запрос "выведи российские модели" или "перечисли японские марки" и программа должна их вывести. Программа должна понимать небольшие опечатки и сокращения (например, "автомобили" и "авто") и не учитывать регистр букв.
 
Представление об этом Я имею какое-то, но как это реализовать в виде программного кода понятия не имею  ,поэтому прошу вас, о, величайшие умы о содействии! Пройти тест надо до следующей недели, а мой уровень знаний в данный момент - определить разницу в условных операторах (величайшее достижение, не каждый сможет )
Подскажите пожалуйста, как мне быть? Информацию ищу уже три дня, безрезультатно.. Спасибо!

[Алексей++]

KOzh1r, что такого ценного дадут по результатам этого теста? А написать придётся небольшой яндекс + ИИ, судя по всему.

Для обучения основам программирования тест так себе 

[Olej]

(начал изучать основы программирования на языке С#).

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

1. Прежде всего - забыть о языке C# как "основах программирования"...
Выберите что-то приличное, дающее идеологию в мозг ... по мере предпочтительности: Python, Go, C, C++, ... Java на худой конец.

2. Далее...
Задача ваша - не сложная, но она - громоздкая, там много рутины... Никто вам её в готовом виде не напишет и нигде вы в точном виде её не найдёте.
Но, хуже того, формулировка задачи - говно  ... такой вот уровень, значит, спрашивающих ... и курсов их вообще:
- задача должна знать только про автомобили? или ещё какие-то предметы объектного мира? или вообще все?
- программа должна анализировать все вариации естественного языка на котором задан вопрос? или только э-э-э-э  ... "небольшие опечатки и сокращения"?
- а в чём и до какой степени могут состоять сокращения? "авто" - это ещё автомобиль? или уже и автомат Калашникова?    

В общем, бросайте вы такие говённые курсы, и возвращайтесь к п.1.

[Алексей++]

Olej, ну, скажем, шарп для начинающих норм в плане стрельбы в ступни (на этом всё)

а вот C из списка я бы убрал точно (и не только для начинающих). C++ для начинающих - тоже под вопросом

[KOzh1r]

KOzh1r, что такого ценного дадут по результатам этого теста?

Ценного не дадут ничего, но закончить курс хочу по причине того, что не люблю незавершенные дела..

1. Прежде всего - забыть о языке C# как "основах программирования"...
Выберите что-то приличное, дающее идеологию в мозг ... по мере предпочтительности: Python, Go, C, C++, ... Java на худой конец.

Вообще, Я планирую заняться питоном, но после того, как освою "азы", а их как раз "преподают" на С# как на" самом лёгком языке для новичков". После этого курса займусь плотно Python.
Далее..
Задача стоит написать программу, которая ответит на два конкретных вопроса про российские и японские автомобили. Вывести конкретный список на не совсем конкретный (с опечатками) запрос.

[Алексей++]

KOzh1r, что они там понимают под опечатками, интересно. Например, можно для каждого слова вывести некий критерий похожести в зависимости от использованных букв и их последовательности. Если критерий совпадает, скажем, процентов на 90 - то слово считается похожим. Порядок слов, опять же, важен или нет. После этого по набору слов тебе нужно понять фразу (запрос от юзера). По фразе выполнить "задание" - вывести нужный список автомобилей

[KOzh1r]

KOzh1r, что они там понимают под опечатками, интересно. Например, можно для каждого слова вывести некий критерий похожести в зависимости от использованных букв и их последовательности. Если критерий совпадает, скажем, процентов на 90 - то слово считается похожим. Порядок слов, опять же, важен или нет. После этого по набору слов тебе нужно понять фразу (запрос от юзера). По фразе выполнить "задание" - вывести нужный список автомобилей

Я думаю сделать что-то простое, по ключевому слову "российские" или "японские". Подскажите пожалуйста, как это реализовать?

[Olej]

а вот C из списка я бы убрал точно (и не только для начинающих).

А вот без C любому, кто собирается работать с Linux, да и с любым другим UNIX like - лучше сразу идти записываться в управдомы... (ну, можете, по желанию, заменить на: "в WEB программисты"  ).

[Алексей++]

Olej, это всё специнструменты, причём обычно для ограниченного в ресурсе железа. А тут речь же про начинающих )

KOzh1r, видимо, перебрать все варианты с опечатками. Для простоты весь текст перевести в один регистр, например нижний

[KOzh1r]

KOzh1r, видимо, перебрать все варианты с опечатками. Для простоты весь текст перевести в один регистр, например нижний

Подскажите, как это сделать на языке С# (это единственный, на котором Я хоть что-то могу написать). Думаю разделить введённый запрос на отдельные слова в переменных и если в нём есть "российские" или "русские" или "рАссийские", то выводить будет определённый, заранее написанный список.

[Алексей++]

KOzh1r, я не использую шарп, поэтому тут вопрос не ко мне )

[Finch]

а вот C из списка я бы убрал точно (и не только для начинающих).

А вот без C любому, кто собирается работать с Linux, да и с любым другим UNIX like - лучше сразу идти записываться в управдомы... (ну, можете, по желанию, заменить на: "в WEB программисты"  ).

Я конечно понимаю Ваше стремление похоливарить. Но Ваше утверждение спорное. Для обычного пользователя не нужно знать С, чтобы полноценно работать на ПК. Тут не важно, какая именно стоит современная ОС. В 70 -х годах прошлого века было модно ставить в качестве старта например Basic. И там да, нужно было, хотя бы знать азы Бэйсика для полноценной работы. Даже чтобы администрировать Linux не обязательно знать С. Вполне достаточно bash скрипты. Для более продвинутого python, perl, ruby, java на усмотрение пользователя.

KOzh1r, Данная задача не для новичка. Как правило подобные задачи решают через нейроные сети.
Чтобы разделить строку на отдельные слова в классе string должна быть функция split
Чтобы перевести символы в нижний или верхний регистр: ToLower или ToUpper
Смотри Help как правильно пользоваться ими.

[Михалыч]

KOzh1r, Данная задача не для новичка. Как правило подобные задачи решают через нейроные сети.
Чтобы разделить строку на отдельные слова в классе string должна быть функция split
Чтобы перевести символы в нижний или верхний регистр: ToLower или ToUpper
Смотри Help как правильно пользоваться ими.

Ой, да бросьте, господа...  
Как же мы раньше то начинали учиться программировать без ИИ и нейронных сетей?  
Может не надо для начинающего таких усложнений?
Я себе вижу эту задачу посложнее, разумеется, чем "Hello World!" но не настолько, чтобы прям нейронные сети...  
Разумное количество шаблонных строк запросов с вариантами оЧепяток, и сообщение типа "Ахтунг! Ай донт андерстенд ю!" когда упоротый пользователь энный раз будет писать галиматью. Вот и весь интеллект и нейронная сеть для самого начала обучения 
Не забывайте, это же курс начинающего... Стало быть и к заданию надо относиться по возможности буквально. Есть два примера запросов? Есть. Реализуем их, разбавляя щедро собственной фантазией на тему содержания. А там и нейронные сети со временем подтянутся. На этой задаче можно долго и упорно учиться, постоянно модифицируя и усложняя ее решение.

[Алексей++]

Михалыч, Finch, можно даже регулярное выражение забацать для всех опечаток, наверное

[Михалыч]

Можно, конечно. 
Вопрос, нужно ли для первой работающей итерации программы? Может и нет.

[Finch]

Михалыч, Тут такая проблема, что в слове "Японский" например можно кучу разных вариантов ошибки совершить. Вплоть до "Йапона". замучаетесь вручную все эти варианты вписывать в код программы. Хотя я один раз видел, как вручную писали код ИИ игры "крестики нолики". 

[Михалыч]

А кто сказал, что надо все варианты?

[darkelf]

Можно написать отдельный файл с синонимами, что Иппонский, Йапона, Япон и т.д., например в виде ini-файла:
[японские]
иппонские
ипонские
йапона
япон

[автомобили]
авто
тазы
модели

[российские]
русские
рассейские

[выведи]
перечисли
расскажи про
покажи

Ну а дальше соорудить код, который пытается все эти синонимы "канонизировать" - сначала переведёт в один регистр, например нижний, и затем из всего многообразия переведёт в одну общую форму путём замены того, что после [] в то, что внутри []. Ну а дальше, можно вообще сделать файл выведи_японские_автомобили.txt следующего содержания:
Mazda
Honda
Toyota

и файл выведи_российские_автомобили.txt
ВАЗ
ГАЗ

Ну а делее - получил из многообразия каноническую строку, заменил в ней пробелы на символ "_", добавил в конце .txt, и попытался из этого файла вывести всё содержимое, если такого файла нет - выдал 404, ну или ещё что-то осмысленное. В итоге в качестве запроса будет ожидаться форма "глагол прилагательное существительное". Базу знаний можно ещё более извращённым способом структурировать - например убрать хранение глагола в файле, а "существительные" сделать каталогами:
автомобили/японские.txt
автомобили/российские.txt
пряники/тульские.txt
пряники/сибирские.txt

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

Главное, чтобы не было двоякого трактования, чтобы не оказалось, что "модели" это не только автомобили, но и например девушки, показывающие моду, и модели телефонов - т.е. определённая узкая специализация.

[Finch]

А где сказано, какие именно опписки могут быть?

[Михалыч]

А где сказано, какие именно опписки могут быть?

Да нигде, судя по всему... Такое задание, которое привел топиккастер, можно формировать только для совсем начинающих. И реализовать, для начала, самый простой вариант.
Даже может и без файлов, просто строками.
Опомнитесь, господа, не усложняйте без нужды.

[Михалыч]

Михалыч, Тут такая проблема, что в слове "Японский" например можно кучу разных вариантов ошибки совершить. Вплоть до "Йапона". замучаетесь вручную все эти варианты вписывать в код программы. Хотя я один раз видел, как вручную писали код ИИ игры "крестики нолики". 

А мне сдается, что и не надо. Сделать 3-4-5 вариантов с ошибками. А на все остальные опечатки - "моя твоя не понимай".
И достаточно для реализации задания и понимания процесса.
А вообще, все зависит от того, на каком этапе обучения возникает такое задание. Контекста не хватает...
Соответственно, пытаемся смотреть максимально возможно шире, а может там еще до работы с файлами даже и не дошло...

[Finch]

Михалыч, Я изначально написал, что в такой постановке задания это не для новичка.

[Sla]

ну почему же, можно и самостоятельно..

Но первоначально нужны словари.
А тут есть куда развернуться
Сбор словаря, организация словаря, словоформы, синонимы, паронимы, морфемы и прочее..

[Olej]

Даже чтобы администрировать Linux не обязательно знать С. Вполне достаточно bash скрипты.

Даже, блин   ... администра-а-а-а-атора  
Так и официанту в ресторане не нужно знать премудрости кулинарии и познания шеф-повара.
А администратор в IT-конторе - это и есть официант ... только часто они не понимают своё место.  

А программист в Linux/UNIX, не знающий C, хотя бы в основах - никогда не поймёт API Linux (POSIX), и цена ему - 3 копейки в базарный день.

[Finch]

Olej, Не верное сравнение. Официант в ресторане не отвечает за безопасность ресторана. Это менеджер по продажам, если переводить на конторский язык. Хотя хороший официант должен разбираться в тонкостях кулинарии, чтобы правильно подобрать для клиента тонкости вкуса. А вот админ отвечает за безопасность и стабильность работы вверенного ему оборудования.

[Olej]

Не верное сравнение.

Ага...  
Системные администраторы - это низшая каста IT-работников, возомнившая о себе невесть что.  

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

[Михалыч]

Olej, Finch, не съезжайте в сторону
Дискуссия у вас не в тему.   От слова совсем.

[RXL]

Ой, да бросьте, господа...  
Как же мы раньше то начинали учиться программировать без ИИ и нейронных сетей?  

У меня на работе коллега очень горит D. Устроился для собственного удовлетворения преподавать его раз в неделю в один столичный ВУЗ. Вроде там уже второй курс и в активе пройденных материалов у студентов значатся и Java и Python, а почти нихрена не понимают. Одним из первых вопросов, был: "можно ли на D написать нейронную сеть". Это у современной молодежи панацея такая от всей работы  

[Михалыч]

RXL, современная молодежь, она такая 
В свое время, когда я немного преподавал (слава богу немного), мне аж поплохело, когда я вдруг понял, что студенты не способны в уме перемножать двухзначные числа. Принципиально не способны без калькулятора... Нет, жить то без этого вполне можно, но... 

[KOzh1r]

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

Именно то, что и хочу сделать. Чтобы было

достаточно для реализации задания

в самом лёгком варианте. Без списков, файлов и ИИ. Просто анализ запроса и вывод из нескольких моделей.

[KOzh1r]

KOzh1r, Данная задача не для новичка. Как правило подобные задачи решают через нейроные сети.
Чтобы разделить строку на отдельные слова в классе string должна быть функция split
Чтобы перевести символы в нижний или верхний регистр: ToLower или ToUpper
Смотри Help как правильно пользоваться ими.

Спасибо! Пойду искать

[Джон]

И тут Остапов понесло...

Читаем внимательно:

Недавно стал открывать

как это реализовать в виде программного кода понятия не имею

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

Быть очень просто:

1. Пишем программу (спокойно, на C#), на вводе которой задаём "А", а программа должна ответить 1.

Это под силам?
Тогда идём дальше

2. Усложняем прогу до
  * на входе Я
  * на выходе список марок японских машин
  * на входе Р
  * на выходе список марок российских машин

Это под силам?
Тогда идём дальше

Основная инфа в предложениях "выведи российские модели" и "перечисли японские марки" - это "российские" или "японские". Ессно, что прога должна выдавать какой-то результат.
Будет ли она его перечислять, выводить и тп ей это фиолетово. "Марки", "модели", "машинки", "тачки" и пр. аналогично. В результате будет то, что находится в БД под соответствующими ключами.

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

Например
 * у "российские" на первом месте согласная, на втором гласная, и уже не важно рАсские, русийские и тп, это уже будет точно не ёпунские.
 * у "японские" наоборот (в данном случае добавим к гласным й, ну и с учётом  йа-панск, проверяем первые две на "гласность", точнее на й и гласную)


Для "выуживания" этой минимальной инфы из запроса, я бы использовал регулярные выражения с немного ужесточённой логикой: какие буквы могут находится в первой, второй, третьей позиции.

Как-то так.

[KOzh1r]

И тут Остапов понесло...

Читаем внимательно:

Недавно стал открывать

как это реализовать в виде программного кода понятия не имею

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

Быть очень просто:

1. Пишем программу (спокойно, на C#), на вводе которой задаём "А", а программа должна ответить 1.

Это под силам?
Тогда идём дальше

2. Усложняем прогу до
  * на входе Я
  * на выходе список марок японских машин
  * на входе Р
  * на выходе список марок российских машин

Это под силам?
Тогда идём дальше

Основная инфа в предложениях "выведи российские модели" и "перечисли японские марки" - это "российские" или "японские". Ессно, что прога должна выдавать какой-то результат.
Будет ли она его перечислять, выводить и тп ей это фиолетово. "Марки", "модели", "машинки", "тачки" и пр. аналогично. В результате будет то, что находится в БД под соответствующими ключами.

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

Например
 * у "российские" на первом месте согласная, на втором гласная, и уже не важно рАсские, русийские и тп, это уже будет точно не ёпунские.
 * у "японские" наоборот (в данном случае добавим к гласным й, ну и с учётом  йа-панск, проверяем первые две на "гласность", точнее на й и гласную)


Для "выуживания" этой минимальной инфы из запроса, я бы использовал регулярные выражения с немного ужесточённой логикой: какие буквы могут находится в первой, второй, третьей позиции.

Как-то так.

Отлично! По силам всё, кроме того, как анализировать именно слово "русские" или "японские"? Я подразумеваю следующий алгоритм:
1. Программа получает запрос в виде предложения
2. Переводит в нижний регистр всё предложение
3. Ищет в предложении нужное слово
4. Обрабатывает это слово и выдаёт ответ.
Проблема только с тем, как реализовать третий пункт..

[Sla]

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

Например
 * у "российские" на первом месте согласная, на втором гласная, и уже не важно рАсские, русийские и тп, это уже будет точно не ёпунские.
 * у "японские" наоборот (в данном случае добавим к гласным й, ну и с учётом  йа-панск, проверяем первые две на "гласность", точнее на й и гласную)

Вы прослушали краткий курс реализации soundex
и алгоритма, если я не ошибаюсь стеммер Портера

[Джон]

Sla,   Я, к сожалению, в последний раз с этим сталкивался когда-то в начале 90-ых. Но тема интересная, наверное, поэтому и запомнилось что-то.

KOzh1r ,

Какие есть возможные комбинации букв на первом, втором и тд местах достаточные для отличия "русских" от "японцев"?

Например: комбинация ххх не удовлетворяет ни одному из условий и в этом случае, а равно в других аналогичных случаях, прога (хорошая) должна сообщить пользователю об этом и предложить правильные варианты ввода/выбора. Ибо, если пользователь уснул на клавише, например, Ж, то никакая AI не распознает, чего же именно он хочет.

[KOzh1r]

Джон, нет, Я не об этом. Как программа поймёт в каком именно слове надо искать различия и анализ какого именно слова необходимо произвести?

[Алексей++]

KOzh1r, регулярное выражение

[Джон]

KOzh1r, регулярное выражение

Лёш, это же тест. А тут ещё такая возможность "пощупать" всё ручками. Обойдёмся простыми if-ами.

Джон, нет, Я не об этом. Как программа поймёт в каком именно слове надо искать различия и анализ какого именно слова необходимо произвести?

Мы знаем, точнее предполагаем, что в одном из слов запроса содержится нужная нам инфа.

Ну хорошо, исходя из предыдущего опыта:

Отлично! По силам всё...

идём дальше.

Усложняем программу
 * на входе ОДНО слово, или сочетание букв
 * на выходе
   1. Р - если распознано слово "российский" (с вариациями)
   2. Я - если распознано слово "японский" (с вариациями)
   3. Н - во всех остальных случаях

Это под силам?
Тогда идём дальше.

Прогоняем все слова запроса через фильтр.

Всё.

[KOzh1r]

Это под силам?

Нет. В том-то и дело, Я не знаю какой код необходимо использовать для поиска определённого слова..

[Finch]

Шарп вроде позволяет прямое сравнение строк.

Код:

string a = "Some string";
if (a == "Some string") {
   DoSomethinf();
} else {
   Doanotherthing():
}

[Джон]

Это под силам?

Нет. В том-то и дело, Я не знаю какой код необходимо использовать для поиска определённого слова..

Покажите код из первых двух версий программы, из того, что было под силу.

Добавлено через 22 часа, 7 минут и 5 секунд:
Эммм сорри, не хотел напугать. Конечно ВЕСЬ код не нужен, достаточно ф-ции (или участка кода), обеспечивающей алгоритм распознавания ОДНОЙ буквы (версия 0.2).

Это будет основой для дальнейшего усложнения программы.

[KOzh1r]

В общем, моих знаний программирования хватило лишь на это:

Код: (C#)

using System;

public class MainClass
{
        public static void Main()
        {
                Console.WriteLine("Введите запрос");
                string a = Console.ReadLine();
                a = a.ToLower();
                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";
                if(a == b || a == c || a == d || a == e)
                {
                        Console.WriteLine("Ваз " + "Газ");
                }
                else
                {
                        Console.WriteLine("Mazda " + "Toyota " + "Nissan");
                }
        }
}

Понимаю насколько смешно вам смотреть на это убожество_))
Задача теперь состоит в том, чтобы из предложения, которое будет вводиться (например, перечисли русские модели) выдернуть конкретное слово "русские" для переменной "а", чтобы начать его сравнивать

[Джон]

Ну, во-первых, ничего смешного и убожественного в этом нет, я, во всяком случае, не вижу.

Во-вторых, небольшая просьба, приводите пожалуйста цитаты из сообщений, только, если это необходимо. И по-возможности цитируйте не ВСЁ, а только существенную часть. Иначе места для полезной инфы мало остаётся, даже на 32" мониторе.

Итак, я включил форматирование Вашего кода для удобочитаемости, содержимое не изменял.

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

Вы представили первую версию программы (0.1), которая что-то получает и что-то выводит. Это уже неплохо. (Я надеюсь работоспособность проверена?)

Не зная Вашего уровня, не могу не спросить:такие вещи как ООП Вам о чём-нибудь говорят? Объектно-ориентированное программирование? Что такое в Вашем коде public class MainClass?

Мы не будем явно использовать (пока ) ООП в программе, но, поскольку, C# является таковым языком, неплохо бы иметь хотя бы общее представление. Если нет, то не страшно. Для начала нам будет достаточно ф-ций, а точнее методов, так называются ф-ции класса/объекта.

Собственно программирование не есть кодирование алгоритма, а сам алгоритм. Как он будет воплощён, на каком языке, это вопрос вторичный. Поскольку у нас C#, то будем "разговаривать" на нём.

Для начала займёмся такой штукой как refactoring, или оптимизацией кода.

Метод Вашего класса Main() содержит код программы, когда он будет полным, даже просто читать такую "простыню" будет неудобно.
Что можно сделать - выделить задания во вспомогательные методы. На результат работы программы это никак не отразится, не должно во всяком случае.

Если представить прогу схематично, то она будет выглядеть примерно так (в скобках я привожу рабочее название методов на англ. тк это принято, да и компилятору легче, если для Вас это сложно, можете использовать русский транслитеративно напр. PloutchitZapros):

Ввод запроса (GetRequest)

Поиск ключа в запросе (FindKey)

Поиск данных по ключу (FindData)

Вывод результата (OutputResult)

Так и запишем

Код: (C#)

public class MainClass
{
        public static void Main()
        {
            string request = GetRequest(); // получаем запрос
            string key = FindKey(request); // передаём запрос в анализатор
            string result = ""; // создаём буфер для результата
            if (!string.IsNullOrEmpty(key)) // так осуществляется проверка пустой строки, её можно будет перенести в метод FindData
            {
                // предположим анализатор возвращает непустую строку, если он распознал ключ
                result = FindData(key); // передаём ключ в метод поиска данных
            }
            OutputResult(result); // выводим результат
      }
}

Вот собственно и всё. Никаких значительных изменений в методе Main мы больше делать не будем.

Задание: закончить оптимизацию кода (перенести строчки кода версии 0.1 в оставшиеся методы)

Подсказка.

№1. Пример первого метода
№2 Методы записываются в пределах фигурных скобок класса

Выглядит это примерно так:

Код: (C#)

public class MainClass
{
        public static void Main()
        {
            string request = GetRequest(); // получаем запрос
            string key = FindKey(request); // передаём запрос в анализатор
            string result = ""; // создаём буфер для результата
            if (!string.IsNullOrEmpty(key)) // так осуществляется проверка пустой строки, её можно будет перенести в метод FindData
            {
                // предположим анализатор возвращает непустую строку, если он распознал ключ
                result = FindData(key); // передаём ключ в метод поиска данных
            }
            OutputResult(result); // выводим результат
      }

      private static string GetRequest()
      {
          Console.WriteLine("Введите запрос");
          string a = Console.ReadLine();
          return a;
      }
}

Результат ничем не должен отличаться от работы версии 0.1 при прочих равных условиях.

[KOzh1r]

Во-вторых, небольшая просьба, приводите пожалуйста цитаты из сообщений, только, если это необходимо. И по-возможности цитируйте не ВСЁ, а только существенную часть. Иначе места для полезной инфы мало остаётся, даже на 32" мониторе.

Я понял, исправляюсь _))

Не зная Вашего уровня, не могу не спросить:такие вещи как ООП Вам о чём-нибудь говорят? Объектно-ориентированное программирование? Что такое в Вашем коде public class MainClass?

ООП не говорит ни о чём, а слова "public class MainClass" в моём коде означают какую-то неведомую штуку, которая очень важна для чего-то не менее важного, чем эти самые слова. _))

Если же говорить серьёзно, то Я только начал постигать азы, поэтому не знаю какую именно функцию несут эти команды. Догадываюсь, что они определяют какой-то принцип, по которому будет происходить алгоритм работы программы. Но какие бывают ещё и чем они отличаются друг от друга мне пока не известно. Равно как и то, что вообще возможно реализовать в рамках программы на С#.

Спасибо Вам за уделённое время, помощь и разъяснения! Написанное Вами мне пока не очень понятно, но Я почитаю и думаю разберусь что к чему. Напишу к каким выводам пришёл, по возможности прошу поправить если ошибусь _))

[Джон]

... слова "public class MainClass" в моём коде означают какую-то неведомую штуку

Рассматривайте пока это просто как контейнер для определённых конструкций кода. Простыми словами, где-то же он, код, должен находиться. С другой стороны, тому кто этот код читает и переваривает (делает из текста что-то, что запускается, выполняется и тд), это тоже должно быть понятно. Поэтому существуют определённые правила, тн синтаксис языка. Есть определённые ключевые слова. Например, class означает декларирование (объявление, описание) объекта, public - публичный доступ к этому объекту, те каждый извне может создавать экземпляры этого объекта. А вот методы, объявленные как private, предназначены для использования только в пределах этого класса. Ну и тд. Ессно, в рамках этой темы не имеет смысла заниматься пересказыванием любой книжки по ООП вообще и C# в частности.
Это мы оставим на домашнее задание.

Есть проблемы с литературой такого рода?

Написанное Вами мне пока не очень понятно

если честно, это было сделано специально, чтобы понять уровень.

Собственно что имелось ввиду.

Любая программа обладает определённым уровнем сложности, как объективно, так и субъективно. Например, совершенно нормально, что это задание Вам кажется сложным (пока) и непонятным (пока). Иначе всё это не имело бы смысла.
В любом случае, гораздо удобней разбить программу на части, в идеале независимые ни от чего, каждая из которых отвечает только за свою конкретную задачу.
Мы начали с самого простого, с методов. Для конечного пользователя это не имеет никакого значения, как я уже говорил, результат ОБЯЗАН быть одинаков в обоих случаях.
Даже если ВЕСЬ код будет написан в одну строчку.

Те это делается для облегчения жизни того, кто программирует. В данном случае для нас. Ведь разбираться с небольшой частью гораздо проще чем со всем целым.
Кроме того, это даст нам возможность изменять эти части, доводя их функциональность до нужного нам уровня, те до тех пор, пока они не заработают так как нам это нужно.
При этом основная программа остаётся неизменной.

К примеру. Пока программа получает запрос из консоли. А что мешает получать его из другого источника? На работу программы (основного алгоритма) это не должно оказывать никакого влияния. Таким образом выделение получения запроса в отдельный метод позволит, по необходимости, изменить логику ввода запроса только в этом методе, основная программа останется нетронутой. Всё что мы должны обеспечить, это чтобы метод Ввод запроса (GetRequest) вернул строку, содержащую этот запрос.

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

Код: (C#)

public class MainClass
{
        public static void Main()
        {
                string a = GetRequest(); // получаем запрос
                a = a.ToLower();
                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";
                if(a == b || a == c || a == d || a == e)
                {
                        Console.WriteLine("Ваз " + "Газ");
                }
                else
                {
                        Console.WriteLine("Mazda " + "Toyota " + "Nissan");
                }
      }

      private static string GetRequest()
      {
          Console.WriteLine("Введите запрос");
          string a = Console.ReadLine();
          return a;
      }
}

Ваше задание, аналогично этому вынести оставшийся код в методы:
 * Поиск ключа в запросе (FindKey)
 * Поиск данных по ключу (FindData)
 * Вывод результата (OutputResult)

Добавлено через 18 минут и 4 секунды:

Я почитаю и думаю разберусь что к чему. Напишу к каким выводам пришёл, по возможности прошу поправить если ошибусь _))

Ну так собственно это и есть цель этого занятия. Представление готового кода не нужно никому. Мне не интересно, да и времени жалко, а для Вас это будет абсолютно бесполезно. Да и самому что-то сделать всегда приятней, пусть даже с помощью, но самому. Я поэтому всегда стараюсь не давать готовых решений, а подталкивать в нужном направлении.

Самое главное, нужно чётко понимать, что нельзя научить программировать. Этому можно только научиться. Если я неделями буду красочно и живописно рассказывать и показвать Вам как махать руками и бултыхать ногами, Вы научитесь плавать? Или, если я буду гонять мимо Вас на велосипеде и говорить, как это легко и просто, Вы сможете только на основании этой инфы самостоятельно сесть и поехать? Но совершенно очевидно, что и тому, и другому лучше учиться, если есть кому помочь, подсказать, посоветовать. Поэтому за советом, за помощью, с непонятками - всегда добро пожаловать.

[KOzh1r]

Ломаю голову над этим который день, и никак не могу сообразить, как мне это

Код: (C#)

using System;

public class MainClass
{
        public static void Main()
        {
                Console.WriteLine("Введите запрос");
                string a = Console.ReadLine();
                a = a.ToLower();
                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";
                if(a == b || a == c || a == d || a == e)
                {
                        Console.WriteLine("Ваз " + "Газ");
                }
                else
                {
                        Console.WriteLine("Mazda " + "Toyota " + "Nissan");
                }
        }
}

реализовать, используя представленный Вами метод?
Я не могу разобраться в этом коде

Код: (C#)

string request = GetRequest(); // получаем запрос
            string key = FindKey(request); // передаём запрос в анализатор
            string result = ""; // создаём буфер для результата
            if (!string.IsNullOrEmpty(key)) // так осуществляется проверка пустой строки, её можно будет перенести в метод FindData
            {
                // предположим анализатор возвращает непустую строку, если он распознал ключ
                result = FindData(key); // передаём ключ в метод поиска данных
            }
            OutputResult

Каким образом сопоставить одно с другим?   Что такое "анализатор", "ключ" и куда мы его передаём?

[Джон]

Вот это понятно было?

Если представить прогу схематично, то она будет выглядеть примерно так (в скобках я привожу рабочее название методов на англ. тк это принято, да и компилятору легче, если для Вас это сложно, можете использовать русский транслитеративно напр. PloutchitZapros):

Ввод запроса (GetRequest)

Поиск ключа в запросе (FindKey)

Поиск данных по ключу (FindData)

Вывод результата (OutputResult)

если ДА
  - какие строчки Вашего оригинального кода (v.0.1) отвечают за каждый их этих четырёх пунктов?
если НЕТ
  - что именно непонятно?

Добавлено через 9 минут и 40 секунд:
зы

Что такое "анализатор", "ключ" и куда мы его передаём?

Назовите его как хотите... Что происходит с вводом? Очевидно, он анализируется. Кто это делает? Я назвал его "Анализатор".

Что мы хотим получить из ввода? Некую строку, известную нашей программе, по которой можно будет сделать однозначный выбор "японцев" или "русских". В программировании, это называется выбор/поиск по ключу. Те вот эта самая однозначная строка и является тем самым ключом. Какая переменная в Вашем оригинальном коде выполняет эту функцию (обладает этим свойством)?

[KOzh1r]

какие строчки Вашего оригинального кода (v.0.1) отвечают за каждый их этих четырёх пунктов?

Я так понимаю, что алгоритм следующий:

Ввод запроса (GetRequest) - получение программой информации. В нашем случае это
string a = Console.ReadLine();

Поиск ключа в запросе (FindKey) - поиск в этой информации какого-то значения, которое нам необходимо
В моём коде отсутствует

Поиск данных по ключу (FindData) - поиск информации, которая относится к значению, которое мы искали в предыдущем пункте
if(a == b || a == c || a == d || a == e)

Вывод результата (OutputResult) - соответственно вывод информации, найденной выше
Console.WriteLine("Ваз " + "Газ");

Моя программа будет работать лишь в одном случае - если на ввод подаётся всего одно слово. И проводится анализ этого слова, исходя из которого выводится результат. Если же поступит запрос из нескольких слов, то результат всегда будет один.

[Джон]

5+  

Именно так всё и есть

Моя программа будет работать лишь в одном случае...

Смотря ЧТО назвать Вашей программой? По сути, через пару дней она так и останется Вашей, немного с другим кодом.

Самое главное, нужно понимать, что программа это то, что Вы описали в последнем сообщении. Алгоритм. Последовательность действий, необходимых для получения желаемого результата.
В этом случае ДА, Ваш оригинальный код (v.0.1) работает очень примитивно и с одним словом и тд и тп.
Но именно это мы и хотим изменить. Основная же программа остаётся без изменений.

Если забыть про код и оставить только задания, и предположить, что каждое задание выполняется как нам нужно, ТО
1. неважно откуда произошёл ввод - за это отвечает "Ввод запроса", а он обязан нам что-то выдать
2. неважно какой ввод (1 слово, 1 буква и тд) - ведь за это отвечает "Поиск ключа" или Анализатор, он тоже обязан нам что-то выдать
3. Неважно где и какие данные мы ищем - за это отвечает "Поиск данных" и он тоже обязан нам что-то вернуть

ну и last but not least  

4. Вывод результата. Опять же, неважно, в косоль, на печатающее устройство и тд, но результат будет куда-то выведен. Для основной программы это не имеет никакого значения, ведь за это отвечает метод "Вывод результата"

Таким образом, введя эти методы, мы абстрагируем реализацию основной программы. Всё её дальнейшее поведение, будет зависеть от реализии отдельных методов. Именно этим мы и будет в дальнейшем заниматься.

Поиск ключа в запросе (FindKey) - поиск в этой информации какого-то значения, которое нам необходимо
В моём коде отсутствует

Правильно, но есть одна строчка, которая ЧТО-ТО делает с вводом, как-то его преобразует. Давайте пока поместим эту строчку в метод "Поиск ключа в запросе".

Теперь Вы сможете добавить необходимые методы в класс MainClass?

[KOzh1r]

Код: (C#)

string request = GetRequest(); // получаем запрос (в переменную "request" записывается "перечисли японские модели")
string key = FindKey(request); // передаём запрос в анализатор (в переменную "key" записывается как-то из "request" слово "японские" )
string result = ""; // создаём буфер для результата (создаётся пустая переменная "result", в неё запишется результат поиска)
result = FindData(key); // передаём ключ в метод поиска данных (в пустую переменную "result" записываются данные, найденные по слову "японские", записанные в "key" )

Смысл Я понял, но как это реализовать в виде кода, Я не знаю. Как задать переменной "key" значение "японские"? Какие команды необходимо написать, чтобы программа поняла что Я хочу?

[Джон]

Посмотрите мой пример с реализацие метода GetRequest. Понятно, что там было сделано? Если нет, что именно непонятно?

Как задать переменной "key" значение "русские"?

А в какой переменной находится это значение в Вашем оригинальном коде? Как вы в нём задали это значение?

Ещё раз, я не требую от Вас НИЧЕГО нового! Никаких новых команд. Просто преобразование ВАШЕГО кода (это называется refactoring) для дальнейшей работы с ним. Это необходимо для того чтобы Вы приобрели навыки:
1. создания методов. Эти методы мы будем добавлять в класс не один раз. В дальнейшем, мы будем добавлять даже целые классы.
2. присваивания переменным имён со смыслом, чтобы потом не гадать: что такое а, б, с.

Ведь именно с этим, как я понимаю, в данный момент у Вас (не)большие проблемы. Или я ошибаюсь?

Какие команды необходимо написать, чтобы программа поняла что Я хочу?

Нет готовых команд для реализации задачи. Эти, как Вы их называете, команды, нам предстоит ещё создать.

[KOzh1r]

Код: (C#)

string a = GetRequest();
a = a.ToLower();
string b = "русские";
string b = FindKey(a);

Как Я понимаю, копаю не в ту сторону?

Вообще мне это напоминает обучение сына чтению. Он тоже понимает что такое буквы, но что такое слова - для него загадка. Вот и Я понимаю что нужно сделать, но как именно это сделать - непонятно. Как внести в переменную слово и одновременно с этим внести в неё этот загадочный FindKey?! Как ребёнок, честное слово.. 

[Джон]

Ок, собственно это и выяснили.

Итак, в оригинальном коде роль ключа выполняет переменная а.

Код: (C#)

string a = Console.ReadLine();

Ей присваивается значение, которое возвращает метод ReadLine объекта Console. Я создал свой собственный метод, который назвал GetRequest, и переместил в него две строчки. Теперь переменная а, получает значение, которое возвращает этот метод. А сам метод отвечает за показ приглашения и получение запроса, который он возвращает

Код: (C#)

      private static string GetRequest()
      {
          Console.WriteLine("Введите запрос"); // показывам приглашение
          string a = Console.ReadLine(); // получаем из консоли ввод пользователя
          return a; // возвращаем его как результат метода
      }

Было (в оригинальном коде)

Код: (C#)

   Console.WriteLine("Введите запрос");
   string a = Console.ReadLine();

Стало

Код: (C#)

string a = GetRequest();

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

[Джон]

Идём дальше.

След. строчка в оригинальном коде

Код: (C#)

a = a.ToLower();

Мы помним, что а - это введённый пользователем запрос. Пока код переводит все символы введённые пользователем в нижний регистр, это необходимо, тк оператор сравнения чувствителен к регистру, т.е. а != А. Таким образом эта строчка ЧТО-ТО делает с вводом, как-то его преобразует. Очень хороший повод поместить и эту строчку в отдельный метод. Конечно, ToLower это не то, что нам понадобится в дальнейшем. Именно над этим методом мы будем тщательно работать. Но пока нам достаточно и этой одной строчки.

Создаём метод, который называем FindKey

Код: (C#)

        private static string FindKey(string request)
        {
            string result = request.ToLower();
            return result;
        }

Метод GetRequest сам заботится о получении ввода, поэтому ему не нужны параметры. А вот FindKey должен что-то сделать с текстом (строкой), введённым пользователем, поэтому он получает этот в текст в качестве параметра (переменной) типа строки, я назвал его
request, и возвращает тоже строку.

Давайте на этом этапе сразу дадим нашим переменным имена со смыслом. До сих пор запрос находился в переменной с благозвучным именем а, которое абсолютно не отражает характера содержащейся в ней информации. Поскольку это запрос, то так и назовём её request, логично предположить, что метод GetRequest возвращает именно request.
Ну в дальнейшем метод FindKey возвращает ключ, поэтому и переменную назовём key

Преобразованный таким образом код на этом этапе выглядит след. образом

Код: (C#)

using System;

public class MainClass
{
        public static void Main()
        {
                string request = GetRequest(); // получаем запрос

                string key = FindKey(request); // передаём запрос в метод поиска ключа в этом запросе

                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";
                if(key == b || key == c || key == d || key == e)
                {
                        Console.WriteLine("Ваз " + "Газ");
                }
                else
                {
                        Console.WriteLine("Mazda " + "Toyota " + "Nissan");
                }
        }

       private static string GetRequest()
       {
           Console.WriteLine("Введите запрос"); // показывам приглашение
           string a = Console.ReadLine(); // получаем из консоли ввод пользователя
           return a; // возвращаем его как результат метода
       }

        private static string FindKey(string request)
        {
            string result = request.ToLower();
            return result;
        }
}

[Джон]

Пока всё понятно? Если нет, что именно непонятно?

[KOzh1r]

Пока всё понятно? Если нет, что именно непонятно?

Код: (C#)

using System;

public class MainClass
{
        public static void Main()
        {
                string request = GetRequest(); // получаем запрос

                string key = FindKey(request); // передаём запрос в метод поиска ключа в этом запросе

                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";
                if(key == b || key == c || key == d || key == e)
                {
                        Console.WriteLine("Ваз " + "Газ");
                }
                else
                {
                        Console.WriteLine("Mazda " + "Toyota " + "Nissan");
                }
        }

       private static string GetRequest()
       {
           Console.WriteLine("Введите запрос"); // показывам приглашение
           string a = Console.ReadLine(); // получаем из консоли ввод пользователя
           return a; // возвращаем его как результат метода
       }

        private static string FindKey(string request)
        {
            string result = request.ToLower();
            return result;
        }
}

Не понял, это один единый код, выполняющий эту программу? Почему
"Console.WriteLine("Введите запрос"); // показывам приглашение
           string a = Console.ReadLine(); // получаем из консоли ввод пользователя
           return a; // возвращаем его как результат метода"
находится внизу хотя вверху есть
"string request = GetRequest(); // получаем запрос"
Это же то же самое? Или нет?

[Джон]

ОК

"внизу" этот код стоит не просто так. Он определён как метод GetRequest. Он не "подвешен" в воздухе, а находится в пределах от

private static string GetRequest()
{

до

}

тело метода всегда находится в фигурных скобках, например, от

public static void Main()
{

до

}

равно как и тело всего класса от

public class MainClass
{

до самой нижней (последней)

}

Методы определяются в теле класса, и по большому счёту неважно "внизу" или "вверху". Таким образом у класса MainClass на данном этапе есть 3 метода:
 * Main - основной метод, который вызывается системой, с него всё и начинается, некая точка входа в программу.
+ два вспомогательных метода
 * GetRequest
 * FindKey

В фигурных скобках можно работать только с известными. Таковыми являются:
 * локальные переменные, которые явно определяются в теле метода (string a)
 * методы известных объектов, например, Console.WriteLine, "известность" им придаёт ключевое слово using в самой верхней части кода, в данном случае using System;
 * методы - члены этого класса, коими и являются GetRequest и FindKey

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

Теперь ответ на "почему?"

Вот чтобы строчка

string request = GetRequest();

сработала, "внизу" должен быть определён этот самый GetRequest. Попробуйте изменить имя метода на GetRequestX (внизу). При компиляции возникнет ошибка: GetRequest неизвестен

[KOzh1r]

Иными словами, под строкой "private static ... " находится метод, который позволяет задать переменной значение, которое выводится в этом методе. После выполнения метода "private static string FindKey(string request)" результатом будет:
a = result.
Я правильно понимаю?
Если да, то значит мне необходимо задать переменной "key" значение переменной "b"? А как это сделать, если "string key = FindKey(request)"? Короче опять похоже не туда Я полез..

Уважаемый Джон, как у Вас хватает терпения разъяснять кажущиеся элементарными понятия? Я бы наверное уже психанул.. 

[Джон]

Стоп, начало совершенно правильное.
Вот до этого места

Я правильно понимаю?

абсолютно ДА!!!

в переменную а будет помещено значение, которое было установлено в методе, в его локальной переменной result.

Дальше Я не совсем понял, особенно с "если"...

Пока забудьте о ключевых словах private и static, и примите их как должное. Нуу типа "тут так принято". А вот дальше...

Пример

У нас есть некий код, строчек 7-8, результатом работы которого является допустим некая строка. В этих строчках объявляются переменные, вызываются некие методы и тд и тп.

У нас возникает необходимость вызывать этот код несколько раз. В одном месте, в другом месте и тд.

Можно конечно скопировать его в нужные места, но! Если эти места находятся в пределах одной пары фигурных скобок, то возникнет проблема с переменными: невозможно объявлять переменные с одинаковыми именами в одном месте, иначе компилятор не будет знать, с какой же именно переменной мы хотим что-то сделать.
Другой недостаток (ну допустим, что мы копируем этот код в разные участки, используем его в разных методах и тд и проблем с переменными у нас не возникает). Спустя какое-то время мы находим ошибку в этом коде, или просто хотим его изменить. Делать это в разных частях программы? Одно и то же?

Вот тут и приходят на помощь методы. Мы выделяем этот код в отдельный блок и "оформляем" его как метод. Что это означает:
 - сначала мы сообщаем тип возвращаемого значения (про private и static мы пока забыли)
 - затем следует имя
 - затем обязательно круглые скобки, неважно, будем мы передавать какое-нибудь значение в качестве параметра или нет
 - после этого следует блок взятый в {} это т.н. тело метода

возвращаемый тип Имя Метода()
{
   тело метода
}

например
string GetText()
{
    int i = 3;
    sring abc = "abc";
}

Метод может и не возвращать никакого значения, в этом случае для возвращаемого используется ключевое слово void.
Если же некое значение возвращается, то в теле метода должна присутствовать команда возврата значения return

Поэтому правильно пример выше выглядит так
string GetText()
{
    int i = 3;
    sring abc = "abc";
    
    return abc;
}

Пример метода без возвращаемого значения
void ShutDown()
{
    int i = 3;
    sring abc = "abc";
}

Ну и если в метод необходимо передать некий параметр, или даже несколько параметров, то они перечисляются в круглых скобках
как при определении переменных, только разделяются не ; а просто запятой
возвращаемый тип Имя Метода( тип параметра1 имя параметра1, тип параметра2 имя параметра2 )
{
   тело метода
}

например
int Sum( int x1, int x2 )
{
    int result = x1 + x2;
    return result;
}

Ну и очевидно, что внутри метода нельзя определять переменные с именами идентичными именам параметров.

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

Ну и "ближе к телу" (с)

С учётом легенды выше вот что я сделал в Вашим кодом

       private static string GetRequest()
       {
           Console.WriteLine("Введите запрос"); // строка перенесена из оригинального кода 1:1
           string a = Console.ReadLine(); // строка перенесена из оригинального кода 1:1
           return a; // возвращаем содержимое локальной переменной а как результат метода
       }

        private static string FindKey(string request)
        {
            string result = request.ToLower(); // строка перенесена из оригинального кода 1:1, я только изменил имя переменной и имя параметра
            return result;
        }

Уважаемый Джон, как у Вас хватает терпения разъяснять кажущиеся элементарными понятия? Я бы наверное уже психанул..  

Пока всё как должно быть. Нет никаких причин психовать.

Добавлено через 7 минут и 39 секунд:
На основании вышеизложенного попробуйте добавить метод OutputResult самостоятельно.
Подсказки
 - какой тип данных возвращается этим методом?
 - нужны ли этому методу параметры? если да, то какие?

FindData немного сложней, но если смелости хватит... то
 - какой тип данных возвращается этим методом?
 - нужны ли этому методу параметры? если да, то какие?
 - как нужно изменить оригинальный код, чтобы его можно было использовать в этом методе?

[KOzh1r]

На основании вышеизложенного попробуйте добавить метод OutputResult самостоятельно.

Код: (C#)

using System;

public class MainClass
{
        public static void Main()
        {
                string request = GetRequest(); // получаем запрос

                string key = FindKey(request); // передаём запрос в метод поиска ключа в этом запросе

                OutputResult(key);
        }

       private static string GetRequest()
       {
           Console.WriteLine("Введите запрос"); // показывам приглашение
           string a = Console.ReadLine(); // получаем из консоли ввод пользователя
           return a; // возвращаем его как результат метода
       }

        private static string FindKey(string request)
        {
            string result = request.ToLower();
            return result;
        }
        private static void OutputResult(string key)
        {
                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";
                if(key == b || key == c || key == d || key == e)
            {
                Console.WriteLine("Ваз " + "Газ");
            }
            else
            {
                Console.WriteLine("Toyota " + "Nissan " + "Mazda");
            }
        }
}

Вот что смог сделать. Пришлось почитать немного, чтобы дойти до результата. По итогу программа делает то же самое, что и ранее 
Как реализовать поиск ключа в запросе Я так и не понял.. Наткнулся в интернете на такой код
 

Код: (C#)

var result = dict.Where(x => x.Value == "Сюда что ищем");
foreach(string data in result)
Console.WriteLine(data);

но так и не понял - это вообще то, что можно использовать или нет? 

[Джон]

Отлично

Теперь к последнему оставшемуся методу.

Ваш метод OutputResult выполняет двойную функцию
 - определяет данные для вывода
 - и делает непосредственно вывод

Не есть гут. Каждый метод должен отвечать только за своё одно задание. Проблема заключается в наглядности, Вы экономите на буферных переменных, что на данном этапе обучения контропродуктивно.

Например

Console.WriteLine("Toyota " + "Nissan " + "Mazda");

можно представить как

string buffer = "Toyota " + "Nissan " + "Mazda";
Console.WriteLine(buffer);

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

Код: (C#)

        private static void OutputResult(string key)
        {
                // >>>>> начиная отсюда

                string result = ""; // это пока пустая строка, которая будет заполнена позднее

                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";

               if(key == b || key == c || key == d || key == e)
               {
                    result = "Ваз " + "Газ";
               }
               else
               {
                   result = "Toyota " + "Nissan " + "Mazda";
               }
               // >>>>> и до сюда, не имеет к выводу никакого отношения, скорее к поиску результата/данных.              

               Console.WriteLine(result); // отвечает 100% названию метода, а что делают строчки до этого места? не лучше ли их вынести в отдельный метод?
        }

[Джон]

По итогу программа делает то же самое, что и ранее

И это замечательно, значит всё делается правильно. Так и должно быть.
Код стал компактным, а значит удобочитаемы и "удоборазбираемым".
Вы получили навыки и теперь можете самостоятельно кодировать логику.

После окончательного refactoring мы познакомимся с парой новых вещей и в принципе на этом всё.
Прога будет готова и Вы сможете самостоятельно усложнять или модифицировать её логику.

Как-то так.

зы

Наткнулся в интернете на такой код

Это ещё повезло, что на такой, а ведь могли бы наткнуться и на что-нить типа такого:

Код: (C#)

            var result = from x in dict
                         where x.Value == "Сюда что ищем"
                         select x;

            result.ToList().ForEach(s => Console.WriteLine(s.Value));

А что Вы поняли из этого кода? Что есть dict? Что такое lambda-выражение, LINQ Вы уже знаете? Так чего же Вы тогда мне голову морочите?
Если нет, то Вы сейчас примерно учитесь даже не летать, а ещё только подпрыгивать. А найденный Вами код можно сравнить с пилотированием космического корабля.

Я немного в замешательстве. Цель всего это начинания? Научиться самостояльно создавать программы?

Или найти в инете хоть какой-нить код, который бы что-то сделал?

Если второе, то смею Вас уверить, такого в инете нет, не бывает. А иначе бы все подобное тестирование, да и сама работа программистов, были бы чистейшим очковтирательством. Нафига платить кучу бабла? Зашёл в инет, скачал готовое решение, компильнул и готово, пользуйся.

[KOzh1r]

Код: (C#)

using System;

public class MainClass
{
        public static void Main()
        {
                string request = GetRequest(); // получаем запрос

                string key = FindKey(request); // передаём запрос в метод поиска ключа в этом запросе

                string data = FindData(key);

                OutputResult(data);
               
        }

       private static string GetRequest()
       {
           Console.WriteLine("Введите запрос"); // показывам приглашение
           string a = Console.ReadLine(); // получаем из консоли ввод пользователя
           return a; // возвращаем его как результат метода
       }

        private static string FindKey(string request)
        {
            string result = request.ToLower();
            return result;
        }
        private static string FindData(string key)
        {
                string result = ""; // это пока пустая строка, которая будет заполнена позднее

                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";

               if(key == b || key == c || key == d || key == e)
               {
                    result = "Ваз " + "Газ";
               }
               else
               {
                    result = "Nissan " + "Toyota " + "Mazda";
               }
               return result;
        }
        private static void OutputResult(string data)
        {
                Console.WriteLine(data);
        }
}

Исправил код, добавив метод. Но мне кажется, где-то в коде какая-то ошибка... 

[KOzh1r]

Наткнулся в интернете на такой код
 

Код: (C#)

var result = dict.Where(x => x.Value == "Сюда что ищем");
foreach(string data in result)
Console.WriteLine(data);

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

В моей ситуации - это лишь попытка найти хоть что-то, что поможет мне реализовать поиск этого самого ключа. Какой-то намёк, тонкая нить, по которой Я смогу выйти на нужный результат.

Но Вы правильно заметили, что найденный мною код для меня - китайские иероглифы. Однако такими же иероглифами, на прошлой неделе было и это:

Код: (C#)

public class MainClass
{
        public static void Main()
        {
            string request = GetRequest(); // получаем запрос
            string key = FindKey(request); // передаём запрос в анализатор
            string result = ""; // создаём буфер для результата
            if (!string.IsNullOrEmpty(key)) // так осуществляется проверка пустой строки, её можно будет перенести в метод FindData
            {
                // предположим анализатор возвращает непустую строку, если он распознал ключ
                result = FindData(key); // передаём ключ в метод поиска данных
            }
            OutputResult(result); // выводим результат
      }
}

[Джон]

Но мне кажется, где-то в коде какая-то ошибка...  

Хм... я ничего такого на первый взгляд не увидел... А что говорит компилятор? Прога должна работать как в версии кода 0.1

Да это по большому счёту и не важно.

Важно
1. Код стал обозримым и более понятным, теперь мы можем с ним работать
2. Вы приобрели необходимые навыки
3. Он уже для Вас не "китайские иероглифы" (с)

Это самое главное. Ещё раз, для проверки: всё понятно в последней версии? Нет никаких сложностей? Иначе мы не сможем двигаться дальше.
На этом этапе не стесняйтесь ошибаться и пороть чепуху, Вы имеете на это право. Гороаздо хуже отмалчивание непоняток. Это приведёт к большим проблемам в будущем.

Насчёт поиска кода в инете.

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

Собственно решение, даже разные варианты, были озвучены уже в первых сообщениях. Конечно, тогда никто не представлял Вашего уровня подготовки.

Ладно. Лирику в сторону.

До сих пор по сути код не изменился, алгоритм остался прежним. Мы просто "разбросали" отдельные куски кода по методам.

Оффтоп, просто для удовольствия или прикола. Такая организация кода позволит нам записать программу в одну строчку.
Если внимательно посмотреть, то каждое возвращённое одним методом значение передаётся в след. метод. Поэтому можно поозорничать

Код: (C#)

        public static void Main()
        {
                OutputResult(FindData(FindKey(GetRequest())));
        }

В реальности этого конечно же лучше не делать. Ибо ТАКОЙ код просто неудобно читать.

Итак, добавляем новинки, о которых я уже упоминал. Сначала я напишу это в коде, а потом объясню.
И ещё одно я теперь не буду приводить весь код, а только необходимые методы, просто чтобы место сэкономить.
По сути методы GetRequest и OutputResult нас ваще не интересуют. Правильно? А после след. добавлений не будет
интересовать и метод Main. Эти методы до самого конца останутся неизменными.

Итак, недостаток существующего кода:
 - ключ по сути определяется в методе поиска данных
 - в качестве ключа используется строка, её надо сравнивать и тд и тп

Исправим эти недостатки

Код: (C#)

using System;
using System.Text; // <<<< это новое, необходимо для "видимости" объекта  StringBuilder

public class MainClass
{
        enum ManufactureCountryKey // <<<<  это новое, эти значения мы будем использовать в качестве ключа
        {
            Undefined,
            Russia,
            Japan
        };

        public static void Main()
        {
                string request = GetRequest();
                ManufactureCountryKey key = FindKey(request); // метод больше не возвращает строковый тип
                string data = FindData(key);  // метод больше не получает строковый тип
                OutputResult(data);
        }

        private static ManufactureCountryKey FindKey(string request)  // изменён тип возвращаемого значения
        {
            if (!string.IsNullOrEmpty(request))
            {
                string key = request.ToLower();

                string b = "русские";
                string c = "российские";
                string d = "рассийские";
                string e = "росийские";

                if (key == b || key == c || key == d || key == e)
                {
                    return ManufactureCountryKey.Russia;
                }
                else
                {
                    return ManufactureCountryKey.Japan;
                }
            }

            return ManufactureCountryKey.Undefined;
        }

        private static string FindData(ManufactureCountryKey key) // тип параметра изменён
        {
            StringBuilder sb = new StringBuilder();

            switch(key)
            {
                case ManufactureCountryKey.Russia:
                    {
                        sb.Append("Ваз");
                        sb.Append(" ");
                        sb.Append("Газ");
                    }
                    break;
                case ManufactureCountryKey.Japan:
                    {
                        sb.Append("Mazda");
                        sb.Append(" ");
                        sb.Append("Toyota");
                        sb.Append(" ");
                        sb.Append("Nissan");
                    }
                    break;
                default:
                    {
                        sb.Append("По Вашему запросу данных нет");
                    }
                    break;
            }

            string result = sb.ToString();
            return result;
        }

}

[Джон]

Не очень большие изменения. Прога ещё работает как до этого.

1. Определён тип данных ManufactureCountryKey, значения этого типа мы будем использовать в качестве ключа (дом. задание: почитать в учебнике про enum)
2. Код для поиска ключа наконец-то находится в методе FindKey, там где он и должен находится.
3. Метод FindData адаптирован под новый тип ключа (раньше это была строка)

О последнем немного подробней в последний раз. В дальнейшем он нас не будет интересовать с точки зрения обучения.
1. Объект StringBuilder  "складывает" строки гораздо эффективней, чем оператор + (оставляем это на домашнее задание)
2. Введён оператор выбора switch, который позволяет выбрать соответствующий ключу код гораздо быстрее чем цепочка if ... else if
3. Очевидно, что такая организация метода позволит, при необходимости расширить его, например, добавить марки машин других стран производителей.

Серьёзные изменения в нём с использованием, например, кода, типа который Вы нашли в инете, необходимо будет сделать если в задаче в качестве источника данных будет некая база данных, или какой другой список. Но пока этого нет, это не требуется.

Есть проблемы с пониманием этих нововведений?

Если нет, то перейдём к последнему и главному методу - нашей цели - FindKey.

[KOzh1r]

Он уже для Вас не "китайские иероглифы" (с)

Исключительно благодаря Вам, за что огромное спасибо!

Есть проблемы с пониманием этих нововведений?

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

[Джон]

Ну тогда вкратце.

Ключ.

Во-первых тип string хоть и является стандартным, но всё-таки это БОЛЬШОЙ объект, с кучей методов и тд. Понятно, что сравнение строки это нетривиальная операция. Необходимо учитывать длину, сравнивать каждый символ (кстати, следущим шагом мы будем практически этим заниматься).
Куда гораздо проще сравнивать целочисленный тип.
Другими словами, если мы скажем, что российские марки получат ключ целочисленного типа 1, японские 2, а неопределённые 0, то все операции с ключом будут производиться с максимальным быстродействием.
С другой стороны, если мы видим в коде 0, 1, 2, что это такое? Тут на помощь приходит enum, который позволяет определить целочисленый тип, но со смысловым именем.

Код: (C#)

        enum ManufactureCountryKey
        {
            Undefined,
            Russia,
            Japan
        };

В данном случае enum это ключевое слово, после которого следует описание = ИмяТипа. В фигурных скобках - возможные элементы, которые имеют целочисленный тип. Называть их можно как угодно, но лучше конечно, вкладывать в имя смысл.

По идее, его можно было бы представить в виде трёх переменных

Код: (C#)

            int Undefined = 0;
            int Russia = 1;
            int Japan = 2;

И пользоваться этими переменными. Тогда бы тип возвращаемого значения метода FindKey и тип параметра метода FindData был бы int. Но в таком случае можно было бы передавать скажем в FindData(9), а девятка у нас не определена. Таким образом enum позволяет избежать случайных ошибок программирования и отлавливать их уже на этапе компиляции. Это повышает надёжность кода и, ессно, программы.

Сложение строк.

Как уже упоминалось выше, строка это сложный объект. Слоожение двух таких объектов может происходить только по одному сценарию. Нам необходимо выделить память размером равным или больше размеров обоих слагаемых. Скопировать поочерёдно содержимое слагаемых в этот буфер и освободить память занимаемую исходными объектами. Если это делать многократно с оператором +, то каждый раз будет выполняться эта цепочка операций.
Если же мы сначала соберём все слагаемые в один список, то перед тем как сделать из них результирующую строку операцию выделения памяти нужно будет сделать только один раз, ведь все размеры в этот момент времени нам известны.

Для этот в .NET есть объект StringBuilder - построитель строк. Сначала мы добавляем в него строки, которые хотим сложить, вызывая метод Append, а для результата вызваем метод ToString.

Если StringBuilder ну уж очень смущает, то от него можно отказаться, введя операции сложения строк с оператором +. В конце концов производительность и быстродействие программы не является нашей целью.

Оператор выбора switch

Можно заменить на

Код: (C#)

                    if(ManufactureCountryKey.Russia == key)
                    {
                        sb.Append("Ваз");
                        sb.Append(" ");
                        sb.Append("Газ");
                    }
                    else if(ManufactureCountryKey.Japan == key)
                    {
                        sb.Append("Mazda");
                        sb.Append(" ");
                        sb.Append("Toyota");
                        sb.Append(" ");
                        sb.Append("Nissan");
                    }
                    else
                    {
                        sb.Append("По Вашему запросу данных нет");
                    }

Так понятней? switch делает тоже самое, только гораздо эффективней. Ну и является одним из стандарных операторов практически каждого языка программирования высокого уровня. Так что это относится к части обучения "расширяем словарный запас".

Вот так выглядит метод FindData без нововведений

Код: (C#)

        private static string FindData(ManufactureCountryKey key)
        {
            string result = "";

            if (ManufactureCountryKey.Russia == key)
            {
                result = "Ваз " + "Газ";
            }
            else if (ManufactureCountryKey.Japan == key)
            {
                result = "Mazda " + "Toyota " + "Nissan";
            }
            else
            {
                result = "По Вашему запросу данных нет";
            }

            return result;
        }

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

зы да, ну и конечно же, без учебника по C# Вам ну никак не обойтись.

[KOzh1r]

Я думаю, что на данном этапе лучше использовать знакомые выражения. В противном случае Я боюсь окончательно запутаться _)) Поэтому, думаю, пока лучше использовать этот вариант:

Код: (C#)

if(ManufactureCountryKey.Russia == key)
                    {
                        sb.Append("Ваз");
                        sb.Append(" ");
                        sb.Append("Газ");
                    }
                    else if(ManufactureCountryKey.Japan == key)
                    {
                        sb.Append("Mazda");
                        sb.Append(" ");
                        sb.Append("Toyota");
                        sb.Append(" ");
                        sb.Append("Nissan");
                    }
                    else
                    {
                        sb.Append("По Вашему запросу данных нет");
                    }

Данный тест необходимо сдать в понедельник, потом он завершится и состоится подведение итогов. Этот этап Я пока пропустил, но хочу успеть его сдать. Как считаете, успею понять и реализовать?

[Джон]

Ну в принципе осталась мелочь. Вам максимум часа на два кодирования.

Забываем про все остальные методы, кроме FindKey. Они выполнют свои ф-ции и что-то в них менять нет необходимости.
С этого момента мы будет работать только с FindKey.

Итак, оригинальная логика проверяет только на "русские" и... остальные. Это неправильно. Она должна выяснить, есть ли в запросе "русские", "японские" и остальные (неопределённые). Это мы уже подготовили. У нас есть три ключа для, соответственно, каждого из вариантов.

Теперь осталось только проанализировать запрос и вернуть нужный ключ. Поскольку запрос задаётся человеком, нам надо "очеловечить" программу. Те анализировать так, как это делает человек.
Предложение состоит из слов, разделённых символами разделения, например пробелом. Слова в свою очередь состоят из букв, которые также являются символами. Мы, человеки, ищем в этом массиве символов нужную нам комбинацию, например мы отличаем "рус" от "япо". С учётом ошибок, таких комбинаций будет несколько, для каждого варианта/страны.

Зададим размер комбинации равный трём. Те исходим из того, что определённая комбинация из трёх букв даст нам однозначный ответ: "русский", "японский", или "неопределён".
Если на каком-то этапе, или в процессе тестирования мы выясним, что трёх недостаточно, мы увеличим их до четырёх, пяти и тд. Или наоборот уменьшим до двух. Принципиального значения это не имеет. Всё ограничивается только нашей фантазией и фантазией пользователя.
По идее можно начать и с одной, но очевидно что одной буквы не хватит, чтобы найти нужную нам информацию.

Для начала нам надо получить доступ к этим отдельным символам запроса.
После этого останется только найти нужную комбинацию в запросе и вернуть соответствующий ключ. Всё.

Тип данных для отдельных символов называется char, а string представляет собой ни что иное, как массив этих самых символов.
Для доступа к элементам массива существует оператор индекса, который записыватся в коде в виде квадратных скобок [], внутри которых записывается индекс (позиция) символа.

ВАЖНО!!! Индекс в квадратных скобках не должен превышать длину массива (в нашем случае строки).
Индексация массивов начинаются с нуля (индекс первого элемента 0), а не с единицы, поэтому условие проверки должно быть строгим. Другими словами, индекс последнего элемента всегда на единицу меньше длины массива.

Пример

в строке string str = "ABC"; три символа
  символ A находится в позиции 0, доступ к нему str[0]
  символ B находится в позиции 1, доступ к нему str[1]
  символ C находится в позиции 2, доступ к нему str[2]

попытка доступа к четвёртому, несуществующему, элементу str[3] приведёт к сбою в работе программы.

Объекты типа char можно добавлять в объекты типа string

Код: (C#)

            char ch = 'C';
            string str = "AB";
            str += ch; // после этого в строке str находится "АВС"

Ну и ещё нам нужен оператор цикла, чтобы "пробегать" по элементам строки (массива). Мы будем использовать for.

Вот как это выглядит в коде

Код: (C#)

            string str = "ABC";
            for (int i = 0; i < str.Length; i++)  // этот оператор увеличивает содержимое переменной i на единицу после каждого выполнения тела цикла, до тех пор, пока не выполнится условие
            {
                // тело цикла
                char ch = str[i]; // здесь в переменную ch будет помещён символ из позиции текущего значения i
                // дальше можно делать с этим символом всё что угодно
            }

int i = 0; -> инициаллизация начального значения
i < str.Length; -> условие работы цикла
        * пока i строго меньше длины массива (она находится в свойстве Length объекта string), выражение истинно и цикл работает
i++ -> увеличение i на единицу, по сути это  i=i+1

Строка str содержит 3 символа, значит длина массива равна 3, следовательно тело цикла будет выполнено...
1-ый раз: i=0, i<3 -> ОК - в переменной ch находится элемент массива (строки) с индексом 0, "A"
2-ой раз: i=1, i<3 -> ОК - в переменной ch находится элемент массива (строки) с индексом 1, "B"
3-ий раз: i=2, i<3 -> ОК - в переменной ch находится элемент массива (строки) с индексом 2, "C"
4-ый раз: i=3, i<3 -> условие ложно, цикл остановлен после третьего раза.


Чтобы Вы поняли как это работает добавим такой код в метод FindKey

Код: (C#)

        private static ManufactureCountryKey FindKey(string request)
        {
            if (!string.IsNullOrEmpty(request))
            {
                // след. код выведет в консоль запрос посимвольно "в столбик"
                for (int i = 0; i < request.Length; i++)
                {
                      char ch = request[i];
                      Console.WriteLine(ch);
                }

                // остальное нас сейчас не интересует
            }

            return ManufactureCountryKey.Undefined;
        }

Ну вот и всё, больше код я писать не буду. Теперь у Вас есть все "инструменты", чтобы максимум за два часа закодировать задание.

Дальше будем обсужать только алгоритм, который в принципе и так понятен.
"Пробегая" по запросу (введённому массиву символов) с помощью оператора цикла мы ищем комбинацию нужных нам букв.
Те является ли первый символ этой комбинации допустимым при использовании его на первом месте для
1. русских
2. японских
если да, проверяем второй символ, является ли он допустимым для использования его на втором месте
и тд

Например, Ъ в любой позиции комбинации не подходит ни для одной из стран.

Лучше сделать два метода, каждый из которых будет отвечать за свою "страну", но можно сделать и один метод, и передавать в него необходимые для "страны" параметры.
Второй способ предпочтительней, но я думаю Вам пока не стоит экономить, а сделать два метода. По функциональности они будут идентичными.
Refactoring мы сделаем потом, если время останется.

Говоря "простым языком", метод FindKey должен выглядеть как-то так

Код: (C#)

        private static ManufactureCountryKey FindKey(string request)  // изменён тип возвращаемого значения
        {
            if (!string.IsNullOrEmpty(request))
            {
                if (HasRUS(request))
                {
                    return ManufactureCountryKey.Russia;
                }
                else if (HasJAP(request))
                {
                    return ManufactureCountryKey.Japan;
                }
            }
            return ManufactureCountryKey.Undefined;
        }

где
 * private static bool HasRUS(string request)
 * private static bool HasJAP(string request)


Разбивать ли запрос сначала на отдельные слова? Это как Вам нравится, с этого момента Вы - программист. Я - только консультант.

Я думаю, что в создании алгоритма бОльшую помощь Вам окажет Ваш сынишка, в силу

Он тоже понимает что такое буквы, но что такое слова - для него загадка.

Попросите его найти нужную комбинацию букв в запросе, а потом запишите в виде кода, как он это сделал.

Ну и ещё я - подсказчик.

Подсказки
 * добавить в код вспомогательный метод выделения из строки-источника определённого количества символов в строку начиная с определённой позиции
   * private static string GetSubString(string source, int startPosition, int length)
   * не забудьте про проверку индекса
 * добавить в код вспомогательный метод для проверки нахождения символа в массиве других смволов
    * private static bool IsCharInArray(char ch, string allowedChars)

[Джон]

Проблемы? Или всё готово?

ps
Меня завтра не будет, и до форума смогу "добраться" только в воскр. в лучшем случае после обеда, в худшем, только к вечеру.

[KOzh1r]

Теперь у Вас есть все "инструменты", чтобы максимум за два часа закодировать задание.

Два дня колупаюсь, но так и не пойму как реализовать это:

После этого останется только найти нужную комбинацию в запросе и вернуть соответствующий ключ.

[Джон]

Ок, есть полчаса. В чём именно проблема? Как цикл работает понятно?

Добавлено через 2 минуты и 14 секунд:
Вы GetSubString уже реализовали?

Добавлено через 13 минут и 43 секунды:
Покажите, что уже сделано. Какие методы реализованы. Начиная с кода FindKey, все остальные нас не интересуют.

Добавлено через 7 минут и 13 секунд:

Два дня колупаюсь, но так и не пойму как реализовать это:

После этого останется только найти нужную комбинацию в запросе и вернуть соответствующий ключ.

Это ВСЁ оставшееся задание.

Делайте постепенно. Методом последовательных приближений. Забудьте на полчаса о ВСЁМ задании, и сконцентрируйтесь на одном методе.

Для начала "вытаскивание" последовательности трёх символов из запроса. Кстати понятно почему и для чего это надо делать? Вы прдеставляете себе алгоритм?

После этого забываем на полчаса обо всём кроме метода проверки наличия символа в массиве.

После этого у Вас остаётся целый час чтобы соединить три вызова этого метода, уже знакомыми Вам if-ами.

Ну это из моей прикидки, что Вам потребуется 2 часа для завершения задания.

[KOzh1r]

Код: (C#)

string rus = "";
            string b = "рус";
            for (int i = 0; i < b.Length; i++)
            {
                char ch = request[i];
                rus = rus + ch;
            }
            string key = request.ToLower();

так реализован метод внесения в переменную rus первых введённых трех символов в request.

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

GetSubString - вообще не понял зачем нужен, если начальная позиция мне не известна?

[Джон]

Вы понимаете, что Вы делаете? Этот код просто копирует переменную b в rus, можете просто записать rus = b;
Но зачем? Что Вам это даёт?

Добавлено через 3 минуты и 33 секунды:
Я так понимаю у Вас проблемы не с кодом, а с алгоритмом.

Забудьте про код, опишите просто словами последовательность действий нахождения в запросе нужной инфы.

На входе "Покажи руския".

Ваши действия?

[Джон]

Я пока "подправлю" Ваш метод. Как цикл работает Вы поняли, это уже хорошо.

[KOzh1r]

1. Вход - "покажи руския"
2. Разбиваем запрос на слова - "покажи" и "руския"
3. Ищем, записанный в переменную, ключ "рус", в этих словах
4. Находим - "руския"
4. Возвращаем слово целиком

[Джон]

Итак

В Вашем коде Вы "вытаскиваете" последовательно символы и сохраняете их в буфер - это то что нам нужно. Только
1. Вытаскивать надо из request, который передаётся в метод в качестве параметра source
2. Сколько? 3 символа, значит цикл должен рабоать только 3 раза

дальше коменты в коде

Код: (C#)

private static string GetSubString(string source, int startPosition, int length)
{
            string rus = ""; // а для вытаскивания японских этот метод разве не подойдёт? ну ладно пусть будет rus
            // string b = "рус"; // это к чему? выкидываем нафик
            for (int i = 0; i < b.Length; i++) // почему с 0? стартовая позиция передаётся в метод в качестве параметра -> startPosition
            // конечное условие? длина последовательности -> length, но нам нужен индекс символа, совершенно очевидно, что если мы начнём "вытаскивать" с 5-ой позиции, то условие 5 < 3 будет изначально ложно, и цикл не выполнится ни разу
            // что надо сделать? пересчитать индекс символа с учётом стартовой позици. Логика: получить 3 символа, начиная с пятой позиции, значит какие индексы нам нужны? 5,6,7 или?
            {
                char ch = request[i];
                rus = rus + ch;
            }

        return rus;
}

Вот и всё... Уже можно работат, НО опасно. А что если метод будет вызван со стартовым индексом 134? Я уже говорил, что доступ к не существующим индексам приведёт к обрушению программы. Значит, перед тем как его запускать
надо проверить стартовое и... конечное значение. Ведь если в метод будет передан индекс последнего символа, то на втором шаге индек будет больше количества элементов в строке.
Это делается простыми известными Вам if-ами

[Джон]

1. Вход - "покажи руския"
2. Разбиваем запрос на слова - "покажи" и "руския"
3. Ищем, записанный в переменную, ключ "рус", в этих словах
4. Находим - "руския"
4. Возвращаем слово целиком

Ок, понятно. Вы всё-таки хотите разбивать на слова. Можно конечно и так. Я так понимаю метод из п.2 готов?

[KOzh1r]

Дело не в том, как Я хочу, но в том, как Я понимаю реализацию. Вот в этом методе Я вообще запутался.

Вытаскиваю Я символы и сую их в буфер. Отлично. Хм, хотя.. А для чего Я это делаю? Допустим Я перебрал и записал три символа с восьмой позиции в запросе "покажи руския ... " и получил "рус". Прога отработала, всё ок. А если запрос будет "какие руския ..."? Тогда Я выдерну уже "уск" и прога не отработает.

Предположим, есть какие-то методы, для поиска нужных символов в запросе. Они сами определят где начинается" руския" и начнут отсчёт именно с этого символа. Но тогда зачем весь этот изначальный перебор? Если программа уже знает, где начинается "руския", то зачем искать это слово заново? В общем Я окончательно запутался..

[Джон]

Всё, мне надо убегать. Посему.

Это даже хорошо, что у Вас "свой" алгоритм. Тогда я могу показать "свой". На его основе, Вы можете реализовать свой. И это не будет "плагиатом".

У Вас в любом случае будут проблемы с п.3. Те необходимо будет рализиовать более простой алгоритм, котороый я предлагал. С тем исключением, что "мой" метод работает дольше, конечно в зависимости от того, как Вы реализуете метод "вытаскивания" слов.

Как дети складывают буквы в слова?

М--А МА  М--А МА... а МАМА

Те это самый простой, "примитивный" способ. Но он работает.

Те нам надо найти определённую последовательность символов, которую мы "узнаем",
точнее предположим, что "узнали"

Допустим есть запрос

"Покажи руския"

Предполагаем, что комбинация из трёх символов может дать нам достаточно инфы, для распознавания.

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

0 Пок
1 ока
2 каж
3 ажи
4 жи
5 и р
6  ру
7 рус
8 уск
и тд

Эту ф-ю будет выполнять метод GetSubString. Те "вытаскивать" из "Покажи руския" поочерёдно по три символа

Определим правила распознавания последовательности.

Какие символы допустимы на первом месте комбинации?
Очевидно, что для "русских" это "Рр"(простой идеальный вариант, а какие могут быть ещё?)
На втором "Уу"(простой идеальный вариант, а какие могут быть ещё?)
На третьем "Сс" (простой идеальный вариант, а какие могут быть ещё?)

Те нам надо проверить наличие символа в каком-то массиве заданных символов.
Эту ф-ю выполняет метод IsCharInArray.
Сначала мы проверяем первый символ, если тест проходит успешно, возрастает вероятность, что это "знакомая" комбинация.
Проверяем второй символ.
Аналогично третий.
Когда все три теста будут успешными, мы с большой долей вероятности, можем сказать, что пользователь хочет посмотреть "русские" марки машин.
Если хотя бы один из тестов выдаёт false, переходим к след. последовательности.
Продолжаем пока не найдём "знакомую" или не переберём все символы ввода.

В примере выше, мы находим нужную комбинацию на восьмом шаге.

Если же мы когда-нибудь решим, что последовательности из 3-х символов недостаточно, мы расширим её до 4-х, 5-ти и тд.
Но на для теста такого уровня достаточно и 3-х.

Абсолютно любой ввод распознать невозможно (пользователь уснул на клавиже Ж).

Это код, который реализует этот алгоритм, в нём необходимо только дополнить разрешённые символы на соответствующих позициях.

[Джон]

Предположим, есть какие-то методы, для поиска нужных символов в запросе. Они сами определят где начинается" руския" и начнут отсчёт именно с этого символа. Но тогда зачем весь этот изначальный перебор? Если программа уже знает, где начинается "руския", то зачем искать это слово заново? В общем Я окончательно запутался..

Хе? Программа не знает НИЧЕГО!!! Точнее, она знает только то, чему мы её научим. Что значит "предположим"? Методы есть только те, которые мы САМИ СОЗДАЁМ. Никакой другой манны небесной нет.

Итак, код

(click to show)

Код: (C#)

        private static bool HasRUS(string request)
        {
            for (int i = 0; i < request.Length; i++)
            {
                string keySequence = GetSubString(request, i, 3);
                if(VerifyRUS(keySequence))
                {
                    return true;
                }
            }
            return false;
        }

        private static bool HasJAP(string request)
        {
            for (int i = 0; i < request.Length; i++)
            {
                string keySequence = GetSubString(request, i, 3);
                if (VerifyJAP(keySequence))
                {
                    return true;
                }
            }
            return false;
        }

        private static bool VerifyRUS(string keySequence)
        {
            if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))
            {
                string allowedInFirstPos = "Рр";
                string allowedInSecondPos = "УуОоАа";
                string allowedInThirdPos = "Сс";

                if (IsCharInArray(keySequence[0], allowedInFirstPos))
                {
                    if (IsCharInArray(keySequence[1], allowedInSecondPos))
                    {
                        if (IsCharInArray(keySequence[2], allowedInThirdPos))
                        {
                            return true;
                        }
                    }
                }
            }
            return false;
        }

        private static bool VerifyJAP(string keySequence)
        {
            if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))
            {
                string allowedInFirstPos = "ЯяИи";
                string allowedInSecondPos = "Пп";
                string allowedInThirdPos = "Оо";

                if (IsCharInArray(keySequence[0], allowedInFirstPos))
                {
                    if (IsCharInArray(keySequence[1], allowedInSecondPos))
                    {
                        if (IsCharInArray(keySequence[2], allowedInThirdPos))
                        {
                            return true;
                        }
                    }
                }
            }
            return false;
        }

[Джон]

До завтра Вам необходимо рализовать метод разбивания запроса на слова и метод поиска нужной последовательности в слове.
За основу можете взять методы Has* и Veirfy*, как Вы заметили они функционально идентичны для стран.

Я думаю, что буду завтра в онлайне где-то часов с 12-13 (GMT+1)

[KOzh1r]

В общем, буду откровенен. В последних Ваших сообщениях Я понимаю только то, что написано русским языком. То, что написано в коде С# до меня не доходит. Равно как и то, что необходимо сделать для того, чтобы всё работало. После того, как Я с Вашей помощью, стал разбирать метод "FindKey", Я вообще перестал понимать код. Может Я настолько глуп, но вот эти фразы типа "IsCharInArray", "!string.IsNullOrEmpty" и "ManufactureCountryKey" вообще не ясны. Вернее, Я понимаю(или думаю что понимаю) что они значат в конкретном участке кода, но их влияние на программу в целом остаётся загадкой. Ранее Я понимал как работает программа по информации, которая выводится. По результату. Теперь Я вообще не контролирую код. Компилятор постоянно выдает ошибки, которые Я допускаю пытаясь собрать код в одно целое. Кроме того, какое-то из этих слов нужно внести наверх кода, в метод Main... Какие? Да черт его знает.

Просто теперь программирование теперь превратилось в попытки угадать что и куда нужно сунуть, не понимая как вообще происходит процесс..

Джон, спасибо Вам большое за помощь и за усердные попытки научить меня понимать и работать с С#. Вы какой-то исключительный человек, чесслово. Не надоело же возиться со мной, с нубом_)) Похоже Я дойду до решения самостоятельно через пару-тройку месяцев. Жаль что тест провалю, столько заданий выполнял и все зря_)) Спасибо Вам ещё раз и уважение! 

[Джон]

Да ладно Вам, насколько я понимаю тест надо сдать завтра? Так ещё не всё потеряно.

В общем, буду откровенен.

Иного я и не жду, в противном случае ничего не получится.

столько заданий выполнял

А какого типа были ЭТИ задания? Те они не были связаны с кодированием?

Компилятор постоянно выдает ошибки, которые Я допускаю пытаясь собрать код в одно целое.

Так покажите Ваш код, на ошибках же учатся.

В последних Ваших сообщениях Я понимаю только то, что написано русским языком.

Ок, ещё раз и по полочкам.

1. Программа не есть код.
Посему сначала пишем программу на понятном нам языке, вместо сигнатур (так называются страшные строчки типа private static bool HasRUS(string request)) я буду писать что происходит в этом месте
Я не буду вдаваться в объяснения, как происходит объявление переменных, или что у них должет быть определён тип. С этим, как и синтаксисом, судя по первому коду, у Вас проблем нет.

Итак

Изначальная точка входа в Программу, здесь всё начинается, а заканчивается после закрывающей скобки, см внизу
{
      буфер запроса, в котором будет храниться введённое значение = Получить введённое пользователем предложение/запрос

      ключ (ключевая информация), который, либо однозначно определяет данные для вывода (рус или яп), либо содержит инфу и нераспознанном запросе = Получить ключ из буфера запроса

      буфер результата = Получить данные по ключу

      Показать буфер результата пользователю

} // когда выполнение дойдёт до этой скобки программа завершится

Всё, это и есть вся программа. Логика, алогоритм и тд и тп. Это то, что мы хотим сделать. Необходимая последовательность действий, выполнив которые мы получим результат.
Такая запись понятна нам, но непонятна компьютеру. Здесь на помощь приходит язык программирования (ЯП). Какой? По сути не важно. Каждый ЯП, так же как и в человеческих языках, имеет свои преимущества и недостатки.
По большому счёту можно выбирать любой. Поэтому в задании Вашего теста не конкретизируется, на каком именно ЯП надо выполнить задание.

Вы выбрали C#. Очевидно, что какие-то вещи из тех, которые нам нужны для "объяснения" нашей задумки компьютеру, уже присутствуют в ЯП. С чего-то надо начинать. А вот остальные нам придётся "добавить" самим.

Запишем нашу программу на языке C#.
К сожалению, большинство компиляторов (это те штуки, которые переводят код ЯП в код понятный процессору) не понимают русский язык. Поэтому будем использовать английские буквы и слова.
Синим цветом выделены уже известные в C# слова.
Коричневым - слова определённые программистом по правилам языка C# и понятные компилятору
Красным -  слова определённые программистом по правилам языка C#, но ещё пока непонятные компилятору

// Изначальная точка входа в Программу, здесь всё начинается, а заканчивается после закрывающей скобки, см внизу
public static void Main()
{
      // буфер запроса, в котором будет храниться введённое значение = Получить введённое пользователем предложение/запрос
      string request = GetRequest();

      // ключ (ключевая информация), который, либо однозначно определяет данные для вывода (рус или яп), либо содержит инфу и нераспознанном запросе = Получить ключ из буфера запроса
       /*тип этой переменной не содержится в стандартном наборе C# и нами тоже пока ещё не определён*/ key = FindKey(request);

      // буфер результата = Получить данные по ключу
      string data = FindData(key);

      // Показать буфер результата пользователю
      OutputResult(data);

} // когда выполнение дойдёт до этой скобки программа завершится

public static void Main() -> известно компилятору изначально, нам это даже зачастую предоставляется средой разработки. Так же как и, например, string.
А вот
 *  GetRequest
 *  FindKey
 *  FindData
 *  OutputResult
неопределены в С#, поэтому нам надо создать их самим. Мы уже сообщили компилятору желаемую последовательность действий. Теперь надо "расшифровать" для него, что же
мы конкретно имеем ввиду под тем или иным действием, до сих пор неизвестным компилятору.

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

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

Небольшой оффтоп, простенькая аллегория:

(click to show)

Представьте себе, что мы создаём неких искусственых собачек, каждая из которых может выполнять определённое задание:
 *  GetRequest: сбегай, принеси, что там пользователь ввёл
 *  FindKey: найди в том, что принёс GetRequest, ключ и принеси его
 *  FindData: понюхай ключ и принеси все данные с таким же запахом
 *  OutputResult: отнеси данные пользователю

Такие программки называются методы.

Как определять в коде методы я уже подробно описал тут.

Если есть непонятки, тогда вернёмся к этому моменту.

Так же я не буду останавливаться на реализации методов GetRequest,  FindData,  OutputResult.

Опять же, если есть непонятки, тогда вернёмся и к этому моменту.

Пока всё ОК?

[KOzh1r]

А какого типа были ЭТИ задания? Те они не были связаны с кодированием?

Задания наподобие
"Напишите программу, которая выводит "Hello, World!" 10 раз, каждый на своей строке"
или
"По данному трехзначному числу, определите образуют ли цифры данного числа возрастающую последовательность"

Пока всё ОК?

Сложности лишь по той причине, что много новой информации. Я не знаю как именно написать (какие слова, методы, переменные или названия использовать) чтобы компьютер понял чего Я хочу. Для какого-то метода есть обязательные условия, один должен что-то возвращать, другой наоборот не должен. Следующий возвращает один тип данных и не возвращает другой. Значит надо как-то изменить этот тип. В общем моих знаний на уровне неофита явно мало.

[Джон]

Делам паузу

Сложности лишь по той причине, что много новой информации. Я не знаю как именно написать (какие слова, методы, переменные или названия использовать) чтобы компьютер понял чего Я хочу. Для какого-то метода есть обязательные условия, один должен что-то возвращать, другой наоборот не должен. Следующий возвращает один тип данных и не возвращает другой. Значит надо как-то изменить этот тип.

Так, стоп, это я всё понял. На этом этапе, Вашего "словарного" запаса достаточно, чтобы СЕГОДНЯ завершить это задание.

А остальное... Вы прочитали предыдущий пост? Моя цель, как раз и объяснить Вам, что МЫ САМИ определяем методы, МЫ САМИ решаем как и что они делают, и что возвращают.
Для этого Вам не нужно знать ничего "нового", мы оперируем только тем, что Вам уже известно. От switch мы (Вы) отказались, как работает for Вы разобрались. Больше нам ничего не надо.
Кстати, какой цикл Вы использовали здесь:

которая выводит "Hello, World!" 10 раз, каждый на своей строке

?

Если уж будет совсем "напряжно", то откажемся и от enum.

Но я пока не потерял надежду, посему...

Продолжаем

Концентрируемся на FindKey.

Начнём с типа ключа, да и вообще с ключа.

В Вашем первом коде различие в марках делалось след. образом

Код: (C#)

               if(key == b || key == c || key == d || key == e)
               {
                    result = "Ваз " + "Газ";
               }
               else
               {
                    result = "Nissan " + "Toyota " + "Mazda";
               }

Те Вы различали определённо русские марки и... все остальные были японскими. Достаточно пользователю было ввести какую-нить чушь (уснуть на кнопке Ж), как он тут же получал японские модели машин. А чтобы Вы делали, если бы в задании требовалось показать российские, японские и немецкие? А ещё французкие? А ещё корейские? И тд.
Как Вы, как человек, различаете, что есть российские, а что японские? (вопрос риторический)
Так а откуда же это ВСЁ должен знать компилятор?

По-хорошему, если бы у нас была некая база данных, то тогда таблица с данными о марках машин содержала бы две, примерно такие, колонки:

Страна-производитель	Марка
Россия	Газ
Япония	Nissan
Япония	Toyota
Япония	Mazda
Россия	Ваз

Очевидно, что в левой колонке находится информация, позволяющая однозначно определить принадлежность марки машики к стране-производителю. Назовём эту инфу "ключ".
Таким образом наш "ключ" может принимать два значения "Россия" и "Япония", но всегда остаётся и третья возможность "ни то, ни то", или "все остальные". Поскольку,
в нашем случае, эта третья возможность получится только тогда, когда мы не распознаем в запросе ни Россию, ни Японию, установим это третье состояние как "Неопределённое".

Если тип данных ключа будет строка, то мравниваение ключа с искомым значением, для нас, человеков, происходит гораздо быстрей, тк мы его читаем. Для компьютера, наоборот,
легче сравнивать, более простые типы данных. Например целочисленный.
Таким образом таблица для компьютера будет выглядеть примерно так

Страна-производитель	Марка
1	Газ
2	Nissan
2	Toyota
2	Mazda
1	Ваз

Но теперь нам непонятно, а что же такое 1, 2 и... помним про "Неопределённое", присвоим ему  0.
Те нужна некая "легенда"

Смысл ключа	Код ключа
Неопределён	0
Россия	1
Япония	2

[Джон]

Продолжу здесь, чтобы сохранить очерёдность постов.

Собственно мы уже подошли к enum.

Он позволяет нам объединить простоту понимания как для человека (мы будем читать в коде понятные нам слова, а не 0 1 2 ), так и для компьютера, который будет работать с 0 1 2.

Внешне конструкция enum выглядит как определения типа. Ну Вы ведь уже пользуетесь такими типами данных как int, string, bool, которые уже находятся в словаре C# и известны компилятору. enum в данном случае позволяет нам определить свой собственный тип и дать ему ЛЮБОЕ имя, которое мы захотим. Я решил дать ему имя ManufactureCountryKey, те это 3 соединёных вместе слова Manufacture Country Key (Производитель Страна Ключ), каждое из которых начинается с большой буквы для удобства прочтения (manufacturecountrykey читать гораздо неудобней). Да, по этой же причине, имена методов состоящие из более чем одного слова
записываются таким же образом (Get Request, Find Key и тд). Такой способ записи имеет своё название CamelCase, но об этом Вы почитаете, когда мы расправимся с заданием.

Итак, первые два "слова" в определении enum понятны? Первое, собственно enum - ключевое слово языка C#, которое сообщает компилятору, что за ним последует определение нового типа данных в общем и непосредственно за ним Именем этого нового типа, которое выбрал программист.

Далее следует "тело" этого нового типа. Как мы уже знаем, "тело" принято брать в фигурные скобки.

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

        enum ManufactureCountryKey
        {
            Undefined = 0,
            Russia = 1,
            Japan = 2
        };

Чтобы долго не заморачиваться, я внёс в этот тип слова, которые соответствуют занчениям нашего ключа. Undefined - Неопределённый, Russia - Россия, Japan - Япония.
Для пущей наглядности, я ввёл целочисленные значения для этих "слов" ( =0, =1, =2), такая запись не является синтаксической ошибкой, но компилятор делает это из без такой записи.
Те и без  =0, =1, =2 реальные значения равны соответствено 0 1 2


Я бы мог кончено определить этот тип и так

        enum KljutchStranyProizvoditelja
        {
            Neopredeljonnyj,
            Rossija,
            Japonija
        };

Но всё-таки лучше использовать английские слова, те слова языка, который худо бедно понимает каждый программис, и следовательно поймёт и Ваш код.
В данном случае, для Вашего теста, Вы можете использовать русский транслитеративно. Может это даже будет полезно, если вы все определённые нами методы, переименуете таким образом.
Ну типа FindKey -> NajtiKljutch.

Всё, после определения такого типа в классе, мы можем им пользоваться везде в коде как и любым другим типом данных (для сравнения я привожу пример использования типа int в аналогичных ситуациях).

Возвращать его как результат метода

Код: (C#)

int FindKey()
ManufactureCountryKey FindKey()

Передавать в качестве параметра в метод

Код: (C#)

string GetData(int key)
string GetData(ManufactureCountryKey key)

Определять переменные этого типа:

Код: (C#)

int keyInt; // переменная типа int
ManufactureCountryKey keyEnum; // переменная типа ManufactureCountryKey

Разница заключается только в том, что переменной типа ManufactureCountryKey может быть присвоено значение ТОЛЬКО из тех, которые перечислены в теле этого типа.

Код: (C#)

int i = 3;

ManufactureCountryKey keyEnum = 0; // ошибка, 0 не находится в списке разрешённых величин
ManufactureCountryKey keyEnum = ManufactureCountryKey.Undefined; // всё ОК

Запись ManufactureCountryKey.Undefined означет, что имеется ввиду величина Undefined именно типа ManufactureCountryKey, а не какого-то другого типа, у которого тоже есть величина с таким именем. Например у меня может быть и другой тип

        enum MyColorEnumType
        {
            Undefined,
            red,
            green,
            blue
        };
тогда
            MyColorEnumType my_color = ManufactureCountryKey.Undefined; // ошибка
            ManufactureCountryKey key = MyColorEnumType.Undefined; // ошибка

аналогично если Вы попробуете сделать след.

Код: (C#)

            string st1 = "";
            int n234 = st1; // ошибка

А так всё в порядке

Код: (C#)

            MyColorEnumType my_color = MyColorEnumType.Undefined;
            ManufactureCountryKey key = ManufactureCountryKey.Undefined;

Ну вроде всё.

Переходим к реализации FindKey? Или есть непонятки?

[Джон]

Итак метод FindKey

Мы помним, что в нашей ПРОГРАММЕ (где всё написано человеческим языком) вторым заданием явлется нахождение ключа

ключ (ключевая информация), который, либо однозначно определяет данные для вывода (рус или яп), либо содержит инфу и нераспознанном запросе = Получить ключ из буфера запроса

Тип ключа мы уже определили, далее в русскоязычном тексте я будут называть его Тип Ключа, в коде C# использовать имя введённого типа ManufactureCountryKey, только цвет у него будет не коричневый, а синий, как и у других типов int, string. Нууу это всё-таки тип, как-никак.

Сначала определяем, что должна делать эта программка-собачка. Совершенно очевидно, что это один из самых простых методов, и по идее можно обойтись и без него, а записать тело метода напрямую в метод Main, но мы учимся создавать хорошо читабельный код, а самое главное - разбивать СЛОЖНЫЕ ЗАДАНИЯ на ПРОСТЫЕ и распределять их по методам-собачкам.

вернуть Тип Ключа после поиска его в буфере запроса
{
      если в буфере запроса содержится хоть какая-нибудь информация (буфер не пуст)
      {
            если проверка наличия ключа для России пройдена успешно
            {
                  возвращаем ключ Россия
            }

            иначе если проверка наличия ключа для Японии пройдена успешно
            {
                  возвращаем ключ Япония
            }
       }
       во всех остальных случаях возвращаем ключ Неопределено
}

Записываем задание на языка C#

Код: (C#)

// вернуть Тип Ключа после поиска его в буфере запроса
private static ManufactureCountryKey FindKey(string request)
{
    // если в буфере запроса содержится хоть какая-нибудь информация (буфер не пуст)
    if (!string.IsNullOrEmpty(request))
    {
        // если проверка наличия ключа для России пройдена успешно
        if (HasRUS(request))
        {
            // возвращаем ключ Россия
            return ManufactureCountryKey.Russia;
        }
        // иначе если проверка наличия ключа для Японии пройдена успешно
        else if (HasJAP(request))
        {
            // возвращаем ключ Япония
            return ManufactureCountryKey.Japan;
        }
    }
    // во всех остальных случаях возвращаем ключ Неопределено
    return ManufactureCountryKey.Undefined;
}

Это понятно?

Компилятору тоже пока всё понятно, или почти всё... Конечно же, он сразу спросит, а что такое HasRUS и HasJAP?

[Джон]

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

Совершенно очевидно, что подпрограммки HasRUS и HasJAP выполняют одну и ту же функциональную задачу, только одна ищет "русский" ключ, вторая "японский".

Поэтому рассмотрим детально только одну из них -> HasRUS.

Итак мы помним, что она должна искать "русский" ключ во введённом пользователм запросе и вернуть истину, если таковой в нём будет найден

вернуть результат проверки наличия "русского" ключа во введённом пользователм запросе
{
    перебираем все позиции в запросе начиная с нулевого
    {
        последовательность для проверки = Получить из запроса последовательность длиной 3 символа, начиная с текущей позиции
        если последовательность соответствует "русскому" ключу
        {
            останавливаем выполнение программки (метода, собачки) и возвращаем результат истина
        }
    }
    "русский" ключ в запросе не найден, возвращем результат ложь
}

Ну и на C#

Код: (C#)

private static bool HasRUS(string request)
// вернуть результат проверки наличия "русского" ключа во введённом пользователм запросе
{
    // перебираем все позиции в запросе начиная с нулевого
    for (int i = 0; i < request.Length; i++)
    {
        // последовательность для проверки = Получить из запроса последовательность длиной 3 символа, начиная с текущей позиции
        string keySequence = GetSubString(request, i, 3);
        // если последовательность соответствует "русскому" ключу
        if(VerifyRUS(keySequence))
        {
            // останавливаем выполнение программки (метода, собачки) и возвращаем результат истина
            return true;
        }
    }
    // "русский" ключ в запросе не найден, возвращем результат ложь
    return false;
}

Совершенно очевидно, что отличие "японской" версии метода (HasJAP) заключается в вызове метода VerifyJAP вместо VerifyRUS.

Поскольку метод VerifyRUS на данном этапе не известен компилятору, необходимо описать и его.

вернуть результат проверки последовательности 3-х символов на соответствие "русскому" ключу
{
    проверку разрешается производить только в том случае, если последовательность состоит из 3-х символов
    {
        определим разрешённые для русского ключа символы на первом месте "Рр"
        определим разрешённые для русского ключа символы на втором месте "УуОоАа"
        определим разрешённые для русского ключа символы на втором месте "Сс"

        если первый символ находится в массиве (списке) разрешённых для первой позиции символов
        {
            проверяем второй символ
            если второй символ находится в массиве (списке) разрешённых для второй позиции символов
            {
                проверяем третий символ
                если третий символ находится в массиве (списке) разрешённых для третьеё позиции символов
                {
                    возвращаем результат истина
                }
            }
        }
    }
    тест как минимум одного символа не пройден, возвращаем результат ложь
}

Теперь всё тоже самое на языке C#

Код: (C#)

// вернуть результат проверки последовательности 3-х символов на соответствие "русскому" ключу
private static bool VerifyRUS(string keySequence)
{
    // проверку разрешается производить только в том случае, если последовательность состоит из 3-х символов
    if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))
    {
        // определим разрешённые для русского ключа символы на первом месте "Рр"
        string allowedInFirstPos = "Рр";
        // определим разрешённые для русского ключа символы на втором месте "УуОоАа"
        string allowedInSecondPos = "УуОоАа";
        // определим разрешённые для русского ключа символы на втором месте "Сс"
        string allowedInThirdPos = "Сс";

        // если первый символ находится в массиве (списке) разрешённых для первой позиции символов
        if (IsCharInArray(keySequence[0], allowedInFirstPos))
        {
            // проверяем второй символ
            // если второй символ находится в массиве (списке) разрешённых для второй позиции символов
            if (IsCharInArray(keySequence[1], allowedInSecondPos))
            {
                // проверяем третий символ
                // если третий символ находится в массиве (списке) разрешённых для третьеё позиции символов
                if (IsCharInArray(keySequence[2], allowedInThirdPos))
                {
                    // возвращаем результат истина
                    return true;
                }
            }
        }
    }
    // тест как минимум одного символа не пройден, возвращаем результат ложь
    return false;
}

Совершенно очевидно, что отличие "японской" версии метода (VerifyJAP) заключается в другом наборе символов разрешённых в позиции 0, 1, 2.
Пока... но возможно его придётся расширить до последовательности 4-х символов, если учесть, что пользователь может написать "йапонские".
Или даже учитывать оба варианта.
Но это уже дело техники, чётко сформулированного задания и очень сильного желания заказчика непомерно увеличить бюджет проекта.

Вот собственно и всё, все шаги нашей ПРОГРАММЫ объяснены компилятору на его "родном" языке. Собачек можно спускать.

Что непонятно?

[Джон]

Если всё понятно, то остаётся только рализовать Ваш метод с разбивкой на слова.

Сначала, на нормальном (русском) языке записываете/описываете новый метод для теста ключа в словах.
Ессно в нём Вы будете вызывать другие "новые" методы. Например вспомогательный метод для "выдёргивания" слова из запроса.

Не старайтесь сделать ВСЁ в одном методе. Сформулируйте конкретную задачу этого метода и разбейте её на подзадачи. Реализуйте методы для подзадач и тд.

После этого "прогоните" метод на его работоспособность (поставьте себя на место компьютера и "выполните" код) и, если всё работает, переходите к кодированию, записи алгоритма на языке C#.

Результат - в студию.

[KOzh1r]

Из непонятного:
Методы GetSubString и IsCharInArray как реализуются?

GetSubString - должен разделять введённую информацию на несколько символов и записывать каждую группу в свою переменную или создавать некий массив, с данными по три символа;

IsCharInArray - должен произвести анализ в переменных, массиве или просто в методе GetSubString?

[KOzh1r]

Пока получается вот это:

Код: (C#)

using System;
using System.Text;

public class MainClass
{
    enum ManufactureCountryKey
    {
        Undefined,
        Russia,
        Japan
    };

    public static void Main()
    {
        string request = GetRequest();
        ManufactureCountryKey key = FindKey(request);
        string data = FindData(key);  
        OutputResult(data);
    }

    private static bool HasRUS(string request)
    {
        for (int i = 0; i < request.Length; i++)
        {
            string keySequence = GetSubString(request, i, 3);
            if (VerifyRUS(keySequence))
            {
                return true;
            }
        }
        return false;
    }

    private static bool HasJAP(string request)
    {
        for (int i = 0; i < request.Length; i++)
        {
            string keySequence = GetSubString(request, i, 3);
            if (VerifyJAP(keySequence))
            {
                return true;
            }
        }
        return false;
    }
    private static bool VerifyRUS(string keySequence)
    {
        if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))
        {
            string allowedInFirstPos = "Рр";
            string allowedInSecondPos = "УуОоАа";
            string allowedInThirdPos = "Сс";
            if (IsCharInArray(keySequence[0], allowedInFirstPos))
            {
                if (IsCharInArray(keySequence[1], allowedInSecondPos))
                {
                    if (IsCharInArray(keySequence[2], allowedInThirdPos))
                    {
                        return true;
                    }
                }
            }
        }
        return false;
    }

    private static bool VerifyJAP(string keySequence)
    {
        if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))
        {
            string allowedInFirstPos = "ЯяИи";
            string allowedInSecondPos = "Пп";
            string allowedInThirdPos = "Оо";

            if (IsCharInArray(keySequence[0], allowedInFirstPos))
            {
                if (IsCharInArray(keySequence[1], allowedInSecondPos))
                {
                    if (IsCharInArray(keySequence[2], allowedInThirdPos))
                    {
                        return true;
                    }
                }
            }
        }
        return false;
    }
    private static string GetRequest()
    {
        Console.WriteLine("Введите запрос");
        string a = Console.ReadLine();
        return a;
    }
    private static ManufactureCountryKey FindKey(string request)
    {        
        if (!string.IsNullOrEmpty(request))
        {
            if (HasRUS(request))
            {
                return ManufactureCountryKey.Russia;
            }
            else if (HasJAP(request))
            {
                return ManufactureCountryKey.Japan;
            }
        }
        return ManufactureCountryKey.Undefined;
    }

    private static string FindData(ManufactureCountryKey key)
    {
        StringBuilder sb = new StringBuilder();

        switch (key)
        {
            case ManufactureCountryKey.Russia:
                {
                    sb.Append("Ваз");
                    sb.Append(" ");
                    sb.Append("Газ");
                }
                break;
            case ManufactureCountryKey.Japan:
                {
                    sb.Append("Mazda");
                    sb.Append(" ");
                    sb.Append("Toyota");
                    sb.Append(" ");
                    sb.Append("Nissan");
                }
                break;
            default:
                {
                    sb.Append("По Вашему запросу данных нет");
                }
                break;
        }

        string result = sb.ToString();
        return result;
    }
    private static void OutputResult(string data)
    {
        Console.WriteLine(data);
    }

}

Не реализованы последние два метода, о которых писал выше. Может Вы увидите еще ошибки?

[KOzh1r]

Код: (C#)

private static string GetSubString (string request)
    {
        string one = "";
        for (int i = 0; i < request.Length; i++)
        {
           char ch = request [i];
           one = one + ch;
        }
    }

Как ограничить "one" тремя символами? Как остановить цикл, когда в переменную попадет нужное значение? Или опять не то делаю Я?

Код: (C#)

private static bool HasRUS(string request)
// вернуть результат проверки наличия "русского" ключа во введённом пользователм запросе
{
    // перебираем все позиции в запросе начиная с нулевого
    for (int i = 0; i < request.Length; i++)
    {
        // последовательность для проверки = Получить из запроса последовательность длиной 3 символа, начиная с текущей позиции
        string keySequence = GetSubString(request, i, 3);
        // если последовательность соответствует "русскому" ключу
        if(VerifyRUS(keySequence))
        {
            // останавливаем выполнение программки (метода, собачки) и возвращаем результат истина
            return true;
        }
    }
    // "русский" ключ в запросе не найден, возвращем результат ложь
    return false;
}

Как Я понимаю, эти два метода действуют аналогично. Только работают с разными вводными данными. Верно?

[Джон]

Как ограничить "one" тремя символами?

Почти правильно, только сигнатура GetSubString у Вас какая? string GetSubString (string request)? А должна быть?

Подсказки
 * добавить в код вспомогательный метод выделения из строки-источника определённого количества символов в строку начиная с определённой позиции
   * private static string GetSubString(string source, int startPosition, int length)
   * не забудьте про проверку индекса

Я пока "подправлю" Ваш метод. Как цикл работает Вы поняли, это уже хорошо.

Итак

В Вашем коде Вы "вытаскиваете" последовательно символы и сохраняете их в буфер - это то что нам нужно. Только
1. Вытаскивать надо из request, который передаётся в метод в качестве параметра source
2. Сколько? 3 символа, значит цикл должен рабоать только 3 раза

дальше коменты в коде

Код: (C#)

private static string GetSubString(string source, int startPosition, int length)
{
            string rus = ""; // а для вытаскивания японских этот метод разве не подойдёт? ну ладно пусть будет rus
            // string b = "рус"; // это к чему? выкидываем нафик
            for (int i = 0; i < b.Length; i++) // почему с 0? стартовая позиция передаётся в метод в качестве параметра -> startPosition
            // конечное условие? длина последовательности -> length, но нам нужен индекс символа, совершенно очевидно, что если мы начнём "вытаскивать" с 5-ой позиции, то условие 5 < 3 будет изначально ложно, и цикл не выполнится ни разу
            // что надо сделать? пересчитать индекс символа с учётом стартовой позици. Логика: получить 3 символа, начиная с пятой позиции, значит какие индексы нам нужны? 5,6,7 или?
            {
                char ch = request[i];
                rus = rus + ch;
            }

        return rus;
}

Вот и всё... Уже можно работать, НО опасно. А что если метод будет вызван со стартовым индексом 134? Я уже говорил, что доступ к не существующим индексам приведёт к обрушению программы. Значит, перед тем как его запускать
надо проверить стартовое и... конечное значение. Ведь если в метод будет передан индекс последнего символа, то на втором шаге индек будет больше количества элементов в строке.
Это делается простыми известными Вам if-ами

Внимательно посмотрите, ниже Вы же сами написали в методе HasRUS:

        // последовательность для проверки = Получить из запроса последовательность длиной 3 символа, начиная с текущей позиции
        string keySequence = GetSubString(request, i, 3);

i - это и есть "текущая" позиция, знчение этой переменной изменяется в цикле, те на первом шаге метод GetSubString будет вызван с параметрами:
GetSubString(request, 0, 3);
GetSubString(request, 1, 3);
GetSubString(request, 2, 3);
и тд

Вспомним пример из описания алгоритма

Это как раз и даёт нам:

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

0 Пок
1 ока
2 каж
3 ажи
4 жи
5 и р
6  ру
7 рус
8 уск
и тд

Таким образом Ваша реализация метода не получает необходимой информации. Она получает только источник = все символы запроса, это правильно. Но,

1. она должна вытаскивать только 3. Можно эти 3 забить жёстко, тогда цикл будет выглядеть так for (int i = 0; i < 3; i++), но это снизит гибкость логики. А если нам потребуется вытаскивать не 3, а 2,4,5... Поэтому лучше передавать длину в качестве параметра.

2. цикл начинается всегда с 0, те сколько бы этот метод не вызывали, он ВСЕГДА будет возвращать только ПЕРВЫЕ три символа. Значит ему надо сообщить дополнительно, с какой позиции надо вытаскивать эти три символа: с нулевой, первой, второй и тд


Добавлено через 3 минуты и 15 секунд:

Как остановить цикл, когда в переменную попадет нужное значение? Или опять не то делаю Я?

В данном случае речь не идёт о "нужном" значении. Или я не понимают что Вы имеете ввиду? Этот метод тупо должен вытаскивать запрошенное количество (в нашем случае 3) символов из переданной в него строки.
Всё. Что это за символы ему по барабану. Анализ не является задачей этого метода.

[Джон]

Как Я понимаю, эти два метода действуют аналогично. Только работают с разными вводными данными. Верно?

Да. Нет.

1. Да, эти методы действуют аналогично
2. Нет, они не получают разные вводные данные

Оба метода, получают строку ввода в качестве параметра, только один пытается найти в ней "русский" ключ, а второй "японский". Единственное отличие HasRUS вызывает VerifyRUS, а HasJPA соответственно VerifyJAP.

private static bool HasRUS(string request) {     for (int i = 0; i < request.Length; i++)     {         string keySequence = GetSubString(request, i, 3);         if (VerifyRUS(keySequence))         {             return true;         }     }     return false; }

private static bool HasJAP(string request) {     for (int i = 0; i < request.Length; i++)     {         string keySequence = GetSubString(request, i, 3);         if (VerifyJAP(keySequence))         {             return true;         }     }     return false; }

И только уже методы VerifyRUS и VerifyJAP, "внутри себя" определяют различные данные для поиска своего собственного ключа. Те самые разрешённые символы в первой, второй и третьей позициях.

private static bool VerifyRUS(string keySequence) {     if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))     {         string allowedInFirstPos = "Рр";         string allowedInSecondPos = "УуОоАа";         string allowedInThirdPos = "Сс";         if (IsCharInArray(keySequence[0], allowedInFirstPos))         {             if (IsCharInArray(keySequence[1], allowedInSecondPos))             {                 if (IsCharInArray(keySequence[2], allowedInThirdPos))                 {                     return true;                 }             }         }     }     return false; }

private static bool VerifyJAP(string keySequence) {     if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))     {         string allowedInFirstPos = "ЯяИи";         string allowedInSecondPos = "Пп";         string allowedInThirdPos = "Оо";         if (IsCharInArray(keySequence[0], allowedInFirstPos))         {             if (IsCharInArray(keySequence[1], allowedInSecondPos))             {                 if (IsCharInArray(keySequence[2], allowedInThirdPos))                 {                     return true;                 }             }         }     }     return false; }

[Джон]

Из непонятного:
Методы GetSubString и IsCharInArray как реализуются?

С первым вроде разобрались.

IsCharInArray - должен произвести анализ в переменных, массиве или просто в методе GetSubString?

Подсказки
 * добавить в код вспомогательный метод для проверки нахождения символа в массиве других смволов
    * private static bool IsCharInArray(char ch, string allowedChars)

Этот метод должен подтвердить или опровергнуть (вернуть true или false) предположение, что символ находится в каком-то массиве символов? А что есть массив символов? Строка.
Те метод получает символ, которой он должен найти, в качестве параметра 1, и 2-ой параметр эта самая строка с разрешёнными (назовём их так) символами.

private static bool IsCharInArray(char ch, string allowedChars)

В цикле перебираем все символы строки и сравниваем каждый с параметром 2. Если они равны, возвращаем true.

[KOzh1r]

Как Я понимаю, эти два метода действуют аналогично. Только работают с разными вводными данными. Верно?

Да. Нет.

1. Да, эти методы действуют аналогично
2. Нет, они не получают разные вводные данные

Оба метода, получают строку ввода в качестве параметра, только один пытается найти в ней "русский" ключ, а второй "японский". Единственное отличие HasRUS вызывает VerifyRUS, а HasJPA соответственно VerifyJAP.

private static bool HasRUS(string request) {     for (int i = 0; i < request.Length; i++)     {         string keySequence = GetSubString(request, i, 3);         if (VerifyRUS(keySequence))         {             return true;         }     }     return false; }

private static bool HasJAP(string request) {     for (int i = 0; i < request.Length; i++)     {         string keySequence = GetSubString(request, i, 3);         if (VerifyJAP(keySequence))         {             return true;         }     }     return false; }

И только уже методы VerifyRUS и VerifyJAP, "внутри себя" определяют различные данные для поиска своего собственного ключа. Те самые разрешённые символы в первой, второй и третьей позициях.

private static bool VerifyRUS(string keySequence) {     if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))     {         string allowedInFirstPos = "Рр";         string allowedInSecondPos = "УуОоАа";         string allowedInThirdPos = "Сс";         if (IsCharInArray(keySequence[0], allowedInFirstPos))         {             if (IsCharInArray(keySequence[1], allowedInSecondPos))             {                 if (IsCharInArray(keySequence[2], allowedInThirdPos))                 {                     return true;                 }             }         }     }     return false; }

private static bool VerifyJAP(string keySequence) {     if (!string.IsNullOrEmpty(keySequence) && (keySequence.Length == 3))     {         string allowedInFirstPos = "ЯяИи";         string allowedInSecondPos = "Пп";         string allowedInThirdPos = "Оо";         if (IsCharInArray(keySequence[0], allowedInFirstPos))         {             if (IsCharInArray(keySequence[1], allowedInSecondPos))             {                 if (IsCharInArray(keySequence[2], allowedInThirdPos))                 {                     return true;                 }             }         }     }     return false; }

Методы "RUS" и "JAP" идентичны, это без проблем понял. Я имел ввиду, что метод GetSubString работает схоже с методом HasRUS.

Не понял сначала, каким образом метод GetSubString работает с данными, которые получает (эти самые три символа) и как именно работает IsCharInArray.