Szybko, krótko i na temat: mam funkcję, która generuje szablony poziomów menu. Przyjmuje jeden parametr - ilość poziomów, które menu będzie posiadało.
Jak byście ją nazwali - SetMenuLevelCount czy SetMenuLevelsCount?
Clean Code 
0
@vpiotr: Dzięki za sugestię, ale trochę się nie zrozumieliśmy.
To, co napisałem o tworzeniu menu było lekkim uproszczeniem, nie chciałem wyjaśniać dokładnie o co chodzi, bo nie ma to teraz większego znaczenia. Chodziło mi raczej o to, czy dać "level" czy "levels" - która opcja Twoim zdaniem jest lepsza?
edytowany 2x, ostatnio: cerrato
Zobacz pozostałe 2 komentarze
Silv
Już przecież zaakceptowałeś odpowiedź, mało tego, ja ją skomentowałem. Ale, tak z ciekawości, może jednak przydałaby się refaktoryzacja, a?
cerrato
wydaje mi się, że jest OK. Tworzenie menu składa się z ciągu wywołań kilku funkcji, z czego każda odpowiada za jedną rzecz. Ta funkcja akurat tworzy strukturę menu w oparciu o przekazany argument. Inne zajmują się zapełnieniem treścią, dodaniem elementów itp. Raczej solid nie jest złamany.
Silv
Ale nazywa się tak, jakby tylko i wyłącznie ustawiała liczbę poziomów menu.
cerrato
No bo właściwie to tak jest :P - ta "struktura" to jest tablica, która przechowuje dane dot. elementów. Potem inna funkcja to zapełnia treścią. Naprawdę, nie ma sensu tego tutaj ciągnąć, nie jest to istotne.
Silv
OK, nie naciskam.
0
@hauleth: w sumie słuszna uwaga, że skoro ten obiekt odpowiada za menu, to nie trzeba doprecyzowywać, czego to są poziomy.
Ale - abstrahując od tego konkretnego przykładu - czy Waszym zdaniem lepiej jest użyć liczby pojedynczej (np. tutaj level) czy mnogiej (levels). Tak mnie naszła po prostu refleksja/wątpliwość przy okazji zabawy z tym menu. Czy dać przykładowo UpdateStudentCount czy UpdateStudentsCount - która wersja jest lepsza? Student czy Students?
- Rejestracja:prawie 11 lat
- Ostatnio:około 2 lata
- Postów:176
- Rejestracja:ponad 5 lat
- Ostatnio:ponad 2 lata
- Lokalizacja:Warszawa
- Postów:95
1
lion137 napisał(a):
Gramatycznie powinna być liczba mnoga, bo są to poziomy, Gdybyś je ustawiał, np., głębokością menu, to byłoby
MenuDepth.
No właśnie w tym wypadku liczba mnoga raczej nie ma znaczenia:
numberOfLevels -> tutaj tak
ale
levelCount
- Rejestracja:około 6 lat
- Ostatnio:prawie 3 lata
- Lokalizacja:Warszawa 🐪
- Postów:1719
0
C.mind napisał(a):
lion137 napisał(a):
Gramatycznie powinna być liczba mnoga, bo są to poziomy, Gdybyś je ustawiał, np., głębokością menu, to byłoby
MenuDepth.No właśnie w tym wypadku liczba mnoga raczej nie ma znaczenia:
numberOfLevels-> tutaj takale
levelCount
Przed snem liczysz barana czy barany?
0
No ale właśnie o to mi chodzi - czy kierować się stricte angielską gramatyką/ortografią i trzymać się liczby mnogiej, czy nie traktować nazw zmiennych/funkcji jako elementów języka naturalnego. Bo jednak angielski komputerowy nie jest angielskim używanym na co dzień, ani nie jest tym samym językiem, którym się pisze książki. Osobiście raczej byłem za levelCount, ale po wpisie @hauleth kilka pięter wyżej - https://4programmers.net/Forum/1641514 zacząłem lekko wątpić.
- Rejestracja:prawie 10 lat
- Ostatnio:5 dni
- Postów:422
4
Tutaj jest wyjaśniona zasada nazywania list i ilości: https://english.stackexchange.com/questions/229406/how-to-write-correctly-chains-of-nouns-with-plurals-like-messages-list-and-ap
Google Translate tłumaczy level count jako liczba poziomów, a levels count jako poziomy się liczą.
Ja bym zostawił nazwę z liczbą mnogą bardziej dla określenia ilości kolekcji poziomów.
edytowany 2x, ostatnio: Burmistrz
- Rejestracja:ponad 5 lat
- Ostatnio:ponad 2 lata
- Lokalizacja:Warszawa
- Postów:95
1
Osobiście raczej byłem za levelCount, ale po wpisie @hauleth kilka pięter wyżej - https://4programmers.net/Forum/1641514 zacząłem lekko wątpić.
Gdyby napisać studentsCount, tak jak było w tym wpisie sugerowane, to taka nazwa wskazywałaby na to, że studenci jako grupa coś liczą. Jeśli chcesz mieć licznik studentów, to wtedy studentCount jest poprawne i nie wskazuje to na licznik dla jednego studenta. Licznikiem czegoś dla jakiegoś jednego studenta byłoby counterForSomeStudent.
Nie patrząc na język – moim zdaniem logiczna sugestia; jeden student nie może "być liczony", tylko wielu, więc od razu wiadomo, o co chodzi.
- Rejestracja:około 6 lat
- Ostatnio:prawie 3 lata
- Lokalizacja:Warszawa 🐪
- Postów:1719
1
@cerrato: No to rzuć monetą i masz spokój. Wygląda na to, że tutaj logiczna argumentacja będzie traktowana na równi z tą z d**y.
Pytanie, które z w/w argumentów są logiczne, a które analne. Bo obie strony piszą w miarę z sensem i bez trollowania.
Robienie na ślepo kalki z języka polskiego w celu udowodnienia poprawności jakiejś konstrukcji angielskiej nie jest logiczną argumentacją.
A czemu nie po prostu
void dupa2()?
tych
dupa() mam już tyle, że przy setce dałem sobie spokój. Kod był przez to coraz mniej czytelny.
To wsadź je do tablicy albo jakiejś mapy i wywołuj po indeksie.
A jak mam nazwać tablicę -
DupaArray czy DupasArray?
Latryna.
@cerrato: haremArray :D
- Rejestracja:ponad 21 lat
- Ostatnio:dzień
- Lokalizacja:Kraków
- Postów:1114
5
To jest właśnie jedna z osobliwości języka angielskiego, poprawna wersja z punktu widzenia gramatycznego to ta bez s, używa się rzeczownika w liczbie pojedynczej chociaż ma znacznie mnogie.
i tak w wpf mamy:
RowDefinitions - które tłumaczymy na polski jako definicje wierszy.
mam tez skopiowana krótkie objaśnienie z jakiejś książki do gramatyki, tylko nie pamiętam której
When we use noun + noun, the first noun is like adjective. It is normally singular, but he meaning is often plural. For example
a bookshop - is a shop where you can buy books
an apple tree - is a tree that has apples
a three-hour journey - a journey that takes three hours
a ten-pound note
a four-week course - The course lasted four weeks.
a six-mile walk
14 year old girl
five star hotels
edytowany 1x, ostatnio: neves
