Warunek ze znakiem (? :) (C++)

Question

(ps[i] > max) ? max = ps[i] : break;

Kompilator pokazuje mi błąd z 'break', dlaczego ?

Answer 1

Masz potrójny operator warunkowy, który w zasadzie ma zwracać wartość. Instrukcja break nie może nic zwracać dlatego wyskoczy błąd. Możesz zrobić tak, ale już lepiej skorzystać z if/else.

template <typename T, typename T2>
void maxn(T * ps, int n, T2 max)
{
	bool exit = false;
	max = ps[0];
	for (int i = 0; i<n && !exit; i++)
		(ps[i] > max) ? max = ps[i] : exit = true;
	std::cout << max << std::endl;
}

W innej sytuacji kompilator w Visual Studio pozwoli na zamieszczenie dwóch wyrażeń o różnych zwracanych typach czyli np int i void (jako np funkcje) czego np clang czy gcc nie zrobi. 

Comments

Dziękuje wykorzystam oba, chciałem spróbować innego sposobu.

A mógłbym prosić o wyjaśnienie 'exit' ? Bo staram się zrozumieć cały zarys pętli z jego wykorzystaniem.

---

Pętla ma warunek, że będzie wykonywać się tak długo jak i < n AND exit == false (lub exit != true, lub !exit w skróconej formie). Gdy z w ciele pętli zmiennej exit zostanie nadana wartość true to przy kolejnym cyklu podczas sprawdzania warunek będzie nieprawdziwy więc pętla zakończy się.

---

Dziękuję  wykorzystywanie warunków to nie taka prosta sprawa.

---

@Hipcio,

... w zasadzie ma zwracać wartość

Nawet nie "w zasadzie" a zawsze zgodny typ. Stąd wynikają ciekawe właściwości tego operatora:

#include <iostream>
#include <vector>

int main() {
    std::vector<int> values{1, 2, 32, 55, 43, 42, 2};
    size_t divided_3{};
    size_t not_divided_3{};
    for(auto& val: values) {
        ++(val % 3 == 0 ? divided_3: not_divided_3);
    }
    std::cout << "Liczb podzielnych przez 3: " << divided_3
        << "\nLiczb nie-podzielnych przez 3: " << not_divided_3
        << "\nRazem liczb jest: " << divided_3 + not_divided_3
        << "\nWielkość kontenera danych: " << values.size()
        << '\n';
}

Każdy sam oceni czy to jest czytelne czy nie... 

---

A co się dzieję w tym kodzie ?(?) Wybacz ale nie za bardzo rozumiem. Spotkałem się kiedyś ze vector ale nie na takim poziomie.

---

Do std::vector wrzucam jakieś wartości, tworzę 2 liczniki, iteruję po wszystkich wartościach z std::vector, wybieram który licznik inkrementować w zależności od tego czy wartość jest podzielna przez 3 czy nie. Dalej prezentacja danych.

Całość zgodna z C++11.

---

Dzięki, wiele się jeszcze muszę nauczyć.

Answer 2

Bo nie jest to ani pętla, ani switch.

Comments

warunek jest ujęty w pętle

template <typename T, typename T2>
void maxn(T * ps, int n, T2 max)
{	
	max = ps[0];
	for(int i=0; i<n; i++)
		(ps[i] > max) ? max = ps[i] : break;
	std::cout << max << std::endl;	
}