Programiranje u C-u - od svega pomalo

1. Ne uvlacis kôd, i ne odvajas cjeline smisleno. Ok, to samo po sebi nije greska, ali smanjuje citljivost.

2. Mjesas formatirani unos sa neformatiranim (cin i cin.get()) to dovodi do toga da ti cin.get() bude potpuno preskocen zato sto cin prije njega ostavlja newline znak u input bufferu. Vise o tome ovdje:

http://www.daniweb.com/software-development/cpp/threads/162903

http://www.daniweb.com/software-development/cpp/threads/90228

3. Kao sto je vec receno treba ici "i--" umjesto "i++"

Evo kako bih ja to napisao (skratio sam neke stvari, i uvukao kôd):

#include <cstdlib>
#include <iostream>

using namespace std;

int main()
{
    char ime1[80], ime2[80], word[80];
    int i;
   
    cout<<"Unesite ime ";
    cin>>ime1;
    strcpy(ime2, ime1);
    cout<<"Drugi niz: "<<ime2<<endl;
   
    cout<<"Upisite svoju rijec ";
    cin>>word;
   
    for (i=strlen(word); i>=0; i--)
        cout<<word[i]<<endl;
   
    system("PAUSE");
    return EXIT_SUCCESS;
}

Ovo je malo Pascalovski stil... Mislim da si mu i to trebao ispraviti - tipično je (da ne kažem nepisani standard) za C-like jezike da se varijabla za brojač/iterator deklarira s petljom, s obzirom da će se u pravilu samo tu i koristiti i nema potrebe da je vidljiva i izvan petlje.

Znači, bilo bi bolje držati se tog nepisanog standarda:

    for (int i=strlen(word); i>=0; i--)
        cout<<word[i]<<endl;

Pa dobro, pričao si za C pa sad... Ja takva saznanja nemam - zapravo bi me iznenadilo da to je tako, s obzirom da C nema odvojene deklarativne dijelove od glavnog koda, kao što to ima Pascal. I kažeš da je "C podržavao deklaracije samo na početku bloka" - to znači neki stari standard ili što? Koga briga za stari, napušteni standard - ako aktualni standardi to omogućavaju, onda tu mogućnost treba i koristiti.  A ovo mi je zbilja prvi glas o tome.

malo kopanja kaže da ansi c standard iz 89te traži deklariranje varijabli na početku scope-a. ali još malo kopanja kaže da c standard 99te dopušta deklaracije svugdje pa definitivno nema smisla deklarirati na početku :)

ANSI C je jos uvijek veoma aktualan standard. Nazalost, kako je C++ popularnoscu pregazio C tako je i razvoj C compilera zapeo, te ih velika vecina podrzava samo neke dijelove C99 standarda. Radi sto vece portabilnosti treba kôd uvijek svoditi na najnizi zajednicki nazivnik.

Ona deklaracija brojaca povrh bloka mi je navika proizasla iz iskustva sa razlicitim C compilerima, nemam nikakvog opravdanja za ovaj konkretan slucaj buduci da se radi o C++ kôdu...

Takve stvari se dogadaju kada pokusavam pisati C++ :D

kako izvući sistemsko vrijeme i spremiti dan, mjesec i god u 3 različite varijable? može i c i c++. ja sam pokušavao, al mi nikako nije uspjelo... :/

mah, mislio sam da ima nešto jednostavnije

na kraju sam sklepao ovo. jel to dobro?

#include <stdio.h>
#include <time.h>

int main () {

    int i, d, y;
    time_t rawtime;
    struct tm *timeinfo;
    char *ttime;
    char mon[4], day[3], year[5];

    time (&rawtime);
    timeinfo = localtime (&rawtime);
    ttime = asctime(timeinfo);
    printf("The current date/time is: %s", ttime);

    for (i=4; i< 7; i++) mon[i-4]=ttime[i];
    for (i=8; i<10; i++) day[i-8]=ttime[i];
    d = atoi(day);
    for (i=20; i<24; i++) year[i-20]=ttime[i];
    y = atoi(year);

    printf("%d %s %d", d, mon, y);

    return 0;
}

ma meni mjesec i treba kao integer. ondje sam ga ostavio kao string samo eto tako. mogu iz iz tog stringa izvuci koji je mjesec, al ajd vidim da je ovo jednostavnije. vec sam napravio funkciju koja ovo obavlja, al ajd promjenit cu je

al me zanima, zasto mi na ovaj nacin izbacuje da je mjesec 3. a ne 4.

#include <iostream>
#include <stdio.h>
#include <time.h>
#include <conio.h>

int main () {
    time_t rawtime;
    struct tm *timeinfo;

    time (&rawtime);
    timeinfo = localtime(&rawtime);
    printf("%d %d %d\n", timeinfo->tm_mday, timeinfo->tm_mon, 1900+timeinfo->tm_year);
    getch();
    return 0;
}

aii, pa da, to sam previdio. hvala! ;)

kao što sam rekao, treba mi integer, tako da mi ovo ne treba

to je sad to. zasad je sve riješeno. hvala na pomoci!

Treba mi opet mala pomoc.

Naime,imam zadatak da napravim program koji provjerava da li je ucitana rijec palindrom.

Ovo je moj kod:

#include <iostream>
#include <stdlib.h>
#include <string>

int n,i,j;

char ulaz[80],izlaz[80];

cout<<"Unesite rijec: ";cin>>ulaz;
n=strlen(ulaz);
j=0;
for(i=n-1;i>=0;i--)
{
izlaz[j]=ulaz[i];
j++;
}
if(strcmp(ulaz,izlaz)==0)
cout<<endl<<"Ucitana rijec JE palindrom "<<endl;
else
cout<<endl<<"Ucitana rijec NIJE palindrom "<<endl;

E sad,kad pokrenem program i ukucam npr rijec neven ispise da ucitana rijec jest palindrom,a kada ukucam npr oko,ana,dud ispise mi da ucitana rijec NIJE palindrom.

Zanima me zasto >.>

Prvo - totalno si me zbunio, jer meni piše da ana i dud JESU palindromi, a riječ neven napiše da nije palindrom. Nego - jasno ti je gdje je problem? U tome da nisi pravilno izokrenuo string, odnosno u nekim slučajevima kad imaš palindrom, ulaz nisi dobro transformirao u izlaz. Ne kužim kako se neki ovdje ne mogu snaći - zašto jednostavno nisi ispisao ulaz i izlaz? Zar je to tako teško? Evo, vidi:

    cout << "U [" << ulaz << "]" << endl
         << "I [" << izlaz << "]" << endl;
         

Da si samo to napravio, već bi vidio da si nešto loše napravio s izlazom, jer u izlazu imaš neke "kerefeke".

Inače, Tom69 ti je dobro rekao.

A također, ne razumijem zašto si išao komplicirati s takvim algoritmom. Zašto bi morao prvo ulazni string transformirati u obrnuti i onda ih uspoređivati? Možeš jednostavno uspoređivati znak po znak:

je li string[i]==string[len(string)-1-i]? Ako za bilo koji znak nema jednakosti označiš neku varijablu u false i prekidaš petlju s break.

Hambrle, što ti znače rijec, rijec2 i rijec3? Zašto nisi imenovao varijable recimo rijecUpisana, rijecBezRazmaka, rijecObrnuta? Zaista ne kužim zašto se početnici toliko boje upotrebljavati nazive varijabli koji zaista jasno govore čemu te varijable služe. To bi vam trebao biti imperativ! Upotrebljavati korektna imena varijabli već i na najmanjim programima, kako bi vam to ušlo u normalan način pisanja koda pa da isto to primjenjujete i u ozbiljnim programima, ako se kasnije odlučite za programersku profesiju.

Također - deklariranje svih varijabli na početku samo zbunjuje jer kad gledaš program ispočetka ne znaš čemu služi koja varijabla (u tom trenutku n, i, m baš ničemu ne služe) i onda moraš tražiti po kodu i gledati gdje se ta varijabla prvi put upotrebljava da bi skužio o čemu se radi. Recimo meni je ovo tvoje deklariranje n i m totalno zbunjujuće - čemu to? Što u tom trenutku znače ti n i m? Također, varijable koje se koriste kao indeksi u petlji najpametnije je deklarirati u samoj naredbi kojom se kreira petlja - da jasno znaš da se ta varijabla odnosi upravo na to.

Znači, nema:

int i;

...

...

...

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

...

nego jednostavno

for(int i=0;i<=n-1;i++)

i gotova stvar.

Napisati program koji ce ispisati koliko rijeci ima ucitana recenica

Mozel pomoc? >.>

Dragi moji programeri, molim vas pomoć oko ovog zadatka. Naime pokušavam skontati proslijeđivanje polja pomoćnoj funkciji, ali ne ide mi baš. Dio sam zadatka riješio i mislim da nije baš sve krivo. Znam da negdje radim debilnu grešku i da će vam biti smiješno, ali patim se s tim već par sati i ne vidim gdje griješim.

Napišite program koji se sastoji od glavne i jedne dodatne funkcije. Dodatna funkcija kao ulazne parametre prima polje maksimalne veličine 100 i broj učitanih brojeva n, a računa zbroj parnih vrijednosti elemenata polja. U glavnoj funkciji se obavlja učitavanje n brojeva u polje i ispis rezultata koji vrati dodatna funkcija. Varijabla n označava koliko brojeva treba učitati u polje preko tipkovnice

#include <stdio.h>
#include <conio.h>

int zbrojparnih(int x, int njiva[100]);

int main ()
{
    int n, i, polje[100];
    printf ("Koliko brojeva zelis unijeti: ");
    scanf ("%d", &n);
    for (i=0; i<n; i++){
        printf ("Unesi broj: ");
        scanf ("%d", &polje[i]);
        }
    printf ("\nZbroj parnih brojeva je %d", zbrojparnih(int n, int polje[100]));
   
    getch();
}

int zbrojparnih(int x, int njiva[100])
    {
                    int j, sum=0;
                    for (j=0; j<x; j++)
                    {
                    if (njiva[j]%2==0)
                    sum+=njiva[j];
                    }
                    return sum;
    }