Kombinere mange id'er til en kort string.

Prøv at sige lidt om, hvordan du "har" dem:
Ligger de i et array eller i enkeltstående variable?
Er der nogen logik i, hvordan de er gemt?
Er det vigtigt at den resulterende streng har en bestemt rækkefølge?

De kommer fra en database, så array er vel mest nærliggende. :)

De bliver gemt i grupper af forskellige størrelser - derfor de 1000 <-> 1 million.

Rækkefølgen er vigtig på den måde, at det skal være muligt at "rode" den ud igen til visning på skærmen. Her i kombination med andre række i tabellen.

Helt overordnet er dette en del af en form for breadcrumb spor, som ikke skal fylde for meget, men stadig skal kunne vises til brugeren. Altså at man kan følge eet id hele vejen.

Ah, og jeg henter dem i nummerorden (ORDER BY), naturligvis! ;)

Nu spurgte du jo til det nemmeste og hurtigste, og her vil jeg mene, at json_encode må være svaret, da den jo klemmer et array ned til en streng -- der så igen kan gøres til et array vha. json_decode.

Men det kan godt være, den ikke helt lever op til dit tilføjede ønske om også at skulle "fylde for meget".

"fylde for meget" skal ses i forhold til at skrive hvert enkelt id i strengen... :)

Du kan flere forslag til koden i https://stackoverflow.com/questions/24103868/converting-array-elements-into-range-in-php

Hmm... bruger jeg dette, så fejler serveren med en 503'er.

Jeg ved godt, at det 10 x hvad jeg bad om, men alligevel...er det koden, serveren eller php der ikke kan klare det?


for($j=0; $j<=1999999; $j++) {
  $arr[] = $j;
}
function newItem($start, $prev)
{
    if ($start == $prev)
    {
        $result = $start;
    }
    else
    {
        $result = $start . '-' . $prev;
    }

    return $result;
}

foreach($arr as $item)
{
    if ($prev)
    {
        if ($item != $prev + 1)
        {
            $newarr[] = newItem($start, $prev);
            $start = $item;
        }
    }
    else
    {
        $start = $item;
    }
    $prev = $item;
}

$newarr[] = newItem($start, $prev);

echo implode(',', $newarr);

Afprøv det med et mindre tal end 1999999. Hvis det virker med 1000, men ikke med det tal, så er det næppe koden.

Bortset fra det, så bør koden ikke køre uden fejl under PHP 7, da $newarr ikke er initialiseret som array.

Jeg har prøvet med mindre, og der virker det fint. :)

Serveren kører 5.6.x

Kan det være max eksekveringstid, der er for lang? Prøv at tilføje som linie 1

ini_set("max_execution_time", 600); // in seconds

Jeg har lige fundet at det er memory grænsen. :D

1848104 er max.

og du kan ikke ændre den med

ini_set("memory_limit", "2048M"); // Mb

Det kunne jeg sørme! :)

Lucky you ;)

Så mangler jeg "bare" at vide mig sikker på, at det er nemt at finde "stien" i disse strenge efterfølgende...ja, det burde jeg nok have startet med! :D

Altså, når jeg har rækker som disse:
1-5,7,9-12,15-16
1-3,9-12
2-5,15-16
1-7,9-12

Og derefter vil se de rækker hvor f.eks. 2 eller 10 er med.

Er det absolut i tekststrengen du vil kunne se efter, eller kan det gemmes lidt smartere?

Jeg er åben for forslag. :)

Noget i stil med

$haystack =
  array(
    array("min" => 1, "max" => 5),
    array("min" => 7, "max" => 7),
    array("min" => 9, "max" => 12),
    array("min" => 15, "max" => 16)
  );

$needle = $_GET["n"];

echo("Is ".$needle." in [".HaystackToString($haystack)."]? ".(isNeedleInHaystack($needle, $haystack)? "Yes": "No"));

function isNeedleInHaystack($needle, $haystack)
{
  foreach($haystack as $item)
  {
    if ($needle < $item["min"]) break;
    if ($needle >= $item["min"] && $needle <= $item["max"]) return true;
  }
  return false;
}

function HaystackToString($haystack)
{
  $s = "";
  foreach($haystack as $item)
  {
    if ($s != "") $s .= ",";
    if ($item["min"] == $item["max"])
      $s .= $item["min"];
    else
      $s .= $item["min"]."-".$item["max"];
  }
  return $s;
}

Hmm...

Mjah, ikke helt.

Der vil være mange række - tusindvis som skal søges i gennem, og hver række hvor id'et findes er en del af resultatet.

Så f.eks. id 2 findes i rækkerne 1, 4, 14, 1007 osv.

Ah, og så er der jo lige en detalje, at rækkerne jo ligger i databasen, så det er jo egentligt SQL jeg skal bruge her...

#18 kan udvides med et snuptag til at give det, du beder om i #19.

Men måske du skulle specificere hvor og hvordan du har data og hvad du ønsker først, og så kan vi svare på det.

Ellers bliver det lidt for meget gætteri, synes jeg.

Jamen, data er jo dem vi laver i starten af spørgsmålet - de skal jo smides ind i en database, for så at kunne hive det ud igen på den nævnte måde. :)

Prekonditioner:
1: id'er er oprettet i databasen i anden tabel - er gjort
2: Nogle id'er skal skifte ejer - dette ejerskifte skal registres for hvert id - er gjort med løsningen i #7, taget fra dit link i #6. :)

Derefter:
3: Søg efter eet id i alle rækker og vis dem

Det her vist det tætteste jeg kan google mig til. Men stadig ikke helt nok. :-/

https://stackoverflow.com/questions/25134413/mysql-query-checking-if-integer-between-range-in-single-column-string

Ej, ved du hvad. Jeg laver sgu et nyt spørgsmål i et db kategori. Det er nok bedre. :)

Jeg takker for hjælpen, og du skal da være velkommen i det nye spørgsmål. :)

Dine data er

1-5,7,9-12,15-16
1-3,9-12
2-5,15-16
1-7,9-12

repræsenteret i din database som

no    range minid maxid
1    1    1    5
2    1    7    7
3    1    9    12
4    1    15    16
5    2    1    3
6    2    9    12
7    3    2    5
8    3    15    16
9    4    1    7
10    4    9    12

SELECT range FROM haystacks WHERE needle >= minid AND needle <= maxid

hvor du erstatter needle med den værdi, du vil søge efter

Hmm....interessant løsning!

Men det giver dog mange flere rækker i databasen...ser du ingen bedre måde? :)

Mange rækker i en database er vel intet problem. Hvordan vil du ellers komme det i databasen? jeg synes dette er at foretrække i forhold til en masse tekststrenge, som du ikke kan søge i.

Mange rækker * mange rækker er RIGTIGT mange rækker. :D

Lol, jamen hvis jeg vidste det, spurgte jeg jo ikke her! :D

Ja, det er jo det interessante ved løsningen. ;)

Fordelen med #25 er, at du kan finde alle nåle i høstakken med én SELECT.

Alternativet er, at hive alle data ud af databasen 8som så har færre rækker) en række ad gangen og køre PHP for at finde ud af om der er et match. Jeg gætter på, at det er meget mindre effektivt.

Men det er ret let at teste:

1. Lav to tomme tabeller i databasen - en med tekst-strenge og en som i #25. husk index på minid og maxid
2. Lav x tusind tilfældige tal i stigende rækkefølge om fyld dem i begge tabeller
3. Kør en test og tag tid
4. Fortæl her om resultatet

Jeg er SLET ikke i tvivl om at hive rækkerne ud og tjekke dem igennem er en dårlig løsning. ;)

Jeg tænker bare, om man kunne lave en funktion i MySQL eller nowwet...måske det ville være hurtigere?

Det vil jeg ikke afvise. Det vigtigste er nok at have index på minid og maxid, samt RAM nok på SQL serveren.

Det er et webhotel, så ressourcerne er jo begrænset...

Anyway, jeg kigger på det imorgen. :)

Og siger tak for hjælpen i dag, det var ikk' så ring' endda. :)

Nå, nu nåede jeg endelig igennem de fixbare bugs, og stødte så på ny: Unsupported operand types

I denne linie:
if ($item != $prev + 1)


Google antyder at det er fordi en af dem er et objekt eller lign.

Hele funktionen:
function CombineSeedIDs($SeedIDs = array()) {
   
    if (!is_array($SeedIDs)) {
        return null;
    }
   
    foreach($SeedIDs as $item) {
        if ($prev) {
            if ($item != ($prev + 1)) {
                $newarr[] = array($start, $prev);
                $start = $item;
            }
        } else {
            $start = $item;
        }
        $prev = $item;
    }

    $newarr[] = array($start, $prev);
   
    return $newarr;
}

og dens data kommer herfra:
$result = mysqli_query($mysqli, 'SELECT SeedID FROM '.UBIC_ProtoUBISeeds.' WHERE CurrentOwnerAID='.$FromAID.' ORDER BY SeedID ASC LIMIT '.$Amount);
if ($result) {
while ($row = $result->fetch_assoc()) {
                $array[] = $row;
            }
            $SeedIDs[] = CombineSeedIDs($array);
...
}

Nå, så kom jeg da fordi dén fejl! :D

if ($item["SeedID"] != ($prev["SeedID"] + 1)) {

Til gengæld for jeg nu et mærkeligt array ud af det:
array(1) { [0]=> array(1) { [0]=> array(2) { [0]=> array(1) { ["SeedID"]=> string(7) "1000001" } [1]=> array(1) { ["SeedID"]=> string(7) "1005000" } } } }

Det burde jo bare være 1000001 og 1005000.

Så hvad pokker går det galt??

$item er et array.

Jeg vil mene, at hver gang du bruger en værdi fra $item, skal der stå ($item["SeedID"]. Til gengæld er ideen, at $prev og $start skal være værdier, så sammenligningen skal være:

if ($item["SeedID"] != ($prev + 1)) {

Endelig er der et problem (der muligvis ikke betyder noget, men er dårlig kode) med initialisering af $newarr og $prev. Det vil i hvert fald ikke køre i PHP 7.

Ja, jeg bruger jo oldtidsphp, så her brokker den sig kun over $prev ;)

Dette fixer det hos mig: $prev = null;

At fjerne ["SeedID"] fra $prev gør $newarr helt tom! :O

    foreach($SeedIDs as $item) {
        if ($prev) {
            if ($item["SeedID"] != ($prev + 1)) {
                $newarr[] = array($start, $prev);
                $start = $item;
            }
        } else {
            $start = $item;
        }
        $prev = $item;
    }

Alle steder, hvor der står $item, skal der stå $item["Seed"]

Du bør inkludere

$newarr = array();

som initialisering sammen med

$prev = null;

...undtagen selvfølgelig i foreach sætningen

ps: har ikke testet

Ah, naturligvis. :) (og så er det SeedID ;p )

Det gav et lidt bedre resultat:
array(1) { [0]=> array(1) { [0]=> array(2) { [0]=> string(7) "1000001" [1]=> string(7) "1005000" } } }

Men ikke helt som det bør være. :-/

post lige hele koden igen, som den ser ud nu

Hehe, godt så. :p


        $result = mysqli_query($mysqli, 'SELECT SeedID FROM '.UBIC_ProtoUBISeeds.' WHERE CurrentOwnerAID='.$FromAID.' ORDER BY SeedID ASC LIMIT '.$Amount);
if ($result) {
            $array = [];
            while ($row = $result->fetch_assoc()) {
                $array[] = $row;
            }
$SeedIDs[] = CombineSeedIDs($array);
            var_dump($SeedIDs);
            // insert the trail rows
            foreach($SeedIDs as $item) {
                if (is_array($item)) {
                    $result = mysqli_query($mysqli, 'INSERT INTO '.UBIC_SeedTrail.' (STID, MinSeedID, MaxSeedID, TID) VALUES (DEFAULT, '.$item[0].', '.$item[1].', '.$TID.')');
                } else {
                    $result = mysqli_query($mysqli, 'INSERT INTO '.UBIC_SeedTrail.' (STID, MinSeedID, MaxSeedID, TID) VALUES (DEFAULT, '.$item.', '.$item.', '.$TID.')');
                }
               
            }
...
}

function CombineSeedIDs($SeedIDs = array()) {
   
    if (!is_array($SeedIDs)) {
        return null;
    }
   
    $prev = null;
    $newarr = array();
   
    foreach($SeedIDs as $item) {
        if ($prev) {
            if ($item["SeedID"] != ($prev + 1)) {
                $newarr[] = array($start, $prev);
                $start = $item;
            }
        } else {
            $start = $item["SeedID"];
        }
        $prev = $item["SeedID"];
    }

    $newarr[] = array($start, $prev);

    //echo implode(',', $newarr);
   
    return $newarr;
}

$start = $item;

skal være

$start = $item["SeedID"];

Ah, godt fanget! ;)

Men det ændrer ikke noget. :-/

og

$SeedIDs[] = CombineSeedIDs($array);

skal være

$SeedIDs = CombineSeedIDs($array);

og den skal dumpe

array(1) { [0]=> array(2) { [0]=> string(7) "1000001" [1]=> string(7) "1005000" } }

Uhh, du go'! :D

Øvelse gør mester ;)

Jaw jaw. ;)

Så er der bare den sidste! :D

            $result = mysqli_query($mysqli, 'UPDATE '.UBIC_ProtoUBISeeds.' SET CurrentOwnerAID = '.$ToAID.' WHERE SeedID IN ('.implode(',',$array).'))');

Den fejler med en Array to string conversion

Det er arrayet fra while løkken.

Harh, ku sel! :p (med lidt gooling, men altså.. ;p )

implode(',', array_map(function ($entry) {
                return $entry['SeedID'];
                }, $array));

Eller dvs...nu er der i hvert fald ikke php fejl. :D

Sql derimod ser ikke ud til at virke korrekt. :-/

Jeg havde ellers den opfattelse, at man kunne opdatere mange rækker vha en WHERE id IN (id, id, id,...). Men det kan man åbenbart ikke? :-/

Ah, bare et parentes problem.

NU spiller det sgu!! :D

Super 😀

Jaw jaw. :)

Så tror jeg, at jeg siger tak for kampen - og tålmodigheden! :) :D

Det var ikke så lidt. :) :)