Problemy z metodami C++

pytanie zadane 13 kwietnia 2016 w C i C++ przez TheFeniks Gaduła (4,690 p.)

Cześć !

Mam dwa problemy otóż:

► Po 1: https://zapodaj.net/cf07ed8a0a2fe.png.html

Nie działa licznik czyli:

Podaj 1 liczbe:

Podaj 2 liczbe:

....

Zamiast tego jest:

Podaj 1 liczbe:

► Po 2:

Program cały czas chce aby wpisywać liczby.

A nie tylko 4 tak jak było na tym scrennie powyżej

W czym leży problem ?

Program jest rodzielony na 3 pliki dwa .cpp i jeden .h

.h :

class Addition //Dodawanie
{
 public:

   float input[30];
   float score;
   int counter;
   int board_story[25];

   float input_M(); //Wejscie

   void control(int);

   float calculate();
};

Metody .cpp :

#include <iostream>
#include <cstdlib>
#include <conio.h>
#include <windows.h>
#include "list.h"
using namespace std;

HANDLE uchwyt = GetStdHandle(STD_OUTPUT_HANDLE);//Uchwyt kolory

float Addition::input_M()
{
    bool l_inCounter=true;
    bool l_input=true;

    while( l_inCounter )
    {
       l_inCounter=true;
       l_input=true;
       system("cls");
       SetConsoleTextAttribute(uchwyt,7);
       cout<<"|=====================|"<<endl; SetConsoleTextAttribute(uchwyt, 9);
       cout<<"       Dodawanie       "<<endl; SetConsoleTextAttribute(uchwyt,7);
       cout<<"    Ile liczb dodac:";
       cin>>counter;
       if(!(counter))
       {
          control(counter);
       }
       else
       {
          l_inCounter=false;
       }
    }
     SetConsoleTextAttribute(uchwyt,7);
     cout<<"|======================|"<<endl;
     for(int i=1; i<=counter; i++)
     {
        l_input=true;
         while( l_input )
         {
           SetConsoleTextAttribute(uchwyt,7);
           cout<<"  Podaj:"<<i<<" liczbe:";
           cin>>input[i];
           control(i);
         }
     }

}

void Addition::control(int i=0)
{
    bool l_inCounter=true;
    bool l_input=true;

    if(!(counter))
    {
      cout<<endl; SetConsoleTextAttribute(uchwyt, 12);
      cout<<" Blad! Nieprawidlowe wejscie!"<<endl;
      Sleep(1000);
      cin.clear();
      cin.ignore(1000,'\n');
    }
    else
    {
      l_inCounter=false;
    }

    if(!(input[i]))
    {
      cout<<endl; SetConsoleTextAttribute(uchwyt, 12);
      cout<<" Blad! Nieprawidlowe wejscie!"<<endl;
      Sleep(1000);
      cin.clear();
      cin.ignore(1000,'\n');
    }
    else
    {
      l_input=false;
    }
}

float Addition::calculate()
{

}

Z góry dzięki za pomoc ! :)

2 odpowiedzi

odpowiedź 13 kwietnia 2016 przez Radfler VIP (101,030 p.)
wybrane 14 kwietnia 2016 przez TheFeniks

Najlepsza

Witaj! Wymienię Ci błędy, które zrobiłeś w kodzie:

1) Funkcje calculate i input_M nic nie zwracają => UB.

2) Linijka 50: argument domniemany nie powinien znaleźć się w tym miejscu. Powinien być w pliku nagłówkowym, w deklaracji funkcji.

3) Funkcja control jest według mnie... bez sensu. Co ona właściwie ma robić? Kontrolować wejście? Ona przecież sprawdza w if-ach czy wartości zmiennych są różne od zera:

!(counter) <=> counter != 0

Jeżeli chodzi tu o coś innego to sprecyzuj proszę.

4) Zamiast pisać wszędzie SetConsoleTextAttribute(uchwyt, 12) zrób sobie funkcję (powiedzmy) set_color i jej używaj. Masz tu moją implementację. Przykład:

set_color(Color::BLUE);

komentarz 13 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)
edycja 13 kwietnia 2016 przez TheFeniks

Funkcja control(); jest po to żeby sprawdzić czy zamiast liczby nie wpisano litery.

W opisie pytania masz dokładny opis problemu. :)

Zmieniłem:

void Addition::control(int i=0)

void Addition::control()

I wyskakują błędy:

||=== Build: Debug in KalkulatorV2 (compiler: GNU GCC Compiler) ===|
methods.cpp||In member function 'float Addition::input_M()':|
methods.cpp|51|warning: no return statement in function returning non-void [-Wreturn-type]|
methods.cpp|53|error: prototype for 'void Addition::control()' does not match any in class 'Addition'|
list.h|15|error: candidate is: void Addition::control(int)|
methods.cpp||In member function 'float Addition::calculate()':|
methods.cpp|88|warning: no return statement in function returning non-void [-Wreturn-type]|
||=== Build failed: 2 error(s), 2 warning(s) (0 minute(s), 4 second(s)) ===|

komentarz 13 kwietnia 2016 przez Radfler VIP (101,030 p.)

Nie no, słowo int musi zostać (w końcu to argument). Przenieś po prostu =0 do pliku nagłówkowego.

komentarz 13 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.)

main.cpp: http://wklej.org/id/2274130/

methods.cpp: http://wklej.org/id/2274127/

list.h : http://wklej.org/id/2274128/

Błędy:

methods.cpp||In member function 'float Addition::input_M()':|

methods.cpp|51|warning: no return statement in function returning non-void [-Wreturn-type]|

methods.cpp|53|error: default argument given for parameter 1 of 'void Addition::control(int)' [-fpermissive]|

ist.h|15|error: after previous specification in 'void Addition::control(int)' [-fpermissive]|

methods.cpp||In member function 'float Addition::calculate()':|

methods.cpp|88|warning: no return statement in function returning non-void [-Wreturn-type]|

||=== Build failed: 2 error(s), 2 warning(s) (0 minute(s), 2 second(s)) ===|

komentarz 14 kwietnia 2016 przez Radfler VIP (101,030 p.)

=0 z pliku .cpp usuń

1	komentarz 14 kwietnia 2016 przez TheFeniks Gaduła (4,690 p.) edycja 14 kwietnia 2016 przez TheFeniks Dobra problem rozwiązany! Dzięki za pomoc ! Pozdrawiam ! :)