Citat:
Hej
Har skrivit klart min kod men behöver lite hjälp nu för den krånglar.
Den ska skriva ut en triangel som är uppbyggd av siffror men om jag skriver in att den ska skriva ut en triangel med hjälp av 1-4 så så skriver den ut såhär:
1
22
333
4444
Det är ju korrekt men problemet är att den skriver ut triangeln fyra gånger vilket blir fel.
Min kod: http://pastebin.com/FFpcg0eS
Skulle vara jättesnällt om någon kan hjälpa mig
Har skrivit klart min kod men behöver lite hjälp nu för den krånglar.
Den ska skriva ut en triangel som är uppbyggd av siffror men om jag skriver in att den ska skriva ut en triangel med hjälp av 1-4 så så skriver den ut såhär:
1
22
333
4444
Det är ju korrekt men problemet är att den skriver ut triangeln fyra gånger vilket blir fel.
Min kod: http://pastebin.com/FFpcg0eS
Skulle vara jättesnällt om någon kan hjälpa mig
Kod:
import java.util.Scanner;
public class noob {
public static void main(String[] args) {
boolean kora = true;
while (kora) {
Scanner skrivbord = new Scanner(System.in);
System.out.println("Ange längden på den 2 lika långa sidorna (Avsluta med -1):");
int valda = 0;
int langd = skrivbord.nextInt();
if (langd == -1)
kora = !kora;
else {
System.out.println("Ska den räta vinkeln vara nedåt (0) eller uppåt (1):");
valda = skrivbord.nextInt();
int i, j;
if (valda == 0) {
//for (i = 1; i <= langd && langd < 10; i++)
for (i = 1; i <= langd; i++)
{
for (j = 1; j <= i; j++) {
System.out.print(i);
}
System.out.print("\n");
}
}
else if (valda == 1)
//for (i = 1; i <= langd && langd < 10; i++)
for (int i1 = (valda == 0 ? 1 : langd); +valda == 0 ? i1 <= langd : i1 > 0; i1 += (valda == 0 ? 1 : -1))
{
for (int j1 = 1; j1 <= i1; j1++)
System.out.print(i1);
System.out.println();
}
}
}
}
}
Avmarkerade onödiga forloopar
"ursäkta" klassnamnet
__________________
Senast redigerad av PG-k 2016-05-07 kl. 17:54.
Senast redigerad av PG-k 2016-05-07 kl. 17:54.
Citat:
Kan inte kolla koden så bra på mobilen. Blir en gröt.
Såg att du har en nedrans massa for loopar.
Eftersom du säger att programmet skriver ut trianglern 4ggr så kolla om någon variabel har värdet 3 eller 4. Kolla sedan om variablen är ett villkor för en av dina loopar.
Kanske glömmer du att nollställa en variabel?
Orkar tyvärr inte kolla koden på mobilen .
Såg att du har en nedrans massa for loopar.
Eftersom du säger att programmet skriver ut trianglern 4ggr så kolla om någon variabel har värdet 3 eller 4. Kolla sedan om variablen är ett villkor för en av dina loopar.
Kanske glömmer du att nollställa en variabel?
Orkar tyvärr inte kolla koden på mobilen .
Löste det nu, tack för att du försökte
Citat:
Raderna 24 och 25 ser lite onödigt redundanta ut vid en snabb titt.
1. Indentera ordentligt. Det blir lättare att läsa koden.
2. Använd alltid måsvingar, {}, efter if/while... satser. Det ger mer konsekvent kod och du minskar risken för fel om du lägger till en extra rad i satsen.
3. Använd vettiga namn på saker och ting. Varför heter det skrivbord och valda?
4. Klistra gärna in koden här, använd spoiler och code-taggar.
1. Indentera ordentligt. Det blir lättare att läsa koden.
2. Använd alltid måsvingar, {}, efter if/while... satser. Det ger mer konsekvent kod och du minskar risken för fel om du lägger till en extra rad i satsen.
3. Använd vettiga namn på saker och ting. Varför heter det skrivbord och valda?
4. Klistra gärna in koden här, använd spoiler och code-taggar.
Tack för tipsen om vad jag kan göra för att få en renare kod
Citat:
Tack
Citat:
Avmarkerade onödiga forloopar
"ursäkta" klassnamnet
Kod:
import java.util.Scanner;
public class noob {
public static void main(String[] args) {
boolean kora = true;
while (kora) {
Scanner skrivbord = new Scanner(System.in);
System.out.println("Ange längden på den 2 lika långa sidorna (Avsluta med -1):");
int valda = 0;
int langd = skrivbord.nextInt();
if (langd == -1)
kora = !kora;
else {
System.out.println("Ska den räta vinkeln vara nedåt (0) eller uppåt (1):");
valda = skrivbord.nextInt();
int i, j;
if (valda == 0) {
//for (i = 1; i <= langd && langd < 10; i++)
for (i = 1; i <= langd; i++)
{
for (j = 1; j <= i; j++) {
System.out.print(i);
}
System.out.print("\n");
}
}
else if (valda == 1)
//for (i = 1; i <= langd && langd < 10; i++)
for (int i1 = (valda == 0 ? 1 : langd); +valda == 0 ? i1 <= langd : i1 > 0; i1 += (valda == 0 ? 1 : -1))
{
for (int j1 = 1; j1 <= i1; j1++)
System.out.print(i1);
System.out.println();
}
}
}
}
}
Avmarkerade onödiga forloopar
"ursäkta" klassnamnet
Det är ett klassnamn som passar mig, riktigt svårt men ändå kul på samma gång med programmeringen
Tack för hjälpen, koden fungerar nu
Stöd Flashback
Flashback finansieras genom donationer från våra medlemmar och besökare. Det är med hjälp av dig vi kan fortsätta erbjuda en fri samhällsdebatt. Tack för ditt stöd!
Swish: 123 536 99 96 Bankgiro: 211-4106
