Räkna procent ACCESS ASP

[climber]

Hej,

Försöker förstå mig på hur jag ska kunna räkna ut procent.
Följande är givet:

Denna kod räknar ut hur många tävlingar en spelare spelat

<% Set AntalV=Conn.Execute("Select Count(*) As antal From tbl_res Where PlayerId= '"& strpID &"' ") %>

och denna kod räknar ut hur många gånger spelaren har slutat med ett resultat mellan 18 och 29 poäng

<% Set AntalB = Conn.Execute("SELECT COUNT(TOTAL) AS AntalBla FROM tbl_res WHERE (TOTAL) BETWEEN 18 AND 29 AND PlayerId= '"& strpID &"'") %>

Det jag nu vill räkna ut är hur många procent spelaren hamnat i det poängintervallet.

Försökte med följande kod

<% SET PB = Conn.Execute("SELECT COUNT(AntalBla) / COUNT(antal) * 100.0 AS PBla WHERE PlayerId= '"& strpID &"'") %>

men det ger bara följande felmeddelande:

Microsoft JET Database Engine error '80040e14'

The SELECT statement includes a reserved word or an argument name that is misspelled or missing, or the punctuation is incorrect.

Vad gör jag för fel? (förmodligen något generalfel men jag är ca 2 veckor gammal i ASP djungeln)

All feedback tas gladeligen emot

[voigtann1]

nu vet jag inte men det känns som du behöver paranterser:

<% SET PB = Conn.Execute("SELECT (COUNT(AntalBla) / COUNT(antal) * 100) AS PBla WHERE PlayerId= '"& strpID &"'") %>

annars kan du ju göra det när du presentera innehållet.

[climber]

Hej voigtann1,

hur menar du att jag kan göra det när jag presenterar innehållet?

[voigtann1]

du sätter AntalBla och Antal till alias och använder dom när du presentera:

<% SET PB = Conn.Execute("SELECT COUNT(AntalBla) as X, Count(antal) as Y WHERE PlayerId= '"& strpID &"'") %>

och när du loopar ut innehållet så använder du ASP för beräkningen:

<%= (PB("x") / PB("Y"))* 100 %>

[climber]

provade det du skrev ovan men fick fortfarande felmeddelandet:

Microsoft JET Database Engine error '80040e14'

The SELECT statement includes a reserved word or an argument name that is misspelled or missing, or the punctuation is incorrect.

Jag fattar inte vad det är jag missar.

[voigtann1]

Du använder reserverade ord i din SQL fråga använd [] runt de fältnamn du har angett som är ett resarverat ord för access

[climber]

Vad är ett reserverat ord för Access då? Är det namnen på tabellerna & kolumnerena?
Eller finns det någon lista på ord som är reserverade?

[voigtann1]

http://office.microsoft.com/en-us/ac...010030643.aspx

[civilpolisen]

<% SET PB = Conn.Execute("SELECT (COUNT(AntalBla) / COUNT(antal) * 100) AS PBla WHERE PlayerId= '"& strpID &"'") %>

<% SET PB = Conn.Execute("SELECT (COUNT(AntalBla) / COUNT(antal) * 100) AS PBla FROM [tbl_res] WHERE PlayerId= '"& strpID &"'") %>

[voigtann1]

ah, inte en form med i sql frågan... men ordet som är reserverad är förmodligen antal och verkligen inte tbl_res

[civilpolisen]

Nej, det tror jag inte heller!
Men låt oss vara tydliga, för omväxlings skull!

<% SET PB = Conn.Execute("SELECT (COUNT([AntalBla]) / COUNT([antal]) * 100) AS PBla FROM [tbl_res] WHERE PlayerId= '"& strpID &"'") %>

[climber]

Hej,

jag löste det på följande sätt:

<%
   Set percentBla = Conn.Execute("SELECT COUNT(TOTAL) AS PBX FROM tbl_res WHERE (TOTAL) BETWEEN 18 AND 29 AND PlayerId= '"& strpID &"'")
%>
<%
   Set totVarv = Conn.Execute("SELECT COUNT(*) As PBY From tbl_res Where PlayerId= '"& strpID &"' ")
%>
   
<%= ROUND(percentBla("PBX") / totVarv("PBY"),3)* 100 %>

La detta i en separat fil. Sedan så includar jag den filen i sidan den ska visas på.


Hmm går det kanske att ha båda Set raderna inom samma (<%) och (%>)?

[voigtann1]

ja, du kan ha allt inne i en och samma <% %> (inte <%=%>, men du kan köra response.write på den raden istället) men är detta verkligen det du vill göra? Att räkna antal rader eller är det en summa du är ute efter? Hur ser tabellerna ut och har du några exempel värden och datan du vill presenterad?

[climber]

Det jag vill är att få ut hur många procent en spelare nått resultatet 18-29 av totala antalet minigolf rundor.

Detta får jag fram genom att först räkna ut (percentBla) hur många gånger spelaren nått resultat mellan 18 och 29.

Sedan så räknar jag ut hur många rundor (totVarv) spelaren spelat.

Därefter kommer uträkningen.

Hoppas det blev lite lättare att förstå vad jag ville uppnå med denna förklaring

[civilpolisen]

Ska du ha samma sak på många sidor?
Då kan du inkludera koden från en annan sida. Annars, lägg bara koden på sidan där den ska visas. Det är enklast för dig i dag och i framtiden.

<%
Set percentBla = Conn.Execute("SELECT COUNT(TOTAL) AS PBX FROM tbl_res "_
& " WHERE (TOTAL) BETWEEN 18 AND 29 AND PlayerId= '"& strpID &"'")

Set totVarv = Conn.Execute("SELECT COUNT(*) As PBY From tbl_res " _
& " Where PlayerId= '"& strpID &"' ")

   
response.write ROUND(percentBla("PBX") / totVarv("PBY"),3)* 100 
%>

***
Sen kan du studera skillnaden mellan INCLUDE FILE och INCLUDE VIRTUAL:

INCLUDE FILE="procent.asp"
INCLUDE VIRTUAL="/includes/procent.asp"

FILE = samma katalog / struktur.
VIRTUAL = från rooten, kan flytta runt i hierarkin hur som helst.

En sidfot är ett exempel på en fil som är vettig att ha samma på alla sidor på en sajt.

INCLUDE VIRTUAL="footer.asp"

[climber]

Tack för tipset på INCLUDE VIRTUAL det hade jag ingen aning om att det fanns ens

[climber]

Kom på en följdfråga som jag inte riktigt hittat något svar på,
Jag har 18 fält i databasen som heter AMT1 AMT2 osv till AMT18.

Om jag nu vill räkna ut hur många gånger spelaren har klarat banorna (AMT) på ett slag och spara detta i ett alias för att sedan kunna räkna ut hur många procent ettor spelaren har haft på antalet banor/hål han spelat, hur skriver jag då koden för detta?

Lyckas inte hitta något som ens liknar vad jag vill göra här

Alla tankar, Idéer ock kodexempel mottages med varm hand

[voigtann1]

Mitt tips är att göra om din databas struktur där du har:

playerId     courseId     score

och för varje barna så skapa du en ny post med poäng... på så sätt isollera du dig inte till 18 (gissa att detta kan ha med golf att göra?) att kolla fält för fält är långt ifrån optimalt.

[climber]

Helt riktigt att det har med golf att göra, mer exakt bangolf.
Det jag jobbar på är klubbens statistiksida där dom skriver in resultat bana för bana.
Detta gör dom för att kunna list top 10 per bana (efter snitt) så därför är det nödvändigt att ha en post för varje bana.

[voigtann1]

Ja, men du kan ha banan som en tabell med info, du ha en tabell för spelare, sen har du en tabell som kopplar spelare och banan med antal poäng på just den banan. Så hade jag gjort för att få en struktur som är lätt att bygga vidare på.

Kolla hur sql Join frågor så kan du få en idé vad jag menar

[civilpolisen]

Detta är roligt!
Bara så vi pratar om samma sak, låt oss reda ut begreppen:
Bangolf = minigolf?
Bana = hål?

OK, spelar ingen roll, förutom att man har inte PAR på minigolf... (vad jag vet i alla fall!!)

***
Jag tror att Vogatini menar nåt liknande som detta, men jag kan ha fel. Nu förstår jag att bygga om din databas kanske inte är prioritet nummer ett i skrivande stund, men om du gör en vettig struktur blir sajten MYCKET roligare och intressantare.

Som ny på ASP och SQL har du en hel del att sätta dig in i. Särskilt med tanke på att ditt projekt har rätt usel design (upplägg) från början. Jag menar att din vän tänkte ett steg längre än man ofta gör när han väljer att spara ID-nummer som text... Det är ju ingen förmildrande omständighet, så att säga!

Egentligen skulle du stänga hela hemsidan och göra om allt det där. Jag är benägen att lägga min röst på att du ska bygga om hela sajten och databasen före jul, i alla fall.

Annars får du dra med den där skiten i tidernas evighet och det är ju ÄNNU jobbigare än att ta sig tid att skriva om projektet.

***
Hur som haver:

[users]
intID -- räknare
strName

[bana / lane / hole] -- välj en av dom
intID -- räknare
strText
intPAR -- om det är aktuellt

[game] -- spelomgång
intID
dteCreated
intUserID

[score]
intID
dteCreated
intGameID
intBanaID

***
Nu lade jag till en ytterligare tabell, men du kan få grymt rolig statistik baserat på detta upplägget. Dessutom, du kan med minimal, eller ingen för att vara mer exakt, ansträngning få en superkul hemsida!

Du kan få månadens snabbaste spel, senaste 30 dagar, årets bästa + snabbaste omgång med mera.

Jag tänker mig att många av dina klubbkompisar har smarta telefoner och laddar in resultatet allt efter hålen de avverkar.

***
Det är viktigt att datum verkligen är datum och inte textfält som du lägger in datum i...

Så här ska det vara: (klassisk ASP + SQL)
text = enkelfjump
datum = brädgård
siffror + boolean (sant/falskt) = inget

SQL = SELECT * WHERE strName = 'Hedenhös' AND dteCreated = #2012-08-17# AND intUserID = 23 AND bolView = TRUE

[voigtann1]

Lite offtopic:
@civilpolisen ett id kan vara en GUID och det är ju inte ett tal och det är unikt

[climber]

@civilpolisen

Bangolf = Minigolf check
Bana = Hål check

Att bygga om hela sidan är inte aktuellt för tillfället eftersom föreningen diskuterar om dom ska bygga en wordpress sida istället och då får man fundera på om man ska försöka återskapa detta system i php istället.
Det är därför i nuläget enklast att bara försöka hålla detta system flytande.

[climber]

Om jag vill räkna ihop alla AMT1-AMT18 där värdet är 1, kan man då använda sig av INNER JOIN eller GROUP BY eller hur beter man sig?
Har fortfarande lite svårt att greppa detta

AMT1-AMT18 ligger i tabellen tbl_res.

Har försökt på detta sätt utan att lyckas: (OBS: Har kortat ner det till AMT1-AMT4 i exemplet)

<%
   SET EttorB1 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB1 FROM tbl_res WHERE AMT1=1 AND PlayerId='"& strpID &"'")
   SET EttorB2 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB2 FROM tbl_res WHERE AMT2=1 AND PlayerId='"& strpID &"'")
   SET EttorB3 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB3 FROM tbl_res WHERE AMT3=1 AND PlayerId='"& strpID &"'")
   SET EttorB4 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB4 FROM tbl_res WHERE AMT4=1 AND PlayerId='"& strpID &"'")
   
   SET TotEttor = Conn.Execute("SELECT SUM(AntalEttorB1+AntalEttorB2+AntalEttorB3+AntalEttorB4) AS Ettor %>
<%= TotEttor("Ettor") %>

och så här

<%
   SET EttorB1 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB1 FROM tbl_res WHERE AMT1=1 AND PlayerId='"& strpID &"'")
   SET EttorB2 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB2 FROM tbl_res WHERE AMT2=1 AND PlayerId='"& strpID &"'")
   SET EttorB3 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB3 FROM tbl_res WHERE AMT3=1 AND PlayerId='"& strpID &"'")
   SET EttorB4 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB4 FROM tbl_res WHERE AMT4=1 AND PlayerId='"& strpID &"'")
   
   SET TotEttor = Conn.Execute("SELECT SUM(AntalEttorB1+AntalEttorB2+AntalEttorB3+AntalEttorB4) AS Ettor FROM tbl_res WHERE PlayerId='"& strpID &"'")
%>
<%= TotEttor("Ettor") %>

Har inte riktigt förstått hur INNER JOIN och GROUP BY är tänkt att fungera så det är lite svårt att prova då.

När det gäller INNER JOIN så tror jag mig ha förstått att det gäller när man vill lägga ihop kolumner från olika tabeller och isf är väl det fel metod?

Jag kanske är helt åt skogen fel ute när det gäller detta

[voigtann1]

climber skrev:

Om jag vill räkna ihop alla AMT1-AMT18 där värdet är 1, kan man då använda sig av INNER JOIN eller GROUP BY eller hur beter man sig?
Har fortfarande lite svårt att greppa detta

AMT1-AMT18 ligger i tabellen tbl_res.

Har försökt på detta sätt utan att lyckas: (OBS: Har kortat ner det till AMT1-AMT4 i exemplet)

<%
   SET EttorB1 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB1 FROM tbl_res WHERE AMT1=1 AND PlayerId='"& strpID &"'")
   SET EttorB2 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB2 FROM tbl_res WHERE AMT2=1 AND PlayerId='"& strpID &"'")
   SET EttorB3 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB3 FROM tbl_res WHERE AMT3=1 AND PlayerId='"& strpID &"'")
   SET EttorB4 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB4 FROM tbl_res WHERE AMT4=1 AND PlayerId='"& strpID &"'")
   
   SET TotEttor = Conn.Execute("SELECT SUM(AntalEttorB1+AntalEttorB2+AntalEttorB3+AntalEttorB4) AS Ettor %>
<%= TotEttor("Ettor") %>

och så här

<%
   SET EttorB1 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB1 FROM tbl_res WHERE AMT1=1 AND PlayerId='"& strpID &"'")
   SET EttorB2 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB2 FROM tbl_res WHERE AMT2=1 AND PlayerId='"& strpID &"'")
   SET EttorB3 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB3 FROM tbl_res WHERE AMT3=1 AND PlayerId='"& strpID &"'")
   SET EttorB4 = Conn.Execute("SELECT COUNT(*) AS AntalEttorB4 FROM tbl_res WHERE AMT4=1 AND PlayerId='"& strpID &"'")
   
   SET TotEttor = Conn.Execute("SELECT SUM(AntalEttorB1+AntalEttorB2+AntalEttorB3+AntalEttorB4) AS Ettor FROM tbl_res WHERE PlayerId='"& strpID &"'")
%>
<%= TotEttor("Ettor") %>

Har inte riktigt förstått hur INNER JOIN och GROUP BY är tänkt att fungera så det är lite svårt att prova då.

När det gäller INNER JOIN så tror jag mig ha förstått att det gäller när man vill lägga ihop kolumner från olika tabeller och isf är väl det fel metod?

Jag kanske är helt åt skogen fel ute när det gäller detta

INNER JOIN kan hjälpa, men tror inte koden kommer bli mindre, men du skall inte jobba med ett recordset till(TotEttor) du har ju redan de recordset du har, det är bara att räkna ihop dom. Om du nu måste göra så som du har gjort varför inte då skapa en funktion som gör allt jobb till dig, är ju inte så kul att skriva samma SQL fråga om och om igen:

<%
Function GetOnce(course, playerId)
	GetOnce = Conn.Execute("SELECT COUNT(*) FROM tbl_res WHERE AMT" & course & "=1 AND PlayerId='"& playerId &"'")(0)
End Function

Dim TotalOnce 


TotalOnce = GetOnce(1, strpID) + GetOnce(2, strpID) + GetOnce(3, strpID) + GetOnce(4, strpID)

Response.Write TotalOnce

%>