Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Для новичков > Не работает программма, вроде бы все правильно

Автор: ctpz 20.1.2012, 18:42

Задача программы вычислять площадь 3d объекта, созданного из треугольников. По какой-то причине программа доходит только до получения информации о точке х. Дальше дело не идет.

Отладчик и компилятор ошибок не выдает. Среда разработки: Dev-cpp. Компилятор: G++

Код


#include <iostream>
#include <cmath>
#include <vector>

using namespace std;

int main()
{
    vector <float> x;
    vector <float> y;
    vector <float> z;
    vector <float> ab;
    vector <float> bc;
    vector <float> ac;
    vector <float> tr_s;
    float s;
    
    string yn;
    
    int i = 0;
    int j = 0;
    int n = 0;
    
    cout << "Начать создание объекта (y/n)" << endl;
    
    cin >> yn;
    
    while(yn == "y"){
        cout << yn << endl;
        
        cout << "Точка " << i << ":" << endl;
        
        // x
        cout << "Координата х... ";
        cin >> x[i];
        cout << endl;
        
        // y
        cout << "Координата y... ";
        cin >> y[i];
        cout << endl;
        
        // z
        cout << "Координата z... ";
        cin >> z[i];
        cout << endl;
        
        cout << "Продолжить (y/n)" << endl;
        cin >> yn;
        
        ++i;
    }
    
    while(x[j + 2] && y[j + 2] && z[j + 2]){
        // xy
        ab[j] = sqrt(pow(x[j] - y[j] - z[j], 2) + pow(x[j + 1] - y[j + 1] - z[j + 1], 2));
        
        // yz
        bc[j] = sqrt(pow(x[j + 1] - y[j + 1] - z[j + 1], 2) + pow(x[j + 2] - y[j + 2] - z[j + 2], 2));
        
        // xz
        ac[j] = sqrt(pow(x[j] - y[j] - z[j], 2) + pow(x[j + 2] - y[j + 2] - z[j + 2], 2));
        
        // tr_s
        tr_s[j] = sqrt(((ab[j] + bc[j] + ac[j]) / 2) * (((ab[j] + bc[j] + ac[j]) / 2) - ab[j]) * (((ab[j] + bc[j] + ac[j]) / 2) - bc[j]) * (((ab[j] + bc[j] + ac[j]) / 2) - ac[j]));
        
        ++j;
    }
    
    while(tr_s[n]){
        s += tr_s[n];
        ++n;
    }
    
    system("PAUSE");
    
    return 0;
}

Автор: feodorv 20.1.2012, 19:01

Честно говоря, не разбирался, но первое, что бросилось в глаза, это странное условие:

Цитата(ctpz @ 20.1.2012, 18:42

)

while(x[j + 2] && y[j + 2] && z[j + 2]){

Как будто координаты не могут принимать нулевого значения!
У Вас есть счётчик числа вершин i, пользуйтесь им ;)

Автор: Albor 20.1.2012, 19:01

Обращение (x[i]) к пустому вектору.

Автор: ctpz 20.1.2012, 19:09

Цитата(feodorv @ 20.1.2012, 19:01)

Честно говоря, не разбирался, но первое, что бросилось в глаза, это странное условие:

Цитата(ctpz @ 20.1.2012, 18:42

)

while(x[j + 2] && y[j + 2] && z[j + 2]){

Как будто координаты не могут принимать нулевого значения!
У Вас есть счётчик числа вершин i, пользуйтесь им ;)

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

Автор: feodorv 20.1.2012, 19:46

Цитата(ctpz @ 20.1.2012, 19:09

)

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

Не совсем понял, что значит пустая координата. Но точка (0,0,0) вполне может являться вершиной какого либо треугольника...

Цитата(Albor @ 20.1.2012, 19:01

)

Обращение (x[i]) к пустому вектору.

Да, наверное, нужен insert...

Автор: ctpz 20.1.2012, 22:26

Цитата(feodorv @ 20.1.2012, 19:46)

Цитата(ctpz @ 20.1.2012, 19:09

)

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

Цитата(Albor @ 20.1.2012, 19:01

)

Обращение (x[i]) к пустому вектору.

Да, наверное, нужен insert...

1. Но будет как-то странно, если будет куча точек (0,0,0)
2. Что такое insert

Автор: feodorv 21.1.2012, 05:37

Цитата(ctpz @ 20.1.2012, 22:26

)

Но будет как-то странно, если будет куча точек (0,0,0)

Достаточно одной!

Цитата(ctpz @ 20.1.2012, 22:26

)

Что такое insert

У Вас в векторе нет памяти для x[0] (аналогично y[0], z[0], x[1], ...), совсем нет. Её либо нужно заказать, либо воспользоваться методом insert.
http://www.cplusplus.com/reference/stl/vector/ smile

Автор: Result 21.1.2012, 09:12

Цитата(feodorv @ 21.1.2012, 05:37)

Её либо нужно заказать, либо воспользоваться методом insert.

Если просто "заказать" памяти и писать в нее, будет ли вектор знать про свой состав ? К примеру if(!vect.empty()), будет работать как ожидается ?

Автор: Albor 21.1.2012, 10:35

Цитата(Result @ 21.1.2012, 08:12

)

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

Автор: feodorv 21.1.2012, 14:19

Цитата(Result @ 21.1.2012, 09:12

)

Если просто "заказать" памяти и писать в нее, будет ли вектор знать про свой состав ?

Нет, конечно не будет. И вообще я погорячился с заказам памяти, потому что если потом делать перезаказ этой самой памяти (под бОльший объём данных), никто не будет копировать старые данные (а они в векторе не будут числиться существующими) в новую область памяти. Поэтому лучше всего (и корректней всего) пользоваться доступными методами класса vector.

И да, vect.empty() (или соответсвующий итератор) значительно лучше чем

Цитата(ctpz @ 20.1.2012, 18:42

)

while(x[j + 2] && y[j + 2] && z[j + 2])

или

Цитата(ctpz @ 20.1.2012, 18:42

)

while(tr_s[n]){

И было бы недурно ввести какой-нибудь класс типа triangle, который описывал бы треугольник сразу (и для задания которого нужно было бы ввести сразу 9 координат трёх его вершин)....

Автор: Result 21.1.2012, 14:25

а нормальная работа проги и не дойдет до распределения памяти (только если явно не ресольвить) ибо n-ый push_back потрет нелегала.