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

[White Dragon]

Обе задачи затыкаются где то в циклах, я понимаю что не хватает знаний но от этого не легче...

задача 1 на строки

/* Символьная строка заканчивается восклицательным знаком (!).
   Слова в ней отделены друг от друга точкой с запятой (;).
   Найти те слова, которые начинаются и оканчиваются одной и той же буквой*/


#include <iostream.h>
#include <conio.h>
#include <LOCALE.H>
#include <string.h>


void main()
{
 clrscr();          //Чистим экран
 setlocale(0,"");  //Чтобы русские буквы были читабельны
 char str [] = "до;точки;с;запятой;страус;после;точки;с;запятой;текст";
 char str1[11],str2[25];



 int i=0; //j=0;
for (i=0; i<strlen(str); i++) /* Цикл работает пока не встретится признак
			  конца строки */
  {
   if (str[i] == ';' || i == 0)
   {
    if ((i == 0 && str[i] == ';')||str[i] == ';')
     //Если точка с запятой в начале строки
    {
     i+=1;
     int j=0;
     for (j=0; (str[i]!=';' || str[i]!='!'); i++,j++)
     // Пока не дойдем до следующей <;> или конца строки <!>
     {
      str1[j] = str[i]; //Сохраняем слово в промежуточную строку
     }
     if (str1[j] == str1[1]) //Если последняя занесенная буква равна первой
			     // тогда сохраняем результат в 3й строке
     {
      strcat (str2,str1);
      strcpy(str1,'\0');  //Чистим строку
     }
    }
    else
    {
     int j=0;
    for (j=0; ((str[i]!=';') || (str[i]!='!'));i++,j++)
     //Если начало строки без точки с запятой
     {
      str1[j] = str[i];
     }
     if (str1[j] == str1[1])
     {
      strcat (str2,str1);
      strcpy(str1,'\0');
     }
    }
   }
  }

cout << str << endl << "Искомые слова: " << str2 ;

getch ();
}

задача 2 на файлы

/* Дан файл, содержащий текст на русском языке
   и некоторая буква. Найти слово, содержащее
   наибольшее количество указанных букв*/

#include <iostream.h>
#include <conio.h>
#include <LOCALE.H>
#include <string.h>
#include <process.h>    //функция exit()
#include <stdio.h>

void main()
{
 clrscr();          //Чистим экран
 setlocale( LC_ALL,"Russian" );  //Чтобы русские буквы были читабельны
FILE *stream;
   stream = fopen("tdoc.txt", "rt");//только чтение, текстовый режим
   int ch,i=0,j=0,k=0;
   char a = 'а', str[15],str1[15];

   if (stream==NULL) {printf("\nФайл не существует");exit(-1);}
   printf("\nФайл успешно открыт " "\n");

/*   fseek(stream, 0, SEEK_SET);//на начало файла, смещение от начала файла 0 ком-
			 //понетнов
   do {
   ch = fgetc(stream);     //получить символ из файла
   cout<<char(ch);    //вывод в виде символа
      } while (ch != EOF); //до конца файла*/

   fseek(stream, 0, SEEK_SET);
   do {
   ch = fgetc(stream);     //получить символ из файла
    if (ch != ' ')
    cout << ch;
    do
       {
	str[i] = ch;
	ch = fgetc(stream);
	if (ch != ' ' || ch != EOF || ch != '\0') i++;
       }while (ch != ' ' || ch != EOF || ch != '\0');

    for (i=0;i<strlen(str);i++)
       if (str[i]==a) j++;

       if (j>k)
	 {
	  strcpy(str1,'\0');
	  strcpy (str1,str);
	 }
      } while (ch != EOF); //до конца файла

   cout << str1 << endl;
   fclose(stream);//закрыть файл

getch ();
}

[Axrik]

Ошибку пока что не искал, но вот это

int j=0;
for (j=0; ((str[i]!=';') || (str[i]!='!'));i++,j++)

можно записать так

for (int j=0; str[i]!=';' || str[i]!='!'; i++, j++)

И не объявляйте j несколько раз, это ни к чему

[White Dragon]

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

Ошибку пока что не искал, но вот это

int j=0;
for (j=0; ((str[i]!=';') || (str[i]!='!'));i++,j++)

можно записать так

for (int j=0; str[i]!=';' || str[i]!='!'; i++, j++)

И не объявляйте j несколько раз, это ни к чему

Просто искал корень зла и решил, что беда в локальной переменной для цикла, однако дело в чем то еще...

[Abstraction]

strcpy(str1,'\0');//Эквивалентно strcpy(str1, NULL), эквивалентно "программа, падай!"

Компилятор должен был выдать предупреждение о приведении типов. Он это сделал? Если да, приучитесь обращать внимание на предупреждения; если нет - найдите в Вашем компиляторе опцию "выдавать все предупреждения" и включите.

[White Dragon]

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

strcpy(str1,'\0');//Эквивалентно strcpy(str1, NULL), эквивалентно "программа, падай!"

Компилятор должен был выдать предупреждение о приведении типов. Он это сделал? Если да, приучитесь обращать внимание на предупреждения; если нет - найдите в Вашем компиляторе опцию "выдавать все предупреждения" и включите.

нет не показывает, консоль Borland C++ Version 3.1 не знаю где там это включить.
Если вы считаете что проблема в этом, как мне тогда корректно очистить строку?

str1[1]='\0';

с таким вариантом прога так же запускается и не пашет...
Или моя ошибка в чем то другом? может я вас не понял? объясните

[Abstraction]

Borland C++ Version 3.1

Мама.

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

[White Dragon]

Криво работает условие

for (j=0; (str[i]!=';' || str[i]!='!'); i++,j++)
     // Пока не дойдем до следующей <;> или конца строки <!>
     {
      str1[j] = str[i]; //Сохраняем слово в промежуточную строку
     }

в переменную зафигачивается вся строка ПОЧЕМУ!!! и не просто зафигачивается а все продолжает и продолжает всовывать в переменную символы... i все растет и растет...

[Abstraction]

А. Ну логично. Если str[i]=='!', первое условие true, второе false, true || false = true.
Если str[i]==';', первое условие false, второе true, false || true = true.
Если что-то третье, то true||true = true.

[White Dragon]

Abstraction подскажете как это обойти?

[Abstraction]

Я подскажу, как в такие ловушки не попадать. Если не можете сообразить условие в уме - выносите в функцию:

//Проверка, является ли символ концом слова
bool IsEndOfWord(char symbol){
  if(symbol == ';') return true;
  if(symbol == '!') return true;
  return false;
}

//В цикле
for(int j=0; !IsEndOfWord(str[i]); ++i, ++j)

В качестве бесплатного бонуса, теперь очень легко модифицировать условие, если вдруг решим добавить новые разделители...