• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Generator haseł - działa ale zwraca undefined + wygenerowane hasło

Object Storage Arubacloud
0 głosów
269 wizyt
pytanie zadane 25 lutego 2019 w JavaScript przez Sobol3k Użytkownik (690 p.)
<html lang="">
<head>
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>generator haseł</title>
    <link href="https://fonts.googleapis.com/css?family=Lato&amp;subset=latin-ext" rel="stylesheet">
</head>
<body>
  <div class="wrapper">
     <div class="header"><h1>Generator haseł</h1></div>
     <div class="how-many-characters">Wybierz z ilu znaków ma się składać hasło:</div>
     <div class="row">
       <label>6 znaków <input type="radio" value="6" name="character"></label>
       <label>8 znaków <input type="radio" value="8" name="character"></label>
       <label>10 znaków <input type="radio" value="10" name="character" class="exception"></label>
     </div>
     <div class="window-with-generated-password">
       <input type="text" value="PASSWORD" class="generated-password">
     </div>
     <div class="button">
       <button type="button">Generuj</button>
     </div>
     <!-- pseudo element -->
     <div class="pseudo-element"></div>
  </div>
  <script>
    let wrapper = document.querySelector('.wrapper');
    let inputGeneratedPassword = wrapper.querySelector(':scope .generated-password');
    let btn = wrapper.querySelector(':scope button');
    let radioInputs = wrapper.querySelectorAll(':scope input[type="radio"]');
      
    let tab = ['a', 'k', 'N', 'W', 'x', '7', 'h', 'o', 'n', '*', '?', '/', 'r', '8', 'j', 'n', '', 'h', '#', '5', '-', 'g', 'b', '*', 'm'];
    
    inputGeneratedPassword.addEventListener("click", function(){
        this.setAttribute('value', '');
    }, false);
    
    btn.addEventListener("click", function(){
        
        for(let i=0; i<radioInputs.length; i++){
            
            if(radioInputs[i].checked){
                
                let value = parseInt(radioInputs[i].value);
                let randomNumbers = [];
                let password;
                
                if(value === 6){
                    
                    for(let i = 0; i<value; i++){
                        
                        randomNumbers[i] = Math.floor(Math.random() * 24);
                        console.log(randomNumbers[i]);
                    }
                    
                    for(let j = 0; j<randomNumbers.length; j++){
                            
                        password += tab[randomNumbers[j]];
                        console.log(tab[randomNumbers[j]]);
                    }
                    
                    console.log(randomNumbers.length, password);
                    inputGeneratedPassword.value = password; 
                }
                
                if(value === 8){
                    
                    for(let i = 0; i<value; i++){
                        
                        randomNumbers[i] = Math.floor(Math.random() * 24);
                        console.log(randomNumbers[i]);
                    }
                    
                    for(let j = 0; j<randomNumbers.length; j++){
                            
                        password += tab[randomNumbers[j]];
                        console.log(tab[randomNumbers[j]]);      
                    }          
                    
                    console.log(randomNumbers.length, password);
                    inputGeneratedPassword.value = password; 
                }
                
                if(value === 10){
                    
                    for(let i = 0; i<value; i++){
                        
                        randomNumbers[i] = Math.floor(Math.random() * 24);
                        console.log(randomNumbers[i]);
                    }
                    
                    for(let j = 0; j<randomNumbers.length; j++){
                            
                        password += tab[randomNumbers[j]];
                        console.log(tab[randomNumbers[j]]);      
                    }          
                    
                    console.log(randomNumbers.length, password);
                    inputGeneratedPassword.value = password; 
                }
                
                break;
            }
            
        }
        
    }, false)
  </script>
</body>
</html>

Czy ktoś mi może wytłumaczyć dlaczego oprócz wygenerowanego hasła dodaje mi na początek undefined analizowałem to jakiś czas, ale nie doszedłem do wniosku czemu to mi się tak generejue. Przepraszam że tyle kodu ale chciałem wstawić już z css i html abyście mieli pogląd na to. Niestety ale jednak skrypt nie pozwala mi wstawić stylu css więc z góry sorry za brak stylów.

1 odpowiedź

+1 głos
odpowiedź 25 lutego 2019 przez ScriptyChris Mędrzec (190,190 p.)
password += tab[randomNumbers[j]];

W 58 linijce zmienna password w trakcie pierwszej iteracji ma wartość undefined (bo zadeklarowałeś ją w 46 linijce bez inicjalizacji):

let password;

Po konkatenacji z wartością uzyskaną z wyrażenia po prawej stronie operatora += otrzymujesz wynik np. "undefined?" - czyli najpierw undefined i potem jakiś losowy znak.

Polecam korzystać z debuggera w przeglądarce.

komentarz 25 lutego 2019 przez Sobol3k Użytkownik (690 p.)
Ale, głupi błąd z mojej strony rzeczywiście bez zdefiniowania wartości zmienna na starcie na undefined.... Dziękuję za pomoc :)

Podobne pytania

0 głosów
2 odpowiedzi 1,141 wizyt
pytanie zadane 8 listopada 2018 w JavaScript przez Sobol3k Użytkownik (690 p.)
0 głosów
1 odpowiedź 332 wizyt
0 głosów
1 odpowiedź 112 wizyt
pytanie zadane 20 stycznia 2023 w JavaScript przez pazo1313 Nowicjusz (140 p.)

92,555 zapytań

141,403 odpowiedzi

319,559 komentarzy

61,940 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Kolejna edycja największej imprezy hakerskiej w Polsce, czyli Mega Sekurak Hacking Party odbędzie się już 20 maja 2024r. Z tej okazji mamy dla Was kod: pasjamshp - jeżeli wpiszecie go w koszyku, to wówczas otrzymacie 40% zniżki na bilet w wersji standard!

Więcej informacji na temat imprezy znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...