adnan best Ответов: 2

Используя строковый указатель, на выходе должно быть имя(должно быть resolve to a, если мы ставим @ и делаем первую букву прописной) и его средняя конечно же отметка


#include<iostream>
#include<string>
using namespace std;
float* f(int n) {
	float* p;
	p = new float[n];
	for (int i = 0; i < n; i++)
	{
		cin >> *(p + i);
	}
	return p;
}

float av(float* a, int j) {
	float sum = 0;
	for (int i = 0; i < j; i++)
	{
		sum = sum + *(a + i);
	}
	return sum / j;
}

string *w( int n) {
	string *j=new string[n];
	for (int i = 0; i < n ; i++)
	{
		getline(cin, *(j+i));
	}
	return j;
}

string *m(string *j, int n) {
	for (int i = 0; i < n ; i++)
	{
		int g = j->find("@");
		while (g != -1)
		{
			j->replace(g, 1, "a");
			g = j->find("@");
		}
		if (i<n-1)
		{
			j++;
		}
		return j;
	}

string * u( string *j, int n ) {
	for(int i=0; i < n; i++) {
		y = j->length();
		for (int f = 0; f < y; f++)
		{
			j->at(0) = toupper(j->at(0));
			if (*(j+f)==" ")
			{
				j->at(f) = toupper(j->at(f));
			}
		}
		if (i < n - 1)
		{
			j--;
		}
	}
	return j;
}

int main() {
	int n,j;
	float max = 0;
	int o = 0;
	cin >> n;
	string *h= w(n);
	string *l = m(h, n);
	string *q = u(l, n);
	for (int i = 0; i < n; i++)
	{
		cout << *(q + i) << endl;
	}
	for (int i = 0; i < n; i++)
	{
		cout << *(q + i) << endl;
		cin >> j;
		float*p = f(j);
		if (i==0)
		{
			 max = av(p, j);
		}
		else
		{
			if (max<av(p,j))
			{
				max = av( p,j);
				o++;
			}
			cout << *(q + o) << endl;
			system("pause");
		}
	}
}


Что я уже пробовал:

я попробовал с j++, но это тоже не сработало

Gerry Schmitz

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

Rick York

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

adnan best

я так и сделал, но обнаружил, что моя проблема связана с выходом:
https://docs.google.com/document/d/1lEeV0VozuhxyfMVshr6KxX0m8468WwNejd3-zQj4JgA/edit?usp=sharing

CPallini

Что, черт возьми, вы пытаетесь сделать (не могли бы вы подробно описать свой сценарий)?

adnan best

https://docs.google.com/document/d/1lEeV0VozuhxyfMVshr6KxX0m8468WwNejd3-zQj4JgA/edit?usp=sharing

2 Ответов

Рейтинг:
2

KarstenK

Вы должны полностью обновить этот код, переименовав функции и переменные в какой-то удобочитаемый формат, например average. И используйте для лучшей читабельности оператор [].

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

Затем вы должны узнать об использовании отладчика и исправить свой код. Подумайте о некотором редизайне с классами или структурами.

PS: никто не любит читать этот код или исправлять беспорядок :-O


Рейтинг:
1

Patrice T

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

Цитата:
я попробовал wihtout j++ это тоже не сработало

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

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]

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


adnan best

сэр это мой выход и код:
https://docs.google.com/document/d/1lEeV0VozuhxyfMVshr6KxX0m8468WwNejd3-zQj4JgA/edit?usp=sharing

Patrice T

Воспользуйся Улучшить вопрос чтобы обновить ваш вопрос.
Чтобы каждый мог обратить внимание на эту информацию.
Описание проблемы - тоже хорошая идея.