Почему выводит лишние символы?

[evgenrpo]

Я новичок помогите!!!

Код: Выделить всё

#include <iostream>

using namespace std;
void main()
{
	char str1[20];
	char str2[20];
	char *str3;
	int coun=0;
	FILE *fil;
	fil = fopen("text.txt","r");
	while(!feof(fil))
		{
	fgets(str1,20,fil);
			if(str1[0]=='$')
			{
				for(int i = 1; i <= 20; i++)
				{
					if(str1[i]=='$')
					{
					break;
					}
					else
					{
				    coun++;
					}
				}
				str3 = new char [coun];
				cout<<" coun = "<<coun<<endl;
				for(int i=0;i<coun;i++)
				{
				str3[i] = str1[i+1];
				}
				cout<<str3<<endl;
				delete []str3;
				coun=0;
				}
	}
	fclose(fil);
}

должно вывести на консоть слова которые находятся у файле .txt отмеченный метками $бла-бла$.............$бла-бла$.......... а выводит ищо непонятные.

у файле

[Decoder]

str3 = new char [coun];
cout<<" coun = "<<coun<<endl;
for(int i=0;i<coun;i++)
{
str3 = str1[i+1];
}

Исправь этот код так:

Код: Выделить всё

    str3 = new char [coun+1];
    cout<<" coun = "<<coun<<endl;
    for(int i=0;i<coun;i++)
    {
        str3[i] = str1[i+1];
    }
    str3[count] = '\0';

Или вместо этого используй функцию strcpy.

[BBB]

Еще и
for(int i = 1; i <= 20; i++)
сменить на
for(int i = 1; i < lstrlen (str1) ; i++)

[Romeo]

А ещё лучше так:

Код: Выделить всё

int len = lstrlen (str1);
for(int i = 1; i < len; ++i)

Вычисление длины строки на каждой итерации цикла не есть гуд

[evgenrpo]

Большое спасибо помогло только так

Код: Выделить всё

str3[coun] = 0;

[azrael]

А ещё лучше так:

Код: Выделить всё

int len = lstrlen (str1);
for(int i = 1; i < len; ++i)

Вычисление длины строки на каждой итерации цикла не есть гуд

Нет предела совершенству
Еще лучше будет так:

Код: Выделить всё

const int len = lstrlen (str1);
for(int i = 1; i < len; ++i)

Использование констант вместо переменных там, где это возможно, повышает читабельность в разы

[Romeo]

azrael, смысл был в том, чтобы избавиться от вычисления длины строки в цикле и выиграть на перфомансе. Добавление const несомненно правильно с точки зрения культуры кода, но не улучшает ни перфоманс, ни оптимизирует размер. На фоне невысокой культуры остального кода примера подобное добавление может даже сбить автора с толку, потому я не рискнул более изменять данный цикл, хотя здесь вообще с самого начала напрашивался алгоритм count_if c inline предикатом.

[azrael]

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