Hejto.pl
Dodaj post

Wpisz coś do wyszukania (minimum 2 znaki)

pumpernikielSpecjalista

Dołączył/a:

  • 11 wpisów
  • 59 komentarzy
  • 0 obserwujących
Hydeparkno skislem xD nie bedziesz robaku wyjezdzac poza europe! enea #patoenergetyka #energetyka
Specjalista1piorunów

@tomasz-frankowski
Important bugs I found

A few things in the original are particularly suspicious:

1. dalej_modal_false = true means this branch is effectively always selected:

} else if (dalej_modal_false) {
modalService.dalejModalFalse();
}

Consequently, dalejModalDefault() is effectively unreachable.

2. These conditions are always truthy:

if (strefaCiszyArr && ...)

An array exists, so strefaCiszyArr itself doesn't test anything.

3. String/number comparison:

if (liczba_n === 0 && liczba_u > 0 ...)

$("_n").val() returns a string. This should use numeric conversion.

4. This appears to be a selector bug:

znizkaWrapper.find("liczba_u2" + (i + 1))

It is missing #.

5. This comparison can never be useful:

var klasa_poc1 = $("_wagonu").find("option:selected").val();
var klasa_poc2 = $("_wagonu").find("option:selected").val();

if (klasa_poc1 !== klasa_poc2) {

Both values come from exactly the same element.

6. This:

.prop("disabled", "disabled")

should be:

.prop("disabled", true)

7. There are many places where the code repeatedly does:

$("#...")
$("....")

and repeatedly reads the same values. This makes the main event handler much harder to reason about.

8. There are several unused variables, including:

rower
v3
v4
v5
v6
arrow_box_ext_info // depending on later usage

Some of those may have historical reasons, so I'd remove them only after checking the rest of the application.

HydeparkWitam państwa z nowego stanu skupienia - bezrobotny :smiley: Jeszcze tylko zakończyć L4 i będzie idealnie. pracbaza
DyskusjeKomuś nieźle pociąg odjechał z wymaganiami dot. dokumentów xDDDDD Ale serio to współczuje dzieciom…
Specjalista0piorunów

@Shagwest Konieczność stworzenia i podpisania 6 dokumentów które tak na prawdę nie są potrzebne, a w rzeczywistości mają pewnie odstraszyć żeby szkoła mogła powiedzieć: nie mamy pracownika wykwalifikowanego, pani Gienia będzie uczyć informatyki bo brakuje jej do etatu. Betonoza w najczystszej postaci - kto na tym straci? Dzieci.

Wiadomości Świat„Głowa i ramiona wystawały przez rozbite okno”. Horror podczas lotu Ryanaira
Specjalista9piorunów

@Mr.Mars Mam nadzieję że nigdy nie dowiem się jak to jest wystawić głowę za okno w samolocie lecącym z prędkością ~500km/h