Witam mam, kiedy możemy zastąpić średnik przecinkiem, bo niezbyt rozumiem dlaczego ten fragment kodu jest tak zapisany?
char a, z;
if(wielkosc_znaku == 0)
a = 'a', z = 'z';
else
a = 'A', z = 'Z';
C++ 
- Rejestracja:ponad 11 lat
- Ostatnio:3 dni
- Postów:1027
2
Przecinek i średnik się różnią znacząco - średnik oddziela dwa wyrażenia, przecinek tworzy jedno wyrażenie, które ewaluuje lewą stronę, i zwraca jako wartość stronę prawą. Tutaj użyto przecinka, gdyż wartość wyrażenia nie jest używana (tzn nikt nie robi int eeee = (a = 'a', z = 'z');), a przez to, że jest to jedno wyrażenie, to nie musisz pisać klamer. Tzn żeby osiągnąć to co trzeba, należy napisać
char a, z;
if (wielkosc_znaku == 0) {
a = 'a';
z = 'z';
} else {
a = 'A';
z = 'Z';
}
dzięki
- Rejestracja:około 9 lat
- Ostatnio:około 2 lata
3
Kompletnie bez sensu użycie przecinka. Zaciemnianie kodu i proszenie się o błędy.
Co więcej, wiele wytycznych kodowania ma zalecenie, żeby po if else for while (itp.) używać {} nawet, jak jest w nich tylko jedna instrukcja (więc formalnie nie trzeba ich wstawiać). Więc tym bardziej tutaj, gdzie opuszcza się klamerki za cenę utraty czytelności kodu (i potencjalnych błędów w przyszłości)...
dzięki, już rozumiem :)
- Rejestracja:ponad 19 lat
- Ostatnio:2 miesiące
2
static struct { char low,high; } tb[]={{'a','z'},{'A','Z'}};
dalej zamiast: a/z
używasz: tb[wielkosc_znaku].low/tb[wielkosc_znaku].high
odpowiednio.
edytowany 3x, ostatnio: _13th_Dragon
ok
- Rejestracja:około 4 lata
- Ostatnio:ponad rok
- Postów:30
0
Mam jeszcze jedno pytanko, chciałbym usunąć plik funkcją remove, jednak przy kompilacji wyskakuje mi bląd
string nazwa_pliku;
cin >> nazwa_pliku;
remove(nazwa_pliku);
a gdy wpisuje w program stałą nazwe pliku wszystko jest ok remove("plik.txt"); . Jak zrobić to poprawnie?
edytowany 1x, ostatnio: 4matey
- Rejestracja:około 9 lat
- Ostatnio:około 2 lata
1
4matey napisał(a):
Mam jeszcze jedno pytanko, chciałbym usunąć plik funkcją remove, jednak przy kompilacji wyskakuje mi bląd
string nazwa_pliku; cin >> nazwa_pliku; remove(nazwa_pliku);a gdy wpisuje w program stałą nazwe pliku wszystko jest ok
remove("plik.txt");. Jak zrobić to poprawnie?
Przeczytaj błąd -- dotyczy niezgodności typów? Ta funkcja jest odziedziczona z C, a w C nie było klasy string (parametr ma być typu const char *). Jeśli już musisz użyć remove, to napisz:
remove(nazwa_pliku.c_str());
error: cannot convert 'std::__cxx11::string*' {aka 'std::__cxx11::basic_string<char>'} to 'const char'
A mogę użyć coś innego zamiast remove by usunąć korzystając ze stringa?
- Rejestracja:około 9 lat
- Ostatnio:około 2 lata
1
@4matey: To chyba już nowy wątek, ale możesz użyć biblioteki filesystem:
https://en.cppreference.com/w/cpp/filesystem/remove
heh, dzięki za pomoc :) Użyję remove będzie prościej, a jest jakaś funkcja w bibliotece standardowej, która mówi czy proces przebiegł pomyślnie?
- Rejestracja:ponad 19 lat
- Ostatnio:2 miesiące
1
Jak wyżej, zaś dla starszych C++: remove(nazwa_pliku.c_str());
Może jeszcze a propos:
kq napisał(a):
Możemy to zrobić w przypadku, gdy chcemy aby nasz kod odrzucono podczas code review. Nie rób tak.
Jeżeli ja prowadzę ten code review, to następujące kody przejdą:
char a, z;
if(wielkosc_znaku) a = 'A', z = 'Z';
else a = 'a', z = 'z';
Ponieważ jak potrzebna będzie dodatkowa zmienna `x`,
a ktoś spróbuje coś głupiego wsadzić bezmyślnie pomiędzy przecinki
to mu się zwyczajnie się nie skompiluje. Poza tym widać kolejność i jak będzie inna ilość zmiennych to będzie widać na pierwszy rzut oka.
* ```c
char a=wielkosc_znaku?'A':'a';
char z=wielkosc_znaku?'Z':'z';
Ponieważ dla każdej wartości mamy osobny wiersz dodatkowa zmienna x
spowoduje dodatkowy wiersz podobny do poprzedniego.
static struct { char low,high; } tb[]={{'a','z'},{'A','Z'}};
Ponieważ dodatkowa zmienna, spowoduje dodatkową wartość w strukturze i tyle.
Zaś kod od @enedil:
```c
char a, z;
if (wielkosc_znaku == 0) {
a = 'a';
z = 'z';
} else {
a = 'A';
z = 'Z';
}
Nie przejdzie, ponieważ łatwo popełnić błąd, np zainicjalizować w if
zaś nie zainicjalizować w else
edytowany 2x, ostatnio: _13th_Dragon
Zobacz pozostałe 5 komentarzy
@enedil, mówię tylko że jeden przeszedł by code review zaś drugi nie, podałem czemu jeden tak zaś drugi nie, kontrargument "
zgodzić się nie mogę" ... sam wiesz jak wygląda.
Jasne, argument jest średni. Dużo lepiej mieć odpowiednią do tego strukturę. I nawet niekoniecznie robić tablicę tych struktur, można by zrobić
struct range range = (wielkosc_znaku == 0) ? (const struct range){'a', 'z'} : (const struct range){'A', 'Z'};
@enedil, Wiele jest sensownych sposobów (włączając podany przez ciebie w poprzednim komentarzu) w tym zagadnieniu (deklaracja przedziału w zależności od warunku) ale to na co zaproponowałeś zamienić pierwotny kod z przecinkami jest najgorszym ze wszystkich przedstawionych.
ja tego nie zaproponowałem, zaprezentowałem zaledwie jak zamienić kod z przecnikami na kod ze średnikiem, aby zaprezentować różnicę (oraz potencjalnie, czemu jeśli nie dam klamer a dam średnik, napotkam błąd kompilacji). Ehh, tłumaczenie intencji jest trudne.
Aha, w takim razie przepraszam, nie zrozumiałem intencji. Z tym że jak ja pisze z jakiejś tam racji zły kod to (no chyba że jednowieszowiec dla nierobów) to pisze dużymi wołami: - NIE PRÓBOWAĆ TEGO W DOMU. :D
{ a = 'a'; z = 'z'; }?