Problem med sökfunktion i c#

Innan du gör sökningen (for-loopen), skapa en bool som anger om den är hittad eller inte, från början satt till false. Om den hittas, sätt variabeln till true (och avbryt loopen, resten är slöseri med CPU-tid). Sen kollar du bara om variabeln är sann eller inte.

För att lägga till information på ovan skrivare: Du ska alltså ta bort else I loopen och sätts en till if efter loopen som kikar ditt nya boom-värde du har skapat enligt ovan skrivare.

Din jämförelse är lite tokig, du ska inte jämföra med ryggans totala längd, utan värdet på de individuella elementen. Du skriver redan ut rygga[i] som innehåller värdet du vill jämföra med, så använd den istället för rygga.Length i din jämförelse.

Console.WriteLine("Skriv in det du söker\n");
                            string sök = Console.ReadLine();// sparar användarens inmatning
                            bool hitta = false;
                            for (int i = 0; i < rygga.Length; i++)// här använder jag en lista som räknar i  index 

                                if (rygga[i] == sök) // om sökningen matchar med listan i index så ska programmet skirva att föremålet hittats.
                                {
                                    hitta = true;
                                    break;
                                }
                            }
                            if(hitta)
                                Console.WriteLine("\nHittades i ryggsäcken.\n");
                            else
                                Console.WriteLine("\nHittades INTE i ryggsäcken.\n");

Är tanken att den ska bryta loopen vid första träff eller ska den lista ut alla träffar?
Sedan kan jag tillägga att sökningen just nu är case sensitive, söker du på "katt" så kommer du inte få en träff på ordet "Katt".

if(string.Equals(rygga[i], sök, StringComparison.OrdinalIgnoreCase)
{
  ...
}

if(rygga[i].ToLower() == sök.ToLower())

Tänk på att denna konvertering kommer att köras för båda strängarna vid varje iteration. Är det många jämförelser kan det vara väldigt tungt att köra. ToLower fungerar inte heller för alla språk då visa konverteringar kan bli knasiga.

Står bara "Sök i ryggsäcken" i menyvalet, jag vet inte kraven på själva uppgiften.
Jag sa det bara som en poängtering så att TS inte sitter och söker på "katt" och förväntas få match på ett föremål av "Katt".
Givetvis får det anpassas till uppgiften och det behöver inte vara korrekt

Prova använda break; när du hittat rätt värde. Det avbryter den närmsta loopen.

För listor har du contains https://docs.microsoft.com/en-us/dotnet/api/system.collection...

if(dinLista.Contains(valueToFind))
{
... 
}

Kod skriven på telefonen, ursäkta om det ser kaos ut.

@Nissan300zx:
Contains gör en linjär sökning, så du behöver inte ha någon for-loop runt Loggboken.Contains(sök); det blir bara kaka på kaka.

För övrigt tror jag inte att ditt felmeddelande rör den biten kod då Loggboken[i] returnerar en string (såvida du inte har skapat en "jagged array", d.v.s. en array av arrayer).

Visual Studio borde tala om för dig vilken rad som felar, dubbelkolla den.

Låter som din lista innehåller arrayer, alltså List<string[]>. Det innebär att du måste gå en nivå djupare för att söka mot strängarna. Simplaste är att nästa två for loopar.

Du har fått bra svar angående sökningar, så jag kommer bara med ett generellt tips inför framtida programmering. Blanda inte språk i koden!

Välj ett språk, och håll dig till det. Gäller både kommentarer och variabler/metoder. Mycket lättare att läsa och förstå koden då.

Och använd code-taggen när du postar kod i forum så får du anyo formatering

Ju lättare vi läsare förstår vad du gjort, desto bättre svar kommer du att få.
Lycka till!

Den lärare som säger Vektor om Array borde få sparken på direkten... Sååå många problem det skapar för de stackars eleverna.

Tänk igenom vad denna raden gör:

                            if (sök == logg[0] || sök == logg[1])

Är det verkligen det du vill testa?
(En hint till: du använder aldrig loopvariabeln i i sökningen. Samt, inte en hint utan observation: "hitta" används inte, men den behövs inte som koden ser ut nu.)

@Nissan300zx: Ett tips är att ta dig en titt på hur debuggern fungerar, här är en kort guide som går igenom grunderna. Med debuggern kan du se exakt vad din kod gör när den körs, så att lägga några minuter på att lära dig använda den kommer spara dig mycket tid i längden.

 case 3:
                        
                       Console.WriteLine("skriv in Titeln du vill söka på: ");
                       string sök = Console.ReadLine();
                        bool hitta = false;
                        for (int i = 0; i < Loggboken.Count; i++)
                            if (sök == logg[i])
                            {
                                hitta = true;
                                Console.WriteLine("\nTitel: " +logg[0] +"\nInlägg: " + logg[1]);

                                Console.WriteLine();
                            }
                        

                        else if (!hitta)
                            {
                                hitta = true;
                                Console.WriteLine("Finns INTE i Loggboken."); 
                            }                                               
                        break;

Vart får du logg[i] ifrån? Du måste loopa den inre arrayen för varje entry.

Edit: Nu såg jag. Du kollar bara värdet i senaste loggen, din log variabel. Du måste nästa en till loop som går igenom LoggBook[i].Lenght för varje string[].
Tror det blir enklare att läsa med en foreach.

Kolla hur du löst case 1.

@Nissan300zx Du använder fel variabel i if-satsen (sök == logg[i]). logg är det senaste du skrivit in (koden i case 2), därav problemet att den bara lyckas hitta den senaste.
Det är Loggboken[i] du vill använda. Det blir dock lite mer komplext eftersom Loggboken[i] är en array med två strängar i sig, så du måste kolla både Loggboken[i][0] (titeln) och Loggboken[i][1] (inlägg).

(Det finns klart snyggare lösningar än [0] och [1], men gissningsvis har ni inte kommit dit ännu, så kör vidare på det där. )

Det går väl att göra. Då loggBok[i] är en string[] så behöver du även iterera indexen i den för att få ut värdet.
Du kan göra det manuellt (i detta fall position 0 som jag antar är titeln) eller med en loop för att gå igenom alla värden.

I vanliga fall så arbetar man inte så här, så det är ingen fara om du tycker det är svårt. Man använder objekt med properties som representerar datan man arbetar med. Tycker det är lite fel av skolan att lära ut på detta sätt, men det verkar vara vanligt i gymnasiet då alla verkar få denna uppgift.

En loop som enbart söker igenom titlarna:

for(int i = 0; i < LoggBoken.Count; i++)
{
    if(LoggBoken[i][0] == sök)
    {
	Console.WriteLine("Hittat!");
    }		   
}

Två loopar som kommer att kolla texten i alla fält i string[]:

foreach(var log in Loggboken)
{
    foreach(var item in log)
    {
         if(item == sök)
         {
              Console.WriteLine("Hittat!");
         }
    }   
}