Minimalizacja skryptu Opcje

[Agusia]

Witajcie, mam pewien problemik i ukłony w stronę osób, które się na tym znają.
Osiągnęłam swój cel, ale zajęło mi to około 350 linijek. Wiem, że da się to rozwiązać w lżejszy sposób.

Założenia:
Skrypt ma sprawdzić zmienną czy jest poprawna pod względem wielkości liter
$zmienna1="JakiSteKsT";
$zmienna2="jakiSteKsT";
Jeżeli nie jest to wykonuje jakąś operacje

[PHP] pobierz, plaintext 
<?
$zmienna="JakaZmienna";
echo $zmienna;
echo "</br>____________________________________________</br>";
 
$A=0;$B=0;$C=0;$D=0;$E=0;$F=0;$G=0;$H=0;$I=0;$J=0;$K=0;$L=0;$M=0;$N=0;$O=0;$P=0;$R=0;$S=0;$T=0;$U=0;$W=0;$X=0;$Y=0;$Z=0;
if(ereg("A", $zmienna))
$A=$A+1;
if(ereg("B", $zmienna))
$B=$B+1;
if(ereg("C", $zmienna))
$C=$C+1;
if(ereg("D", $zmienna))
$D=$D+1;
if(ereg("E", $zmienna))
$E=$E+1;
if(ereg("F", $zmienna))
$F=$F+1;
if(ereg("G", $zmienna))
$G=$G+1;
if(ereg("H", $zmienna))
$H=$H+1;
if(ereg("I", $zmienna))
$I=$I+1;
if(ereg("J", $zmienna))
$J=$J+1;
if(ereg("K", $zmienna))
$K=$K+1;
if(ereg("L", $zmienna))
$L=$L+1;
if(ereg("M", $zmienna))
$M=$M+1;
if(ereg("N", $zmienna))
$N=$N+1;
if(ereg("O", $zmienna))
$O=$O+1;
if(ereg("P", $zmienna))
$P=$P+1;
if(ereg("R", $zmienna))
$R=$R+1;
if(ereg("S", $zmienna))
$S=$S+1;
if(ereg("T", $zmienna))
$T=$T+1;
if(ereg("U", $zmienna))
$U=$U+1;
if(ereg("W", $zmienna))
$W=$W+1;
if(ereg("X", $zmienna))
$X=$X+1;
if(ereg("Y", $zmienna))
$Y=$Y+1;
if(ereg("Z", $zmienna))
$Z=$Z+1;
 
$zmienna2="Jakaszmienna";
echo $zmienna2;
echo "</br>____________________________________________</br>";
 
$AA=0;$BB=0;$CC=0;$DD=0;$EE=0;$FF=0;$GG=0;$HH=0;$II=0;$JJ=0;$KK=0;$LL=0;$MM=0;$NN=0;$OO=0;$PP=0;$RR=0;$SS=0;$TT=0;$UU=0;$WW=0;$XX=0;$YY=0;$ZZ=0;
if(ereg("A", $zmienna2))
$AA=$AA+1;
if(ereg("B", $zmienna2))
$BB=$BB+1;
if(ereg("C", $zmienna2))
$CC=$CC+1;
if(ereg("D", $zmienna2))
$DD=$DD+1;
if(ereg("E", $zmienna2))
$EE=$EE+1;
if(ereg("F", $zmienna2))
$FF=$FF+1;
if(ereg("G", $zmienna2))
$GG=$GG+1;
if(ereg("H", $zmienna2))
$HH=$HH+1;
if(ereg("I", $zmienna2))
$II=$II+1;
if(ereg("J", $zmienna2))
$JJ=$JJ+1;
if(ereg("K", $zmienna2))
$KK=$KK+1;
if(ereg("L", $zmienna2))
$LL=$LL+1;
if(ereg("M", $zmienna2))
$MM=$MM+1;
if(ereg("N", $zmienna2))
$NN=$NN+1;
if(ereg("O", $zmienna2))
$OO=$OO+1;
if(ereg("P", $zmienna2))
$PP=$PP+1;
if(ereg("R", $zmienna2))
$RR=$RR+1;
if(ereg("S", $zmienna2))
$SS=$SS+1;
if(ereg("T", $zmienna2))
$TT=$TT+1;
if(ereg("U", $zmienna2))
$UU=$UU+1;
if(ereg("W", $zmienna2))
$WW=$WW+1;
if(ereg("X", $zmienna2))
$XX=$XX+1;
if(ereg("Y", $zmienna2))
$YY=$YY+1;
if(ereg("Z", $zmienna2))
$ZZ=$ZZ+1;
if ($A==$AA){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($B==$BB){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($C==$CC){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($D==$DD){
//jest lux
}
else
{
   $zmieniamy="1";
}
 
if ($E==$EE){
//jest lux
}
else
{
   $zmieniamy="1";
}
 
if ($F==$FF){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($G==$GG){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($H==$HH){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($I==$II){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($J==$JJ){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($K==$KK){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($L==$LL){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($M==$MM){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($N==$NN){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($O==$OO){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($P==$PP){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($R==$RR){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($S==$SS){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($T==$TT){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($U==$UU){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($W==$WW){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($X==$XX){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($Y==$YY){
//jest lux
}
else
{
   $zmieniamy="1";
}
if ($Z==$ZZ){
//jest lux
}
else
{
   $zmieniamy="1";
}
 
if($zmieniamy==1)
{
   echo "Zmieniamy URLA gdyż zmienne różnią się";
}
 
   ?>	
 
[PHP] pobierz, plaintext 

Ma ktoś jakiś pomysł ?

Ten post edytował Agusia 11.12.2015, 11:42:08

[nospor]

Chyba nie ogarniam...

a czemu poprostu nie zrobisz:

if ($zmienna1 != $zmienna2) echo 'Sie roznia';

[Crozin]

Możesz napisać jakie są założenia tego skryptu? Podaj jakieś przykłady danych wejściowych i wyjściowych.

[Agusia]

skrypt sprawdza dwie zmienne takie same lecz różniące się małymi i wielkimi literami.
no i jeżeli obydwie są takie same (małe i duże litery się zgadzają) to nie robi nic
jeżeli jest różnica wielkości znaków to zmienna przyjmuje wartość ustaloną
generalnie to działa tylko jest chyba zbyt duże.

Czy skrypt w 1 poście jest tym samym co ?

[PHP] pobierz, plaintext 
<?
$zmienna="Cos";
$zmienna2="cos";
 
if($zmienna == $zmienna2)
{
	echo "jest ok";
}
else
{
	  echo "Zmieniamy URLA gdyż zmienne różnią się";
;}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Agusia 11.12.2015, 12:04:36

[nospor]

No to juz ci podalem rozwiazanie... jedna linia zamiast tego calego Twojego metliku

edit: tak, to jest to samo

[KsaR]

jeżeli obydwie są takie same (małe i duże litery się zgadzają) to nie robi nic

Musisz skorzystać z wbudowanej funkcji strcasecmp, gdyż operatory nie patrzą na wielkość.

[PHP] pobierz, plaintext 
if (0 === strcasecmp($zmienna, $zmienna2))
{
    # identczne
}
else
{
   # różnią się wielkoscia czy ogółem. (IMG:style_emoticons/default/biggrin.gif) 
}
[PHP] pobierz, plaintext 

Ten post edytował KsaR 11.12.2015, 12:14:52

[Crozin]

Chcesz porównać tylko te ~26 znaków (A-Z) czy ogólnie chcesz porównać wszystkie znaki?

[Regz]

Agusi chodziło o coś takiego:

[PHP] pobierz, plaintext 
<?php
 
	function URLtoLower() 
	{
 
		$protocol = strpos(strtolower($_SERVER['SERVER_PROTOCOL']),'https') === FALSE ? 'http' : 'https';            
		$host     = $_SERVER['HTTP_HOST'];   
		$script   = $_SERVER['SCRIPT_NAME']; 
		$params   = $_SERVER['QUERY_STRING'];
 
		$URL = $protocol . '://' . $host . $script . '?' . $params;
 
		if (preg_match("/[A-Z]/", $URL))
		{
			header('Location: '. strtolower($URL));
			die();
		}
	}
 
	URLtoLower();
[PHP] pobierz, plaintext 

Ten post edytował Regz 11.12.2015, 12:19:42

[Agusia]

@Regz nie zależało mi na Lowerowaniu

---
Jakiś czas temu założyłam serwis zawierający gify i obrazki animowane i niedawno GoogleWebmasterTools wykrył u mnie zduplikowane strony.
Założyłam sobie że urle będą wyglądały w ten sposób

Kod

http://animowane-gify.pl/pobierz_SpieszmysieKochacludzitakszybkoodchodza_714

Problem w tym że jeżeli ktoś gdzieś umieścił odnośnik z małymi literami w linku to google traktował to jako 2 identyczne strony z tym samym contentem = duplicate content.

No i pisząc skrypt i później pytając się Was chciałam przekierować URL do poprawnej wersji z dużymi literami.
Błędne adresy URL naprawione poniższym przykładem. Może jeszcze komuś się przyda.

[PHP] pobierz, plaintext 
$URLPOPRAWNY = $seoURL;
$URLAKTYWNY = $_SERVER['REQUEST_URI'];
$URLAKTYWNY = explode("_", $URLAKTYWNY);
$URLAKTYWNYOK = $URLAKTYWNY[1];
$zmienna=$URLPOPRAWNY;
$zmienna2=$URLAKTYWNYOK;
	if($zmienna == $zmienna2)
    {
    	//echo "jest ok";
    }
    else
    {
    	  $zmieniamy=1;
    ;}
	if($zmieniamy==1)
	{
	echo "Zmieniamy  błędnego urla.";
	$urlik='poprawnyurl.pl';
	echo $urlik;
	header("HTTP/1.1 301 Moved Permanently");
	header("Location: ".$urlik."");
	}
[PHP] pobierz, plaintext 

naprawdę dziękuję za pomoc. I życzę wszystkim miłego dnia.

Ten post edytował Agusia 11.12.2015, 12:49:30

[Pyton_000]

Musisz skorzystać z wbudowanej funkcji strcasecmp, gdyż operatory nie patrzą na wielkość.

Ekhm....
Zanim napiszesz to sprawdź (IMG:style_emoticons/default/wink.gif)

Kod

php -r "var_dump('ala' === 'Ala');"
bool(false)

php -r "var_dump('ala' === 'ala');"
bool(true)

PHP 5.6.11

[KsaR]

Ekhm....
Zanim napiszesz to sprawdź (IMG:style_emoticons/default/wink.gif)

Kod

php -r "var_dump('ala' === 'Ala');"
bool(false)

php -r "var_dump('ala' === 'ala');"
bool(true)

PHP 5.6.11

Masz racje, bo uzyles operatora === ktory sprawdza takze typ..
http://stackoverflow.com/questions/3333353...using-vs-strcmp

Co do cytatu to gdzies musieli zle napisac ale nie moge odszukac gdzie, z glowy bym tego nie wymyslil. A moze jakies glebsze testy by trzeba np. Z emoji, ruskimi znakami itd. Ale pewnosci juz nie mam (IMG:style_emoticons/default/tongue.gif)

A tak napisalem przy okazji bo nad moim postem byl operator == zamiast === tak czy inaczej (IMG:style_emoticons/default/biggrin.gif)

[kapslokk]

Masz racje, bo uzyles operatora === ktory sprawdza takze typ..

Przecież w obu przypdkach typ jest ten sam - string.
Poza tym, u mnie z 2 = też działa:

vagrant@debian-jessie:~$ php -r "var_dump('ala' == 'Ala');"
Command line code:1:
bool(false)
vagrant@debian-jessie~$ php -r "var_dump('ala' == 'ala');"
Command line code:1:
bool(true)

Ten post edytował kapslokk 11.12.2015, 13:19:05

[KsaR]

kapslokk a sprawdziles link ktory dalem?
Tam bylo tez cos takiego (odpal to)

[PHP] pobierz, plaintext 
<?php
 
var_dump('0XAB' == '0xab');
var_dump('0XAB' === '0xab');
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/cool.gif) , jest string? - jest.
Tak więc operatory jak widać też coś robią :-P

EDIT.
Jak dodalem jeszcze do testu
var_dump(0 === strcasecmp('0XAB', '0xab'));
To wygrywa samo ===, (IMG:style_emoticons/default/biggrin.gif) ale gdzies na jakims blogu tak czy inaczej tak widzialem, nie pamietam jakie stringi byly porownywane (IMG:style_emoticons/default/tongue.gif) P

Edit 2.
http://stackoverflow.com/questions/3506706...ring-comparison
Tu jest tez ze

== is case sensitive.

You can use strcasecmp for case insensitive comparison

....

Ten post edytował KsaR 11.12.2015, 13:44:38

[Pyton_000]

Ba...

[PHP] pobierz, plaintext 
php -r "var_dump("0XAB" === "0xab");"
bool(true)
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/haha.gif)