C# den där j*kla ryggsäcken

Hej! Jag redigerade till ditt inlägg så att koden är enklare att läsa.

Använd gärna [code] före och [/code] efter din kod så blir den lättare att läsa med indrag.

Du har ju skrivit din kod som att du lägger till två objekt, så det är inte konstigt att du får en fråga om att skriva in två objekt att lägga till i ryggsäcken. Det du behöver är en variabel som räknar efter hur många objekt du har i ryggsäcken. Arrayen är alltid lika stor (säg 100 platser) men sen sparar du en variabel som säger hur många objekt du har, och därmed vet du också vilket "index" du ska lägga in ett nytt objekt i.

Båda dina if satser körs så det är helt rätt att både rygga[1] och rygga[2] delarna kommer att köras.
Använd if else, eller switch om du bara vill hantera två eller ett fåtal element
Bättre är att göra en generell if sats utan hårdkodad array position

Skippa loopen och sätt index 1 till texten bara?
Vet inte vad programmet ska göra dock, ska du ha en case för varje position i arrayen? verkar lite märkligt.
För övrigt så använder du inte loop variabeln i for-loopen heller.

Annars för att avbyta en loop tidigare använder du break;

Skickades från m.sweclockers.com

Så du ska bara lägga till om det är tomt?
För det första så måste du ha i som index i rygga, annars har ju loopen ingen funktion.

Ska du bara lägga till en sak i taget?
I så fall måste du bryta loopen efter att du lagt till något.
Ingen ide att någon annan löser uppgiften åt dig, men läs på om break; och return; när det gäller loopar.

 case 1:
Console.write("Lägg till ");
rygga[1] = Console.ReadLine()
Console.write("Lägg till ");
rygga [2]=Console.ReadLine();
rygga[3] osv..

Vad är menyn?
Du får exempelvis hålla din huvudmeny i en while loop som körs så länge programmet är i gång.

var isRunning = true;

while(isRunning)
{
 //... skriv ut meny
 //... välj input

// ... Switch sats med cases

// ... en knapp för att avsluta programmet (isRunning = false)
}

Vet inte exakt hur du menar, case 1 i switchen ska lägga till ett item i ryggsäcket oavsätt vilken position i arrayen man är?
Detta är ju en vanlig for-loop där du kommer loopa arrayen och första värdet som är null kommer få ditt värde. Vet inte om det är så du menar?

for(int i = 0; i < arr.length; i++)
{
    if(arr[i] == null)
    {
        arr[i] = Console.ReadLine();
        break;
    }       
}

Ja om jag fattar rätt så gör du bara som inlägget ovan. Dvs loopa igenom tills du hittar Första tomma platsen i arrayen och sedan stoppar du in din string där

Skickades från m.sweclockers.com

Så strunta i for-loopen och ha bara ett index som pekar på den senast ifyllda positionen i arrayen?

Så istället för index i, så kör du

if(arr[lastIndex] == null)
    {
        arr[i] = Console.ReadLine();
        lastIndex++;
    }

EDIT: Ingen break om man nu tar bort for-loopen

Låter sjukt märkligt, break ska avbryta nuvarande loop och fortsätta till nästa kodrad. Behöver se mer kod för att kunna se vad som då strular. Låter som det är någon annan logik som strular i detta fall.

Skickades från m.sweclockers.com

if (rygga[i] == null)
                                    {
                                        Console.Write("Lägg till: ");
                                        rygga[i] = Console.ReadLine();
                                        if (rygga[i] != null)
                                        {
                                            Console.WriteLine("Du har lagt till: " + rygga[i]);
                                            i = rygga.Length;
                                        }

Det är ju precis den lösningen jag gav två inlägg ovanför...

Du har ju fått lösningen ovan men om du vill använda if satserna som du skrivit det behöver du använda "else"

if (rygga[1] == null)
    {
        Console.Write("Lägg till: ");
        rygga[1] = Console.ReadLine();

    }     
    else  //Denna gör att bara en av dem körs.                           
    if (rygga[2] == null)
    {
        Console.Write("Lägg till: ");
        rygga[2] = Console.ReadLine();
    }