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

[kpo4u]

Pусловие:
Напечатать в порядке возрастания корни уравнений , 3ex+x=0, x ln(1+x)=0.5. Определить процедуру нахождения корней уравнения.

программа:

// 6.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <iostream>
#include <cmath>

using namespace std;

double f1(double x)
{
	return 1/(1+x*x) - x;
}

double f2(double x)
{
	return 3*exp(x) + x;
}

double f3(double x)
{
	return x*log(1+x) - 0.5;
}

double Middle(double (*f)(double),double xLeft, double xRight, double epsilon)
{
      double xMiddle; 
      while((xRight - xLeft) > epsilon) 
      { 
           xMiddle = (xLeft + xRight) / 2.0; 

           if(f(xLeft) * f(xMiddle) < 0) 
                xRight = xMiddle; 
           else if (f(xRight) * f(xMiddle) < 0) 
                xLeft = xMiddle; 
           else 
                break; 
      } 
  return xMiddle;
}

int _tmain(int argc, _TCHAR* argv[])
{
	system("CLS");
	setlocale(LC_ALL,"Russian");
	double x1 = Middle(&f1,0,10,0.00001);
	double x2 = Middle(&f2,-10,10,0.00001);
	double x3 = Middle(&f3,0,10,0.00001);
	double root[3];
	root[0] = x1;
	root[1] = x2;
	root[2] = x3;
	for (int i = 0; i < 2; i++)
		for (int j = i+1; j <=2; j++)
			if (root[i] > root[j])
			{
				double swap = root[i];
				root[i] = root[j];
				root[j] = swap;
			}
			cout << "Корни в порядке возрастания: " << endl;
			for (int i = 0; i <= 2; i++)
				cout << root[i] << endl;
	system("PAUSE");
	return 0;
}

преподу не понравилось то что выделено, как исправить ?


Условие:
Дана последовательность символов. Проверить, правильно ли в них расставлены скобки (т.е. находится ли справа от каждой открывающей скобки закрывающая). (Тип string запрещен для использования)

программа:

// 7.cpp : Defines the entry point for the console application.
//

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

using namespace std;


int _tmain(int argc, _TCHAR* argv[])
{
	system("CLS");
	setlocale(LC_ALL,"Russian");
	cout << "Введите строку: ";
	char str[255];
	gets(str);
	int c = 0;
	for (int i = 0; i < strlen(str); i++)
	{
		if (str[i] == '(') c++;
		if (str[i] == ')') c--;
		if (c < 0)
			break;

	}
	if (c == 0)
		cout << "Порядок и количество скобок впорядке" << endl;
	else
		cout << "Скобки расставлены неправильно" << endl;
	system("PAUSE");
	return 0;
}

препод написал "использование необоснованно" как исправить ?



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

программа:

uses crt;
Var str: array[1..20] of string;
    i,j,n: integer;
    f: text;
    c: integer;
Begin
 clrscr;
  assign(f,'1.txt');
  reset(f);
   n:=0;
   while not eof(f) do
    begin
     inc(n);
     readln(f,str[n]);
    End;

  close(f);
  for i:=1 to n do
   Begin
    for j:=1 to length(str[i]) do
     if str[i,j] = '(' then
      inc(c)
     else if str[i,j] = ')' then
      dec(c);
      if c<0 then
       break;
   End;
  if c = 0 then
   writeln(' В файле со скобками все нормально')
  else
   writeln(' В файле не все впорядке со скобками');

 readln;
End.

^организовать без массива


Заранее спасибо за помощь

[Zer0]

в первой видимо избыточность:

	double x1 = Middle(&f1,0,10,0.00001);
	double x2 = Middle(&f2,-10,10,0.00001);
	double x3 = Middle(&f3,0,10,0.00001);
	double root[3];
	root[0] = x1;
	root[1] = x2;
	root[2] = x3;

Почему нельзя сразу присваивать к root?

[kpo4u]

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

в первой видимо избыточность:

	double x1 = Middle(&f1,0,10,0.00001);
	double x2 = Middle(&f2,-10,10,0.00001);
	double x3 = Middle(&f3,0,10,0.00001);
	double root[3];
	root[0] = x1;
	root[1] = x2;
	root[2] = x3;

Почему нельзя сразу присваивать к root?

исправил

[kpo4u]

// 7.cpp : Defines the entry point for the console application.
//

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

using namespace std;


int _tmain(int argc, _TCHAR* argv[])
{
	system("CLS");
	setlocale(LC_ALL,"Russian");
	cout << "Введите строку: ";
	char str[255];
	gets(str);
	int c = 0;
	for (int i = 0; i < strlen(str); i++)
	{
		if (str[i] == '(') c++;
		if (str[i] == ')') c--;
		if (c < 0)
			break;

	}
	if (c == 0)
		cout << "Порядок и количество скобок впорядке" << endl;
	else
		cout << "Скобки расставлены неправильно" << endl;
	system("PAUSE");
	return 0;
}

эту задачу решил

[Mad_Cat]

^организовать без массива

uses crt;
Var str:string;
    i,j,n: integer;
    f: text;
    c: integer;
Begin
 clrscr;
  assign(f,'1.txt');
  reset(f);
   n:=0;
   while not eof(f) do
    begin
     inc(n);
     readln(f,str);
for i:=1 to n do
   Begin
    for j:=1 to length(str) do
     if str[i] = '(' then
      inc(c)
     else if str[i] = ')' then
      dec(c);
   End;
   end;
  if c = 0 then
   writeln(' В файле со скобками все нормально')
  else
   writeln(' В файле не все впорядке со скобками');
 readln;
End

[kpo4u]

Ребята огромное спасибо