Problem z programem z tablicami

Question

Witam wykonuję właśnie taki program: program, który wyznaczy wszystkie pary kolejnych elementów tablicy o tej własności, że jeden element z pary jest kwadratem drugiego, lecz nie wiem dlaczego dla nie których par np. 5 i 25, 9 i 81, 15 i 225, program ich nie wypisuje w konsoli. Może ktoś sprawdzić co robię źle bądź co jest nie tak? O to mój kod:

#include <iostream>
#include <math.h>
using namespace std;

int main()
{
    int n;
    cout<<"Podaj rozmiar tablicy: ";
    cin>>n;

    int a[n];

    for(int i=0; i<n; i++)
    {
        cout<<"Podaj element "<<i<<" tablicy: ";
        cin>>a[i];
    }

    int b[n],c[n],i=0,k=1,m=0;

    do
    {
        if(a[k]==pow(a[i],2))
        {
            b[m]=a[k];
            c[m]=a[i];
            m++;
        }
        k++;
        i++;
    }while(k<n);

    cout<<endl<<"Pary kolejnych elementow tablicy spelniajacych warunek: ";

    for(int i=0; i<m; i++)
    {
       cout<<endl<<c[i]<<" "<<b[i];
    }


    return 0;
}

 

Answer 1

Tutaj masz działający kod, zobacz sobie i porownaj co u Ciebie jest zle.

#include <iostream>
#include <math.h>
#include <conio.h>

using namespace std;

int main()
{
    int n;
    cout<<"Podaj rozmiar tablicy: ";
    cin>>n;

    double *a = new double[n];

    for(int i=0; i<n; i++)
    {
        cout<<"Podaj element "<<i<<" tablicy: ";
        cin>>a[i];
    }

    for(int i1 = 0; i1<n; i1++)
       for(int i2 = i1+1; i2<n; i2++)
           if(a[i2] == pow(a[i1],2) || a[i1] == pow(a[i2],2))
               cout<<a[i1] << " oraz "<<a[i2] <<endl;


    getch();
    return 0;
}

 

Comments

Tak wyglada teraz moj kod: 

#include <iostream>
#include <math.h>
#include <cmath>
using namespace std;

int main()
{
    int n;
    cout<<"Podaj rozmiar tablicy: ";
    cin>>n;

    int a[n];

    for(int i=0; i<n; i++)
    {
        cout<<"Podaj element "<<i<<" tablicy: ";
        cin>>a[i];
    }

    for(int i=0; i<n; i++)
    {
        for(int j=1; j<n; j++)
        {
            if(a[i]==pow(a[j],2)||a[j]==pow(a[i],2))
                cout<<"Pary to "<<a[i]<<" oraz "<<a[j]<<endl;
        }
    }

    return 0;
}

Wydruk z konsoli, nie wiem dlaczego nie drukuje tej pary?

---

Szczerze mowiac nie widze co jest nie tak, ale zarwalem nocke na rybach i ogolnie nie bardzo ogarniam. Moj kod dziala, wiec najwyzej zmodyfikuj go sobie tak zeby Ci pasowalo :)

---

Brakło nawiasów do warunku :)

 if((a[i]==pow(a[j],2))||(a[j]==pow(a[i],2)))

 

---

U mnie to dziala bez nawiasow... zreszta w tym ukladzie one nic nie powinny zmieniac

---

Hmm, nie wiem, może to problem kompilatora? Dodałem jeszcze nawiasy i dalej jest to samo, programuje w CodeBlocks 13.12

---

A wklejales moj kod bez zmian? Jesli moj kod u Ciebie nie dziala, to cos jest nie tak

---

Teraz już wszystko, działa, problem się rozwiązał gdy zmieniłem typ tablicy z int na double. Dzięki za pomoc.

Answer 2

Moim zdaniem powinienes to zrobic w dwoch petlach for np:

for(int i1 = 0; i1<n; i1++)

   for(int i2 = i1+1; i2<n; i2++)

       if(a[i1] == pow(a[i2],2))
           cout<<a[i1] << " oraz "<<a[i2] <<endl;

zobacz sobie po kolei jakie wartosci maja zmienne "i" i "k", nie sprawdzasz wszystkich mozliwych zaleznosci  i tu jest blad Twojego programu

Comments

Wykonałem tak jak zaleciłeś, dalej jest to samo co w moim programie jak wpisuje 9 i 81 to nie widzi takiej pary, nie wiem w czym jest problem?

---

wklej kod jak to masz teraz zrobione