C++ uppgift

Det är inte så att du glömt en måsvine eller semikolon? De brukar pumpa upp dina antal fel enligt din kompilator. 

Du kan alltid posta koden här så kan vi kika på det. För övrigt när du kodar, försöka bygga din kod efter varje liten ändring du gör för att försäkra dig att du inte gjort något knasigt. Det räcker med ett litet slarvfel så blir det jobbigt sedan att hitta när du ska försöka bygga din applikation.

TItta på det första felet först. Många av de andra kan vara följdfel.

Här är de första felen. På rad 6 och 9. Det står ”expect an expression” 

undrar lite förståelse om private och public med :) 

"private" och "public" kan du bara ha i en klassdefinition. Inuti en funktion saknar de mening. Ta bort dem.

Ska jag ta bort dom helt. Alltså själva int index och int size också eller ska jag bara ta bort själva private och public ”orden” 

Bara rad 6 och 9.

Det är då en bokhandel jag ska göra. 
jag döpte den till storebook = newbook osv… 

det står att denna är oidentifierad. 

undrar också vad int idex egentligen betyder. :) 

Det är för mig oklart vad det är du försöker göra.

Det ser ut som att du försöker göra en struct i dina tidigare bilder? 

Idex är förmodligen index och size är hur många böcker du har. Men det är för mig oklart varför du behöver en index pekare. Är detta någon sorts stack rller kö?

Det ser nästan ut som att index och Size bara borde vara en och samma variabel av den lilla snutten jag ser.

Lägg gärna upp uppgiften i dens helhet så blir det enklare att förstå vad det är du försöker åstadkomma.

Här är själv uppgiften så är det kanske lättare att förstå.

jag har lite svårt med programmering då jag tycker det är väldigt svårt så har bara gjort mitt bästa försök. https://docs.google.com/document/d/1SutWT5cgf2H2cl2RhuE_Rx2Zgi15qyr1eB0uxeDT87o/edit

Den bryr sig bara om vad du skriver, inte vad du menar. Du menar new Book, men har skrivit newBook.

plommonjuice87 skrev:

Här är själv uppgiften så är det kanske lättare att förstå.

 

jag har lite svårt med programmering då jag tycker det är väldigt svårt så har bara gjort mitt bästa försök. https://docs.google.com/document/d/1SutWT5cgf2H2cl2RhuE_Rx2Zgi15qyr1eB0uxeDT87o/edit

Går inte att komma åt innehållet på länken (låst för de som inte har åtkomst)

Men jag får lite grann känslan att du har lagt till massa kod som du inte har en aning om vad den gör. Jag förstår att det känns stressigt när man har en uppgift man vill hinna klart med, men det blir tyvärr inte gjort snabbare om du inte har koll på vad du gör.

Mitt tips är att du går tillbaka ett steg och fokuserar på att bli mer säker på grunderna först, till exempel att du känner dig bekväm med hur funktioner och variabler fungerar. När du sen har gjort det kommer det bli betydligt lättare att lösa uppgiften. Programmering bygger på att tillämpa olika verktyg för att lösa problem, och då är det viktigt att man vet hur verktygen fungerar.

Ett tips också, det finns en knapp du kan använda för att klistra in din kod så att den också behåller formateringen, det blir lättare för oss att hjälpa till om vi ser din kod:

ja jag tycker det är lite svårt. Men jag känner att jag lär mig mest om vi går igenom uppgiften då denna uppgiften tar liksom allt. 
https://docs.google.com/document/d/1SutWT5cgf2H2cl2RhuE_Rx2Zgi15qyr1eB0uxeDT87o/edit
nu ska det gå att gå in på länken.
jag förstår inte heller hur du kan lägga in en kod på pluggakuten då det alternativet inte kommer upp för mig. 

Men iallafall: jag har ett stort problem om att mycket är odefinierat. Storebook och newbook är odentifiearad som i inlägg 8 .  

Du läste inte mitt svar?

Laguna skrev:

Du läste inte mitt svar?

Jo jag bara inte förstod vad du menade? Ska jag skriva ett mellanslag bara? Det blev fel då också.

Det är möjligt, men det blir väl inte samma fel?

Förstod du inte att det behövs ett mellanslag?

Aha ja new book löste sig. Men det står fortfarande att storeBook är identifierad. Sätter jag mellanslag där så blir ordet ”store” identifierad 

även mellan new och Book så står det att den förväntar sig en type specifier. Vad betyder det? 

Så du får fixa en sak i taget. Felmeddelandet om storebook har du inte visat.

Självklart. Tack för all hjälp! Den högst upp då. 

Vilken rad, och hur ser det ut runt den raden?

Såhär ser det ut nu :) 

Är det rad 10?

Det står storeBook där, men storebook längre ner. Det kanske ska stå storebook.

Men jag ser ingen klassdefinition alls. Då kommer inte this-> att betyda något. Det hela borde vara en klass, med attribut storebook, index osv. och då har public: och private: sin plats.

Ja det är rad 10 men den blev oidentifierad ändå. 

vill du snabbt läsa uppgiften jag har länkat så förstår du kanske lite bättre för det gör inte jag. 🥲. 

jag vet inte om man ska ha en klassdefinnation osv. 

Uppgiften verkar inte kräva en klass, men det kanske ändå är meningen att du ska använda klasser. Tala med din lärare.

Var kommer din kod ifrån? Har du skrivit den?

Ja jah har kollat på gamla lektioner och på massa tutorials online. Men jag tror nog inte man behöver en klass men vet inte hur jag annars kan lösa den

Jag tittade lite som hastigast på din uppgift och vad du har gjort. Jag skulle föreslå följande:

Skapa en class som heter Book, tydligen skall du ha en array av sådana klasser (storeBook).

class Book
{
public:
  Book() = default;

  void setTitle(std::string title) { m_title = title; }
  void setAuthor(std::string author) { m_author = author; }
  void setPrice(int price) { m_price = price; }

private:
  std::string m_title;
  std::string m_author;
  int m_price{0};
};

(Jag brukar skriva av gammal vana m_ framför klassmember variabler)

Sen gör en rutin för att läsa från tangenbordet, du behöver en meny för de olika funktionerna så något sätt att läsa meny valet behövs.

Sen gör du en switch-statement där de olika funktionerna görs

int menyVal = getMenyVal();

switch (menyVal)
{
  case 1:
    insertNewBook(storeBook,index); // läs title,author och price inne i den istället.
    break;
  case 2:
    displayBooks(storeBook, index);
    break;

  osv.
}

// sedan

void insertNewBook(Book* storeBooks, int& numberOfBooks)
{ ... ++numberOfBooks; }

void displayBooks(Book* storeBooks, int numberOfBooks)
{ ... }

...