[estplayer]

Tere!
Pidage meeles, et olen alles algaja ja ei saa paljust veel aru. Pean tegema programmi C++'s, mis arvutaks inimese kehamassi indeksi, kui ta on oma andmed sisestanud. Praegult on mul nii palju tehtud:

#include <stdio.h>

main()
{
char nimi[100];
printf("Sisestage nimi:\n");
scanf("%s", &nimi);
char P;
printf("Sisestage pikkus sentimeetrites:\n");
scanf("%s", &P);
char K;
printf("Sisestage kaal kilogrammides:\n");
scanf("%s", &K);
float I;
I = K/(P/100)^2;
printf("%f", I);
}
Programmi compiledes ühtegi errorit ei tule, kuid programm jookseb kokku, pärast seda kui sisestada oma kaal kilogrammides. Olen üsna kindel, et viga on mul valemiga seotud, ei oska seda õigesti programmi kirjutada.
Äkki keegi suvatseb aidata?
Aitäh

[RaidoR]

Ei ole küll c++'ga väga kursis, kuid kas pikkus ja kaal mitte double'id ei peaks olema?

[Timukas0]

Valemis ka viga. ^ on XOR tehe, ruutu tõstmiseks kõige lihtsam lihtsalt arv iseendaga korrutada (või pow funktsiooni kasutada math.h teegist).

[keevitaja]

arvud peaks defineerima int või double?

[Ho Ho]

int = täisarv = ümardamine. Igasugu matemaatika jaoks üldjuhul ikka mõttekas float või double kasutada

Btw, see programm on sul siiski seal C's, mitte c++'s. C++'s käiks klaverilt asjade lugemine cin'i ja ekraanile kirjutamine cout'iga. Lisaks crashimine on põhjustatud sellest, et sa üritad uneid väärtusi sisse lugeda stringidena (%s) kuid mälu eraldasid üheainsa märgi jagu. Eriti ei imestaks kui programm crashiks ka siis, kui pikkusesse kirjutada vähe rohkem numbreid.

Samuti peaks sa sealt mingit sorti warninguid saama. Näiteks mul ütles GCC kohe ära:

[Supiplex]

Alustame crashist.

[Ho Ho]

Nagu eespool kirjutasin, kuna eesmärk on kasutada c++'i siis viska printf/scanf minema ja kasuta cout/cin
http://www.cplusplus.com/doc/tutorial/basic_io/

[estplayer]

[Ho Ho]

Vahet pole, ise eelistaks säärase asja jaoks siiski c++'i kuna pole mingeid jamasid inputi lugemisega.

Muidugi kui on vaja teatud kindlas formaadis teksti väljastada siis võibolla mõtleks veel ümber kuid seda sul praegu siin pole

[estplayer]

Proovin siiski C's teha seda, kuna olen C järgi õppinud siiani (C++ õpetusi ei olegi).

[Urmet]

Kui eeldada, et kasutaja mingeid komadega asju ei sisesta, siis tee nii

[estplayer]

Veel üks küsimus, kuidas ma saan compiletud programmi pausile panna? St. et pärast viimast andmete sisestamist ma näen, et mingi tekst käib eest läbi, aga seda lugeda ei jõua, kuna programm läheb iseenesest kinni.

[keevitaja]

aga küsi näiteks veel ühte muutujat kasutajalt. ala, kas sulgede programm? (y/n)

[Fukiku]

[Timukas0]

getchar(); main funktsiooni lõppu on kõige lihtsam lahendus.

[Fukiku]

keevitaja ja Timukas0 pakutud lahendused töötavad, aga ei ole tegelikult õiged. Sedalaadi käsurea programmil ei ole mingit põhjust jääda pärast oma töö lõpetamist mokk töllakil vahtima ja veel üht nupuvajutust ootama. Samuti peaks iga vähegi programmeerimisega tegelev inimene (olgu siis oma huvist või sunni korras, vahet pole) siiski viima ennast vähemalt elementaarsel tasemel kurssi oma lemmik operatsioonisüsteemi käsurea keskkonnaga ning jooksutama oma käsurea jaoks mõeldud programmijupikesi nende loomulikus keskkonnas.

[Timukas0]

Käsurea tundmise kohapealt olen nõus, aga näiteks getchar kasutamine ja käsurea tundmine ei välista üksteist. Minu meelest üsna mugav, kui saab programmi igal hetkel topeltklõpsuga käima panna ja kasutada, selleasemel et kogu aeg käsureale ronida.

[Ho Ho]

Kui juba "häkkida" siis teha korralikult. getch() ei pruugi alati programmi ootama jätta, eriti kui viimane sisestus ei koristanud newline'i ära klaveripuhvrist. Ise kasutan sellistel puhkudel system("pause"). Loomulikult eriti porditav see pole.

[Urmet]

[estplayer]

Tõesti, suur tänu abi eest, sain hulga targemaks! Kehamassi indeksi arvutamise programmil sain system("pause") ilusti tööle aga järgneval programmil see ei tööta, äkki saate seletada miks.
#include <stdio.h>
#include <math.h>

int main()
{
float S;
float Y;
float P;
float K;
float T;
printf("Sisestage laenusumma eurodes: \n");
scanf("%f", &S);
printf("Sisestage laenuperiood aastates: \n");
scanf("%f", &Y);
printf("Sisestage intressimäär eurodes: \n");
scanf("%f", &P);
K = (S * (P/100/12)) / (1 - pow((1 + (P/100/12)),(-Y*12)) );
T = K * Y * 12;
printf("Laenusummaga %.2f eurot %.2f intressimääraga te peate maksma %.2f eurot kuus %.2f aastat. Ja kokku maksate pangale %.2f eurot.\n", S, P, K, Y, T);
system("pause");
}

Aitäh
E: Leidsin oma vea, salvestasin kogemata .cpp, mitte .c

[Genert]

Mis sa muutujaid nii pikalt teed.
Lihtsamini =
float S,Y,P,K,T;

Ning
I = K/(P/100)^2;
Asemel paneks
#define NUMBER 2

I = K/(P/100) << NUMBER;
Aga jah,vale kalkulatsioon

[estplayer]

Nii, oleks veel üks küsimus. Kuidas ma teen nii, et kui kasutaja poolt sisestatud arv jagub 4-ga, siis aktiveerub üks valiklause, kui ei jagu 4-ga siis ei aktiveeru.
näide;
#include<stdio.h>
main()
{
int A;
printf("Sisesta aasta: ");
scanf("%d", &A);

if(vot siia ma ei teagi mida kirjutada, peaks olema et aktiveerub ainult siis kui A jagub 4-ga)
{
printf("Aasta on liigaasta");
}
else
{
printf("Aasta ei ole liigaasta");
}
return 0;
}

Muidugi tulevad siia hiljem veel mõned tegurid mängu aga tahaks selle selgeks saada kõigepealt.
Aitäh!

[Fukiku]

[estplayer]

[Fukiku]

[Ho Ho]

C's on stringide võrdluseks strncmp funktsioon ning sääraselt staatilisse massiivi scanf'iga lugemine lõppeb heal juhul vajalike andmete üle kirjutamisega, halvemal programmi crashimisega.

Sama asi c++'s oeks oluliselt töökindlam implementeerida. Prindid std::cout'iga, loed std::cin'iga ning ka võrdlusi saad teha otse =='ga eeldusel, et char*'i asemel kasutad std::string'i.

Kinda-OT'ish:
C++ on tegelikult üks pagana võimas asi ning üks töö- ja lollikindlamaid keeli üldse kui kasutada standardteeki + smart pointereid ning exceptioneid. C++11'ga tuleb veel kamaluga ülifancyt nänni juurde ning mul tekib juba vägisi soov ka oma koduleht php/pythoni asemel selles kirjutada

[estplayer]

Suur aitäh teie kõigi abi eest! Te olete kindlasti juba tüdinud minu algaja lihtsatest probleemidest, kuid äkki suudate veel natuke aidata. Kuidas lisada C++'s komakohti? C's ma seda oskasin, C++ kohta ei tea. Näide siis selline:

[Ho Ho]

http://www.cplusplus.com/reference/iostream/manipulators/setprecision/

Seal lehel on üldse palju head infot.

[estplayer]

Nii, järgmine probleem. Pean arvutama kasutaja isikukoodi põhjal tema soo ja vanuse. Praegult on nii palju tehtud:

[Ho Ho]

Probleem on, et sa ei saa sedasi vahemikke võrrelda. Näiteks p2ev == 01 - 31 tõlgendub seal kui 1-31 ehk võrreldakse, kas p2ev muutujas on väärtus -30

Pead võrdlema eraldi nii alumist kui ülemist otsa, a'la suurem-võrde 1'st ja väiksem-võrdne 31'st

[estplayer]

Tere! On järjekordne probleem. On selline programm:

[Ho Ho]

No siis lihtsalt selle asemel, et välja kirjutada et pole algarv kirjutad järjest välja kõik nood arvud millega ta jagub.

Muidu kui natuke viitsid nuputada on võimalik su algarvude otsimise algoritmi röögatult palju kiiremaks teha mõningase lisa mälukulu arvelt. Omal ajal panin Pascalis vana P1@188MHz otsima kõik arvud 2-1000000 mõne sekundiga vs sinu stiilis otsing mis võttis 2-100k puhul juba mitu minutit

[estplayer]

Nüüd siis vaja selline programm teha, mis küsib kasutajalt arvu ning genereerib kõigepealt nii mitu juhuslikku arvu (vahemikus 1 kuni miljon) kui kasutaja sisestas ning seejärel paneb need kasvavasse järjeokrda. For example:
input: 5.
output:
Juhuslik jada:
1276, 305, 7, 28, 2000.
Järjestatud jada:
7, 28, 305, 1276, 2000.

Praegult on nii palju tehtud:

[Ho Ho]

std::sort


Samuti on sul koodi treppimine kergelt öeldes katki

[estplayer]

[Fukiku]

Mida sa oled teinud selleks, et mullitamist realiseerida? Wikis on pseudokoodis näited olemas - ei ole väga keeruline seda C-sse ju tõlkida. Kui sa kuskilt otsast pihta ei hakka, ega ta siis valmis ka ei saa. Seega näita senitehtu välja ja räägi oma probleemidest.

[estplayer]

Proovisin siis kuidagi mullsorti kasutada.

[Fukiku]

Põhimõtteliselt teeb su kood praegu mullisordis ühe iteratsiooni. Korrektses mullisordis on kaks tsüklit - sisemine ja välimine. Sul on hetkel olemas ainult sisemine, mis kõnnib mööda massiivi ja vahetab vastavalt vajadusele elemente omavahel - see on täitsa õige, aga ühe korra massiivist üle käies saame halvima stsenaariumi korral täpselt ühe elemendi õigesse kohta paika.

Lisaks peab olema ka välimine tsükkel, mis sisemist tsüklit kordab senikaua, kuni ühtegi kohavahetust enam massiivis ei tehtud (järelikult on siis kõik juba õiges järjekorras). Selle üle arvepidamiseks oled sa juba muutuja exchanges loonud, kuid ei tee sinna kogutud infoga hetkel midagi.

Seega mine loe wikit uuesti, vaata minu kommentaari ja nuputa edasi.

PS. sorteerimise ajal massiivi elementide väljatrükkimine vist suurt tulu ei anna .. mõistlik oleks lõpus massiiv välja sülitada, kui ta juba sorditud on.

PPS. code tag on postituses hea, syntax on parem.

[estplayer]

Suur tänu, sain selle korda Läheksin nüüd korraks tagasi ühe teise programmi juurde.

[Ho Ho]

Tüüpiliselt sellistel puhkudel olen ma kirjutanud esimese väljnudi tsükli väliselt ning tsükli sees eraldusmärke (sinu puhul komad) väljundist enne (ehk siis sul cout << ", " << p/i; ) ja peale tsükli lõppemist siis lõpumärgi, kui seda vaja on. Standardvahenditega seda koma seal lõpus "üle kirjutada" ei saa.


Muidu noist goto'dest saaks seal vägagi lihtsa vaevaga lahti. Kui veel pisut lisamälu tarbida ning kõik prinditavad väärtused kusagil alles hoida enne väljastust saaks ka koodi hulga vähemalt 2x lühemaks ning lihtsamini jälgitavaks

[estplayer]

Aitäh abi eest Nüüd uus programm juba. Kuidas ma saaksin teha nii, et programmi käivitades näitab viite esimest raamatut, kui kasutaja vajutab "j" tähte, näitab viite järgmist, kui vajutab "e" tähte, näitab viite eelmist ning kui vajutab "v" tähte, siis väljub programmist?

[Ho Ho]

gah. Loodan sügavalt, et mõtlesid sellise raamatuinfo koodis kasutamise välja iseeseisvalt ja ei võtnud mingist õpetusest. Kui võtsid siis viska see kiirelt minema ja otsi midagi normaalset.

Esiteks tee selgeks funktsioonid kuhu näiteks söödad sisse structi reference'i ning parameetrite soovitud väärtused ning too kopeerib sinna reference'i muutujatesse nood parameetrid. Või veel "õigem" oleks seda teha klassidega. Saab ka structile constructorit kirjutada kuid see ei pruugi mõnele väga meeldida stiili poolest.
See tehtud siis võta uuri välja mida kujutavad endast ette konteinerid. Ideaalis lahendaksin säärase ülesande vektoritega kuid saaks ka massiividega hakkama, ehkki tulemus poleks nii "puhas" ning oleks mõnevõrra keerukam.
Neid kaht kombineerides saad selle rakenduse tehtud max ~35 koodireaga ning saadav kood on ka reaalselt kasutatav kuna ei pea manageerima eri nimega muutujaid vaid saad lihtsalt konteineri indeksi järgi hetke asukohta arvestada ning konteineris edasi-tagasi liikuda.

(CSV) failist võid ka muidugi seda infot lugeda kuid iseenesest pole see antud funktsionaalsuse vajaduste juures väga oluline kuidas täpselt teed.


Üks asi mida pead silmas pidama on, et standardvahendeid kasutades pead kasutama sisestamiseks ka enterit, lihtsalt j'i vajutades ei jõua kusagile ning tuleb vajutada j+enter.

[estplayer]

Nii mässasin natuke vektorite ja konteineritega ja sain sellise tulemuse:

[Ho Ho]

Never ever ei tohiks tsüklimuutuja väärtust muuta kusagil mujal kui tsükli päises.
Samuti for tsükkel ei kõlba säärase voojuhtimise jaoks. Pigem tee miski lõpmatu tsükkel (while (true)) ning sinna sisse sisendi küsimine ning sellele vastavalt käitumine. Tsüklist saad break'iga välja. Ei mingeid goto'sid pole seal juures samuti tarvis.

[neros]

Nõustun eelkõneleja seisukohaga. Kuna juba C++'i kasutada, siis miks mitte juba OOP teed minna. Kõigepealt soovitaks tekitada näiteks BookCollection klassi, kus on funktsioon Add, mis võtab parameetriteks samad väärtused mis su structi constructor. Saaks ära jätta inetud container.push_back(BOOK(...)); read ning asendada näiteks Books.Add(...);

Teiseks, while loop. Use it. Love it.

Kolmandaks, funktsioonid. Use them. Love them. Üldiselt on reegel selline, et "funktsioon peaks tegema ühte asja" ilma naeruväärseks mineva abstraheerimiseta. Klassid peaks tegema väga konkreetseid asju. Kusagil oli isegi videoloeng selle kohta, üritan tänase päeva jooksul üles leida... kui õnnestub.

Poolikult pole mõtet programmerimist õppida - hiljem kahetsed. Pigem natuke rohkem vaeva näha alguses ja õppida korralikku koodi kirjutama kui pärast spagetist pannkooke teha.

[Ho Ho]

[neros]

Näpuharjutuse jaoks ma mõtlesingi Kui mul on mingi programm mis üle ühe faili ei nõua et loetav püsida, ei hakka ka abiklasse ehitama, pole lihtsalt mõtet kui kiirelt midagi tarvis meisterdada, vähemalt C++'ga. C#'ga oleks vastasel juhul liiga palju staticuid vaja ja staticute nägemine mujal kui extensionmethodites ajab lihtsalt segadusse (on erandeid muidugi). Ning jällegi abstraheerimise poole pealt - liiga palju head asja on paha küll, aga alustajale on sobilik just seetõttu, et saab vähemalt asja natuke selgemaks.

As for all-caps nimetused... esimest korda koodist üle lastes tekkis koheselt küsimus "oot, mis konstandikasutus see selline on" aga uuesti vaadates tuli välja et hoopis structi construct

[estplayer]

Aitäh abi eest! Nüüd tahaksin teha programmi, mis loob 3 4*4 massiivi ja täidab need naturaalarvudega 0 kuni 100. Siis väljastada kõik 3 massiivi ekraanile ning seejärel leida, milline neist on elementide väärtuselt suurim ning siis ka see eraldi väljastada. Omal jõul jõudsin ma kuni 3 massiivi väljastamiseni, elementide summat eriti ei oska arvutada. Samuti sooviks teada, kas seda kõike saab kuidagi lühemalt ka kirja panna? Praegult tundub mul kõik väga laialt välja kirjutatud.

[Ho Ho]

Tee omale funktsiooniparameetrid selgeks

Käigult kirjuatud kood, ei pruugi kompileerida kuid idee peaks enam-vähem edasi andma.

[neros]

Tegelikult saaks vähem häkiliselt kui 4 asendada igal pool nt konstandiga MAX.

Lisaks mis masiivi typedefimisel viga? typedefe kasutatakse kõikjal, ei näe põhjust, miks ka siin ei võiks

E: kuigi vectoritega saaks ilmselt funktsionaalsema koodi