Rzutowanie char na int

Question

Witam, rozwiązując zadanie, mam napisać program, który:

1. Może pobrać do 10 wartości
2. Jeśli użytkownik poda wartość inną niż liczba, program kończy działanie
3. Na koniec program sumuje wartości.

Do oceny czy na wejściu jest znak bądź liczba użyłem funkcji isdigit(). Następnie w

#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
using namespace std;

int main()
{
    char z[10];
    int suma=0;

    for(int i=0; i<10; i++ )
    {
        cout<<"Podaj wartosc: ";
        cin>>z[i];

        if(isdigit(z[i]))
        {
            int(z[i]);
            suma=+z[i];

            continue;
        }
        else
        {
            cout<<"Podano znak; koncze program";
            break;
        }


}
cout<<suma;
    return 0;
}

ykonuje rzutowanie z char na int. Niestety, program pokazuje wartość suma ciągle tą samą, błędną  cyfrę. W czym jest błąd? 

Pozdrowienia.

Answer 1

Po pierwsze zamiast char z[10], to możesz zapisać string a z tego co zrozumiałem, bo chodzi tylko o to ile wartości wpisze użytkownik. A tutaj masz sposób jak zamienić string na int. A tutaj jak poprawnie rzutować zmienne ;)

Answer 2

int v, sum = 0;

while (std::cin >> v) {
    sum += v;
}

std::cout << sum << '\n';

Prościej...

Comments

Po prostu zapis jest nieprawidłowy(+ dodatkowo błędy) i następuje konwersja znaku do odpowiadającej mu liczby. Przykładowo konwertując znak '2' konwertujesz go do liczby 50. Odsyłam do : http://www.algorytm.edu.pl/wstp-do-c/ascii.html . Próbujesz rzutować w stylu C.W C++ robi się to inaczej . Widać ,że programujesz w C++ ,więc wykorzystuj zalety tego języka. Użyj jednoelementowego stringa . Podłącz nagłówek <string> i spójrz na to rozwiązanie:

std::string str{}; // zabezpieczenie przed niezainicjalizowanym //stringiem 
	int suma{ 0 };
	for (int i = 0; i < 10; i++)
	{
		std::cout << "Podaj wartosc: ";
		std::cin >> str;

		if (isdigit(str[0]))
		{
			suma +=  std::stoi(str); // konwersja strumienia znaków na liczbę całkowitą 
		}
		else
		{
			std::cout << "Podano znak; koncze program";
            std::terminate();// funkcja w C++ natychmiastowo
            //"wyłączająca program"              
            // wyskoczy ci błąd, ale to prawidłowe zachowanie tej 
            //funkcji
		}
    std::cout << std::endl;
    std::cout << suma;
	}

 

---

Sorki ,niechcący kliknąłem w komentarz

---

@j23,

W zadaniu niestety nie chodzi o prostotę rozwiązania, a o użycie innych instrukcji, w ramach ćwiczeń; nie mniej, dziękuje za odpowiedź. 

 

---

Super, o to mi chodziło