ImageCounter von http://www.tommyherrmanndesign.com

Fragen und Diskussionen rund um Internet, Software und PC
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

Wunderschönen guten Tag.

Ich habe mir den im Titel besagten Imagecounter erneut in meine Seite eingebaut, er funktionierte bereits vor einer woche einwandfrei, dann verlor ich meine kleine seite, nun habe ich sie fix erneut aufgebaut, und wieder den script eingebaut:

http://www.tommyherrmanndesign.com/nof/ ... torial.php

Es funktioniert einwadfrei, aber aus irgend einem Grund bekomme ich folgende Meldung:

Code: Alles auswählen

Warning: Undefined array key 1 in D:\xampp\htdocs\wsagapps.php on line 36
Siehet selber:
http://www.wsa-gapps.de/wsagapps.php

In der Gemeldete Zeile steht folgender code:

Code: Alles auswählen

 $clicks[$temp[0]] = $temp[1];

Und das ist der Gesammte Code:

Code: Alles auswählen

<?php
 // Bilderklickzähler
 // Das Script stammt von W. Zenk
 // Homepage: www.homepage-total.de
 // Das Script darf frei genutzt werden,
 // solange dieser Vermerk nicht entfernt wird!

$clicks_file = file('daten.txt');
for ($i = 0; $i < count($clicks_file); $i++) {
 $temp = explode('|', $clicks_file[$i]);
 $clicks[$temp[0]] = $temp[1];
}

function get_clicks($id) {
 global $clicks;
 return $clicks[$id];
}
?>

was funktioniert da nicht mehr?
Benutzeravatar
Tommy Herrmann
Site Admin
Site Admin
Beiträge: 5744
Registriert: So 6. Dez 2020, 07:37
Wohnort: Berlin
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Tommy Herrmann »

Hallo,

und willkommen bei uns im Forum :hallo:

Ich bin gerade unterwegs und kann deswegen heute nicht wirklich helfen. Ich versuche mal den Autor Werner Zenk für Dich zu erreichen.

Hast Du denn die Datei “daten.txt” auch angelegt?
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

Ja, es funktioniert ja wie gesagt alles einwand frei, doch troz alle dem wird mir diese fehlermeldung ausgegeben hehe :) Lustig Kurios

Vielen dank :)
Benutzeravatar
Werner-Zenk.de
Mitglied (Level 10)
Mitglied (Level 10)
Beiträge: 755
Registriert: Di 8. Dez 2020, 19:42
Wohnort: Bamberg
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Werner-Zenk.de »

Die ID des Datensatzes muss vorhanden sein, wenn diese aufgerufen wird!
<?= get_clicks(1);?>
Screenshot 2023-05-07 190954.png
Screenshot 2023-05-07 190954.png (9.93 KiB) 763 mal betrachtet
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

Wie gesagt es ist alles korrekt, und funktioniert einwandfrei, aber aus unerfindlichen Gründen wird der von mir oben gennante fehler gemeldet, oder versteh ich was nicht?
Benutzeravatar
Tommy Herrmann
Site Admin
Site Admin
Beiträge: 5744
Registriert: So 6. Dez 2020, 07:37
Wohnort: Berlin
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Tommy Herrmann »

Moin,

mache doch auch mal einen Screenshot vom Inhalt Deiner Datei:

daten.txt

Werner hat diesen Counter ja damals erstellt - noch unter "homepage-total.de" (seiner alten Website vom Jahr 2010), heute ist das die Website vom Werner:

https://werner-zenk.de/

Ich habe dieses alte Skript doch auch überall, auch auf ganz neuen Seiten, erfolgreich im Einsatz - inzwischen mit PHP 8.0.28, wie z.B. hier in einer Seite, die mit der Software "Mobirise" erstellt wurde:

https://www.mobirise-tutorials.com/Kontakt-Formular.php

daten.txt hat geschrieben:
1|2785|../download/2017-09-04-Mobirise_Mailscript.zip
2|1086|../download/2017-09-23-Mobirise_Mailscript.zip
3|336|../download/2020-10-12-popup_ein_aus.zip
4|4030|../download/2021-04-24-mailscript_phpmailer640.zip
5|1365|../download/2021-05-05-mailscript_2_phpmailer640.zip
6|656|../download/2021-07-21-mailscript_phpmailer650_smtp_mobirise.zip
7|587|../download/2021-10-16-mailscript_phpmailer650_smtp_mobirise_spam1.zip

Der Link zum Zähler kann dann bei der id=4 zum Beispiel so aussehen:

Code: Alles auswählen

<a href="http://www.mobirise-tutorials.com/downloads/imagecounter/zaehler.php?id=4">Download Mailscripts</a>
... und die Anzeige des Zählers zum Beispiel so:

Code: Alles auswählen

<p>Downloads insgesamt seit 04.09.2017:&nbsp;<strong><?php echo get_clicks(4)?></strong></p>

Auch ich verwende immer noch exakt das gleiche Skript, das Du oben gepostet hast - ohne Fehlermeldung.
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

Code: Alles auswählen

1|4|downloads/WSA_mit_Gapps_von_wsa-gapps_2303.40000.5.0_x64.rar
2|1|downloads/WSA_mit_Gapps_von_wsa-gapps_2303.40000.5.0_arm64.rar

Code: Alles auswählen

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"http://www.w3.org/TR/html4/loose.dtd">

<head>
<meta http-equiv="content-Type" content="text/html; utf-8" />
<meta http-equiv="Pragma" content="cache" />
<meta name="robots" content="INDEX,FOLLOW" />
<meta http-equiv="content-Language" content="de" />
<meta name="description" content="Windows Subsystem für Android mit Google Playstor" />
<meta name="keywords" content="wsa,windows,subsystem,android,google,playstore,gapps" />
<meta name="author" content="Wilhelm Zeiger" />
<meta name="publisher" content="Wilhelm Zeiger" />
<meta name="copyright" content="©wsa-gapps" />
<meta name="audience" content="Profis" />
<meta name="page-type" content="Software Download" />
<meta name="page-topic" content="Computer" />
<meta http-equiv="Reply-to" content="wsagappsde@gmail.com" />
<meta name="expires" content="" />
<meta name="revisit-after" content="2 days" />
<title>WSA-Gapps</title>

</head>

<body>
<?php
 // Bilderklickzähler
 // Das Script stammt von W. Zenk
 // Homepage: www.homepage-total.de
 // Das Script darf frei genutzt werden,
 // solange dieser Vermerk nicht entfernt wird!

$clicks_file = file('./daten.txt');
for ($i = 0; $i < count($clicks_file); $i++) 
{
 $temp = explode('|', $clicks_file[$i]);
 $clicks[$temp[0]] = $temp[1];
}

function get_clicks($id) {
 global $clicks;
 return $clicks[$id];
}
?>

</body>





<style>

body { 
margin:0; 
padding:0; 
font-family:verdana,
arial,helvetica,
sans-serif; 
font-size:0.8em; 
}

.logo {
		background-color: #009bde;
	background-image: url("images/logo.png"); 
width: 1024px;
  height: 158px;
  border-radius : 0px 25% 10px;
  box-shadow: 0px 0px 5px black;
  

}

.inhalt_mitte {
	background-color: #009bde;
width: 1024px;
  height: 1000px;
border-radius: 10px;
box-shadow: 0px 0px 5px black;
  position: relative;
  top: 8px;

}



.button {
  background-color: #d9cc00; 
  border: none;
  color: white;
  padding: 5px 10px;
  text-align: center;
  text-color: black;
  text-decoration: none;
  display: inline-block;
  font-size: 16px;
  border-radius: 10px;
  position: relative;
  top: 120px;
   left: 352px;
}

.content {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 200px;
 
  font-color: withe;
  padding: 5px;
  
   position: relative;
  top: 10px;
}

.content_social {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 100px;
  box-shadow: 0px 0px 5px black;
  font-color: withe;
  padding: 5px;
  
   position: relative;
  top: 20px;
}

.content_download {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 150px;
  box-shadow: 0px 0px 5px black;
  color: #ffffff;
  text-align: left;
  padding: 5px;
  
   position: relative;
  top: 32px;
}

.content_update {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 200px;
  box-shadow: 0px 0px 5px black;
  color: #ffffff;
  text-align: left;
  padding: 5px;
  
   position: relative;
  top: 45px;
}

.wsa_bild {
	
border-radius: 10px;
border-color: #000000;

position: relative;

   left:-285px;


}

.wsa_text {

border-radius: 10px;
border-color: #000000;
color: #ffffff;

position: relative;

top: -150px;
left: 250;
text-align: left;
font-size: 20px;

}


.klickzaehler {

border-radius: 10px;
border-color: #000000;
color: #ffffff;

position: relative;

top: 100px;
left: 5px;



}

.download {

border-radius: 10px;
border-color: #000000;
color: #ffffff;

position: relative;

top: 100px;
left: 780px;



}

.download_button {
  background-color: #d9cc00; 
  border: none;
  color: black;
  padding: 5px 10px;
  text-align: center;
  text-color: black;
  text-decoration: none;
  display: inline-block;
  font-size: 16px;
  border-radius: 10px;
  position: relative;
  top: -25px;
   left: -77px;
}

</style>

<center>
<div class="logo">
<a href="index.php"><div class="button">Startseite</div></a>  <a href="wsagapps.php"><div class="button">Download</div></a> <a href="anleitung.php"><div class="button">Anleitung</div></a>


</div>


<div class="inhalt_mitte">





<div class="content_download">WSA mit Gapps 2303.40000.5.0 x64<br>
<hr>
<div class="klickzaehler">Heruntergladen: <?php echo get_clicks(1)?></div><div class="download"><a href="./download.php?id=1" target_"top"><div class="download_button">Download</div></a></div>
 </div>
 <br>

<div class="content_download">WSA mit Gapps 2303.40000.5.0 arm64<br>
<hr>
<div class="klickzaehler">Heruntergeladen: <?php echo get_clicks(2)?></div><div class="download"><a href="./download.php?id=2" target_"top"><div class="download_button">Download</div></a></div>

</div>






</center>
Benutzeravatar
Tommy Herrmann
Site Admin
Site Admin
Beiträge: 5744
Registriert: So 6. Dez 2020, 07:37
Wohnort: Berlin
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Tommy Herrmann »

Moin,

also - das liegt wohl wahrscheinlich nicht daran, aber Dein Quelltext ist vollkommen falsch aufgebaut :eek:

Die Style-Anweisung gehört vor das schließende </head> Tag:

Code: Alles auswählen

<style>

body { 
margin:0; 
padding:0; 
font-family:verdana,
arial,helvetica,
sans-serif; 
font-size:0.8em; 
}

.logo {
		background-color: #009bde;
	background-image: url("images/logo.png"); 
width: 1024px;
  height: 158px;
  border-radius : 0px 25% 10px;
  box-shadow: 0px 0px 5px black;
  

}

.inhalt_mitte {
	background-color: #009bde;
width: 1024px;
  height: 1000px;
border-radius: 10px;
box-shadow: 0px 0px 5px black;
  position: relative;
  top: 8px;

}



.button {
  background-color: #d9cc00; 
  border: none;
  color: white;
  padding: 5px 10px;
  text-align: center;
  text-color: black;
  text-decoration: none;
  display: inline-block;
  font-size: 16px;
  border-radius: 10px;
  position: relative;
  top: 120px;
   left: 352px;
}

.content {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 200px;
 
  font-color: withe;
  padding: 5px;
  
   position: relative;
  top: 10px;
}

.content_social {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 100px;
  box-shadow: 0px 0px 5px black;
  font-color: withe;
  padding: 5px;
  
   position: relative;
  top: 20px;
}

.content_download {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 150px;
  box-shadow: 0px 0px 5px black;
  color: #ffffff;
  text-align: left;
  padding: 5px;
  
   position: relative;
  top: 32px;
}

.content_update {
	background-color: #009bde;
border-radius: 10px;
border-color: #000000;
width: 800px;
  height: 200px;
  box-shadow: 0px 0px 5px black;
  color: #ffffff;
  text-align: left;
  padding: 5px;
  
   position: relative;
  top: 45px;
}

.wsa_bild {
	
border-radius: 10px;
border-color: #000000;

position: relative;

   left:-285px;


}

.wsa_text {

border-radius: 10px;
border-color: #000000;
color: #ffffff;

position: relative;

top: -150px;
left: 250;
text-align: left;
font-size: 20px;

}


.klickzaehler {

border-radius: 10px;
border-color: #000000;
color: #ffffff;

position: relative;

top: 100px;
left: 5px;



}

.download {

border-radius: 10px;
border-color: #000000;
color: #ffffff;

position: relative;

top: 100px;
left: 780px;



}

.download_button {
  background-color: #d9cc00; 
  border: none;
  color: black;
  padding: 5px 10px;
  text-align: center;
  text-color: black;
  text-decoration: none;
  display: inline-block;
  font-size: 16px;
  border-radius: 10px;
  position: relative;
  top: -25px;
   left: -77px;
}

</style>
... und der Inhalt vor das schließende </body> Tag:

Code: Alles auswählen

<center>

<div class="logo">
<a href="index.php"><div class="button">Startseite</div></a>  <a href="wsagapps.php"><div class="button">Download</div></a> <a href="anleitung.php"><div class="button">Anleitung</div></a>

</div>

<div class="inhalt_mitte">

<div class="content_download">WSA mit Gapps 2303.40000.5.0 x64<br>
<hr>
<div class="klickzaehler">Heruntergladen: <?php echo get_clicks(1)?></div><div class="download"><a href="./download.php?id=1" target_"top"><div class="download_button">Download</div></a></div>
 </div>
 <br>

<div class="content_download">WSA mit Gapps 2303.40000.5.0 arm64<br>
<hr>
<div class="klickzaehler">Heruntergeladen: <?php echo get_clicks(2)?></div><div class="download"><a href="./download.php?id=2" target_"top"><div class="download_button">Download</div></a></div>

</div>

</center>
... und die <html> Webseite </html> Tags fehlen völlig :eek:
Benutzeravatar
Tommy Herrmann
Site Admin
Site Admin
Beiträge: 5744
Registriert: So 6. Dez 2020, 07:37
Wohnort: Berlin
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Tommy Herrmann »

... im Prinzip sieht der Aufbau einer Seite mit dem gängigen DOCTYPE "HTML5" heute so aus, egal ob PHP-Inhalte oder nicht. Lediglich die Datei-Erweiterung muss bei PHP dann .php sein (anstatt z.B. .html)

Code: Alles auswählen

<!DOCTYPE html>
<html lang="de">
 <head>
 
  <meta charset="UTF-8"> 
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <meta name="description" content="Beschreibung der Seite - max. 160 Zeichen">

  <title>Titel der Seite - max. 65 Zeichen</title>
  
  <!-- Hier z.B. Dateipfad zu eventuellen CSS Dateien -->
  <link rel="stylesheet" href="./css/bootstrap.min.css">

  <!-- inline Style-Anweisungen -->
  <style>
    body, textarea {
       font-family: Verdana, Sans-Serif;
       font-size: 1rem;
       background-color: #efefef;
       padding-left: 20px;
       padding-right: 20px;
    }
  </style>

 </head>
 
<body>

<h1>Aufbau einer Webseite</h1>
<p>Nur ein Beispiel</p><br>
<p><a href="./kontakt.php">Link zu meiner Kontakt-Seite</a></p>

<?php
  PHP Skript könnte z.B. hier stehen ...
?>

<!-- eventuelle Verweise zu JavaScript Dateien -->
<script src="./js/script.js"></script>

</body>
</html>
Benutzeravatar
Werner-Zenk.de
Mitglied (Level 10)
Mitglied (Level 10)
Beiträge: 755
Registriert: Di 8. Dez 2020, 19:42
Wohnort: Bamberg
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Werner-Zenk.de »

Da wird halt auf eine ID in einem Datensatz (Array) zugegriffen, der nicht existiert. Und nix anderes sagt die Fehlermeldung aus.
Bei mir kommt die gleiche Meldung, wenn die ID nicht vorhanden ist:

Screenshot 2023-05-08 182259.png
(mit PHP 8.2.0)
Tommy hat geschrieben:Die Style-Anweisung gehört vor das schließende </head> Tag
Das stimmt so nicht mehr, aktuelle Browser lassen auch CSS zu, der erst im BODY-Tag definiert wird.
Ich glaube Mozilla waren die ersten die das unterstützten!
Ich mache das aber auch nicht, und lasse die CSS-Anweisungen im Head-Tag, da kann ich schnell darauf zugreifen, ohne den Quelltext nach CSS-Anweisungen zu durchforsten.

Problematisch finde ich jedoch die <center>-Tag Markierung, diese lässt sich durch:
<div style="text-align: center;"></div> ersetzen.

Zurück zum Thema:
Hier, mit diesem Script habe ich das getestet:
imagecounter_aktuell.zip
(1.19 KiB) 27-mal heruntergeladen
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

Keine änderung, so sieht meine data.txt aus

Code: Alles auswählen

1|5|downloads/WSA_mit_Gapps_von_wsa-gapps_2303.40000.5.0_x64.rar
2|2|downloads/WSA_mit_Gapps_von_wsa-gapps_2303.40000.5.0_arm64.rar

Ab er die beiden ID 1 und 2 sind ja vorhanden
Benutzeravatar
Werner-Zenk.de
Mitglied (Level 10)
Mitglied (Level 10)
Beiträge: 755
Registriert: Di 8. Dez 2020, 19:42
Wohnort: Bamberg
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Werner-Zenk.de »

Kannst du mir das mal als Zip-Datei senden (nur das nötigste).
Hast du schon mein aktuelles Script getestet?
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

ja
habe ich getestet
wsa-gapps.rar
(2.13 KiB) 22-mal heruntergeladen
PrexSL
Mitglied (Level 1)
Mitglied (Level 1)
Beiträge: 7
Registriert: So 7. Mai 2023, 15:42

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von PrexSL »

Hab den fehler gefunden, alles gut jetzt, hatte in der data.txt mehrere leere zeilen, die muss der code wohl als id angesehen haben
Benutzeravatar
Tommy Herrmann
Site Admin
Site Admin
Beiträge: 5744
Registriert: So 6. Dez 2020, 07:37
Wohnort: Berlin
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Tommy Herrmann »

Moin,

na ein Glück - ich hätte jetzt auch keinerlei Idee mehr gehabt.

Natürlich wird eine Leerzeile, wie jede andere Zeile, über die for Schleife mit ausgelesen und dann fehlt die ID.
Benutzeravatar
Tommy Herrmann
Site Admin
Site Admin
Beiträge: 5744
Registriert: So 6. Dez 2020, 07:37
Wohnort: Berlin
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Tommy Herrmann »

Werner-Zenk.de hat geschrieben: Mo 8. Mai 2023, 19:02
Tommy hat geschrieben:Die Style-Anweisung gehört vor das schließende </head> Tag
Das stimmt so nicht mehr, aktuelle Browser lassen auch CSS zu, der erst im BODY-Tag definiert wird.
@ Werner,

aber hast Du mal den Quelltext seiner Seite angeguckt? Das ist doch dennoch grundsätzlich falsch.

Er hängt den Style und auch den Inhalt ganz unten an die Datei ran (unterhalb vom </body>), anstatt das in die entsprechenden HTML-Tags einzupflegen.
Benutzeravatar
Werner-Zenk.de
Mitglied (Level 10)
Mitglied (Level 10)
Beiträge: 755
Registriert: Di 8. Dez 2020, 19:42
Wohnort: Bamberg
Kontaktdaten:

Re: ImageCounter von http://www.tommyherrmanndesign.com

Ungelesener Beitrag von Werner-Zenk.de »

Ach, die CSS-Anweisungen stehen nach dem </body> Tag ... so etwas habe ich auch noch nicht gesehen :eek:
Man sollte schon validen Code schreiben, dieser wird dann korrekt angezeigt und wird auch schnell im Browser geladen.
Antworten

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 13 Gäste