[Javascript] Generator liczb

Strona 1 z 1

Nie możesz napisać tematu
Dodaj odpowiedź

#1 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 01 listopad 2011, 18:58

Na lekcjach infy robimy strony. Ja robię stronę sklepów i program lojalnościowy.
Moim problemem jest to, że chcę aby po wejściu na stronę dolacz.html Java generowała mi numer klienta w sposób pokazany na obrazku:
http://forum.kompute...st&attach_id=91

Mój problem polega na tym, że mam kod dolacz.html(fragment związany z generowaniem kodu):

<html>
<head>
<title>Testowanie formularza</title>,
<script type="application/javascript" src="gen.js">
</script>
</head>
<body>
<script language="JavaScript" type="text/javascript">
<!--
generujNrKlienta()
//-->
</script>
<form method="post" enctype="text/plain">
<table>
<tr><td>Twój numer klienta</td>
<td><input type="text" name="NrKlienta"></td></tr>

i gen.js

function generujNrKlienta(){
form = document.opcje
var sings = new Array ("")

{
sings.push("1", "2", "3", "4", "5", "6", "7", "8", "9", "0")
}

NrKlienta = ""
for ( i=1 ; i<=13; i++ ) {
los = Math.floor(Math.random() * sings.length);
litera = sings[los];
NrKlienta += litera;
}


form.NrKlienta.value = NrKlienta;

return true;
}

Niestety te kody nie działają. Chciałbym także, aby wygenerowany kod nie pojawiał się w input tylko statycznie, ale był możliwy do przekazania PHP.
Ktoś wie co poprawić?

Użytkownik OldShaterhan edytował ten post 01 listopad 2011, 19:00

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#2 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 01 listopad 2011, 20:03

Istnieje prawdopodobieństwo, że inny klient otrzyma ten sam ID. Zobacz: http://phpjs.org/functions/uniqid:750

Jeżeli chcesz mieć ten ID po stronie PHP, musisz go przesłać w formularzu albo wygenerować po stronie PHP.

Jeżeli koniecznie chcesz mieć liczbowy identyfikator:

var numer = 10000000 + Math.floor(Math.random() * 90000000); //numer 8-cyfrowy

Twój kod też jest dobry, ale ma błędy składni:

function generujNrKlienta(){
  var form = document.forms['opcje']; //jeżeli formularz ma id="opcje"
  var signs = new Array("1", "2", "3", "4", "5", "6", "7", "8", "9", "0");
  var NrKlienta = '';
  for (var i=1; i<=13; i++) {
    NrKlienta = NrKlienta.concat(signs[Math.floor(Math.random() * 10)]);
  }
  form.NrKlienta.value = NrKlienta;
  return true;
}

Przygoda w Biurowej Dzielnicy - TODA :: Przyszłość sieci Webb i informatyzacja Polski
_{There is a road to freedom. Its milestones are Obedience, Endeavor, Honesty, Order, Cleanliness, Sobriety, Truthfulness, Sacrifice, and love of the Fatherland.}

#3 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 02 listopad 2011, 06:51

Podmieniłem ten kod JS na Twój, ale się nic nie pojawia w polu inputa.

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#4 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 02 listopad 2011, 19:16

Adres strony w obecnym stanie(strona będzie co jakiś czas aktualizowana): http://kwas.kasztanek.tk/dolacz.html

Użytkownik OldShaterhan edytował ten post 02 listopad 2011, 19:18

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#5 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 03 listopad 2011, 09:30

Nie do końca podmieniłeś. Zamień type="application/javascript" na type="text/javascript", bo starsze przeglądarki tego nie odczytają. Wewnątrz funkcji generujNrKlienta() odnosisz się do form. Nie nadałeś formularzowi żadnego identyfikatora! Jeżeli jesteś pewny, że ten formularz będzie pierwszy w kodzie, możesz też użyć zapisu:

document.forms[0].NrKlienta.value = NrKlienta;

To jest skrócony zapis. Jeżeli występuje konflikt nazw między polami a własnościami formularza, używa się:

document.forms[0].elements['NrKlienta'].value = NrKlienta;

#6 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 03 listopad 2011, 15:36

Nie mam pojęcia co robię źle, ale nadal nie działa ;/...

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#7 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 03 listopad 2011, 20:35

Ten kod umieść dopiero przed zamknięciem </body>

<script type="text/javascript">
generujNrKlienta()
</script>

W gen.js zapomniałeś o klamrze zamykającej } na końcu funkcji.

#8 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 04 listopad 2011, 15:19

W gen.js klamry mi się zgadzają (2:2).
Przeniesienie kodu pomogło. A jeszcze jedno pytanie - załóżmy, że chciałbym jedna czyfra, aby była stała. Co wówczas mam zrobić? Powiedzmy, że ta stała to pierwsza cyfra.

Użytkownik OldShaterhan edytował ten post 04 listopad 2011, 15:21

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#9 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 04 listopad 2011, 19:04

Odnajdź miejsce, gdzie deklarujesz zmienną NrKlienta. Tam możesz ustawić wartość początkową.

var NrKlienta = 'A'; //gdy pierwszy znak to A, tak samo z cyframi

#10 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 05 listopad 2011, 16:19

Ok działa. A jeszcze (z czystej ciekawości) - jakbym chciał, aby ten określony znak był na innym miejscu?

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#11 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 05 listopad 2011, 19:02

Możesz na kilka sposobów:

1. Dodaj dodatkowy warunek w pętli:

if(i == 5)
{
   //dodaj stały znak
}
else
{
   //losuj znak
}

2. Wstaw stały znak do istniejącego numeru klienta (zwiększy długość o 1)

NrKlienta = NrKlienta.substr(0,3) + 'X' + NrKlienta.substr(3);

3. Stwórz 2 pętle losujące znaki, a pomiędzy nimi dodaj stały znak

for(i=0; i<10; i++) { /* losuj */ }
NrKlienta += 'X';
for(i++; i<10; i++) { /* losuj */ }

4. Podmień wygenerowany losowo znak na stały:

NrKlienta[3] = 'X';

5. Utwórz gotowy ciąg znaków ze stałą, a potem przypisuj losowe znaki:

NrKlienta = '   X      ';
for(i=0; i<10; i++) { if(NrKlienta[i] == '') { /* losuj */ } }

#12 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 06 listopad 2011, 20:41

Ok, dzięki. Jeszcze jedno - 13 znak ma być przedstawiony nastepująco:
Kolejne cyfry kodu - NrKlienta[1]*1 dodać NrKlienta[2]*3 dodać NrKlienta[4]*1 dodać NrKlienta[5]*3 itd. do 12. Następnie wyciągnąć z tego resztę(dzielenie przez 10; suma mod 10). Od 10 odjąć tę resztę i ponownie wyciągnąć resztę przez 10 (iloraz mod 10). Po prostu ma to być zgodne z EAN-13.
Interesuje mnie tylko jak zapisać działanie w JS - czy normalnie, czy jakoś inaczej.

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#13 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 06 listopad 2011, 20:48

Dodawanie, mnożenie, odejmowanie i dzielenie jak w innych językach: + - * /
Reszta z dzielenia: %

Napisz kod i przetestuj go. Jeżeli coś będzie źle, poprawimy

#14 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 06 listopad 2011, 21:42

Jeszcze nie działa(testuję kod generatorem kodów kreskowych

)

//gen.js
function generujNrKlienta(){
  var signs = new Array("1", "2", "3", "4", "5", "6", "7", "8", "9", "0");
  var NrKlienta = '7';
  for (var i=1; i<=12; i++) {
    NrKlienta = NrKlienta.concat(signs[Math.floor(Math.random() * 10)]);
  }
  NrKlienta[13]= (10-(NrKlienta[1]*1+NrKlienta[2]*3+NrKlienta[3]*1+NrKlienta[4]*3+NrKlienta[
5]*1+NrKlienta[6]*3+NrKlienta[7]*1+NrKlienta[8]*3+NrKlienta[9]*1+NrKlienta[
10]*3+NrKlienta[11]*1+NrKlienta[12]*3)%10)%10
  document.forms[0].elements['NrKlienta'].value = NrKlienta;
  return true;
}

Wprowadzenie var przed NrKlienta[13] nie pomogło(przestało generować).

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#15 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 06 listopad 2011, 22:15

Użyj konstrukcji NrKlienta += albo NrKlienta = NrKlienta.concat()

NrKlienta[13] jeszcze nie istnieje, więc nie możesz ot tak przypisać.

#16 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 06 listopad 2011, 22:35

Mnie wyszło, że istnieje - 7 jako pierwszy znak i 12 kolejnych - jest ich 13.

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#17 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 06 listopad 2011, 22:36

Zamień NrKlienta[13]= na NrKlienta +=

#18 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 07 listopad 2011, 18:16

Generuje np: 792467343148NaN, 716796775082NaN(NaN jest trzynastym znakiem, bo zmieniłem i<=12 na i<=11).
Kod:

function generujNrKlienta(){
  var signs = new Array("1", "2", "3", "4", "5", "6", "7", "8", "9", "0");
  var NrKlienta = '7';
  for (var i=1; i<=11; i++) {
    NrKlienta = NrKlienta.concat(signs[Math.floor(Math.random() * 10)]);
  }
  NrKlienta+= (10-(NrKlienta[1]*1+NrKlienta[2]*3+NrKlienta[3]*1+NrKlienta[4]*3+NrKlienta[
5]*1+NrKlienta[6]*3+NrKlienta[7]*1+NrKlienta[8]*3+NrKlienta[9]*1+NrKlienta[
10]*3+NrKlienta[11]*1+NrKlienta[12]*3)%10)%10
  document.forms[0].elements['NrKlienta'].value = NrKlienta;
  return true;
}

Użytkownik OldShaterhan edytował ten post 07 listopad 2011, 18:49

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#19 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 07 listopad 2011, 18:56

Problem jest w tym, że NrKlienta jest łańcuchem znaków. Aby skonwertować go na liczbę całkowitą, musisz użyć funkcji parseInt()

NrKlienta += parseInt(NrKlienta[1]) + parseInt(NrKlienta[2])*3 + ...

Liczba powinna skonwertować się automatycznie jako znak przy dołączeniu do zmiennej NrKlienta.

#20 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 07 listopad 2011, 19:10

Ja teraz zauważyłem, że w gen.js po tym kodzie z += nie ma średnika. Dodać? Jeśli tak to przeniesienie części działania do nowej linii mogę zrobić normalnie?

Użytkownik OldShaterhan edytował ten post 07 listopad 2011, 19:11

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#21 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 07 listopad 2011, 19:14

Nie powinno być problemów. Po prostu sprawdź, czy po przeniesieniu do nowej linii skrypt wykona się, czy wyrzuci błąd. Po co chcesz przenosić działania do nowej linii, jeśli nie ma potrzeby?

#22 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 07 listopad 2011, 19:24

Po prostu w notatniku kod strasznie w prawo poleciał, i nie lubię jak są długie linie, a ta taką jest.
Dodałem parsowanie i nadal NaN'y robi. Chyba, że ja coś źle zrobiłem.

function generujNrKlienta(){
  var signs = new Array("1", "2", "3", "4", "5", "6", "7", "8", "9", "0");
  var NrKlienta = '7';
  for (var i=1; i<=11; i++) {
    NrKlienta = NrKlienta.concat(signs[Math.floor(Math.random() * 10)]);
  }
  NrKlienta+= (10-(parseInt(NrKlienta[1])+parseInt(NrKlienta[2])*3+parseInt(NrKlienta[3])+parseInt(NrKlienta[
4])*3
  +parseInt(NrKlienta[5])+parseInt(NrKlienta[6])*3+parseInt(NrKlienta[7])+parseInt(NrKlienta[
8])*3
  +parseInt(NrKlienta[9])+parseInt(NrKlienta[10])*3+parseInt(NrKlienta[11])+parseInt(NrKlienta[
12])*3)%10)%10;
  document.forms[0].elements['NrKlienta'].value = NrKlienta;
  return true;
}

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#23 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 07 listopad 2011, 22:59

Testuję kod w IE 8. Nie obsługuje notacji zmienna[5]. Pozostaje zmienna.charAt()

parseInt(NrKlienta.charAt(5))

Możesz sobie ułatwić zadanie, np. liczyć ostatnią cyfrę jeszcze w pętli lub rozbić NrKlienta na tablicę znaków i zamienić masowo na liczby całkowite. A nie lepiej tworzyć ID klienta po stronie PHP? Przecież przed wysłaniem formularza użytkownik może zmienić numer na inny, nawet jak ustawisz atrybut readonly! Do serwera można wysłać dowolne dane.

Lektury do przeczytania
https://developer.mo..._Objects/String
https://developer.mo...nt_i_parseFloat
https://developer.mo...l_Objects/array
https://developer.mo...s/Array/forEach

#24 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 08 listopad 2011, 16:23

function generujNrKlienta(){
  var signs = new Array("1", "2", "3", "4", "5", "6", "7", "8", "9", "0");
  var NrKlienta = '7';
  for (var i=1; i<=11; i++) {
    NrKlienta = NrKlienta.concat(signs[Math.floor(Math.random() * 10)]);
  }
  NrKlienta+= (10-(parseInt(NrKlienta.charAt(1))+parseInt(NrKlienta.charAt(2))*3+parseInt(NrKlienta.charAt(3))+parseInt(NrKlienta.
charAt(4))*3
  +parseInt(NrKlienta.charAt(5))+parseInt(NrKlienta.charAt(6))*3+parseInt(NrKlienta.charAt(7))+parseInt(NrKlienta.
charAt(8))*3
  +parseInt(NrKlienta.charAt(9))+parseInt(NrKlienta.charAt(10))*3+parseInt(NrKlienta.charAt(11))+parseInt(NrKlienta.
charAt(12))*3)%10)%10;
  document.forms[0].elements['NrKlienta'].value = NrKlienta;
  return true;
}

Nadal NaN. A może problem tkwi nie w tym, a w błędzie w działaniu?
A jakby ew. spróbować tego generowania w PHP?
Bo chcę jeszcze zrobić, że jak zacznie działać generacja kodu poprawnie, to chcę dodać wysyłanie maila z PDF'em, w którym ma być "karta tymczasowa"

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

#25 Ferrari

Tajny agent USB

Grupa: Moderatorzy
Postów 3531
Rejestracja: 21-marzec 04

Napisano 08 listopad 2011, 17:00

Numerujesz indeksy od 1, a trzeba od 0. Wszędzie w charAt() obniż indeks o 1.

Jak to zrobić w PHP?

$znaki = range('0', '9');
$NrKlienta = '';
for($i=0; $i<12; $i++)
{
  $NrKlienta .= $znaki[array_rand($znaki)];
}
$NrKlienta .= 10 - ($NrKlienta[0] + $NrKlienta[1]*3 + ...

Nie musisz konwertować typów. PHP powinien sam skonwertować. Liczenie ostatniego znaku możesz sobie ułatwić np. w pętli. Poczytaj:

http://pl.php.net/array_rand
http://php.net/manua...expressions.php

#26 OldShaterhan

Początkujący Uczestnik

Grupa: Czytelnicy
Postów 172
Rejestracja: 22-czerwiec 09

Napisano 08 listopad 2011, 17:06

Dzięki

- zaczęło generować 13. A powiedz jeszcze - czy w takim wypadku mogę uprościć kod (jeszcze JS) na NrKlienta[0-11]?

Jeżeli możesz, spróbuj odpowiedzieć na moje pytanie w tym temacie.
Dołączona grafika

Strona 1 z 1

Nie możesz napisać tematu
Dodaj odpowiedź

Szybka odpowiedź

Użytkownicy przeglądający ten temat: 1
0 użytkowników, 1 gości, 0 anonimowych

Zmień widoczność shoutboxa Shoutbox

emikyou :	(16 kwiecień 2012 - 13:26) @Ferrari: Na PŁ korzystają z "OpenGL SuperBible: Comprehensive Tutorial and Reference" i "OpenGL Programming Guide: The Official Guide to Learning OpenGL, Versions 3.0 and 3.1"
wtomczak :	(10 kwiecień 2012 - 09:27) masz problem z komputerem napisz na facebooku na stronie WT654, na pewno pomoże
Maciej :) :	(08 kwiecień 2012 - 09:36) A co do reklam, to mi na operze adBlock coś nie służy, szczególnie z postami bota redakcji od kiedy go dali, to zaczęło się walić..
Maciej :) :	(08 kwiecień 2012 - 09:35) Hej witam no, jak przestałem kupować Eksperta to wzięli i wstrzymali ;\| także nie mam tylko numeru pierwszego i ostatniego :p trochę szkoda, parę lat się było...
neo5628 :	(05 kwiecień 2012 - 21:57) Help http://forum.ks-ekspert.pl/topic/140746-problem-z-dzwiekiem/ Proszę
PiKey :	(21 marzec 2012 - 18:24) ludzie, pomóżcie http://forum.ks-ekspert.pl/topic/140418-jak-dziala-sprawdzanie-wykonania-zadania-na-stronach-typu-doladujse/
PiKey :	(21 marzec 2012 - 18:24) ds
A:-)Brunuś :	(17 marzec 2012 - 19:17) jest tu ktoś? http://forum.ks-ekspert.pl/topic/140400-firewire-obudowa-do-dysku-25-—-200-zlo/
Ferrari :	(16 marzec 2012 - 11:12) Polećcie dobrą książkę do OpenGL albo kurs w sieci
lisz55 :	(15 marzec 2012 - 21:44) przez ktore reklamy? nie widze zeby jakies nowe byly
Pretender :	(15 marzec 2012 - 18:22) :/
Pretender :	(15 marzec 2012 - 18:22) az sie nie chce czytac tego forum przez te parszywe reklamy...ale dzidostwo zrobili...
yojo2 :	(12 marzec 2012 - 13:26) wchodz na forum przez lynxa :P
Ferrari :	(10 marzec 2012 - 21:21) Jak tu nie korzystać z AdBlocka, kiedy Forum Eksperta wywala reklamę na całą stronę?
lisz55 :	(09 marzec 2012 - 21:41) siedze i czytam calkiem niezle ale mogl by ktos wytlumaczyc o co chodzi z dark knightem nie widzialem tego filmu
lisz55 :	(09 marzec 2012 - 21:40) to moze rozruszam troche czat http://moviesascode.net/
Pretender :	(06 marzec 2012 - 16:43) rozmowy tak jak Enty prowadziły w powieści Tolkiena :P
dawido90 :	(05 marzec 2012 - 15:03) to już poprostu przyzwyczajenie
yojo2 :	(04 marzec 2012 - 22:25) mailem przestac tu wchodzic, ale stwierdzilem ze skoro i tak zwykle tu tylko wchodze i wychodze, to wszystko jedno :P
Ferrari :	(04 marzec 2012 - 21:16) Już myślałem, że ruch na forum spada zera, a jednak pochodna zmieniła znak

Komputer Świat Ekspert - Forum: [Javascript] Generator liczb - Komputer Świat Ekspert - Forum