Programiranje u C++-u - pitanja i odgovori

Mozel neko oobjasniti gdje je ovjdje greska hitno je!!!

#include <cstdlib>
#include <iostream>

using namespace std;
void sortBubble(float a[],int n){
int i,j;
float pom;
for(int i=0;i<n;++i)
{
for(int j=0;j<n-i;++j)
if(a[j]>=a[j+1])
{
pom=a[j];a[j]=a[j+1];a[j+1]=pom;
}
}
}

int main()
{
void displej(float a[],int n);
int n;
cout<<"Unesite br. clanova niza\n";
cin>>n;
float a[n];
cout<<"UNOS\n";
for(int i=0;i<n-1;++i)
{
        cout<<i+1<<".clan\n";
        cin>>a[i];
}
sortBubble(a,n);
displej(a,n);
cout<<endl;
    system("PAUSE");
    return 0;
}
void displej(float a[],int n)
{
 for(int i=0;i<n;++i){
 cout<<a[i]<<" ";}
    
}

Što si ga zakomplicirao, nisi dinamički alocirao memoriju, a morao si, a onda si i u sortu malo pobrkao lončiće, a malo si i u petlji kod unosa niza pogriješio.

#include <cstdlib>
#include <iostream>

using namespace std;
void sortBubble(float *a,int n){
float pom;
for(int i=0;i<n-1;++i)
{
for(int j=0;j<n-1-i;++j)
if(a[j]>a[j+1])
{
pom=a[j];a[j]=a[j+1];a[j+1]=pom;
}
}
}

int main()
{
void displej(float *a,int n);
int n;
cout<<"Unesite br. clanova niza\n";
cin>>n;
float *a = new float[n];
cout<<"UNOS\n";
for(int i=0;i<n;++i)
{
        cout<<i+1<<".clan\n";
        cin>>a[i];
}
sortBubble(a,n);
displej(a,n);
cout<<endl;
delete [] a;
    system("PAUSE");
    return 0;
}
void displej(float *a,int n)
{
 for(int i=0;i<n;++i){
 cout<<a[i]<<" ";}
   
}

for(int i=0;i<n;++i)
{
        cout<<i+1<<".clan\n";
        cin>>a[i];
}

ovo će mu ispisati prvo "2.clan" koliko ja vidim...i ako ima 5 članova taj niz a[] onda je prvi a[0] a zadnji a[4], ne postoji onda a[5] a čini mi se da ga on tamo želi ispisati (i upisati)... stavi i++ ne znam zašto okrećeš možeš samo pogreške dobiti

i ovo je puno elegantnije i ljepše za viditi u sortiranju

for(i=0;i<n-1;i++)
    {
       for(j=i+1;j<n;j++)
          if(x[j]<x[i])
          {
             pom=x[i];
             x[i]=x[j];
             x[j]=pom;
          }
    }

dobro, neće doći (ako je npr n=5) do a[5] jer će provjeriti da i nije manji od n, ali će prvi član biti upisan u a[1] zar ne?

cin>>a[i];

tu je i=1 odmah ako se ne varam (zbog ++i), tj imat će 1 član manje ukupno?

U For petlji među njima razlike nema. Zašto?, pogledaj kako se izvršava For petlja:
1. početna vrijetnost - i=0
2. uvjet - i<=10
3. naredba/blok nardebi
4. inkrementacija/dekrementacija varijable i
Pogledaj sljedeći kod i rezultat ispisa:
#include<iostream>#include<cstdlib>using namespace std;int main(){    for(int i=0; i<=10; i++)    {       cout<<i<<endl;       if(i==10) cout<<endl<<endl;    }    for(int i=0; i<=10; ++i)    {       cout<<i<<endl;    }    system("PAUSE");    return 0;}

Statistička funkcija lower incomplete gamma funkcija se računa prema slici 1 a kod je sljedeći:

double incgamma (double x, double a){ 

double sum=0; 

double term=1.0/a; 

int n=1; 

while (term != 0){ 

sum = sum + term; 

term = term*(x/(a+n)); 

n++; 

} 

return pow(x,a)*exp(-1*x)*sum; 

} 

Meni je sada potrebna upper incomplete gamma funkcija koja se računa prema slici 2, a razlika je jedino u granicama određenog integrala, pa da li netko zna kako to izmjeniti?

e ekipa imam pitanje u jednom zadatku. Znači unosim kvadratnu matricu i sada treba prvojeriti da li je ona jedinična ili ne. (Matrica je jedinična ako su joj na galvnoj dijagonali jedinica, a svi ostali elementi matrice 0.) za sve treba koristiti funkcije. i mene sada muči taj uvjet kako ga postaviti. ako netko zna bio bih mu zahvala! :)

Da znam kako provjeriti to za glavnu dijagonalu, ali treba provjeriti i ostale elemente niza koji moraju biti 0.

for (i=0; i<n; i++)

  for (j=0; j<n; j++)

{

if (i==j)

{

     if (a[i][j]!=1)

             c++; //prije toga je postavljeno na 0.

}

     else

      {

                if (a[i][j]!=0)

                     k++;

        }

if (c==0 && k==0)

cout<<"Matrica je jedinična."<<endl;

else

cout<<"......nije jedinična."<<endl;

evo sada sam tako rješio, sad nezz da li će biti točna za sve unesene kombinacije ali...

hvala! :)

Da to. Na to sam posve zaboravio.

for varijable ti nemaju vitice koje označavaju blok naredbi.

/*An Internet service provider has three different subscription packages for
its customers:
-Package A: For $9.95 per month 10 hours of access are provided. Additional
hours are $2.00 per hour.
-Package B: For $14.95 per month 20 hours of access are provided. Additional
hours are $1.00 per hour.
-Package C: For $19.95 per month unlimited access is provided.
Write a program that calculates a customer’s monthly bill. It should input
customer name, which package the customer has purchased, and how many hours were
used. It should then create a bill that includes the input information and the
total amount due. The bill should be written to a file.
*Input Validation: Be sure the user only selects package A, B, or C. Also, the
number of hours used in a month cannot exceed 744.
21.
Modify the program in problem 20 so it also displays how much money Package A
customers would save if they purchased packages B or C, and how much money
package B customers would save if they purchased package C. If there would be no
savings, no message should be printed……*/

#include <iostream>
#include <fstream>
#include <stdlib.h>
using namespace std;

void Print(string strName, char chPackage, int nHours, float fPrice, int nTime, int nAdditional)
{
    cout << "Customer: " << strName << endl;
    cout << "Package: " << chPackage << endl;
    cout << "Hours used: " << nHours << endl;
    cout << "TOTAL: " << fPrice + (nHours - nTime) * nAdditional << " $" << endl;
}

int main()
{
    cout << "Please select a subscription package:" << endl << endl;
    cout << "Press 1 for package A: For $9.95 per month 10 hours of access " <<
         "are provided." << endl;
    cout << "Additional hours are $2.00 per hour." << endl << endl;
    cout << "Press 2 for package B: For $14.95 per month 20 hours of access " <<
         "are provided." << endl;
    cout << "Additional hours are $1.00 per hour." << endl << endl;
    cout << "Press 3 for package C: For $19.95 per month unlimited access " <<
         "is provided." << endl;
    int nX;

    do
    {
        cout << "Enter your choice: " << endl;
        cin >> nX;
    }while(nX<1 || nX>3);

    char chPackage;

    //default cijena paketa
    float fPrice;

    //vrijeme dobiveno u pojedniom paketu
    int nTime = 0;

    //cijena svakog sata koji prelazi nTime pojedinog paketa
    int nAdditional = 0;

    if(nX==1)
    {
        cout << "Package A selected." << endl;
        chPackage = 'A';
        fPrice = 9.95;
        nTime = 10;
        nAdditional = 2;
    }
    else if(nX==2)
    {
        cout << "Package B selected." << endl;
        chPackage = 'B';
        fPrice = 14.95;
        nTime = 20;
        nAdditional = 1;
    }
    else if(nX==3)
    {
        cout << "Package C selected." << endl;
        chPackage = 'C';
        fPrice = 19.95;
    }

   cout << "Enter your name:" << endl;
   string strName;
   cin >> strName;
    int nHours;

    do
    {
        cout << "Enter how many hours you have used (max. 744 hours):" << endl;
        cin >> nHours;
    }while (nHours < 0 || nHours > 744);

    char chYN;
   cout << "Print bill? <Y/N>" << endl;
   cin >> chYN;
   cout << endl;
   if(chYN == 'y' || chYN == 'Y')
       Print(strName, chPackage, nHours, fPrice, nTime, nAdditional);
   else if(chYN == 'n' || chYN == 'N')
       exit (1);

   ofstream File;
   File.open("ISP_bill.txt");
   File << "Customer: " << strName << endl;
   File << "Package: " << chPackage << endl;
   File << "Hours used: " << nHours << endl;
   File << "TOTAL: " << fPrice + (nHours-nTime) * nAdditional << " $" << endl;
   File.close();

    //float fA/fB/fC su cijene po paketima s uracunatim iskoristenim satima
    float fA, fB, fC;
    if(nHours > nTime)
    {
        fA = 9.95 + (nHours - 10) * 2;
        fB = 14.95 + (nHours - 20) * 1;
        fC = 19.95;
    }
    else
    {
        fA = 9.95;
        fB = 14.95;
        fC = 19.95;
    }


    if(nX==1)
    {
        if( fA > fB && fB > fC )
            cout << "If you had purchased package C you would have saved " <<
                fA - fC << " dollars." << endl;

        if( fA > fB && fB < fC )
            cout << "If you had purchased package B you would have saved " <<
                fA - fB << " dollars." << endl;
    }
    else if(nX==2)
    {
        if( fB > fC)
            cout << "If you had purchased package C you would have saved " <<
                fB - fC << " dollars." << endl;
    }

    return 0;
}

Ovaj kod radi ali mislim da bi se mogao malo poboljsati i uljepsati.

Ako se nesto od ovoga moze krace napisati, slusam. Stvari koje me zanimaju sam oznacio zelenim markerom ali primam i kritike na racun svega ostaloga, pogotovo na stil pisanja. Znam da je zadatak malo duzi od prosijeka na ovoj temi, ali ako netko moze pogledati i dati koji savjet

1. zeleno: kada unesem razmak jednostavno prekoci taj dio i normalno nastavi dalje. getline(cin, strName) ne prihvaca. Htio bih napraviti da se moze unijeti ime i prezime a da ne moram to podijeliti na zasebne varijable.

2. zeleno: htio sam to napisati kao do-while kod cega bi while bio (chYN != 'y' || chYN != 'Y'|| chYN != 'n' || chYN != 'N'); ali iz nekog razloga to ne radi. Tj. radi ali ne kako treba, zapne. I kada unesem y/Y/n/N i dalje vrti onaj do.

3. zeleno: se moze to nekako krace napisati? Ili bolje?

for varijable ti nemaju vitice koje označavaju blok naredbi.

Ma zaboravio sam kod prepisivanja programa napisati vitičaste zagrade (jer mi ne radi ovaj stil kod), ali u programu sam ih stavio ;)

Koristi getline, ja mislio da si to već pokušao pa sam zato predložio vector kao iduće ali vidim sada nema nigdje getline gore :)

evo link http://www.cplusplus.com/forum/articles/6046/