[C++] Chcę napiać coś co mi sprawdzi czy w tablicy nie ma znaku

Mam coś takiego:

Kod:

---

          if(
          for (c=0; c<51;){
          if(wpisane[c]!=znak)
          c++;} ) {wykonano=wykonano +1;}

---

wpisane- to tablica,W której są litery, które podało się wcześniej
znak- to aktualna podana literka
c-licznik
wykonano- to zmienna, w której przechowuje się ile razy wydrukowano znak

Mam taki pomysł na funkcje:

Kod:

---

prototyp: czyniema (char wpisane, char znak);
wywołanie: czyniema (char wpisane, char znak);
funckja:int czyniema(char wpisane, char znak){
        int c=0;
        for (c=0; c<51;){
                if(wpisane[c]==znak) {return powtorka=1;}
        }
}

---

Ale to mi nie działa, bo chyba nie mogę w if'a, dawać pętli ? Chciałem to zrobić funkcją, ale nie wiem jak przekazać te parametry, w ogóle przeczytałem, że funkcja nie może zwrócić wartości tablicy... A może je przyjąć?

Ogólnie to muszę sprawdzić czy aktualnie podany znak już nie został wcześniej podany...

W ifa nie możesz dać pętli, bo pętla nie ma żadnej wartości logicznej.
A w "pomyśle" twoja pętla nie ma kroku, więc będzie się cały czas wykonywać z c == 0. Zmienna powtorka nie jest zadeklarowana. Wpisane nie jest tablicą tylko znakiem, a potem używasz jak tablicę. Brakuje returna.

Funkcja może przyjąć tablicę jako argument, jak również zwracać tablicę.

WTF ?

Piszesz i tłumaczysz tak chaotycznie, że ciężko cokolwiek ogarnąc o kodzie nie wspomne .

Cytuj:

---

Hardware napisał

Ale to mi nie działa, bo chyba nie mogę w if'a, dawać pętli ?

---

możesz dawać ifa w pętli

Cytuj:

---

Hardware napisał

że funkcja nie może zwrócić wartości tablicy

---

może

Cytuj:

---

Hardware napisał

A może je przyjąć?

---

może

//

Napisanie tego co chcesz osiągnąć to funkcja mająca 5 linijek

Cytuj:

---

Qsty napisał

możesz dawać ifa w pętli

---

Ale jemu chodzi o pętlę wewnątrz ifa.
Też na początku źle przeczytałem, bo prawda - strasznie chaotycznie napisane.

Kod:

---

#include <iostream>

using namespace std;

void czyniema (char wpisane[], int rozm)
{
    char znak;
    cout << "Podaj znak: ";
    cin >> znak;
    for (int i=0; i < rozm; i++)
    {
        if(znak == wpisane[i])
        {
            cout << "znak jest juz w tablicy";
        }

    }

}

int main()
{
    char c[] = { 'a', 'b', 'c', 'd', 'e' };
    int rozm = sizeof(c);

    czyniema(c,rozm);
}

---

ogólnie musisz podać do funkcji tablice ze znakami które już były i rozmiar tej tablicy

Jeszcze raz. Trochę przeczytałem i walczę ciągle. Ale coś mi nie wychodzi.

Kod:

---

prototyp: int czyniema (char q[50], char w);
wywołanie: czyniema (wwpisane[50], zznak);
funkcja:

int czyniema(char q[50], char w){
        int c=0;
        int powtorka=0;
        for (c=0; c<51;){
                if(q[c]==w) {return powtorka=1;}
                c++;
               
        }
}

---

q i w to odpowiedniki <<wwpisane i zznak
wwpisane i zznak to zmienne do których przepisałem wartości zmiennych wwpisane[50]=wpisane[50]; zznak=znak;
w znaku mam aktualnie wciśniętą literkę, wpisane zbiera wszystkie wpisane literki, mam to już napisane w programie głównym
w funkcji musi być na końcu return 0;? Z tej funkcji jeśli litera się powtarza, chcę zwrócić powtórkę.

Aktualnie kompilator wywala mi tylko jeden błąd:
Załącznik 341287

@up

http://bfy.tw/2M2e

i naucz się pisać fora.

ma być:

Kod:

---

for (c=0; c<51;c++) {
//instrukcje
}

---

bo to co Ty piszesz to połączenie fora z whilem ;o

Przy okazji powiem, że wśród programistow przyjęło się iterowanie za pomocą zmiennych i, j, k. Nie jest to jakiś błąd, że piszesz c, ale skoro sie uczysz to wyrabiaj sobie dobre nawyki.

Dzięki wielkie za pomoc. Poszły reputy, już prawie mam wszystko ok. Mam problem, żeby funkcja zwróciła wartość powtórka. Normalnie wartość powtórki się zmienia wewnątrz funkcji, ale niestety nie działa to już na zewnątrz. Powtorka ma ciągle wartość 0.

Kod:

---

        if (znak == s[t])
                {
          cout<<znak;
          powtorka=0;
          czyniema (wpisane, znak, powtorka);
          if(powtorka==0) {wykonano=wykonano +1;}
          wpisane[i]= znak;
          gotoXY (x, y+21);
                  cout<<"Wykonano:"<<wykonano;
        }

int czyniema(char q[50], char w, int powtorka){
        int c=0;
        int x, y;
        for (c=0; c<51;){
                if(q[c]==w) {powtorka=1;}
                c++;
                }
                return powtorka;
}

---

Cytuj:

---

Qsty napisał

@up

http://bfy.tw/2M2e

i naucz się pisać fora.

ma być:

Kod:

---

for (c=0; c<51;c++) {
//instrukcje
}

---

bo to co Ty piszesz to połączenie fora z whilem ;o

Przy okazji powiem, że wśród programistow przyjęło się iterowanie za pomocą zmiennych i, j, k. Nie jest to jakiś błąd, że piszesz c, ale skoro sie uczysz to wyrabiaj sobie dobre nawyki.

---

Ty a jak się daje c++ na początku funkcji to, to nie przeszkadza? Tzn. chodzi mi o to, czy nie zwiększa argumentu od razu o jeden? Taki problem pamiętam jak się uczyłem na początku programowania i wole sobie to dawać na końcu pętli.

http://ideone.com/jVVsc9
najpierw wykonuje się ciało pętli, potem ostatnia część fora

Cytuj:

---

Hardware napisał

Ty a jak się daje c++ na początku funkcji to, to nie przeszkadza? Tzn. chodzi mi o to, czy nie zwiększa argumentu od razu o jeden? Taki problem pamiętam jak się uczyłem na początku programowania i wole sobie to dawać na końcu pętli.

---

Ale tutaj dajesz w miejscu które definiuje całego fora (wartosc poczatkowa;warunek zakonczenia petli;krok petli) wykonujesz wszystko co jest w {} i wykonujesz krok, sprawdzając czy warunek zakończenia nie jest spełniony.
Ofc możesz to pominąć i np napisać (c=0;;c++) czy coś takiego, tak samo możesz napisać (c=0;c<51;) ale jak nie ogarniasz jak tego używać to będą błędy xd

Cytuj:

---

Qsty napisał

i naucz się pisać fora.

ma być:

Kod:

---

for (c=0; c<51;c++) {
//instrukcje
}

---

bo to co Ty piszesz to połączenie fora z whilem ;o

---

To też jest działająca pętla for, bo zasadniczo, to

Kod:

---

for (c=0; c<51;c++) {
//instrukcje
}

---

da taki sam wynik, co

Kod:

---

for (c=0; c<51;) {
//instrukcje
c++;
}

---

jak on tam miał. A jako że zaraz przed forem ma c = 0; to mogłoby być nawet

Kod:

---

int c = 0;
for (;c<51;) {
//instrukcje
c++;
}

---

Co nie zmienia faktu, że pisanie w ten sposób jest bez sensu.

Cytuj:

---

Hardware napisał

Ty a jak się daje c++ na początku funkcji to, to nie przeszkadza? Tzn. chodzi mi o to, czy nie zwiększa argumentu od razu o jeden? Taki problem pamiętam jak się uczyłem na początku programowania i wole sobie to dawać na końcu pętli.

---

Nie. To się wykonuje po każdym obrocie pętli. I nieważne czy będzie c++ czy ++c.
A co do twojego kodu, to nic nie idzie zrozumieć z niego.
Po co przekazujesz tę zmienną powtorka do funkcji, która potem zwraca jej wartość? Dlaczego deklarujesz jakies int x i y, jak potem ich nie używasz? Dlaczego ta pętla się nie kończy po pierwszym podstawieniu powtorka = 1; skoro potem już nic się nie może zmienić? Poza tym, for(c = 0; c < 51; c++) sprawia, że wyjeżdżasz poza zakres swojej tablicy.

http://www.programiz.com/c-programming/c-for-loop

A tak ogólnie to polecam najpierw poczytać a później brać się za programowanie. Później wychodzą takie kwiatki.

Cytuj:

---

Hardware napisał

Dzięki wielkie za pomoc. Poszły reputy, już prawie mam wszystko ok. Mam problem, żeby funkcja zwróciła wartość powtórka. Normalnie wartość powtórki się zmienia wewnątrz funkcji, ale niestety nie działa to już na zewnątrz. Powtorka ma ciągle wartość 0.

---

Powtorka wynosi ciagle 0, bo powtorka w funkcji to nie to samo co powtorka w mainie. Funkcja pracuje na kopii zmiennej powtorka i po jej wykonaniu zmienna ta znika. Mozesz uzyć referencji i przekazać argument int &powtorka i wtedy będziesz pracował na tej samej zmiennej. Tak jak tutaj:

Kod:

---

void czyniema(char q[50], char w, int &powtorka){
        for (int c=0; c<51; c++){
        if(q[c]==w) powtorka = 1;
    }
}

---

Ale lepiej będzie jak zrobisz tak jak tutaj:

Kod:

---

int czyniema(char q[50], char w){
        for (int c=0; c<51; c++){
                if(q[c]==w) return 1;
                }
                return 0;
}

---

Twoja funkcja czyniema zwraca int, ale do niczego go nie przypisujesz. Aby wykonać funkcje i zapisać wynik musisz napisać zmienna = funkcja(). Wtedy funkcja się wykona i zapisze zwróconą wartość do zmiennej.

Kod:

---

if (znak == s[t])
                {
          /* jakis tam kod */
          powtorka = czyniema (wpisane, znak);
          /* jakis tam kod */
        }

---