Skracanie kodu :: 4programmers.net

Rejestracja:prawie 6 lat
Ostatnio:ponad 5 lat
Postów:18

Witam mam pewne pytanko ponieważ siedzę przy c# od niedawna i nie znam jeszcze wielu funkcij i dlatego mam pytanko czy da sie skrocic fragment kodu ktory wklejam ponizej? A jeżeli się da a wydaje mi sie ze sie da to jak to zrobic ?

Kopiuj

                if (b.Location.X >= 0 && b.Location.X <= 100 && b.Location.Y >= Convert.ToInt32(position[0]) && b.Location.Y <= Convert.ToInt32(position[1])) // 0 to 135
                {
                    b.Location = new Point(50 - (b.Width / 2), 68 - (b.Height / 2));
                }
                else if (b.Location.X >= 0 && b.Location.X <= 100 && b.Location.Y >= Convert.ToInt32(position[1]) && b.Location.Y <= Convert.ToInt32(position[2])) // 135 to 270
                {
                    b.Location = new Point(50 - (b.Width / 2), 204 - (b.Height / 2));
                }
                else if (b.Location.X >= 0 && b.Location.X <= 100 && b.Location.Y >= Convert.ToInt32(position[2]) && b.Location.Y <= Convert.ToInt32(position[3])) // 270 to 405
                {
                    b.Location = new Point(50 - (b.Width / 2), 340 - (b.Height / 2));
                }
                else if (b.Location.X >= 0 && b.Location.X <= 100 && b.Location.Y >= Convert.ToInt32(position[3]) && b.Location.Y <= Convert.ToInt32(position[4])) // 405 to 540
                {
                    b.Location = new Point(50 - (b.Width / 2), 476 - (b.Height / 2));
                }
                else if (b.Location.X >= 0 && b.Location.X <= 100 && b.Location.Y >= Convert.ToInt32(position[4]) && b.Location.Y <= Convert.ToInt32(position[5])) // 540 to 675
                {
                    b.Location = new Point(50 - (b.Width / 2), 612 - (b.Height / 2));
                }
                else if (b.Location.X >= 0 && b.Location.X <= 100 && b.Location.Y >= Convert.ToInt32(position[5]) && b.Location.Y <= Convert.ToInt32(position[6])) // 675 to 810
                {
                    b.Location = new Point(50 - (b.Width / 2), 748 - (b.Height / 2));
                }

Delor

2019-05-06 22:22

Delor2019-05-06 22:22

Rejestracja:ponad 6 lat
Ostatnio:około 2 lata

Tak na szybko:
Początek warunku jest identyczny wszędzie więc można go wyciągnąć.
Potem wrzucić w pętlę ze zmienną przechodzącą od 0 do 5. W ciele pętli pozostałe warunki i wyjście przy którymkolwiek spełnionym.

Kopiuj

if (b.Location.X >= 0 && b.Location.X <= 100)
{
  for (int i = 0; i <= 5; i++)
  {
    if(b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
    {
      b.Location = new Point(50 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
      break;
    }
  }

abrakadaber

2019-05-06 22:35

~~abrakadaber~~2019-05-06 22:35

Rejestracja:ponad 12 lat
Ostatnio:7 miesięcy
Postów:6610

Kopiuj

var cos = new int[] { 68, 204, 340, 476, 612, 748 };
            if (b.Location.X >= 0 && b.Location.X <= 100)
            {
                for (int i = 0; i <= 5; i++)
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(50 - (b.Width / 2), cos[i] - (b.Height / 2));
                        break;
                    }
            }

ech, prawie identyczny kod :p. Chociaż te magiczne numerki wypadało by stałymi zastąpić

edytowany 1x, ostatnio: ~~abrakadaber~~ 2019-05-06 22:36

Delor 2019-05-06 22:39

Pierwsza wersja była identyczna. Potem zauważyłem stały odstęp między wartościami.

Zimny Krawiec

2019-05-07 11:03

Zimny Krawiec2019-05-07 11:03

Rejestracja:prawie 7 lat
Ostatnio:4 miesiące
Postów:273

Kod powinien być krótki w tym sensie że nie powinien się niepotrzebnie powtarzać ale powinien również być prosty w czytaniu .Możesz np. używać nawiasów w wyrażeniach nawet jeśli nie są potrzebne. Łatwiej będzie ci się potem czytało kod.
Co z tego że ktoś potrafi 5 linijek kodu zapisać w jednej jak potem on sam albo ktoś inny będzie tracił czas na odszyfrowywanie kodu. Często to jak zapiszemy kod nie ma żadnego wpływu na wynik końcowy ani na wydajność programu.

obscurity

2019-05-07 14:56

obscurity2019-05-07 14:56

Rejestracja:około 6 lat
Ostatnio:około 9 godzin

wydaje się że chcesz po prostu wyśrodkować element w komórkach po 135 pikseli, kod powinien wyglądać mniej więcej tak:

Kopiuj

int cellHeight = 135;

if (b.Location.X >= 0 && b.Location.X <= 100)
{
    var yAligned = (b.Location.Y / cellHeight) * cellHeight;
    b.Location = new Point(50 - b.Width / 2, yAligned + cellHeight / 2 + b.Height / 2);
}

podejrzewam że to samo chcesz zrobić z b.Location.X

edytowany 3x, ostatnio: obscurity 2019-05-07 15:00

Jan33336

2019-05-07 16:08

Jan333362019-05-07 16:08

Rejestracja:prawie 6 lat
Ostatnio:ponad 5 lat
Postów:18

obscurity napisał(a):

wydaje się że chcesz po prostu wyśrodkować element w komórkach po 135 pikseli, kod powinien wyglądać mniej więcej tak:
Kopiuj
int cellHeight = 135;

if (b.Location.X >= 0 && b.Location.X <= 100)
{
 var yAligned = (b.Location.Y / cellHeight) * cellHeight;
 b.Location = new Point(50 - b.Width / 2, yAligned + cellHeight / 2 + b.Height / 2);
}
podejrzewam że to samo chcesz zrobić z b.Location.X

Bardzo dziękuje i z góry chciał bym tez podziękować innym za bardzo pomoce porady i wlasnie tutaj nawiazujac chcialem sie spytac czy dalo by sie rowniez skrocic to b.Location.X bo ja wykorzystuje te sktory to wychodzi cos takiego

Kopiuj

            if (b.Location.X >= 0 && b.Location.X <= 100)
            {
                for (int i = 0; i <= 500; i++)
                {
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(50 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
                        break;
                    }
                }
            }

            if (b.Location.X >= 100 && b.Location.X <= 200)
            {
                for (int i = 0; i <= 500; i++)
                {
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(150 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
                        break;
                    }
                }
            }

            if (b.Location.X >= 200 && b.Location.X <= 300)
            {
                for (int i = 0; i <= 500; i++)
                {
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(250 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
                        break;
                    }
                }
            }

            if (b.Location.X >= 300 && b.Location.X <= 400)
            {
                for (int i = 0; i <= 500; i++)
                {
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(350 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
                        break;
                    }
                }
            }

            if (b.Location.X >= 400 && b.Location.X <= 500)
            {
                for (int i = 0; i <= 500; i++)
                {
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(450 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
                        break;
                    }
                }
            }

            if (b.Location.X >= 500 && b.Location.X <= 600)
            {
                for (int i = 0; i <= 500; i++)
                {
                    if (b.Location.Y >= Convert.ToInt32(position[i]) && b.Location.Y <= Convert.ToInt32(position[i + 1]))
                    {
                        b.Location = new Point(550 - (b.Width / 2), (68 + (136 * i)) - (b.Height / 2));
                        break;
                    }
                }
            }

AnyKtokolwiek

2019-05-07 17:20

AnyKtokolwiek2019-05-07 17:20

Rejestracja:ponad 6 lat
Ostatnio:2 dni
Postów:3561

Przeczuwam możliwość skrócenia i uczynienia kodu bardzie czytelnym, gdyby tablica position była tablicą par liczb X,Y (???), lub czwórek.(X,Y, coś, coś)
Nie podajesz deklaracji ani komentarza co w niej jest

Jan33336

2019-05-07 17:32

Jan333362019-05-07 17:32

Rejestracja:prawie 6 lat
Ostatnio:ponad 5 lat
Postów:18

Są w niej zapisane przedziały od do gdzie przycisk ma się wyśrodkować

Plik position fragment

Kopiuj

Delor 2019-05-07 17:44

Czyli można napisać index * 135 zamiast position[index].

Jan33336 2019-05-07 18:03

Słusznie już to zmieniłem

obscurity

2019-05-07 17:46

obscurity2019-05-07 17:46

Rejestracja:około 6 lat
Ostatnio:około 9 godzin

Kopiuj

int cellHeight = 135;
int cellWidth = 100;

var yAligned = (b.Location.Y / cellHeight) * cellHeight;
var xAligned = (b.Location.X / cellWidth) * cellWidth;
b.Location = new Point(xAligned + cellWidth / 2 + b.Width / 2, yAligned + cellHeight / 2 + b.Height / 2);

edytowany 3x, ostatnio: obscurity 2019-08-02 05:30

Jan33336

2019-05-07 18:02

Jan333362019-05-07 18:02

Rejestracja:prawie 6 lat
Ostatnio:ponad 5 lat
Postów:18

obscurity napisał(a):

Kopiuj

int cellHeight = 135;
int cellWidth = 100;

var yAligned = (b.Location.Y / cellHeight) * cellHeight;
var xAligned = (b.Location.X / cellWidth) * cellWidth;
b.Location = new Point(xAligned + cellWidth / 2 + b.Width / 2, yAligned + cellHeight / 2 + b.Height / 2);

Niestety ten kod jeżeli chodzi o mnie nie działa ponieważ jak przesuwam element (przycisk) to nie jest on na środku tylko poza prostokątem

Jan33336

2019-05-07 18:55

Jan333362019-05-07 18:55

Rejestracja:prawie 6 lat
Ostatnio:ponad 5 lat
Postów:18

Dobrze po lekkiej mojej modyfikacji kodu @obscurity wyszedł mi kod który według mnie jest już na sensownym poziomie w porównaniu do tego co było na początku bardzo dziękuje za wszelka pomoc wszystkich którzy pisali porady i wskazówki jestem wam za to bardzo wdzięczny chociaż wiem ze to dopiero początek mojej długiej przygody z c# mam nadzieje ze kiedyś również będę mógł dawać porady początkującym tak jak wy dawaliście mi jeszcze raz dziękuje :)

Poniżej wklejam modyfikacje która wykonałem

Kopiuj

            int cellHeight = 135;
            int cellWidth = 100;

            var yAligned = (b.Location.Y / cellHeight) * cellHeight;
            var xAligned = (b.Location.X / cellWidth) * cellWidth;
            b.Location = new Point((b.Width / 2) - 74 + xAligned + cellWidth / 2, yAligned + cellHeight / 2 + b.Height / 2 - 21);

edytowany 1x, ostatnio: Jan33336 2019-05-07 18:55

obscurity

2019-05-08 04:42

obscurity2019-05-08 04:42

Rejestracja:około 6 lat
Ostatnio:około 9 godzin

miało być oczywiście "- b.Width / 2" i "- b.Height / 2", ale można też powpisywać magiczne liczby tak żeby się zgadzało ;)

programmer123

2019-05-08 10:08

programmer1232019-05-08 10:08

Rejestracja:prawie 6 lat
Ostatnio:ponad 5 lat
Postów:10

Warto przeczytać to: https://docs.microsoft.com/pl-pl/dotnet/csharp/whats-new/csharp-8

Liczba odpowiedzi na stronę

Treść *

Treść

Podgląd

Kliknij, aby dodać treść...

Kliknij, aby dodać załącznik lub wklej ze schowka.

Instrukcja obsługi Markdown

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj underline.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, , , <kbd>, <del>, , <dfn>, <pre>, <blockquote>, <hr/>, ,  oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H2O i m2 otrzymasz: H₂O i m².

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript document.write('Hello World'); ```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

Alt+L - zaznaczenie całej linii
Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

Ctrl+Enter - dodaj post
⌘+Enter - dodaj post (MacOS)

Skracanie kodu

abrakadaber

obscurity napisał(a):

obscurity napisał(a):

Smileys & People

Animals & Nature

Food & Drink

Travel & Places

Activities

Objects

Symbols

Typografia

Linki

Wewnętrzne odnośniki

Wspomnienia użytkowników

Znaczniki HTML

Skróty klawiszowe

Indeks górny oraz dolny

Składnia Tex

Kod źródłowy

Krótkie fragmenty kodu

Kod wielolinijkowy

Tabelki

Lista uporządkowana i nieuporządkowana

Składnia Markdown

Skróty klawiszowe