• 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

Mały hosting, OGROMNE możliwości
0 głosów
422 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,170 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,623 wizyt
pytanie zadane 8 listopada 2018 w JavaScript przez Sobol3k Użytkownik (690 p.)
0 głosów
1 odpowiedź 535 wizyt
0 głosów
1 odpowiedź 366 wizyt
pytanie zadane 20 stycznia 2023 w JavaScript przez pazo1313 Nowicjusz (140 p.)

93,717 zapytań

142,629 odpowiedzi

323,261 komentarzy

63,261 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

Twierdza Linux. Bezpieczeństwo dla dociekliwych

Aby uzyskać rabat -10%, użyjcie kodu pasja-linux, wpisując go w specjalne pole w koszyku.

...