4629. PTwPZ Kalkulator

[rafalx200]

Co jest w tym kodzie nie tak? Dopiero zaczynam uczyć się C# i pewnie to jakiś głupi błąd


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Kalkulator
{
class Program
{

    static void Main(string[] args)
    {
        int N = int.Parse(Console.ReadLine());
         string wyrazenie;
        for (int i = 0; i < N; i++)
        {

            wyrazenie = Console.ReadLine();
            int suma = wyrazenie[0]- 48;

            for (int j = 2; j<wyrazenie.Length; j+=2)
            {
                if (wyrazenie[j - 1] == '+')
                {

                    suma += wyrazenie[j] - 48;
                }
                else if (wyrazenie[j - 1] == '-')
                {  

                    suma -= wyrazenie[j] - 48;
                }
            }
            Console.WriteLine(suma);
            Console.Read();

        }


    }
}

}

[quenthui]

Odpal swój kod dla zestawu, który zawiera kilka linijek danych testowych (ale innego niż przykładowy zestaw z zadania. Ten pechowo działa, pomimo błędnego kodu).
Np.
2
1+1
1+1

[rafalx200]

Dziwne , bo mi żaden z zestawów nie działa poprawnie

[rafalx200]

To będzie jakiś problem ze znakiem enter? Nie wiem jak temu zaradzić.

[sig]

Console.Read();

Wywal tą linijkę i będzie dobrze, spoj nie toleruje takiego "zatrzymywania konsoli" więc będziesz musiał zmienić sposób testowania albo pamiętać o jej usuwaniu. W IDE powinieneś mieć opcję pozostawienia widocznej konsoli, możesz też skorzystać z http://ideone.com.

ps jak już ci ruszy to usuń kod z forum.

[rafalx200]

To na pewno wina zatrzymywania konsoli? Bez tej linijki przez ułamek sekundy widziałem,że wyniki miałem prawidłowe. Z nią się nie zgadzały - wychodziły jakieś ujemne liczby. Ktoś powie co było nie tak ? Jak w takim razie zatrzymywać okno konsoli i jaki miał oto wpływ na wynik?

[sig]

Co prawda nie znam C#, ale po usunięciu tej linijki dostałem AC (a wiem że spoj ze względu na sprawdzanie maszynowe czegoś takiego nie toleruje).

Co do "pozostawiania" konsoli, to zależy od twojego IDE, mi np w code::blocks "sama z siebie" czeka aż wcisnę enter

[jasiu2012]

Cześć

Co jest nie tak z tym kodem ? https://ideone.com/wbpXU4

[pawoj20]

Program zle oblicza.
W ideone przekopiuj test z tresci zadania do okienka stdin.
Ideone sluzy do testowania. Prawda?

[jasiu2012]

Nie pomogłeś mi za bardzo. Wiem, że program źle oblicza, w przeciwnym wypadku dostałbym AC. Mój program zaczyna się psuć dla wielu zestawów danych. Dla jednego zestawu podaje dobre wyniki.
Myślę, że może być problem z enterem, ale nie jestem do końca przekonany. W jaki sposób zerwać pętlę po naciśnięciu enter tak aby suma została wypisana na ekran ?

[quenthui]

Twój program nie działa nawet dla zestawu wejściowego: https://ideone.com/hJqeU4
Podejrzewam, że problemem jest wczytywanie danych. Mieszanie wczytywanie int’ów i char’ów nigdy nie kończy się dobrze.
Najlepiej najpierw wczytaj liczbę testów
cin >> n
a potem cały wyrażenie wczytaj do string’a lub tablicy.
Potem odpal swój algorytm dla tak zapisanego działania

[hipcia]

Myślę, że może być problem z enterem, ale nie jestem do końca przekonany. W jaki sposób zerwać pętlę po naciśnięciu enter tak aby suma została wypisana na ekran ?

int n;
cin >> n;
while (n--) {
	int suma = 0, liczba;
	char znak;
	cin >> suma;
	while (cin >> znak >> liczba) {
		/* ... */

		if (cin.peek() == '\n')
			break;
	}
	cout << suma << "\n"; 
}

Funkcja peek zwraca kolejny znak znajdujący w strumieniu. Jeżeli będzie nim znak nowej linii to przerywamy pętlę (break). Analogicznie można podpiąć wszystko do wyrażenia warunkowego pętli:

while (cin.peek() != '\n' and cin >> znak >> liczba) {